『プリンシプル オブ プログラミング』を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. 次のコードは、グローバル変数に状態を持っており、並行リクエストが増えると値の整合性が崩れやすくなります。信頼性とスケーラビリティの観点から、もっとも適切な改善はどれでしょうか?

let currentCount = 0;

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

2. 次のコードは、設定を参照するたびに localStorage を読み、JSON.parse を実行しています。高頻度で呼ばれると性能に影響します。非機能要件(性能・保守性)の観点から、もっともよい改善はどれでしょうか?

function getConfig() {
  const raw = localStorage.getItem('config');
  return raw ? JSON.parse(raw) : {};
}

3. 次のコードは、送金処理にトランザクションを使っておらず、途中でエラーが起きたときに残高が不整合になる危険があります。信頼性(一貫性)の観点から、もっとも重要な改善はどれでしょうか?

async function transfer(db, fromId, toId, amount) {
  await db.execute(
    'UPDATE accounts SET balance = balance - ? WHERE id = ?',
    [amount, fromId]
  );
  await db.execute(
    'UPDATE accounts SET balance = balance + ? WHERE id = ?',
    [amount, toId]
  );
}

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

5. 次のコードは、エラーが発生した場合にローディング表示が消えず、ユーザーが操作不能な状態になる可能性があります。ユーザビリティと信頼性の観点から、もっとも重要な改善はどれでしょうか?

async function loadData() {
  showLoading();
  const res = await fetch('/api/data');
  const data = await res.json();
  hideLoading();
  render(data);
}

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

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

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

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

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

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

9. 次のコードは、ユーザーごとのステータス更新を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' })
    });
  }
}

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

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

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

正答率:0%

-

-

参考文献・出典

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

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

このサイトの運営者

もちもちみかん(社内SE)

運営者は、グループ企業向けの業務アプリを要件定義から運用まで担当している社内SEです。PHP・JavaScript・MySQL・CSSを使った実務経験をもとに、一次情報や実際の運用で得た気づきを整理しています。

AIにコードを書かせて終わりではなく、どこで迷ったか・どこを人がレビューすべきかまで含めて、 やさしく噛みくだいてまとめています。

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

得意:PHP・JavaScript・MySQL・CSS

制作・運用中:フォーム生成基盤・クイズ学習プラットフォーム・htmx逆引きレシピ 等

AI時代のエンジニアタイプ診断:CSPF/とろとろみかん

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

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

もちもちみかん.comとは


もちもちみかん.com は、AI時代に必要なプログラミングの基礎・設計・改善の考え方を、やさしく学べるサイトです。


『プリンシプル オブ プログラミング』の原則まとめ、用語集、クイズ、htmx逆引きレシピを通じて、「分かったつもり」で終わらず、実際に使える判断軸まで身につくようにしています。


フォームジェネレーターなどの便利ツールも用意しつつ、AIにコードを書かせる時代にこそ大切な設計・責務・レビューの視点を、実務と検証の目線で届けます。


むずかしい内容でも親しみやすく学べるように、「もちもちみかん」らしいやわらかさも大切にしています!