『プリンシプル オブ プログラミング』を10問で学べるクイズサイト

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

公開日:
最終更新日:

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

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

今すぐクイズに挑戦
(10問・約3分・解説つき)
プリンシプルオブプログラミング
AI時代を生き抜く「101の設計原則」

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

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

  1. プログラミング クイズ一覧へ
  2. コードレビュー道場 目次へ
  3. 次のクイズへ(#1 DRY/KISS:AIコードをリファクタせよ!)
  4. 前のクイズへ(#2 凝集度/結合度:疎結合へリファクタ!)
  5. 『プリンシプル オブ プログラミング』101の原理・原則総まとめ

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

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

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

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

2. 次のコードは、すべてのジョブが毎分ぴったりのタイミングで重い処理を走らせる設計になっています。他のサービスも同じタイミングで動くと、一時的な負荷スパイクを招く可能性があります。非機能要件(安定性・スループット)の観点から、もっともよい改善はどれでしょうか?

// 毎分0秒に実行される
setInterval(runJob, 60 * 1000);

function runJob() {
  // 重い集計処理
}

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

const logs = [];

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

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

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

5. 次のコードは、典型的な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;
}

6. 次のコードは、ユーザーごとのステータス更新を1件ずつAPIに投げています。対象件数が増えると処理時間と負荷が急増します。性能とスケーラビリティの観点から、もっともインパクトの大きい改善はどれでしょうか?

async function updateStatuses(users) {
  for (const user of users) {
    await fetch('/api/status', {
      method: 'POST',
      body: JSON.stringify({ id: user.id, status: 'active' })
    });
  }
}

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

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

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

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');
}

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

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

10. 次のコードは、重い処理のたびに詳細なログを出しており、高負荷時にログI/O自体がボトルネックになる恐れがあります。性能と運用性のバランスをとるために、もっとも適切な改善はどれでしょうか?

function processItem(item) {
  console.log('START PROCESS', item);
  doHeavyWork(item);
  console.log('END PROCESS', item);
}

落ちるコードを非機能要件で鍛え直せ!
採点結果

正答率:0%

-

-

参考文献・出典

  • プリンシプル オブ プログラミング ~3年目までに身につけたい一生役立つ101の原理原則~
  • 上田 勲(著)/秀和システム/第1版14刷/2025年/ISBN978-4-7980-4614-3

※本ページは学習支援を目的とした要約です。実務適用時は原典もご参照ください。

このページの著者

もちもちみかん(システムエンジニア)

社内SEとしてグループ企業向けの業務アプリを要件定義〜運用まで一気通貫で担当しています。

経験:Webアプリ/業務システム

得意:PHP・JavaScript・MySQL・CSS

個人実績:フォーム生成基盤クイズ学習プラットフォーム

詳しいプロフィールはこちら!  もちもちみかんのプロフィール

もちもちみかん0系くん
TOPへ

もちもちみかん.comとは


このサイトでは、コーディングがめんどうくさい人向けのお助けツールとして、フォームやCSSをノーコードで生成できる、
 もちもちみかん.forms
 もちもちみかん.css1
 もちもちみかん.css2
と言ったジェネレーターを用意してます。

また、このサイトを通じて、「もちもちみかん」のかわいさを普及したいとかんがえてます!