Haramaki0326
commented
2 years ago 参考
前提 ソースコードを変更する目的に問題はないか ソースコードの変更項目に問題はないか コミットの粒度は問題ないか コミットメッセージは問題ないか ソフトウェアの動作 画面の表示に問題はないか 文言に問題はないか 満たすべき環境すべてで問題なく動作するか 設計方法 変更項目と実装内容に齟齬はないか ロジックは合っているか よりよい設計はないか オブジェクト指向設計の原則は守られているか 実装方法 不要になったソースコードはないか DRYに書けているか 将来必要になるかもしれないものを実装していないか 導入したライブラリに問題はないか 使えそうなライブラリはないか 標準のAPIにある機能を再発明していないか 使用できる既存のモジュールはないか APIのリクエストやレスポンスに問題はないか リトライ処理は行えているか。問題ないか データの型は適切か データの不整合は起こらないか。トランザクションは張れているか ログは適切に記録されているか デバッグ用のコードは残っていないか よりよい書き方はないか 命名に問題ないか 名前から副作用の有無は想定できるか 名前と実装は合っているか 可読性に問題はないか ガード節を用いているか アクセス修飾子は適切か スコープは適切か スタイルに問題はないか ひとつのメソッドの処理は多すぎないか ネストは深くないか 運用 ユーザに影響はないか、対応はとれているか メンバーに影響はないか、対応はとれているか システムに影響はないか、対応はとれているか ドキュメントの更新は十分か 法的、あるいは利用規約上の問題はないか テスト テストケースに問題はないか テストの実装方法に問題はないか テストはパスしているか セキュリティ上の問題はないか パフォーマンス パフォーマンス上の問題はないか N+1問題は発生しないか 計算量は問題ないか、あるいは減らせないか クエリは最適化できないか
まずは大事そうな要素を列挙してみる