结对编程-队友代码分析

Posted zxpbp

tags:

篇首语:本文由小常识网(cha138.com)小编为大家整理,主要介绍了结对编程-队友代码分析相关的知识,希望对你有一定的参考价值。

 按照老师的要求,对队友的代码进行解析,先说缺点在看优点,改正缺点学习优点,一起进步!  

1.首先打开代码的时候,关于账号密码的存储,将账号和密码直接存储在了一个数组中。按照“用户名 密码"的格式,这在之后的登陆检测中确实可以方便检测,但是个人感觉将账号密码直接存储在一起会造成一些安全问题。

string allUser[]={"张三1 123","张三2 123","张三3 123","李四1 123","李四2 123","李四3 123","王五1 123","王五2 123","王五3 123"};
// 用户账号密码信息

2.在用户登录的部分,默认了只拥有9个用户,不考虑增加用户的问题,用户的类型选择只是用了用户编号除三的选择方式,在增加用户后可能会导致问题,拓展性不强。

string UserType[]={"小学","初中","高中"}; //用户类型信息

    while(getline(cin,Account)){ //用户登录处理 
        for(user=0;user<9;user++){
            if(Account==allUser[user]){
                cout<<"当前选择为"<<UserType[user/3]<<"出题"<<endl;
                flag_Account=1;
                break;
            }
            else if(user==8){
                cout<<"请输入正确的用户名、密码"<<endl;
            }
        }
        if(flag_Account==1)
        break;
    }
    usertype=user/3; //0-2号用户为小学,3-5号用户为初中,6-8号用户为高中

3.关于高耦合的问题,没有将生成不同阶段题目的代码分离出main函数,只在main函数中使用了一个switch语句控制,导致main函数非常庞大,不利于修改和扩展。

4.在生成特殊符号的三个函数部分,观察到了许多相似的代码,或许可以专门分离出一个函数解决冗余问题。

关于函数的优点

1.在生成特殊符号的部分,使用了二维数组的方式来解决,将每个数字的座位二维数组的一行,用二维数组内的0,1值来判断是否会出现特殊符号,与我的暴力判断输出不同,这样的方式更加的精巧,虽然也会出现一些问题,但是能更加清晰的了解问题进行修改。

int Creat_list_bra(int oper_num,int list_bra[][2]){ //产生括号标志数组,数组每一行代表一个操作数,第一个元素为1表示操作数左边有左括号,第二个元素为1表示操作数右边有有括号 
    while(1){
        int sum_fbra=0,sum_bbra=0;
        for(int i=0;i<oper_num;i++){
            if(i!=oper_num-1){
                list_bra[i][0]=rand()%2;
                if(list_bra[i][0]==1){
                    list_bra[i][1]=0;
                    sum_fbra++;
                }
                else if(list_bra[i][0]==0 && i!=0 && sum_fbra>sum_bbra){
                    list_bra[i][1]=rand()%2;
                    if(list_bra[i][1]==1)
                    sum_bbra++;
                }
            }
            else{
                if(list_bra[0][0]!=1)
                {
                    list_bra[i][1]=rand()%2;
                    if(list_bra[i][1]==1)
                    sum_bbra++;
                }
            }
        }
        if(sum_fbra==sum_bbra){  //当左右括号数量相等时结果合法 
            return 1;
        } 
    }
}

 

2.关于变量和函数的命名方式十分简洁易懂,规范使用,不会对读代码的人造成误解。

3.代码内的缩进十分规范,在一些部分的分割也让人看得非常舒服,代码的注释部分也解释的十分到位,没有多余的部分。

以上是关于结对编程-队友代码分析的主要内容,如果未能解决你的问题,请参考以下文章

结对编程-队友代码分析

结对编程队友代码分析

结对编程----分析队友代码

结对编程-队友代码分析

结对编程队友代码分析

结对编程之队友代码分析