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

公開日：2025年12月13日

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

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

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

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

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

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

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

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

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

1. 次のコードは、ユーザー名ラベルを作る処理です。DRYの観点から、最も問題がある部分はどこでしょうか？

function createUserLabel(user) {
  return user.lastName + ' ' + user.firstName + '（' + user.id + '）';
}

function createOrderLabel(order) {
  const user = order.user;
  return user.lastName + ' ' + user.firstName + '（' + user.id + '）';
}

A. lastName と firstName をスペースでつなげていること
B. createUserLabel と createOrderLabel で同じラベル生成ロジックをコピペしていること
C. ユーザーIDをカッコで囲んでいること
D. 変数 user を再代入していないこと

2. 次のコードでは、バリデーションメッセージがコピペされています。DRYの観点から、どこを改善するとよいでしょうか？

function validateName(name) {
  if (!name) {
    return '名前は必須です';
  }
  if (name.length > 20) {
    return '名前は必須です';
  }
  return '';
}

A. 20 の制限をなくす
B. エラーメッセージをそれぞれ別の文言にする
C. 同じメッセージを返している部分をまとめ、条件だけ分岐させる
D. validateName 関数を削除する

3. 次の if 文は、KISSの観点から少し読みにくい書き方になっています。最も改善したいポイントはどれでしょうか？

function canShip(order) {
  if (order && order.paid && !order.canceled && !order.refunded && order.items && order.items.length > 0) {
    return true;
  }
  return false;
}

A. order を引数に渡していること
B. 1つの if 条件に多くのチェックを詰め込みすぎていること
C. !order.canceled と !order.refunded を使っていること
D. return true / false を使っていること

4. 次の関数は、画面ごとにほぼ同じローディング表示ロジックをコピペしています。DRYの観点から、最も適切な改善はどれでしょうか？

function showUserLoading() {
  const el = document.getElementById('user-section');
  el.innerHTML = '読み込み中...';
}

function showOrderLoading() {
  const el = document.getElementById('order-section');
  el.innerHTML = '読み込み中...';
}

A. innerHTML を textContent に変える
B. 'loading' クラス名を変える
C. ローディング表示用の共通関数を定義し、IDだけ引数で渡す
D. ローディング表示をやめる

5. 次の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();
    });
}

A. エラーメッセージを英語にする
B. エラーメッセージ部分を共通の handleError 関数に切り出し、APIごとの差分（どのリソースか）だけ渡す
C. res.ok のチェックを削除する
D. fetch を XMLHttpRequest に変える

6. 次の関数は、バリデーション・保存・画面遷移までを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('送信に失敗しました');
  });
}

A. createdAt を削除する
B. fetch を同期処理にする
C. バリデーション・保存ロジック・画面遷移を役割ごとに分割し、小さな関数にする
D. location.href の代わりに console.log を使う

7. 次の2つの関数は、とても似たバリデーション処理を行っています。DRYの観点から、最も適切な改善はどれでしょうか？

function isValidUserName(name) {
  return typeof name === 'string' && name.length >= 3 && name.length <= 20;
}

function isValidProjectName(name) {
  return typeof name === 'string' && name.length >= 3 && name.length <= 20;
}

A. ユーザー名とプロジェクト名は別物なので、このまま重複していてよい
B. typeof チェックを削除する
C. 長さチェックを別の共通関数に切り出し、再利用する
D. 3 と 20 の値を変える

8. 次のコードでは、配列のフィルタリングとマッピングが重複しています。DRYとKISSの両方を意識した改善として、最もよいものはどれでしょうか？

const activeUserNames = users
  .filter(user => user.active)
  .map(user => user.name);

const activeUserEmails = users
  .filter(user => user.active)
  .map(user => user.email);

A. filter と map を1行にまとめて三項演算子で処理する
B. active なユーザー一覧を一度変数に格納し、それを元に name と email をそれぞれ map する
C. active プロパティを削除する
D. map の代わりに forEach を使う

9. 次の関数は、複数のモードに応じて処理を切り替えています。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');
  }
}

A. mode を文字列で管理していること
B. 例外を投げていること
C. 1つの関数で多くのモードを扱い、if-else が増え続ける設計になっていること
D. data を引数に取っていること

10. 次のコードは、「とりあえずなんでも詰め込んだユーティリティ関数」です。KISSの観点から、どのような改善方針がよいでしょうか？

function doMagic(obj, options) {
  // ログ出力
  console.log('start', obj, options);

  // バリデーション
  // ...

  // データ変換
  // ...

  // API呼び出し
  // ...

  // 画面更新
  // ...
}