結對作業一
阿新 • • 發佈:2018-04-06
序列 管理 void inf lag case語句 include net 不可
一、題目要求。
(1). 首先在同學中找一個同伴,範圍不限,可以在1~5班中隨意組合,建議盡量不要找同組的成員,女同學盡量找男同學結對,但是不做強制要求;
(2). 從以往個人完成的項目中選擇一個作品,例如:以往的數據結構課程設計或者其它具有比較完整功能的小系統,代碼至少要大於100行;
(3). 將代碼上傳至個人GitHub或Coding.net系統中,並將代碼地址交給對方;
(4). 對同伴的作品進行代碼復審,並參照C/C++代碼審查表和 Java代碼審查表 這兩篇博文的內容自行設計代碼審查表並填寫內容;
(5). 將對夥伴審查的結果以表格的形式寫到自己的博客作業裏,博客中應該附有夥伴作業的GitHub或Coding.net的代碼地址;
(6). 對同伴的代碼寫一篇500字以上的評論,介紹同伴的優缺點。
二、審查表。
功能模塊名稱 | 實現並演示堆排序的排序算法和排序過程 | ||
審查人 | 賀延俏 | 審查日期 | 2018/4/5 |
代碼名稱 | 堆排序 | 代碼作者 | 張慶民 |
文件結構 | |||
重要性 | 審查項 | 結論 | |
頭文件和定義文件的名稱是否合理? | 合理 | ||
頭文件和定義文件的目錄結構是否合理? | 合理 | ||
版權和版本聲明是否完整? | 無 | ||
重要 | 頭文件是否使用了 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)沒有正確地關閉文件嗎? | 否 | ||
三、代碼。
#include<stdio.h>
#include<iostream>
#include<stdlib.h>
using namespace std;
typedef int sqlist;
int flag1; //用於區別2功能與3功能是否需要輸出過程
int flag2=0; //用於判斷有無輸入以保證2 3功能的成功進行
int fog; //用於實現逐步演示步驟
sqlist R2[1000]; //用於備份
void sift(sqlist R[],int low,int high) //調整堆列
{
int i=low,j=2*i;
sqlist temp=R[i];
while(j<=high)
{
if(j<high && R[j]<R[j+1])
j++;
if(temp<R[j])
{
R[i]=R[j];
i=j;
j=2*i;
}
else break;
}
R[i]=temp;
}
void heapsort(sqlist R[],int n) //排序的主體
{
int i,j;
sqlist temp;
for(i=n/2;i>=1;i--) //初始堆
{
sift(R,i,n);
}
if(flag1)
{
printf("初始堆:");
for(j=1;j<=n;j++)
printf("%d ",R[j]);
putchar(10);putchar(10);
printf("觀看下一步輸入 1 ;直接看結果輸入 0\n");
cin>>flag1;
}
for(i=n;i>=2;i--)
{
temp=R[1];
R[1]=R[i];
R[i]=temp;
sift(R,1,i-1);
if(flag1 && i!=2)
{
printf("第%d步:",(n-i+1));
for(j=1;j<=n;j++)
printf("%d ",R[j]);
putchar(10);putchar(10);
printf("觀看下一步輸入 1 ;直接看結果輸入 0\n");
cin>>flag1;
}
}
}
void inflow(sqlist R[],int n) //輸入
{
memset(R,0,sizeof(sqlist));
memset(R2,0,sizeof(sqlist));
int i;
printf("請輸入您的整數序列,整數與整數間用空格隔開\n");
for(i=1;i<=n;i++)
{
cin>>R[i];
R2[i]=R[i];
}
}
void boundar() //界面
{
printf("┏━━━━━━━━━━━━┓\n");
printf("┃ 堆排序演示程序 v1.0 ┃\n");
printf("┣━━━━━━━━━━━━┫\n");
printf("┃ 1.輸入整數序列 ┃\n");
printf("┣━━━━━━━━━━━━┫\n");
printf("┃ 2.查看結果 ┃\n");
printf("┣━━━━━━━━━━━━┫\n");
printf("┃ 3.觀看排序過程 ┃\n");
printf("┣━━━━━━━━━━━━┫\n");
printf("┃ 4.結束程序 ┃\n");
printf("┗━━━━━━━━━━━━┛\n");
}
int choose() //輸入想實現的功能序號
{
int l;
boundar();
printf("請輸入想使用的功能序號:\n 例如:1\n");
cin>>l;
return l;
}
int main()
{
int n,l,i;
sqlist R[1000];
while(1)
{
l=choose();
switch(l)
{
case 1:
{
system("cls");
boundar();
flag2=1;
printf("請輸入您想排序的整數個數:\n");
cin>>n;
inflow(R,n);
system("cls");
break;
}
case 2:
{
system("cls");
printf("\n\n結果為:");
if(!flag2)
{
printf("抱歉您還未輸入整數序列!\n");
break;
}
else
{
flag1=0;
heapsort(R,n);
printf(" ");
for(i=1;i<=n;i++)
printf("%d ",R[i]);
printf("\n\n");
}
break;
}
case 3:
{
system("cls");
boundar();
if(!flag2)
{
printf("抱歉您還未輸入整數序列!\n");
break;
}
else
{
for(i=1;i<=n;i++)
{
R[i]=R2[i];
}
flag1=1;
heapsort(R,n);
printf("結果為:");
for(i=1;i<=n;i++)
printf("%d ",R[i]);
putchar(10);putchar(10);
}
break;
}
case 4:
{
system("cls");
boundar();
printf("感謝您的使用!單擊任何鍵以退出。\n");
return 0;
}
default:
{
system("cls");
printf("輸入錯誤。\n");
break;
}
}
}
return 0;
}
代碼鏈接點這裏~
四、評價及總結。
拿到張慶民同學的代碼之後,在不知道他的代碼是實現什麽功能的情況下,我第一時間是進行了測試,首先,他的界面如下:
先不說代碼寫的如何,單單是用戶體驗就很好,界面演示共分為四小部分:(1)輸入整數序列 (2)查看結果 (3)觀看排序過程 (4)結束程序。雖然這只是一次課程設計,但是可以看出他的用心。
在測試了代碼沒問題之後,我開始仔細審查他的代碼,分析各部分代碼的功能。由於他的每部分代碼都有註釋,所以看起來比較容易,這個習慣很好。整體程序分為四大模塊:調整堆列,堆列排序,輸入功能序號,界面演示。其中,調整堆列是前提,其他三個模塊的實現都要依靠調整堆列來實現。其他3個模塊相互獨立。
張慶民同學的算法大致是通過將無序序列建成一個堆,得到關鍵字最大的記錄;輸出堆頂的最大值後,使剩余的n-1個元素重又建成一個堆,則可得到n個元素的次小值;重復執行,得到一個有序序列。輸出堆頂元素之後,以堆中最後一個元素替代之;然後將根結點值與左、右子樹的根結點值進行比較,並與其中小者進行交換;重復上述操作,直至葉子結點,將得到新的堆,稱這個從堆頂至葉子的調整過程為“篩選”。如此反復,便可得到最後排好序的整數序列。
張慶民同學的代碼整體給人的感覺就是簡潔明了,我們知道代碼的可讀性很重要,在結對編程中尤為重要,如果一個人寫的代碼只有他自己能看懂,甚至過一段時間自己都忘了,想改都無從下手,這樣的代碼是不能稱之為好代碼的。所以註釋很重要,結構也很重要。
這次代碼審查,讓我學習到了別人身上的優點,更多的是發現了自身的不足,沒有對比就沒有傷害,看到人家的代碼很規範,寫註釋的習慣不是一天兩天就能養成的,說明別人比自己花費了更多的時間和精力。自己以後也會努力向其他人學習,養成好的編程習慣。
結對作業一