代码审查 —— 第二次个人编程作业

这个作业属于哪个课程	软件工程 (福州大学至诚学院 - 计算机工程系)
这个作业要求在哪里	第二次个人编程作业
这个作业的目标	和同学之间互相提出issue和改进代码
作业正文	我的GitHub仓库地址
其他参考文献	GitHub帮助

我的commit提交信息

“让我看看你的代码！”

issues

1. 吴秋悦的GitHub仓库地址

   • BUG：一开始就输入看看 “字符串”，没有输出字符串。已解决
     

   • 建议：读取字符串一般不加地址符，数组名表示首地址。已解决
     

2. 朱传彬的GitHub仓库地址

   • 不符合题目需求：代码单纯为解决样例1而写，无法解决样例2。
     

3. 林航的GitHub仓库地址

   • 建议：定义数组char *str[11]，可改成char str[11][3]，就不会有一堆warning。已解决
     

4. 蔡泽华的GitHub仓库地址

   • 建议：在输出后加个换行符。已解决
     
   • BUG：一开始就输入看看 “字符串”，没有输出字符串。
     

5. 连捷的GitHub仓库地址

   • 不符合题目需求：只能在开始时定义一个变量，之后不能再定义变量，无法完成样例2，需要改进。已解决
     

6. 林连坤的GitHub仓库地址

   • 建议：更改输入方式，而不是靠main()来一个一个传参。已解决
     
   • 不符合题目需求: 代码输入“十”会报错，只能输入“一零”。建议改进。
     

7. 张体杰的GitHub仓库地址

   • 不符合题目需求: 输入 看看 “字符串” 时，输出没有去掉引号。已解决
     

8. 冯志成的GitHub仓库地址

   • 不符合题目需求: 当输入看看 “字符串”时，却输出了零，而不是字符串。
     

9. 程昱的GitHub仓库地址

   • BUG：程序无法运行，需要再完善完善。
     

10. 陶佳杰的GitHub仓库地址

    • BUG: 代码只是为了迎合样例1的结果，printf("你好，世界");、printf("冻死我了");???需要改进。
      

11. 李思锦的GitHub仓库地址

    • BUG：一开始就输入看看 “字符串”，没有输出字符串。
      
    • 输出不符合题目需求：比如应该是“三”、“十”，却输出“零三”、“一十”。
      
    • 建议：字符串输入时，不需要加&取址符。
      

pull requests

1. 帮助吴秋悦同学解决定义变量后，不能重复 看看 “字符串” 的问题。

   因为flag2再次使用时，没有初始化。

   

2. 帮助陶佳杰同学解决在判断语句中正确输出“字符串”的问题。
   
   

3. 帮助朱传彬同学删除了szz函数，用char num[][]数组替代，优化了一些代码。
   

   

“容我再改一改！”

字符串的引号只考虑英文引号，没考虑到中文引号。

解决方案

@朱传彬 增加了对中文双引号的判断，if (var.substr(0,2) == "“" && var.substr(var.size()-2,2) == "”")

功能请求

解决方案

@张体杰 使用map容器，增加对变量名的判断，若非字符串且不存在，则输出“无此变量”并跳过后面的步骤。

map<string, int> mp; 

if (!mp.count(var)) {
    if (var[0] == '"' && var[var.size()-1] == '"') 
        cout << var.substr(1,var.size()-2) << endl;
    else if (var.substr(0,2) == "“" && var.substr(var.size()-2,2) == "”") 
        cout << var.substr(2,var.size()-4) << endl; 
    else cout << "无此变量：" << var << endl; 
	continue; // 跳过计算步骤 
}

“让我再看一看！”

1. 吴秋悦
   
   
   改进：

   • 去除开头的定义变量。
   • 增加了flag2 = 0

2. 张体杰
   
   改进：

   • 去除了输出的字符串的引号。

3. 连捷
   改进：

   • 增加了对样例2的处理。

4. 林连坤
   
   改进：

   • 改进了输入方式，而不是靠main()来一个一个传参。

5. 林航
   
   改进：

   • 定义数组char *str[11]改成char str[11][3]