Webフロントエンドのコードレビューメモ2

また、以下の記事の続きを書こうと思います。

ビジネスロジック

アプリケーションのビジネスロジックに関することについて書きます。

• 複数パラメータによる条件分岐がある場合、不都合なパターンが存在しないか
  • 3つ4つと増えると、都合が悪いパターンが稀に発生するため
• アクセスして良い画面（認証）であるか、参照して良いデータ（認可）であるか
  • 許可していないのにアクセスできたり、許可しているのにアクセスできなかったり
• 対象ドメインを適切に表す型であるか、嘘をついていないか
  • 無駄な型だと、余計なガード処理を入れる必要がでる
  • 嘘をついている場合、保守する際にバグを生む可能性がある
• 必要十分なデータのみ扱っているか（過剰ではないか）
  • APIのデータフェッチや、オブジェクトへ保持されるデータなど
• 過去・現在・未来と考慮できているか
  • 設計をそのまま未来へスライドしても不都合がないか
    • 今は良いけど、近い未来に運用上困らないか。トイルは生まれないか。
  • 現在の設計により過去の設計に不都合がないか
    • データを整えたり、移行手順を用意できているか
• 登場人物のコンテキストに配慮できているか
  • 誰が使うものか、どういう状況で使うものか

レビュー

レビュー時に考慮していることについて書きます。

• レビューコメントに、質問意図は添えれているか
  • レビュイへの負担を減らせているか
• imo,ask,nits,must,fyi と色分けできているか
  • 質問なのか共有なのか、読み方が変わるため
• 具体的なものを書いて、わかりやすくコメントできているか
  • コードスニペットか、パッチを当てるPRを出すなど
  • Storybook 等で、指摘したい箇所を画像・動画を残せているか
  • 時間軸が大事であれば、タイムラインをリスト形式で書けているか
  • 具体的なドメインデータ（登場人物名や商品名など）で会話できているか
• 何十行も長文ではなく、端的にコメントできているか
  • 読む時間や読む負荷を下げれているか
• 良い点や学びがある点など、ポジティブなフィードバックを出せているか
  • レビュイへの敬意を忘れていないか
  • 提案頂いたことへ感謝すること
• レビュー中に確認したことについては、コメントを残せているか
  • 動作確認の報告や、関係者へ確認したSlackリンクなど

終わりに

また思いつくことがあれば、書こうと思います！

-

読者になる

|

シェアする

|

ブログを応援する

この記事がよかったら、お布施という形で応援してもらえるとうれしいです。

おふせぼたん

関連する記事

タグ「フロントエンド」の記事

写経テストコードを全部消した

以下で書いた通り、プロダクトコードを写経したテストコードを削除しました。 "こぶりー" ( https://kobliy.vercel.app/ ) という個人ブログを読むアプリのコードです。 https://silverbirder.gi

2026年06月08日

AI

フロントエンド

テスト

念の為に、手動確認をしよう

最近のお悩みは、Webのソフトウェア開発におけるテストコードが爆増したことにより、 テスト成功による過度な安心感 によって手動確認するのが減っているのかもと思ったりしています。 例えば、Webのフォーム画面に小さな改修があったとして、その修

2026年06月03日

AI

フロントエンド

モバイルアプリからPWAアプリへ切り替え

以下で書いた個人ブログを読むアプリ（個人ブログライブラリ、略して "こぶりー" ）をモバイルアプリで開発していました。 https://silverbirder.github.io/blog/contents/20260419/ 審査関連で

2026年05月11日

フロントエンド

← ブログ一覧へ