试卷生成程序优缺点分析

Posted on 2018-09-26 19:42  derzom  阅读(452)  评论(0编辑  收藏  举报

 ·程序说明

队友的个人项目完成了项目的所有功能要求。

他使用了如下函数来实现需求:

void kuohao(int B[] ,int numA);         //预定公式在括号中的位置

string Pschooltext(int A[],int numA);     //生成一条小学的试题

string Jschooltext(int A[],int numA);     //生成一条初中的试题

string Sschool(int A[],int numA);        //生成一条高中的试题

int searchtext(string a[],string strname);  //打开文件夹读取所有的历史试题并返回题目数量

string ltime();                       //获取当前时间

 

 

 ·优缺点分析

细看代码后发现了有些不错的处理:

1、首先是生成括号的函数,由于题目限定了操作数个数,所以生成括号函数的操作,他才用了一个特别的int型数组来记录该操作数左右方括号的情况,具体是:

int B[6];//记录括号的位置 ,1单(,2单),3((,4)),0代表没有括号

括号位置确定代码:

void kuohao(int B[] ,int numA){
	int k,n;
	if(numA<=2) k=0;
	else k=numA/2;
	n=rand()%(k+1);
	while(n>0)
	{
		int a,b;
		a=rand()%numA+1;
		b=rand()%(numA-a+1)+a;
		if(a==1 && b== numA)
		continue;
		else if(a!=b){
			if(B[a]==0 && B[b]==0){
				B[a]=1;
		        B[b]=2;
		        n--;
			}
			else if(B[a]==1 && B[b]==0){
				B[a]=3;
				B[b]=2;
				n--;
			}
			else if(B[a]==0 && B[b]==2){
				B[a]=1;
				B[b]=4;
				n--;
			}
		}
		
	}

}

对应下标就是第几个操作数,下标1就是第一个操作数,然后在函数中用随机函数模拟,并分好不同的情况,最后再生成运算式中加进去。

 

2、他在处理如何打开一个文件夹中的所有txt文件进行查重的时候,每生成一套卷只需要进行一次,因为第一次将所有的题目都存在一个string数组中,然后每生成一道题只需要和数组对比一次,之后没有重复的题目就放进数组中(数组设立最大上限),所以打开操作只需一次。

文件读取函数:

int searchtext(string a[],string strname){
	int i=0;
	string textname="F:\\输出文件\\"+strname+"\\*.txt";
    struct _finddata_t fileinfo;
    string in_name;
    long handle;
    if ((handle = _findfirst(textname.c_str(), &fileinfo)) == -1L)
    {
        cout << "没有找到匹配文件!" << endl;
        return 0;
    }
    else
    {
        in_name = "F:\\输出文件\\"+strname + "\\" + fileinfo.name;
        ifstream fin(in_name.c_str());
            if (!fin)
            {    
             cerr << "open file error" << endl;
             exit(-1);
            }
            string str;
            while (getline(fin, str))
             {
              a[i]=str;
              i++;
             }
        while (!(_findnext(handle, &fileinfo)))
        {
        	in_name ="F:\\输出文件\\"+strname + "\\" + fileinfo.name;
        	ifstream fin1(in_name.c_str());
            if (!fin1)
            {    
             cerr << "open file error" << endl;
             exit(-1);
            }
            string str;
            while (getline(fin1, str))
             {
              a[i]=str;
              i++;
             }
        }
        _findclose(handle);
    }
    for(int j=0;j<i;j++)
    {
    	if(a[j]!="")
    	a[j]=a[j].substr(3);
	}
    return i;
}

 获取历史试题:

string a[10000];
int sum = searchtext(a,strname);

sum返回的是数组中存放题目的个数.

 

同时,我认为他的代码有以下几点不足之处:

1、他的代码有些冗长,其实在生成3中难度题目中的很多代码都可以写成一个函数进行重复调用

2、输出格式,不够美观,题号后面的“.”应该加个空格和题目分开

3、他的代码有个的BUG,就是假如随机生成的操作数如果在1到9之间,生成出来就是一个乱码的数:

产生Bug的代码:

char s1,s2;
if(A[i]/10!=0)
s1=A[i]/10+'0';
s2=A[i]%10+'0';
string S1,S2;
S1=s1; S2=s2; number+=S1+S2;

Bug产生原因:

由于没有考虑char型没有初始的时候是随机一个乱码,当随机数是19中,s1就会没有被赋值,就会出现错误(该部分代码是将生成的随机数转化为string型)。

Bug截图:

后来经他本人debug,就把这个BUG很快地修正了;修正后代码:

char s1,s2;
string S1,S2;
s2=A[i]%10+'0';
S2=s2;
if(A[i]/10!=0)
{
  s1=A[i]/10+'0';
  S1=s1;
  number+=S1+S2;
}
else{
  number+=S2;	
}

 

·程序运行结果

控制台界面:

输出文件: