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

AIコードをDRY/KISSで徹底リファクタ!
コードレビュー道場 #1:DRY/KISS

公開日:
最終更新日:

このページでは、AIが生成しがちな冗長・重複・やりすぎなコードを題材に、DRY / KISSを武器にしたコードレビュー形式の4択クイズで実戦力を鍛えます。理屈を読むだけでなく、「どこがまずい?どう直す?」を自分の判断として身につけるのが狙いです。

4択クイズ(10問・全問解説付き)で腕試し → 解説でなぜその修正が効くのかを腹落ち。「読む→解く→わかる」で、AIコードをスリムに整えるリファクタの勘所を鍛えていきましょう!

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

#1 DRY/KISS:AIコードをリファクタせよ! 目次

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

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

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

コードレビュー道場 #1~DRY/KISSで太りすぎた関数を救え!~

1. 次の関数は、バリデーション・保存・画面遷移までを1つに詰め込んだ例です。KISSの観点から、最も優先して行うべき改善はどれでしょうか?
function submitForm(data) {
  // 入力チェック
  if (!data.name) {
    alert('名前は必須です');
    return;
  }
  if (!data.email) {
    alert('メールアドレスは必須です');
    return;
  }

  // 保存用データ整形
  const payload = {
    name: data.name,
    email: data.email,
    createdAt: new Date().toISOString()
  };

  // 保存API
  fetch('/api/contacts', {
    method: 'POST',
    body: JSON.stringify(payload)
  }).then(() => {
    alert('送信しました');
    location.href = '/thanks';
  }).catch(() => {
    alert('送信に失敗しました');
  });
}

2. 次の2つの関数は、データ取得後のエラーハンドリングがコピペになっています。DRYの観点から、どう改善するのがよいでしょうか?
function loadUsers() {
  return fetch('/api/users')
    .then(res => {
      if (!res.ok) {
        throw new Error('ユーザーの取得に失敗しました');
      }
      return res.json();
    });
}

function loadProjects() {
  return fetch('/api/projects')
    .then(res => {
      if (!res.ok) {
        throw new Error('プロジェクトの取得に失敗しました');
      }
      return res.json();
    });
}

3. 次の2つの関数は、ほとんど同じメール送信処理を行っています。DRYの観点から、最も良い改善はどれでしょうか?
function sendWelcomeMail(user) {
  const subject = 'ようこそ!';
  const body = `こんにちは ${user.name} さん`;
  mailer.send(user.email, subject, body);
}

function sendByeMail(user) {
  const subject = 'ご利用ありがとうございました';
  const body = `さようなら ${user.name} さん`;
  mailer.send(user.email, subject, body);
}

4. 次の if 文は、KISSの観点から少し読みにくい書き方になっています。最も改善したいポイントはどれでしょうか?
function canShip(order) {
  if (order && order.paid && !order.canceled && !order.refunded && order.items && order.items.length > 0) {
    return true;
  }
  return false;
}

5. 次のコードでは、バリデーションメッセージがコピペされています。DRYの観点から、どこを改善するとよいでしょうか?
function validateName(name) {
  if (!name) {
    return '名前は必須です';
  }
  if (name.length > 20) {
    return '名前は必須です';
  }
  return '';
}

6. 次のコードは、「分かりやすさ」よりも「1行で書くこと」を優先してしまっています。KISSの観点から、どの書き方がより望ましいでしょうか?
const isAdult = (age) => age != null && age >= 18 ? true : false;

7. 次のコードでは、同じ「合計金額のフォーマット処理」が重複しています。DRYの観点から、最も適切な改善はどれでしょうか?
function showCartTotal(cart) {
  const total = cart.items.reduce((sum, item) => sum + item.price * item.quantity, 0);
  const label = total.toLocaleString() + '円(税込)';
  document.getElementById('cart-total').textContent = label;
}

function showOrderTotal(order) {
  const total = order.items.reduce((sum, item) => sum + item.price * item.quantity, 0);
  const label = total.toLocaleString() + '円(税込)';
  document.getElementById('order-total').textContent = label;
}

8. 次の関数は、複数のモードに応じて処理を切り替えています。KISSの観点から、最も問題になりやすい点はどれでしょうか?
function handleAction(mode, data) {
  if (mode === 'create') {
    // 作成処理
    // ...
  } else if (mode === 'update') {
    // 更新処理
    // ...
  } else if (mode === 'delete') {
    // 削除処理
    // ...
  } else if (mode === 'copy') {
    // コピー処理
    // ...
  } else if (mode === 'archive') {
    // アーカイブ処理
    // ...
  } else {
    throw new Error('Unknown mode');
  }
}

9. 次のコードは、API呼び出しのオプションを毎回ベタ書きしています。DRYの観点から、最も効果的な改善はどれでしょうか?
function fetchUsers() {
  return fetch('/api/users', {
    method: 'GET',
    headers: { 'Content-Type': 'application/json' },
    credentials: 'include'
  });
}

function fetchOrders() {
  return fetch('/api/orders', {
    method: 'GET',
    headers: { 'Content-Type': 'application/json' },
    credentials: 'include'
  });
}

10. 次の関数は、条件分岐が深くネストしています。KISSの観点から、どのような改善が最も効果的でしょうか?
function process(user) {
  if (user) {
    if (user.isActive) {
      if (!user.isBanned) {
        // メイン処理
        doSomething(user);
      }
    }
  }
}

DRY/KISSで太りすぎた関数を救え!
採点結果

正答率: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にコードを書かせる時代にこそ大切な設計・責務・レビューの視点を、実務と検証の目線で届けます。


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