AIでも壊れない堅牢コードへ鍛え直す！
コードレビュー道場 #3：非機能要件

公開日：2025年12月13日

最終更新日：2026年02月22日

このページでは、AIが見落としやすい非機能要件（性能・信頼性・保守性など）の観点で、実際のコード断片を読むコードレビュー形式の4択クイズを体験します。「動く」だけでは守れない、本番で踏ん張る品質を鍛えるのが狙いです。

4択クイズ（10問・全問解説付き）で腕試し → 解説でなぜ落ちるのか／なぜ運用がつらいのかを言語化。「読む→解く→わかる」で、AI生成コードをプロ品質へ引き上げる現場の勘所を身につけていきましょう。

今すぐクイズに挑戦
（10問・約3分・解説つき）

プリンシプルオブプログラミング
AI時代を生き抜く「101の設計原則」

#3 非機能要件：落ちるコードを鍛え直せ！目次

クイズに挑戦する(10問)

参考文献・出典
このページの著者
シェア

コードレビュー道場 #3～落ちるコードを非機能要件で鍛え直せ！～

1. 次のコードは、リトライ戦略が単純で、ネットワークが一時的に不安定な場合と恒久的に落ちている場合を区別していません。非機能要件（信頼性）の観点から、より良い改善はどれでしょうか？


async function fetchWithRetry(url) {
  for (let i = 0; i < 2; i++) {
    try {
      const res = await fetch(url);
      if (!res.ok) throw new Error('bad status');
      return res;
    } catch (e) {
      await new Promise(r => setTimeout(r, 100));
    }
  }
  throw new Error('failed after 2 retries');
}

A. リトライ回数を大幅に増やして、諦めるまで長く粘るようにする
B. エラーの種類（タイムアウト・一時的エラー・4xxなど）に応じてリトライ可否と回数・間隔を変える
C. エラー時には常に同じステータスコードを返すようにする
D. リトライのたびにログを出しておき、後から原因を分析しやすくする

2. 次のコードは、グローバル変数に状態を持っており、並行リクエストが増えると値の整合性が崩れやすくなります。信頼性とスケーラビリティの観点から、もっとも適切な改善はどれでしょうか？


let currentCount = 0;

async function handleRequest(req, res) {
  currentCount++;
  // いろいろな処理...
  res.end('count: ' + currentCount);
}

A. currentCount を const に変更し、書き換えを禁止する
B. currentCount の更新前後でログを出力し、値の変化を追えるようにする
C. 共有ミュータブル状態を避け、リクエストごとに独立した値で処理するか、カウンタを専用ストアやミドルウェアの責務として切り出す
D. res.end のメッセージ形式を JSON に変更する

3. 次のコードは、機能フラグを毎回リモートから取得しており、外部サービス障害時にアプリ全体が影響を受けやすくなっています。信頼性と性能の観点から、もっとも適切な改善はどれでしょうか？


async function getFeatureFlags() {
  const res = await fetch('https://flags.example.com/config.json');
  return res.json();
}

A. フラグの取得に失敗した場合は、常にすべての機能を無効にする
B. 結果を常に localStorage に保存しておき、次回起動時にだけそこから読む
C. 一定間隔または起動時にフラグを取得してキャッシュし、通常の処理ではキャッシュ済み値を参照するようにする
D. 外部サービスのレスポンス時間をログに出し、遅い場合はアラートを上げる

4. 次のコードは、多数のURLに対して同時に fetch を投げており、クライアントや相手サーバーの負荷スパイクを引き起こす可能性があります。非機能要件（スケーラビリティ・安定性）の観点から、もっとも適切な改善はどれでしょうか？


async function fetchAll(urls) {
  const promises = urls.map(url => fetch(url));
  return Promise.all(promises);
}

A. すべてのリクエストを直列に処理し、1件ずつ順番に送る
B. リクエスト前後で必ずログを出し、どのURLが遅いかを可視化する
C. 同時実行数に上限を設け、一定数ずつ並列に処理するキューやプールの仕組みを導入する
D. 失敗したURLだけを後で再試行するバッチを用意する

5. 次のコードは、ダッシュボード表示のたびに重い集計処理を実行しています。性能とスケーラビリティの観点から、もっとも有効な改善はどれでしょうか？


function renderDashboard(userId) {
  const stats = calculateAllStats(userId); // 重い処理
  return buildHtml(stats);
}

A. calculateAllStats の中でログを出し、どこが遅いか把握できるようにする
B. buildHtml の処理を最適化し、テンプレートエンジンを高速なものに変える
C. 一定期間ごとに集計結果をバッチ処理で更新してキャッシュし、画面表示時にはキャッシュ済みデータを参照する
D. ダッシュボードの表示頻度を制限する

6. 次のコードは、ユーザー入力に対するバリデーションがなく、異常値で例外や異常動作を起こしやすくなっています。信頼性の観点から、もっとも重要な改善はどれでしょうか？


function calculate(pageSize) {
  return 1000 / pageSize;
}

A. 計算結果を Math.floor で丸める
B. pageSize が 0 や負の値、極端に大きい値などの場合はエラーとして扱い、安全な経路で呼び出し元に伝える
C. 1000 という値を定数に切り出す
D. 計算結果が Infinity や NaN のときだけログに出す

7. 次のサーバーコードは、リクエストのたびに同期的なファイル読み込みを行っています。負荷が高まると応答が極端に遅くなる危険があります。スループットと応答性の観点から、もっとも有効な改善はどれでしょうか？


const fs = require('fs');

function handleRequest(req, res) {
  const content = fs.readFileSync('./template.html', 'utf8');
  res.end(content);
}

A. template.html のファイルサイズをできるだけ小さくする
B. readFileSync を非同期版の readFile に置き換えるだけにする
C. アプリ起動時などにテンプレートを読み込んでキャッシュし、リクエスト時はメモリから返すようにしつつ、I/Oは非同期で扱う
D. 応答の直前に処理時間をログ出力しておく

8. 次のコードは、重い計算をクリックイベント内で同期的に実行しており、UIを長時間フリーズさせる可能性があります。ユーザビリティと応答性の観点から、もっとも適切な改善はどれでしょうか？


function handleClick() {
  let sum = 0;
  for (let i = 0; i < 1_000_000_000; i++) {
    sum += i;
  }
  alert(sum);
}

A. ループ回数を減らし、計算自体を軽くする
B. 計算の前後でローディングスピナーを表示し、ユーザーに待ち時間を意識してもらう
C. Web Worker やサーバー側に重い計算を逃がし、メインスレッドでは結果が届いたときだけUIを更新するようにする
D. alert ではなく画面上の要素に結果を表示する

9. 次のコードでは、ログを永続化せずメモリ上の配列に貯め続けています。長時間稼働時の安定性を高めるために、もっとも重要な改善はどれでしょうか？


const logs = [];

function logEvent(event) {
  logs.push({
    at: new Date().toISOString(),
    event
  });
}

A. ログのタイムスタンプを UNIX時間にする
B. logs をオブジェクトではなく Map に変更する
C. 保持件数に上限を設け、超えた分を外部に吐き出すか破棄する仕組みを入れて、メモリ使用量を制御する
D. logEvent 内でログ件数を監視し、一定件数ごとに警告ログを出す

10. 次のコードは、典型的なN+1クエリ問題を抱えています。性能とスケーラビリティの観点から、もっともインパクトの大きい改善はどれでしょうか？


async function listOrdersWithUser(db) {
  const orders = await db.query('SELECT * FROM orders');
  for (const order of orders) {
    const rows = await db.query(
      'SELECT * FROM users WHERE id = ?',
      [order.user_id]
    );
    order.user = rows[0] ?? null;
  }
  return orders;
}