- 题目：程序设计第三次作业
- 程序设计第三次作业附加
我的程序设计第三次作业附加 代码规范
更新时间：2016/3/23
第一次尝试：(此时题目尚未更改，我按要求把-和数字连起来输出)
我在学习完相关的C++类的知识还有STL中stack和queue的用法之后，大概捋清了题意。用不到十五分钟的时间敲出了代码1的大体，但还是由于自己对相关知识的不熟悉，修改大大小小的错误花费了我一天的时间，代码1如下。
- 代码1
.cpp:
#include<stdlib.h>
#include<iostream>
#include<string>
#include<queue>
#include<cstdio>
using namespace std;
class Scan
{
// private section
public:
void ToStringQueue(string input);
};
void Scan::ToStringQueue(string input)
{
string s;//Define a string "s",and store(存储) something by using it.
queue<string>que;//Define a queue "que".
int m,n,i,j,t=0;
n=input.size();//Define int n,"n = string's length".
for(i=0;i<n;i++)//mistake:i=1;i<=n;
{
if(input[i]=='-')
{
t=1;
que.push(s);//
s.clear();//clear s
s+=input[i];
continue;
}
else
{
if(t==1)// '-'
{
if(input[i]=='('||input[i]==')'||input[i]=='+'||input[i]=='*'||input[i]=='/')
{
t=0;
que.push(s);
s.clear();
s+=input[i];
que.push(s);//que.push(input[i]);
s.clear();
}
else
{
s+=input[i];
continue;
}
}
else
{
if(input[i]=='('||input[i]==')'||input[i]=='+'||input[i]=='*'||input[i]=='/')
{
que.push(s);
s.clear();
s+=input[i];
que.push(s);//que.push(s);
s.clear();
}
else
{
s+=input[i];
continue;
}
}
}
}
m=s.size();
if(m!=0)que.push(s);//check s
s.clear();
while(que.empty()==0)//mistake:j<n;
{
cout<<que.front()<<endl;
que.pop();
}
}
int main()
{
int n=0,i,j;
Scan sc;//实例化对象miss
string b;
cin>>b;
sc.ToStringQueue(b);
return 0;
}
//example:
//-100+(98-97)*2
//-100+(98+97)*2
//100*(99-98)
//100*10+1000
//100*10-100*(98*97-1)+100
//
- 输入样例：
100*10-100*(98*97-1)+100- 运行结果：
思路分析：
大概来阐述一下我的思路，在
void Scan::ToStringQueue(string input)接收完队列input以后，首先定义一个空字符串s和一个类型为queue<string>的队列que(定义语句：queue<string>que;),其次利用string的函数input.size()把字符串input的长度(一共有几个字符)赋值给int类型的n，再利用for循环对一个个字符进行判断。我还定义一个int类型的变量t：
int t=0;，并且利用t的值进行状态判断。这里使用bool变量进行判断也是可以的。如果此时for循环处理的字符
input.[i]是-(这里利用==判断两个字符是否相同),把t的值转换成1。而语句que.push(s);s.clear();是把s之前储存的内容push到队列que，再利用函数s.clear()清除字符串里的字符。然后把-存储到字符串s里：s+=input[i];(这里我犯了一个错误)，continue。然后对下一个元素进行判断，如果是数字再利用+=存储到字符串s，利用continue，一直到下一个字符串input的元素不是数字为止，比如-100+，s存储到-100的时候,把s的内容push到队列里，再清空字符串s，对下一个元素进行判断。如果不是数字，而是+,/,*,(,)这样的符号的话，和上面同理，先把s先前存储的内容push，清空s，再利用+=把此时判断的字符存储到字符串s上，再push到队列que，清空s。
- 简单来说，如果遇到
-，把t的值转为1，再看下一个元素，如果下一个元素是数字再进行存储的操作；如果是其他字符就把s存储的先前内容push，清空s，再把s现在要存储的东西push，清空s，把t的值转为0。- 在for循环结束后，有如下代码：
m=s.size();
if(m!=0)que.push(s);//check s
s.clear();
利用函数
s.size()把此时字符串的长度赋值给int类型的m，判断m是否为0，如果不为0，说明s仍有存储的内容尚未push，把剩下的内容push并清空s。
根据上面的测试结果，读者会发现，输出结果多了几个行，而且输出并没有按照要求放在一个类里面。
为什么会多输出
\n(换行符)呢，这个问题我们先放一放。关于第二个问题，代码1是我的最开始的测试代码，它用于验证我的大体思路和想法是否正确，与最后的成品相比亘长且不成熟。
在代码1的编写过程中我遇到了大大小小的问题，下面即是我对这些问题的一个总结和反思。
问题1：for(i=1;i<=n;i++)
这个问题跟我之前给数组元素赋值有关，如果有n个变量需要我输入，我喜欢从a[1]开始(假如定义了一个数组a)，这给我带来了方便，但是也是一个坏习惯。
而此处，我输入一个字符串input，比如是-100+99，-就是由input.[0]存储的。
我在发现这个问题以后把i=1改成了i=0，但是i<=n忘记更改了，于是输出结果的后面莫名多了个a。
最后发现它的时候我也是醉了...马虎过头。
- 解决方法：
for(i=0;i<n;i++);
问题2：que.push(input[i]);
这是我之前的push语句，编译并没有通过，主要原因是队列que存储的是string类型，定义que的语句：
queue<string>que;，而input[i]是字符类型，与函数push传递的变量类型string不相符。
- 解决方法：自定义一个空字符串s，来储存字符
input.[i]。
问题3：主函数里未实例化对象。
这是C++类的知识，在这个.cpp文件里我定义了一个Scan类，并且类外定义了函数：
void ToStringQueue(string input),这与::运算符有关。
而在主函数中如果想要调用Scan类中声明的函数ToStringQueue，就必须实例化一个Scan类的对象。
参考：实例化对象
- 解决方法：在主函数里面定义一个对象sc：
Scan sc;然后通过调用函数ToStringQueue：sc.ToStringQueue(input)- 关于"::"运算符
第二次尝试(-单独一行)：
在第一次尝试的铺垫和理解下，第二次尝试我试着把class的声明内容放到.h文件内，而其他定义内容则在.cpp文件内进行。
代码2:
.h：
#ifndef CALCULATOR_H
#define CALCULATOR_H
#include<string>
#include<queue>
#include<stdlib.h>
#include<iostream>
using namespace std;
/*
* No description
*/
class Print
{
public:
void pout();
};
class Scan
{
// private section
public:
void ToStringQueue(string input);
};
#endif // CALCULATOR_H
.cpp:
#include "calculator.h" // class's header file
#include<string>
#include<queue>
#include<stdlib.h>
#include<iostream>
using namespace std;
queue<string>que;
string s;
void Scan::ToStringQueue(string input)
{
int n=input.length(),m=0,i,j;//n是字符串input的长度
for(i=0;i<n;i++)//n has no define;
{
if(input[i]=='-'||input[i]=='+'||input[i]=='('||input[i]==')'||input[i]=='/'||input[i]=='*')
{
que.push(s);
s.clear();
s+=input[i];
que.push(s);
s.clear();
}
else
{
s+=input[i];
continue;
}
}
m=s.size();//利用m判断s是否为空串
if(m!=0)que.push(s);
s.clear();
}
void Print::pout()//输出函数pout单独声明在out类里
{
while(que.empty()==0)
{
cout<<que.front()<<endl;
que.pop();
}
}
int main()
{
Scan Sc;//实例化对象Sc
Print Put;//实例化对象Put
string input;
cin>>input;
Sc.ToStringQueue(input);
Put.pout();
return 0;
}
//example：-100+(98-97)*2
- 输入样例：
-100+(98-97)*2- 运行结果：
思路分析：
.cpp文件与代码1类似，我这里仍然利用一个空字符串s来存储input的元素。不同的是，由于题目做出了改变，我这里并没有对-进行判断。- 而代码2比较特别的地方在于它的
.h文件，我把对于Scan和Print类的声明放在了.h文件里，那么.h文件里就是声明类的部分，.cpp文件里是定义函数及其他的部分。
- 注意，我创建的
.h和.cpp文件都是在同一个工程里(工程1)，在.cpp文件的开头需加上：#include"calculator";来包括calculator.h这个头文件。
问题：
因为在这之前有了代码1的实战体验，敲代码2比较快，而且错误少。在刚开始我忘记对n进行赋值(但是对它进行了初始化：
n=0;)，导致队列que内并没有push进东西：
没有输出说明循环出了问题，很快我就找到了n并解决了这个问题：n=input.length();
到目前为止，我把代码的大体已经成功的写出来了，现在仍然还有等待我解决的问题和要求：
要求：
- 1）当输入的数字超过10位（包括小数位）时，报错。
- 能够处理不超过10位的任意实数。
- 代码要有注释，变量名要有意义。
- 遵循编码规范(编码规范参考)，可读性尽可能好。
以下是我根据要求做出的改进：
代码3：
.h文件:
#ifndef CALCULATOR_H
#define CALCULATOR_H
#include<string>
#include<queue>
#include<stdlib.h>
#include<iostream>
using namespace std;
/*
* No description
*/
class Print
{
public:
void pout();
};
class Scan
{
// private section
public:
void ToStringQueue(string input);
};
#endif // CALCULATOR_H
.cpp文件：
#include "calculator.h" // class's header file
#include<string>
#include<queue>
#include<stdlib.h>
#include<iostream>
using namespace std;
queue<string>que; // Define a <string>queue "que";
string s; // Define string s;用空串s存储字符
int t=0; // Define t=0; 来计算位数
// Using global variables here.
void Scan::ToStringQueue(string input)
{
int n=input.length(),m=0,i,j;
for(i=0;i<n;i++) // Error: n has no define;
{
if(t>10) // such as "10,000,000,000"
{
cout<<"Error"<<endl;
break;
}
if(input[i]=='-' || input[i]=='+' || input[i]=='(' || input[i]==')' || input[i]=='/' || input[i]=='*')
{
t=0; // stop the count
que.push(s);
s=""; // 相当于s.clear();
s+=input[i];
que.push(s);
s="";
}
else
{
t++;
s+=input[i];
continue;
}
}
if(t<=10) // ifnormal
{
m=s.size();
if(m!=0)que.push(s);
s="";
}
}
void Print::pout()
{
if(t<=10) // ifnormal
{
while(que.empty()==0) // "que" is not empty
{
cout<<que.front()<<endl;
que.pop();
}
}
}
int main()
{
Scan Sc; // Define a "Scan" object:"Sc".
Print Put; // Define a "Print" object "Put".
string input;
cin>>input;
Sc.ToStringQueue(input);
Put.pout();
return 0;
}
//-100+(98-97)*2
代码3我使用的是定义全局变量的方法，这样在
void Scan::ToStringQueue(string input)里把应该存储到队列que的值push到que内，再在void Print::pout中输出队列que存储的内容。
这样是比较方便，但是全局变量有其短缺和不利的地方。那么就有我改进它的理由。
第三次尝试：
“调用Scan对象的ToStringQueue(string input)，将输入传进去以获取队列。接着将这个队列传入Print对象的方法中，得到输出。”
代码4：
.h文件：
#ifndef CALCULATOR_H
#define CALCULATOR_H
#include<string>
#include<queue>
#include<stdlib.h>
#include<iostream>
using namespace std;
/*
* No description
*/
class Print
{
public:
void pout(queue<string>que);//type:queue<string>
};
class Scan
{
// private section
public:
queue<string>ToStringQueue(string input);//type:queue<string>
};
#endif // CALCULATOR_H
.cpp文件：
#include "calculator.h" // class's header file
#include<string>
#include<queue>
#include<stdlib.h>
#include<iostream>
using namespace std;
int t=0;//Using global variable "t";
queue<string>Scan::ToStringQueue(string input)//type:queue<string>
{
int n=input.length(),m=0,i,j;
queue<string>que;
string s;
for(i=0;i<n;i++) // Error: n has no define;
{
if(t>10) // such as "10,000,000,000"
{
cout<<"Error"<<endl;
break;
}
if(input[i]=='-' || input[i]=='+' || input[i]=='(' || input[i]==')' || input[i]=='/' || input[i]=='*')
{
t=0; // stop the count
que.push(s);
s=""; // s.clear();
s+=input[i];
que.push(s);
s="";
}
else
{
t++;
s+=input[i];
continue;
}
}
if(t<=10) // ifnormal
{
m=s.size();
if(m!=0)que.push(s);
s="";
}
return que;
}
void Print::pout(queue<string>que)
{
if(t<=10) // ifnormal
{
while(que.empty()==0) // "que" is not empty
{
cout<<que.front()<<endl;
que.pop();
}
}
}
int main()
{
queue<string>que;//type:queue<string>
Scan Sc; // Define a "Scan" object:"Sc".
Print Put; // Define a "Print" object "Put".
string input;
cin>>input;
que=Sc.ToStringQueue(input);
Put.pout(que);
return 0;
}
//-100+(98-97)*2
问题探究：
这次与前两次不同的地方在于函数ToStringQueue的类型：
queue<string>。在调用该函数之后，函数返回存储字符串的队列que，再传入到函数pout。
这里我也是栽了几次,首先最开始时我是这样定义的：queue ToSringQueue<string input>，编译器很明显告诉我没有queue这个类型，不让我通过。
在定义存储string类型的时候我使用了queue<string>que;这样的语句，类比于之前定义整形变量的int a;的“类型+变量名”，我想queue<string>会不会就是我想要的答案呢？经过编译器的肯定，证明了我的想法是正确的。
解决方法：
queue<string>Scan::ToStringQueue(string input)
样例测试：
样例：
-100+(98-97)*2
运行结果：
在尝试过程中，测试样例无一是令我满意的，原因我之前也有提到，就是多输出了空行。
那么问题出在哪里呢，我首先怀疑的是输入字符串之后我敲的一个回车，但我很快发现这并不影响。于是就把问题锁定在了队列queue上了，输出有空行，说明在入队的时候出现了情况。
测试：
样例1：-100+(98-97)*2
输出结果：
样例2：
10000000000000-10
输出结果：
样例3：
10+(999-10000000000*2)-10*1
输出结果：
由样例2和样例3可以验证“当输入的数字超过10位（包括小数位）时报错”这一部分是正确的。
经过比较长时间的寻找，我得到了答案：
原来的代码片段：
t=0; // stop the count
que.push(s);
s=""; // s.clear();
s+=input[i];
que.push(s);
s="";
在我停止计数以后(即遇到了除数字之外的其他字符)，队列que进行了
que.push(s)操作，倘若之前s存储了内容(比如100)，这当然是正确的。那么如果s是空串，传入队列que的也是空字符串，也就造成了输出结果存在空行的情况。
解决方法：
t=0; // stop the count
if(s!="")
que.push(s);
s=""; // s.clear();
s+=input[i];
que.push(s);
s="";
测试：
样例1:-100+(98-97)*2
输出结果：
样例2:
100*10-100*(98*97-1)+100
输出结果：
可以看到，成功解决了输出之间夹空行的问题。
最终的代码：
.h.文件：
#ifndef CALCULATOR_H
#define CALCULATOR_H
#include<string>
#include<queue>
#include<stdlib.h>
#include<iostream>
using namespace std;
/*
* No description
*/
class Input
{
public:
string Get();
};
class Print
{
public:
void pout(queue<string>que);//type:queue<string>
};
class Scan
{
// private section
public:
queue<string>ToStringQueue(string input);//type:queue<string>
};
#endif // CALCULATOR_H
.cpp文件
#include "calculator2.h" // class's header file
#include<string>
#include<queue>
#include<stdlib.h>
#include<iostream>
using namespace std;
int t=0;//Using global variable "t";
string Input::Get()
{
string s;
cin>>s;
return s;
}
queue<string>Scan::ToStringQueue(string input)//type:queue<string>
{
int n=input.length(),m=0,i,j;
queue<string>que;
string s;
for(i=0;i<n;i++) // Error: n has no define;
{
if(t>10) // such as "10,000,000,000"
{
cout<<"Error"<<endl;
break;
}
if(input[i]=='-' || input[i]=='+' || input[i]=='(' || input[i]==')' || input[i]=='/' || input[i]=='*')
{
t=0; // stop the count
if(s!="")//avoid null string
que.push(s);
s=""; // s.clear();
s+=input[i];
que.push(s);
s="";
}
else
{
t++;
s+=input[i];
continue;
}
}
que.push(s);
s.clear();
/*if(t<=10) // ifnormal
{
m=s.size();
if(m!=0)que.push(s);
s="";
} */
return que;
}
void Print::pout(queue<string>que)
{
if(t<=10) // ifnormal
{
while(que.empty()==0) // "que" is not empty
{
cout<<que.front()<<endl;
que.pop();
}
}
}
int main()
{
queue<string>que;//type:queue<string>
Scan Sc; // Define a "Scan" object:"Sc".
Print Put; // Define a "Print" object "Put".
Input Ge;// Define a "Input" object "Ge".
string input;
input=Ge.Get();//receieve input.
que=Sc.ToStringQueue(input);//receieve que.
Put.pout(que);//printf que.
return 0;
}
//-100+(98-97)*2
最后我把输入也放在单独的一个class
Input里面了。
- 小结：在我做完这篇随笔的时候，寒假已经接近尾声了，回顾这个不短不长的寒假，我或多或少收获了一些东西。拿这篇随笔来说，我动笔的时候我还不清楚说在这次作业里需要用到queue和string的一些功能，但现在也算是了解了它。当时容易犯下的诸如忘记在main函数里实例化类的对象这样的错误，现在已经很少见了。这次作业让我第一次对项目有所了解和感触，完成它的期间我参阅了一些对我帮助颇深的资料，也有很多贵人相助，感激之余也对下学期的学习生活充满了遐想，自己通过下学期的学习会达到一个怎样的高度？我所面对的挑战和压力(像这次作业)我能不能很好的解决它？像这次的作业，从发布时的一无所知，到翻阅资料初有了解，再到通过不断的实践完成它，我想，这虽不足以回答我的问题，但是就像是一个引领的坐标，使我有了走下去的动力和勇气。
2016/2/22
附：本文参考资料及书籍：
- 1.面向对象程序设计-C++网易公开课 授课人：翁恺
- 2.C++远征封装篇 授课人：james_yuan
- 3.Git教程 廖雪峰的官方网站
- 4.《挑战程序设计竞赛》第二版
感谢 @Sxiaopeng 和 @chs97 朋友的帮助和指导
附：git操作：
- git config --global user.name"Wasdns"
- git config --global user.email"952693358@qq.com"
- git init(可以省略,因为之前对
object-oriented已经执行了该命令)- git add calculator
- git commit -m'calculator'
- git push -u origin master
