軟件工程第五次作業——第二次結對編程
題目要求
本次作業要求兩個人合作完成,駕駛員和導航員角色自定,鼓勵大家在工作期間角色隨時互換,這裏會布置兩個題目,請各組成員根據自己的愛好任選一題。
題目1:
我們在剛開始上課的時候介紹過一個小學四則運算自動生成程序的例子,請實現它,要求:
能夠自動生成四則運算練習題
可以定制題目數量
用戶可以選擇運算符
用戶設置最大數(如十以內、百以內等)
用戶選擇是否有括號、是否有小數
用戶選擇輸出方式(如輸出到文件、打印機等)
最好能提供圖形用戶界面(根據自己能力選做,以完成上述功能為主)
角色分配
本次作業我作為領航員,呂偉華同學為駕駛員。
測試環境
操作系統:Windows 10
開發語言:C++
測試環境:Qt Creator 5.7.1 windows 64bit
Coding代碼地址:https://coding.net/u/Erick_Lv/p/Qt_Calculator/git/tree/master/software_homework?public=true
測試截圖
開始界面
隨機出題(10以內加減法)
繼續出題,並選擇100以內的乘除法
這裏出現了一個Bug,選擇出混合無符號數題目時不論後面選擇出題數量為多少,都只會出一道題
接下來制造有括號題目
選擇保存數據後
給出“保存成功”的提示
查看保存的文件
代碼分析
界面模塊
Calculator::Calculator(QObject *parent) : QObject(parent)
{
m_nType=0; // 默認加減法
m_nNum=10; // 默認10道題
m_nSize=10; // 默認10以內
}
void Calculator::setAddSub()
{
m_nType=0;
}
void Calculator::setMulDiv()
{
m_nType=1;
}
void Calculator::setFixedNoBrace()
{
m_nType=2;
}
void Calculator::setFixedBrace()
{
m_nType=3;
}
void Calculator::getSize(QString n)
{
bool ok;
m_nSize=n.toInt(&ok,10);
}
void Calculator::getNum(int n)
{
m_nNum=n;
}
void Calculator::CalRes()
{
QString val;
// 所有的要求題目發送給TextBrowser
for(int i=1;i<=m_nNum;++i)
{
if(m_nType==0)
val+=addSub();
else if(m_nType==1)
val+=mulDiv();
else if(m_nType==2)
val=fixedNoBrace();
else
val+=fixedBrace();
if(i%3==0) // 每三個換行
val+=‘\n‘;
else
val+=" "; //
}
// 信號發送
emit sendValue(val);
}
生成加減法函數
QString Calculator::addSub()
{
while(true)
{
QString value;
int a=qrand()%(m_nSize+1);
int b=qrand()%(m_nSize+1);
int c=0;
value+=QString::number(a,10);
// 隨機進行加減,但是不能出現負數或者超過範圍
if(qrand()%2)
{
c=a+b;
if(c>m_nSize)
continue;
value+="+";
}
else
{
c=a-b;
if(c<0)
continue;
value+="-";
}
value+=QString::number(b,10);
value+="=";
value+=QString::number(c,10);
return value;
}
}
生成乘除法函數
QString Calculator::mulDiv()
{
while(true)
{
QString value;
int a=qrand()%(m_nSize+1);
int b=qrand()%(m_nSize+1);
int c=0;
value+=QString::number(a,10);
// 隨機進行乘除,但是不能出現超過範圍或者出現小數
if(qrand()%2)
{
c=a*b;
if(c>m_nSize)
continue;
value+="*";
}
else
{
if(b==0||a<b||a%b!=0)
continue;
value+="/";
c=a/b;
}
value+=QString::number(b,10);
value+="=";
value+=QString::number(c,10);
return value;
}
}
生成混合無括號題目函數
QString Calculator::fixedNoBrace()
{
while(true)
{
QString value;
int a=qrand()%(m_nSize+1);
int b=qrand()%(m_nSize+1);
int c=qrand()%(m_nSize+1);
int d=0;
if(qrand()%2) // 乘除在左側
{
value+=QString::number(c,10);
if(qrand()%2) //乘法
{
value+="*";
d=c*a;
}
else // 除法
{
value+="/";
if(a==0||c%a!=0) // 必須保證整除
continue;
d=c/a;
}
value+=QString::number(a,10);
if(qrand()%2) // 加法
{
d=c*a+b;
if(d>m_nSize) // 溢出
continue;
value+="+";
}
else // 減法
{
d=c*a-b;
if(d<0||d>m_nSize) // 溢出或者小於0
continue;
value+="-";
}
value+=QString::number(b,10);
value+="=";
value+=QString::number(d,10);
}
else // 乘除在右側
{
QString tmp;
tmp+=QString::number(b,10);
if(qrand()%2) // 乘法
{
d=b*c;
if(d>m_nSize)
continue;
tmp+="*";
}
else // 除法
{
if(c==0||b%c!=0)
continue;
tmp+="/";
d=b/c;
}
tmp+=QString::number(c,10);
value+=QString::number(a,10);
if(qrand()%2) // 加法
{
d+=a;
if(d>m_nSize) // 溢出
continue;
value+="+";
}
else
{
d-=a;
if(d<0||d>m_nSize)
continue;
value+="-";
}
value+=tmp;
value+="=";
value+=QString::number(d,10);
}
return value;
}
}
生成混合有括號題目函數
QString Calculator::fixedBrace()
{
while(true)
{
QString value;
int a=qrand()%(m_nSize+1);
int b=qrand()%(m_nSize+1);
int c=qrand()%(m_nSize+1);
int d=0;
if(qrand()%2) // 括號在左側
{
value+="(";
value+=QString::number(a,10);
if(qrand()%2) // 加法
{
if(a+b>m_nSize)
continue;
value+="+";
d=a+b;
}
else
{
if(a<b)
continue;
value+="-";
d=a-b;
}
value+=QString::number(b,10);
value+=")";
if(qrand()%2) //乘法
{
value+="*";
if(d*c>m_nSize)
continue;
d*=c;
}
else // 除法
{
value+="/";
if(c==0||d%c!=0)
continue;
d/=c;
}
value+=QString::number(c,10);
value+="=";
value+=QString::number(d,10);
}
else //括號在右側
{
QString tmp;
tmp+="(";
if(qrand()%2) // 右側加
{
if(a+b>m_nSize)
continue;
d=a+b;
tmp+=QString::number(a,10);
tmp+="+";
tmp+=QString::number(b,10);
}
else // 右側減
{
if(a<b)
continue;
d=a-b;
tmp+=QString::number(a,10);
tmp+="-";
tmp+=QString::number(b,10);
}
tmp+=")";
value+=QString::number(c,10);
if(qrand()%2) // 乘法
{
d*=c;
if(d>m_nSize)
continue;
value+="*";
}
else // 除法
{
if(d==0||c%d!=0)
continue;
d=c/d;
value+="/";
}
tmp+="=";
tmp+=QString::number(d,10);
value+=tmp;
}
return value;
}
}
代碼審查
功能模塊名稱 | 小學四則運算自動生成程序 | ||
審查人 | 陳鑫 | 審查日期 | 2018.4.17 |
代碼名稱 | Calculate | 代碼作者 | 呂偉華 |
文件結構 | |||
重要性 | 審查項 | 結論 | |
頭文件和定義文件的名稱是否合理? | 合理 | ||
頭文件和定義文件的目錄結構是否合理? | 合理 | ||
版權和版本聲明是否完整? | 不完整 | ||
重要 | 頭文件是否使用了 ifndef/define/endif 預處理塊? | 是 | |
頭文件中是否只存放“聲明”而不存放“定義” | 否 | ||
程序的版式 | |||
重要性 | 審查項 | 結論 | |
空行是否得體? | 是 | ||
代碼行內的空格是否得體? | 是 | ||
長行拆分是否得體? | 是 | ||
“{” 和 “}” 是否各占一行並且對齊於同一列? | 是 | ||
重要 | 一行代碼是否只做一件事?如只定義一個變量,只寫一條語句。 | 否 | |
重要 | If、for、while、do等語句自占一行,不論執行語句多少都要加 “{}”。 | 否 | |
重要 | 在定義變量(或參數)時,是否將修飾符 * 和 & 緊靠變量名?註釋是否清晰並且必要? | 否 | |
重要 | 註釋是否有錯誤或者可能導致誤解? | 否 | |
重要 | 類結構的public, protected, private順序是否在所有的程序中保持一致? | 是 | |
命名規則 | |||
重要性 | 審查項 | 結論 | |
重要 | 命名規則是否與所采用的操作系統或開發工具的風格保持一致? | 否 | |
標識符是否直觀且可以拼讀? | 否 | ||
標識符的長度應當符合“min-length && max-information”原則? | 否 | ||
重要 | 程序中是否出現相同的局部變量和全部變量? | 是 | |
類名、函數名、變量和參數、常量的書寫格式是否遵循一定的規則? | 是 | ||
靜態變量、全局變量、類的成員變量是否加前綴? | 否 | ||
表達式與基本語句 | |||
重要性 | 審查項 | 結論 | |
重要 | 如果代碼行中的運算符比較多,是否已經用括號清楚地確定表達式的操作順序? | 是 | |
是否編寫太復雜或者多用途的復合表達式? | 是 | ||
重要 | 是否將復合表達式與“真正的數學表達式”混淆? | 否 | |
重要 | 是否用隱含錯誤的方式寫if語句? 例如 | ||
(1)將布爾變量直接與TRUE、FALSE或者1、0進行比較。 | 否 | ||
(2)將浮點變量用“==”或“!=”與任何數字比較。 | 否 | ||
(3)將指針變量用“==”或“!=”與NULL比較。 | 是 | ||
如果循環體內存在邏輯判斷,並且循環次數很大,是否已經將邏輯判斷移到循環體的外面? | 否 | ||
重要 | Case語句的結尾是否忘了加break? | 否 | |
重要 | 是否忘記寫switch的default分支? | 否 | |
重要 | 使用goto 語句時是否留下隱患? 例如跳過了某些對象的構造、變量的初始化、重要的計算等。 | 否 | |
常量 | |||
重要性 | 審查項 | 結論 | |
是否使用含義直觀的常量來表示那些將在程序中多次出現的數字或字符串? | 否 | ||
在C++ 程序中,是否用const常量取代宏常量? | 否 | ||
重要 | 如果某一常量與其它常量密切相關,是否在定義中包含了這種關系? | 是 | |
是否誤解了類中的const數據成員?因為const數據成員只在某個對象生存期內是常量,而對於整個類而言卻是可變的。 | 否 | ||
函數設計 | |||
重要性 | 審查項 | 結論 | |
參數的書寫是否完整?不要貪圖省事只寫參數的類型而省略參數名字。 | 完整 | ||
參數命名、順序是否合理? | 合理 | ||
參數的個數是否太多? | 否 | ||
是否使用類型和數目不確定的參數? | 否 | ||
是否省略了函數返回值的類型? | 否 | ||
函數名字與返回值類型在語義上是否沖突? | 否 | ||
重要 | 是否將正常值和錯誤標誌混在一起返回?正常值應當用輸出參數獲得,而錯誤標誌用return語句返回。 | 是 | |
重要 | 在函數體的“入口處”,是否用assert對參數的有效性進行檢查? | 否 | |
重要 | 使用濫用了assert? 例如混淆非法情況與錯誤情況,後者是必然存在的並且是一定要作出處理的。 | 否 | |
重要 | return語句是否返回指向“棧內存”的“指針”或者“引用”? | 是 | |
是否使用const提高函數的健壯性?const可以強制保護函數的參數、返回值,甚至函數的定義體。“Use const whenever you need” | 否 | ||
內存管理 | |||
重要性 | 審查項 | 結論 | |
重要 | 用malloc或new申請內存之後,是否立即檢查指針值是否為NULL?(防止使用指針值為NULL的內存) | 否 | |
重要 | 是否忘記為數組和動態內存賦初值?(防止將未被初始化的內存作為右值使用) | 否 | |
重要 | 數組或指針的下標是否越界? | 否 | |
重要 | 動態內存的申請與釋放是否配對?(防止內存泄漏) | 是 | |
重要 | 是否有效地處理了“內存耗盡”問題? | 否 | |
重要 | 是否修改“指向常量的指針”的內容? | 否 | |
重要 | 是否出現野指針?例如(1)指針變量沒有被初始化;(2)用free或delete釋放了內存之後,忘記將指針設置為NULL。 | 否 | |
重要 | 是否將malloc/free 和 new/delete 混淆使用? | 否 | |
重要 | malloc語句是否正確無誤?例如字節數是否正確?類型轉換是否正 確? | 是 | |
重要 | 在創建與釋放動態對象數組時,new/delete的語句是否正確無誤? | 是 | |
C++ 函數的高級特性 | |||
重要性 | 審查項 | 結論 | |
重載函數是否有二義性? | 否 | ||
重要 | 是否混淆了成員函數的重載、覆蓋與隱藏? | 否 | |
運算符的重載是否符合制定的編程規範? | 是 | ||
是否濫用內聯函數?例如函數體內的代碼比較長,函數體內出現循環。 | 否 | ||
重要 | 是否用內聯函數取代了宏代碼? | 否 | |
類的構造函數、析構函數和賦值函數 | |||
重要性 | 審查項 | 結論 | |
重要 | 是否違背編程規範而讓C++ 編譯器自動為類產生四個缺省的函數: | ||
(1)缺省的無參數構造函數; | 否 | ||
(2)缺省的拷貝構造函數; | 否 | ||
(3)缺省的析構函數; | 否 | ||
(4)缺省的賦值函數。 | 否 | ||
重要 | 構造函數中是否遺漏了某些初始化工作? | 否 | |
重要 | 是否正確地使用構造函數的初始化表? | 是 | |
重要 | 析構函數中是否遺漏了某些清除工作? | 否 | |
是否錯寫、錯用了拷貝構造函數和賦值函數? | 否 | ||
重要 | 賦值函數一般分四個步驟: | ||
(1)檢查自賦值; | 否 | ||
(2)釋放原有內存資源; | 否 | ||
(3)分配新的內存資源,並復制內容; | 是 | ||
(4)返回 *this。是否遺漏了重要步驟? | 否 | ||
重要 | 是否正確地編寫了派生類的構造函數、析構函數、賦值函數? | 是 | |
註意事項: | |||
(1)派生類不可能繼承基類的構造函數、析構函數、賦值函數。 | 是 | ||
(2)派生類的構造函數應在其初始化表裏調用基類的構造函數。 | 否 | ||
(3)基類與派生類的析構函數應該為虛(即加virtual關鍵字)。 | 否 | ||
(4)在編寫派生類的賦值函數時,註意不要忘記對基類的數據成員重新賦值 | 否 | ||
類的高級特性 | |||
重要性 | 審查項 | 結論 | |
重要 | 是否違背了繼承和組合的規則? | ||
(1)若在邏輯上B是A的“一種”,並且A的所有功能和屬性對B而言都有意義,則允許B繼承A的功能和屬性。 | 是 | ||
(2)若在邏輯上A是B的“一部分”(a part of),則不允許B從A派生,而是要用A和其它東西組合出B。 | 否 | ||
其它常見問題 | |||
重要性 | 審查項 | 結論 | |
重要 | 數據類型問題: | ||
(1)變量的數據類型有錯誤嗎? | 否 | ||
(2)存在不同數據類型的賦值嗎? | 否 | ||
(3)存在不同數據類型的比較嗎? | 否 | ||
重要 | 變量值問題: | ||
(1)變量的初始化或缺省值有錯誤嗎? | 沒有 | ||
(2)變量發生上溢或下溢嗎? | 不 | ||
(3)變量的精度夠嗎? | 夠 | ||
重要 | 邏輯判斷問題: | ||
(1)由於精度原因導致比較無效嗎? | 否 | ||
(2)表達式中的優先級有誤嗎? | 否 | ||
(3)邏輯判斷結果顛倒嗎? | 否 | ||
重要 | 循環問題: | ||
(1)循環終止條件不正確嗎? | 否 | ||
(2)無法正常終止(死循環)嗎? | 否 | ||
(3)錯誤地修改循環變量嗎? | 否 | ||
(4)存在誤差累積嗎? | 否 | ||
重要 | 錯誤處理問題: | ||
(1)忘記進行錯誤處理嗎? | 沒忘記 | ||
(2)錯誤處理程序塊一直沒有機會被運行? | 否 | ||
(3)錯誤處理程序塊本身就有毛病嗎?如報告的錯誤與實際錯誤不一致,處理方式不正確等等。 | 無 | ||
(4)錯誤處理程序塊是“馬後炮”嗎?如在被它被調用之前軟件已經出錯。 | 不是 | ||
重要 | 文件I/O問題: | ||
(1)對不存在的或者錯誤的文件進行操作嗎? | 否 | ||
(2)文件以不正確的方式打開嗎? | 否 | ||
(3)文件結束判斷不正確嗎? | 否 | ||
(4)沒有正確地關閉文件嗎? | 否 | ||
總結
呂偉華同學是一個學習能力很強的人,為了做界面在短時間內自學了Qt,而且做出來的程序方便易用,界面足夠友好以至於可以很輕松的上手。在這裏還是有幾條建議要提,希望能完善程序。
1.如果可以的話,可以將生成的題目分為有答案版和無答案版分別輸出出來,這樣可以方便用於打印,符合家長或是老師的進一步需求。
2.每次生成的題目希望都有一個提示或是空行,這樣可以分清楚每批題目分別屬於那一次,看上去也不會很亂給人不想看下去的感覺。
3.Linux系統和Windows下系統的編譯器環境等可能不兼容,最好能做成平日裏用的多的系統的版本,畢竟一般人不會用命令行。
通過這次的結對編程作業,我意識到程序員是一個需要不斷地學習的一個職業,畢竟就像老師上課講的那樣,程序員是要隨著客戶的要求去做事,相當於讓你去做什麽你就去做什麽,不會那就自己去學,而不是讓客戶去適應你。。。。。。。。現在網絡的發達也讓信息變得更流通,有什麽困難及時檢索或者去問同學。駕駛員和領航員最好水平都一樣,這樣的話領航員在一邊就能看懂代碼中是否有毛病,而不是理解就需要半天。而且一個人的體力和精力是有限的,可能駕駛員連續敲一定時間的代碼後就難受了,這時候就應該角色互換了,可以保持最大的效率。
(ノ=Д=)ノ┻━┻ 另外一定改掉一邊敲代碼一邊喝酒的習慣
下面就是結對編程的具體圖片
軟件工程第五次作業——第二次結對編程