2020年ソフトウェアエンジニアリングペアワーク

1.関連する操作要件

ブログ情報 瀋陽航空宇宙大学コンピュータ工学部2020ソフトウェア工学
仕事の要件 https://edu.cnblogs.com/campus/sau/Computer1701-1705/homework/10616
コースの目的 「高品質」ソフトウェアの開発プロセスに精通している
宿題の目標 コード仕様とペアワイズ相互レビューに精通

第二に、レビューコードの詳細なレビューフォーム

査読者 李建洪 レビュー日 2020.4.13
シリアル番号 重要性 レビューアイテム おわりに
1 重要な ヘッダファイルと定義ファイルの名前は妥当ですか? はい
2 重要ではない ヘッダーファイルと定義ファイルのカタログファイルが妥当かどうか はい
重要ではない バージョンとバージョン宣言が完了しているかどうか
4 重要な ヘッダーファイルはifndef / define / endif前処理ブロックを使用していますか?
5 重要ではない ヘッダーファイルに「宣言」のみが格納され、「定義」は格納されていないか
6 重要な 空行はまともですか? はい
7 重要ではない コード行のスペースは適切ですか? はい
8 重要な 長い行の分割は適切ですか? はい
9 重要な 「{」と「}」は1つの行を占め、同じ列に配置されますか? はい
10 重要ではない 1行のコードで実行できることは1つだけですか?変数が1つだけ定義されている場合は、ステートメントを1つだけ記述します。 はい
11 重要な for、while、doおよび他のステートメントが1行にある場合、実行されるステートメントの数に関係なく、「{}」を追加する必要があります。 はい
12 重要な 変数(またはパラメーター)を定義するとき、修飾子*および&は変数名の横にありますか?コメントは明確で必要ですか? はい
13 重要な コメントに誤りがあるか、誤解を招く可能性がありますか?
14 重要な クラス構造のパブリック、保護、プライベートの順序はすべてのプログラムで一貫していますか?
15 重要な 命名規則は、使用されているオペレーティングシステムまたは開発ツールのスタイルと一致していますか?
16 重要な 識別子は直感的でスペル可能ですか? はい
17 重要ではない 識別子の長さは「最小長&&最大情報」の原則に準拠する必要がありますか? はい
18 重要な 同じローカル変数とすべての変数がプログラムに表示されますか?
19 重要な クラス名、関数名、変数とパラメーター、定数は特定の規則に従っていますか?
20 重要ではない 静的変数、グローバル変数、およびクラスメンバー変数の接頭辞はありますか?
21 重要な コードの行に多くの演算子がある場合、括弧を使用して式の演算の順序を明確に決定していますか?
22 重要な 複雑すぎる、または多目的な複合式を記述していますか? いいえ
23 重要な 複合式は「実際の数式」と混同されますか?
24 重要な ifステートメントは暗黙のエラーで記述されていますか?例
25 重要な ブール変数をTRUE、FALSE、または1、0と直接比較します。
26日 重要な 将浮点变量用“==”或“!=”与任何数字比较。
27 重要 将指针变量用“==”或“!=”与NULL比较。
28 重要 如果循环体内存在逻辑判断,并且循环次数很大,是否已经将逻辑判
29 不重要 判断移到循环体的外面?
30 重要 Case语句的结尾是否忘了加break?
31 重要 是否忘记写switch的default分支?
32 重要 使用goto 语句时是否留下隐患? 例如跳过了某些对象的构造、变量的初始化、重要的计算等。
33 不重要 是否使用含义直观的常量来表示那些将在程序中多次出现的数字或字符串?
34 不重要 在C++ 程序中,是否用const常量取代宏常量?
35 重要 如果某一常量与其它常量密切相关,是否在定义中包含了这种关系?
36 重要 是否误解了类中的const数据成员?因为const数据成员只在某个对象
37 重要 生存期内是常量,而对于整个类而言却是可变的。
38 不重要 参数的书写是否完整?不要贪图省事只写参数的类型而省略参数名字。
39 不重要 参数命名、顺序是否合理?
40 不重要 参数的个数是否太多?
41 重要 是否使用类型和数目不确定的参数?
42 重要 是否省略了函数返回值的类型?
43 不重要 函数名字与返回值类型在语义上是否冲突?
44 重要 是否将正常值和错误标志混在一起返回?正常值应当用输出参数获得,而错误标志用return语句返回。
45 重要 在函数体的“入口处”,是否用assert对参数的有效性进行检查?
46 重要 使用滥用了assert? 例如混淆非法情况与错误情况,后者是必然存在的并且是一定要作出处理的。
47 重要 return语句是否返回指向“栈内存”的“指针”或者“引用”?
48 不重要 是否使用const提高函数的健壮性?const可以强制保护函数的参数、返回值,甚至函数的定义体。“Use const whenever you need”
49 重要 用malloc或new申请内存之后,是否立即检查指针值是否为NULL?(防止使用指针值为NULL的内存)
50 重要 是否忘记为数组和动态内存赋初值?(防止将未被初始化的内存作为右值使用)
51 重要 数组或指针的下标是否越界?
52 重要 动态内存的申请与释放是否配对?(防止内存泄漏)
53 重要 是否有效地处理了“内存耗尽”问题?
54 重要 是否修改“指向常量的指针”的内容?
55 重要 是否出现野指针?例如(1)指针变量没有被初始化;(2)用free或delete释放了内存之后,忘记将指针设置为NULL。
56 重要 变量的数据类型有错误吗?
57 重要 存在不同数据类型的赋值吗?
58 重要 存在不同数据类型的比较吗?
59 重要 变量的初始化或缺省值有错误吗?
60 重要 变量发生上溢或下溢吗?
61 重要 变量的精度够吗?
62 重要 由于精度原因导致比较无效吗?
63 重要 表达式中的优先级有误吗?
64 重要 逻辑判断结果颠倒吗?
65 重要 循环终止条件不正确吗?
66 重要 无法正常终止(死循环)吗?
67 重要 错误地修改循环变量吗?
68 不重要 存在误差累积吗?
69 重要 忘记进行错误处理吗?
70 重要 错误处理程序块一直没有机会被运行?
71 重要 错误处理程序块本身就有毛病吗?如报告的错误与实际错误不一致,处理方式不正确等等。
72 不重要 错误处理程序块是“马后炮”吗?如在被它被调用之前软件已经出错。
73 不重要 对不存在的或者错误的文件进行操作吗?
74 不重要 文件以不正确的方式打开吗?
75 重要 文件结束判断不正确吗?
76 不重要 没有正确地关闭文件吗?

三、代码优缺点的总结

优点:1.变量包括函数的命名都很直观,有特殊的含义,便于后期的阅读;例如参数lchild,rchild的命名,代表含义:左孩子,右孩子;函数GreateBiTree(), Enqueue(),代表含义:创建二叉树,插入结点操作。
2.结构化清晰,整体布局很好。对于程序中需要的用到的函数全部进行了单独的编写,在主函数里面只是进行简单的调用,和一些简单的操作。
3.没有多余的变量存在以及2句或2句以上的语句处于同一行,读起来很清晰。
缺点:没有一些简单的注释,导致在某一点读起来还是不是很理解,希望以后可以改正。

四、相关的审查代码链接

https://github.com/lyt1234567/soft/blob/master/树.cpp

おすすめ

転載: www.cnblogs.com/LIJIH/p/12684697.html