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

这个作业属于哪个课程 软件工程 (福州大学至诚学院 - 计算机工程系)
这个作业要求在哪里 第二次个人编程作业
这个作业的目标 和同学之间互相提出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]
posted @ 2020-02-10 21:38  oeong  阅读(309)  评论(0编辑  收藏