代碼審查 —— 第二次我的編程做業

這個做業屬於哪一個課程	<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]