代码审查 —— 第二次我的编程做业

这个做业属于哪一个课程	<a href="https://edu.cnblogs.com/campus/fzzcxy/SE/" target="_blank">软件工程 (福州大学至诚学院 - 计算机工程系)</a>
这个做业要求在哪里	<a href="https://edu.cnblogs.com/campus/fzzcxy/SE/homework/10289?tdsourcetag=s_pctim_aiomsg" target="_blank">第二次我的编程做业</a>
这个做业的目标	和同窗之间互相提出issue和改进代码
做业正文	个人GitHub仓库地址
其余参考文献	GitHub帮助

个人commit提交信息

“让我看看你的代码！”

issues

1. 吴秋悦的GitHub仓库地址

   • BUG：一开始就输入看看 “字符串”，没有输出字符串。<span style="color:red">已解决</span>

   • 建议：读取字符串通常不加地址符，数组名表示首地址。<span style="color:red">已解决</span>

2. 朱传彬的GitHub仓库地址

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

3. 林航的GitHub仓库地址

   • 建议：定义数组char *str[11]，可改为char str[11][3]，就不会有一堆warning。<span style="color:red">已解决</span>

4. 蔡泽华的GitHub仓库地址

   • 建议：在输出后加个换行符。<span style="color:red">已解决</span>
   • BUG：一开始就输入看看 “字符串”，没有输出字符串。

5. 连捷的GitHub仓库地址

   • 不符合题目需求：只能在开始时定义一个变量，以后不能再定义变量，没法完成样例2，须要改进。<span style="color:red">已解决</span>

6. 林连坤的GitHub仓库地址

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

7. 张体杰的GitHub仓库地址

   • 不符合题目需求: 输入 看看 “字符串” 时，输出没有去掉引号。<span style="color:red">已解决</span>

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容器，增长对变量名的判断，若<span style="color:red">非字符串且不存在</span>，则输出“无此变量”并跳事后面的步骤。

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]