指摘・レビュークイズ

コードの誤りを見つけて一言で指摘する、レビューで指摘を受けて適切に返す、打ち間違いを訂正する。現場で毎日起きるやりとりを、エンジニアの一般的な作法にもとづいて出題します。

出題範囲

  • SQL・コマンド・設定ファイルの誤りを見つけて指摘する
  • 指摘の妥当性を判断する(論点のずれ・存在しない問題)
  • レビューで指摘を受けたときの返し方
  • 打ち間違えたコマンドや誤った情報を訂正するときの書き方
  • 伝わる指摘の書き方(事実・理由・提案)

問題一覧

各問題の解説ページです。正答と解説、公式ドキュメントへの参考リンクを載せています。

番号問題形式難易度
code-review-001この SQL を正しく直して書いてください。記述基礎
code-review-002name と age の 2 列を出したい。この SQL を正しく直して書いてください。記述標準
code-review-003削除されていない(deleted_at が NULL の)行を出したい。この SQL を正しく直して書いてください。記述標準
code-review-004この SQL を正しく直して書いてください。記述基礎
code-review-005この SQL を正しく直して書いてください。記述基礎
code-review-006名前が tanaka の行を出したい。この SQL を正しく直して書いてください。記述基礎
code-review-007このSQLへの指摘として正しいものは?4 択基礎
code-review-008部署ごとの人数を出したい。このSQLへの指摘として正しいものは?4 択標準
code-review-009このSQLへの指摘として正しいものは?4 択標準
code-review-010注文とその注文者を並べたい。このSQLへの指摘として正しいものは?4 択標準
code-review-011このSQLへの指摘として正しいものは?4 択基礎
code-review-012注文の無いユーザーを出したい。このSQLへの指摘として正しいものは?4 択応用
code-review-013/var/log に移動したい。このコマンドを正しく直して書いてください(先頭の $ は不要)。記述基礎
code-review-014デプロイスクリプトの 1 行。指摘として正しいものは?4 択応用
code-review-015このコマンドへの指摘として正しいものは?4 択基礎
code-review-016共有の main ブランチへの git push -f は、他の人のコミットを消すおそれがあるため指摘すべき操作である。○×基礎
code-review-017このコミットへの指摘として正しいものは?4 択基礎
code-review-018この YAML への指摘として正しいものは?4 択基礎
code-review-019この JSON への指摘として正しいものは?4 択基礎
code-review-020ログから「connection refused」を探したい。指摘として正しいものは?4 択標準
code-review-021既存の設定を残したまま PATH を 1 行足したい。このコマンドを正しく直して書いてください(先頭の $ は不要)。記述標準
code-review-022このコードへの指摘として正しいものは?4 択基礎
code-review-023「SELECT * は遅いので常に禁止」という指摘は、どんな場面でも正しい。○×基礎
code-review-024「変数名 tmp では中身が分からない」という指摘は、動作に影響しないので出すべきではない。○×標準
code-review-025このSQLへの指摘として妥当なものは?4 択標準
code-review-026次の指摘のうち、マージ前に必ず直すべきものは?4 択基礎
code-review-027好みや軽微な点の指摘には「nit:」などの印を付け、必須の修正と区別できるようにするとよい。○×基礎
code-review-028PR「ログイン画面のボタン文言の誤字修正」への指摘で、論点がずれているのは?4 択標準
code-review-029指摘された箇所に実際は問題が無いと確かめられたら、根拠を添えて「このままにしたい」と返してよい。○×標準
code-review-030仕様は「20 歳以上なら許可」。このコードへの指摘として正しいものは?4 択応用
code-review-031指摘「WHERE の後にスペースが抜けています」への返信として適切なのは?4 択基礎
code-review-032指摘「ここ、もう少し何とかなりませんか」の意図が分からない。返信として適切なのは?4 択標準
code-review-033指摘「このループは map に書き換えてください」。途中で抜ける必要がある。返信として適切なのは?4 択応用
code-review-034指摘を受けて直したら、返信せずに push だけするのが、相手の時間を取らない最もよい対応である。○×基礎
code-review-035指摘「テストが 1 件落ちています」への返信として適切なのは?4 択基礎
code-review-036指摘「この関数は 200 行あって読みにくいです」の受け止め方として適切なのは?4 択基礎
code-review-0375 件の指摘をまとめて直した。返信の仕方として適切なのは?4 択標準
code-review-038レビュー指摘のスレッドを「解決(Resolve)」する扱いとして適切なのは?4 択標準
code-review-039誤った指摘「SQLite では = でなく == を使わないと比較になりません」への返信は?4 択応用
code-review-040指摘「ついでにこのモジュール全体の命名を見直してほしい」への返信は?4 択応用
code-review-041チャットで git reset --soft のつもりが --hard と書いてしまった。訂正として適切なのは?4 択基礎
code-review-042チャットに共有したコマンドの誤りに気づいた。訂正の出し方として適切なのは?4 択基礎
code-review-043誤りのあるメッセージを編集して直せば、すでに読んだ人にも訂正が伝わる。○×基礎
code-review-044手順書の rm -rf /var/app/* は誤りで、正しくは /var/app/tmp/*。訂正として適切なのは?4 択標準
code-review-045共有した git checkout -b feature/loign の綴りが誤りだった。訂正として適切なのは?4 択標準
code-review-046「staging に反映済み」と書いたが、実際に反映したのは dev だった。訂正として適切なのは?4 択基礎
code-review-047README のポート番号が誤っていた(3000 と書いたが正しくは 8080)。直し方として適切なのは?4 択標準
code-review-048訂正では、何が誤りで正しくは何かに加えて、誤った内容に従った人への影響や対処も書くとよい。○×基礎
code-review-049誤ったコマンドを数人が実行済みかもしれない。訂正メッセージの冒頭に書くべきことは?4 択応用
code-review-050削除 SQL の条件列を expires_at でなく created_at と共有してしまった。訂正は?4 択応用

よくある質問

指摘・レビューのクイズは何が出ますか?

SQL・コマンド・設定ファイルの誤りを見つけて指摘する、指摘の妥当性を判断する(論点のずれ・存在しない問題)、レビューで指摘を受けたときの返し方、打ち間違えたコマンドや誤った情報を訂正するときの書き方、伝わる指摘の書き方(事実・理由・提案)を出題範囲にしています。1 問 10 秒前後で解ける基礎的な内容が中心で、暗記ではなくコードを読んで判断する形式が多いです。

何問ありますか?

現在 50 問です。すべてオリジナルで、ビルド時に構造検証を通したものだけを掲載しています。

回答は送信されますか?

問題の番号・正誤・選んだ選択肢の番号・かかった時間を、問題改善のための統計として Google アナリティクスに送信します。記述式(SQL・コマンド・英単語を書く問題)は、書いた答えも送信します。結果画面でエンジニア歴を答えた場合は、その区分も送信します(任意)。