结队编程之——阅读分析队友的代码（C++自动生成数学试卷）

队友用c++写的的，其中使用到的函数名称如下

 

bool enter(string account,string password) //登录验证 
int makefile(int number,string account,string topic[60][30])//创建文件
int primaryschool(string account) //生成小学题目 
int juniorschool(string account) //生成初中题目 
int seniorschool(string account) //生成高中题目 
int main()

　　

功能实现都没有问题，条例也很清晰，尤其是创建文件的部分让我学到了很多

 

int makefile(int number,string account,string topic[60][30])
{
    time_t t = time(0);
    char file[30];
    strftime(file, sizeof(file), "%Y-%m-%d-%H-%M-%S.txt", localtime(&t));    //以固定格式的时间为文件名
    ofstream openfile((account + '/' + file).c_str());   //打开文件并在文件末尾写
    for(int i=0;i 
 

因为其实我对C++的创建文件部分不太熟，看别人的代码，总有收获

 

不过还是有一些缺陷，主要是在函数的调用方面

体会一下看到这段代码的感受

bool enter(string account,string password)    //登录验证 
{
	if(account=="张三1"&&password=="123")
	{
		return true;
	}
	if(account=="张三2"&&password=="123")
	{
		return true;
	}
	if(account=="张三3"&&password=="123")
	{
		return true;
	}
	if(account=="李四1"&&password=="123")
	{
		return true;
	}
	if(account=="李四2"&&password=="123")
	{
		return true;
	}
	if(account=="李四3"&&password=="123")
	{
		return true;
	}
	if(account=="王五1"&&password=="123")
	{
		return true;
	}
	if(account=="王五2"&&password=="123")
	{
		return true;
	}
	if(account=="王五3"&&password=="123")
	{
		return true;
	}
	
	return false;
}

看到这的时候我心里面就“咯噔”一下，因为这个源程序足足有400多行，但是显然，如果只是生成试卷的话是不可能用这么多的

我理解的情况。生成试卷应该是封装成一个函数的，然后再写“小学”，“初中”，“高中”三个函数来调用

但是队友可能比较赶时间，

int primaryschool(string account) //生成小学题目 
int juniorschool(string account) //生成初中题目 
int seniorschool(string account) //生成高中题目 

这三个函数都有大量的重复~

而且，里面也用了很多if，变量的命名也让我看不太懂

int primaryschool(string account)                //生成小学题目 
{
	cout<<"准备生成小学数学题目，请输入生成题目数量(10-30)"<>number;
	while(number<10||number>30)
	{
		cout<<"请输入有效输入范围：10-30"<>number;
	}
	string topic[60][30];
	for(int i=0;i<60;i++)
	{
		for(int j=0;j<30;j++)
		{
			topic[i][j]="";
		}
	}
	srand((unsigned)time(NULL));
	int flag=0;            //对是否有括号进行标记 
	int flag2=0;           //对括号是否只括了一个操作数进行标记 
	for(int i=0;i>topic[i][0];
		topic[i][1]=". ";
		for(int j=0;j0)       //左括号 
			{
				topic[i][z]="(";
				z++;
				flag=1;
			}
			int operand_data=(rand()%(100-1+1))+1;
			stringstream ss;
			ss<>topic[i][z];
			if(flag==1)
			{
				flag2++;
			}
			z++;
			int r_bracket=(rand()%(1-0+1));
			if(flag==1&&r_bracket==1&&flag2>1)     //右括号 
			{
				topic[i][z]=")";
				z++;
				flag=0;
				flag2=0;
			}
			if(j==operand-1&&flag==1)
			{
				topic[i][z]=")";
				z++;
				flag=0;
				flag2=0;
			}
			if(j 
 

 这个代码看着其实挺累人的

所以这让我对面向对象有了更深的理解

每个方法只做一件事，就像老师说的，每个方法最多20行，只需要调用就行，修改的时候非常方便，可扩展性强

但是这有一个缺陷，就是调来调去容易晕，也好解决，两条

1.命名规范，让人一看命名就知道这个方法是干什么的

2.事先想清楚方法怎么调用，脑袋里面有个大致的框架

 

我的代码也很难看，所以还得努力呀