AIが迷わない疎結合設計へリファクタ！
コードレビュー道場 #2：凝集度/結合度

公開日：2025年12月13日

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

このページでは、AIが作りがちな密結合な設計を題材に、凝集度／結合度の観点で読み解くコードレビュー形式の4択クイズに挑戦します。AIに任せるほど、境界と責務を人間が設計できるかが差になります。

4択クイズ（10問・全問解説付き）で腕試し → 解説で責務の切り方・依存の減らし方を復習。「読む→解く→わかる」で、AIが迷わない疎結合設計へ導く視点を鍛えましょう！

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

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

#2 凝集度/結合度：疎結合へリファクタ！目次

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

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

コードレビュー道場 #2～密結合な設計を疎結合に変えよ！～

1. 次のクラスは、ログ出力と業務ロジックが混ざっています。凝集度/結合度の観点から、どのような改善が望ましいでしょうか？


class PaymentService {
  constructor(http) {
    this.http = http;
  }

  async pay(order) {
    console.log('PAYMENT START', new Date(), order.id);

    const result = await this.http.post('/payments', { orderId: order.id });

    console.log('PAYMENT END', new Date(), order.id, result.status);

    return result;
  }
}

A. console.log を減らし、1か所だけにする
B. ログ出力を別のロガーに委譲し、PaymentService は「支払い処理」に集中させる
C. result を返さないようにする
D. this.http をグローバル変数にする

2. 次のコードは、複数の外部サービスに直接依存しています。結合度の観点から、どのような問題がありますか？


function notifyUser(user, message) {
  Slack.post('#alerts', message);
  Email.send(user.email, message);
  Sms.send(user.phone, message);
}

A. メソッド名が英語で書かれていること
B. 3つのサービスに直に依存しており、通知手段の変更がしづらい高結合な設計になっている
C. user を引数で取っていること
D. 3つの通知を同時に送っていること

3. 次のコードは、メール送信の方法が直接固定されています。結合度の観点から、どの改善が望ましいでしょうか？


function sendPasswordReset(user, token) {
  const url = `https://example.com/reset?token=${token}`;
  const message = `こちらからパスワードを再設定してください: ${url}`;
  SmtpClient.send(user.email, message);
}

A. URLを https ではなく http にする
B. SmtpClient に直接依存せず、メール送信を抽象化したインターフェースに依存する
C. message を短くする
D. user.email を使わないようにする

4. 次のコードは、1つの関数が多くのフラグ引数に依存しています。凝集度/結合度の観点から、どのような問題がありますか？


function renderList(items, showHeader, showFooter, highlightActive, allowDelete) {
  // たくさんの条件分岐で描画を切り替える
}

A. items が配列であること
B. 関数の引数としてフラグが多すぎて、1つの関数に多くの役割が詰め込まれている
C. highlightActive を true にしていること
D. allowDelete を false にしていること

5. 次のコードは、ドメインロジックがUIの具体的なIDに依存しています。結合度の観点から、最も問題となる点はどれでしょうか？


function updateUserName(user) {
  const input = document.getElementById('user-name-input');
  user.name = input.value;
}

A. getElementById を使っていること
B. user.name を直接書き換えていること
C. ドメインロジックが特定のDOM要素IDに直接依存しており、表示とロジックが密結合していること
D. 関数名が長いこと

6. 次の2つのクラスは、互いにお互いを new しています。結合度の観点から、どのような問題がありますか？


class A {
  constructor() {
    this.b = new B(this);
  }
}

class B {
  constructor(a) {
    this.a = a;
  }
}

A. クラス名が1文字で短すぎる
B. A と B が循環参照しており、互いに強く結合している
C. constructor を使っていること
D. this を保存していること

7. 次の関数は、複数のオブジェクトにまたがって状態を更新しています。結合度/凝集度の観点から、どのような設計リスクがありますか？


function completeTask(task, user, project) {
  task.completed = true;
  task.completedAt = new Date();

  user.completedTaskCount++;

  project.lastActivityAt = new Date();
}

A. Date コンストラクタを2回呼んでいること
B. task, user, project の3つの集約にまたがる更新ロジックを1つの関数にまとめており、複数のものと強く結合している
C. completed フラグを true にしていること
D. project に lastActivityAt を持たせていること

8. 次のクラスは、「メール送信」と「テンプレート生成」を同じ場所で行っています。凝集度/結合度の観点から、どの設計が望ましいでしょうか？


class MailService {
  constructor(client) {
    this.client = client;
  }

  sendWelcomeMail(user) {
    const body = `ようこそ ${user.name} さん`;
    this.client.send(user.email, body);
  }

  sendByeMail(user) {
    const body = `ご利用ありがとうございました ${user.name} さん`;
    this.client.send(user.email, body);
  }
}

A. MailService を2つに分割し、1つは send 専用、1つはテンプレート専用にする
B. テンプレート生成を別クラスや関数に切り出し、MailService は「送る」責務に集中させる
C. user.name を使わないようにする
D. client を new するようにする

9. 次のコードは、内部データ構造に強く依存した取り方をしています。結合度の観点から、最も問題となる点はどれでしょうか？


function getFirstItemPrice(cart) {
  return cart.items[0].price;
}

A. price プロパティ名を使っていること
B. items 配列のインデックス0に直接アクセスしていること
C. cart を引数で受け取っていること
D. 関数名の先頭が get になっていること

10. 次のコードでは、テストしづらい結合が生まれています。結合度の観点から、どこが問題でしょうか？


function calculateDiscount(user, now = new Date()) {
  if (user.isPremium && now.getDay() === 5) {
    return 0.2;
  }
  return 0;
}

A. user.isPremium を参照していること
B. Date をデフォルト引数として new しており、時間に強く結合していること
C. 割引率が 0.2 であること
D. getDay を使っていること

次のクイズ(#3 非機能要件：落ちるコードを鍛え直せ！)へ

SPECIAL

2026.03.28 公開

『AI時代のエンジニアタイプ診断』を新規公開！
全16タイプで設計スタイル・強み・相性が分かります！

抽象派/具現派、品質重視/速度重視、AI協働派/自力精査派など4つの指標から診断し、強み・弱み・相性のいいタイプや今読むべき記事まで分かるようにしました！

診断してみる

SUB SPECIAL

2026.04.20 公開

『コーディング性格診断』を新規公開！
32問で精密採点＆8軸レーダーチャートで分析！

あなたの実装は何点？ 8軸レーダーチャートで強み・注意点・改善点まで見える化。みかん称号・改善アドバイス・AIプロンプトまで持ち帰れる、登録不要・完全無料の診断です。気楽にやってみてね！

診断してみる

今すぐクイズに挑戦プログラミングクイズ一覧へ

参考文献・出典

プリンシプルオブプログラミング～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/とろとろみかん

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

AIが迷わない疎結合設計へリファクタ！
コードレビュー道場 #2：凝集度/結合度

#2 凝集度/結合度：疎結合へリファクタ！目次

コードレビュー道場 #2～密結合な設計を疎結合に変えよ！～

2. 次のコードは、複数の外部サービスに直接依存しています。結合度の観点から、どのような問題がありますか？

`function notifyUser(user, message) { Slack.post('#alerts', message); Email.send(user.email, message); Sms.send(user.phone, message); }`

5. 次のコードは、ドメインロジックがUIの具体的なIDに依存しています。結合度の観点から、最も問題となる点はどれでしょうか？

`function updateUserName(user) { const input = document.getElementById('user-name-input'); user.name = input.value; }`

6. 次の2つのクラスは、互いにお互いを new しています。結合度の観点から、どのような問題がありますか？

`class A { constructor() { this.b = new B(this); } } class B { constructor(a) { this.a = a; } }`

9. 次のコードは、内部データ構造に強く依存した取り方をしています。結合度の観点から、最も問題となる点はどれでしょうか？

`function getFirstItemPrice(cart) { return cart.items[0].price; }`

10. 次のコードでは、テストしづらい結合が生まれています。結合度の観点から、どこが問題でしょうか？

`function calculateDiscount(user, now = new Date()) { if (user.isPremium && now.getDay() === 5) { return 0.2; } return 0; }`

『AI時代のエンジニアタイプ診断』を新規公開！
全16タイプで設計スタイル・強み・相性が分かります！

『コーディング性格診断』を新規公開！
32問で精密採点＆8軸レーダーチャートで分析！

密結合な設計を疎結合に変えよ！
採点結果

参考文献・出典

このサイトの運営者

もちもちみかん.comとは

AIが迷わない疎結合設計へリファクタ！コードレビュー道場 #2：凝集度/結合度

#2 凝集度/結合度：疎結合へリファクタ！ 目次

コードレビュー道場 #2～密結合な設計を疎結合に変えよ！～

2. 次のコードは、複数の外部サービスに直接依存しています。結合度の観点から、どのような問題がありますか？ function notifyUser(user, message) { Slack.post('#alerts', message); Email.send(user.email, message); Sms.send(user.phone, message); }

5. 次のコードは、ドメインロジックがUIの具体的なIDに依存しています。結合度の観点から、最も問題となる点はどれでしょうか？ function updateUserName(user) { const input = document.getElementById('user-name-input'); user.name = input.value; }

6. 次の2つのクラスは、互いにお互いを new しています。結合度の観点から、どのような問題がありますか？ class A { constructor() { this.b = new B(this); } } class B { constructor(a) { this.a = a; } }

9. 次のコードは、内部データ構造に強く依存した取り方をしています。結合度の観点から、最も問題となる点はどれでしょうか？ function getFirstItemPrice(cart) { return cart.items[0].price; }

10. 次のコードでは、テストしづらい結合が生まれています。結合度の観点から、どこが問題でしょうか？ function calculateDiscount(user, now = new Date()) { if (user.isPremium && now.getDay() === 5) { return 0.2; } return 0; }

『AI時代のエンジニアタイプ診断』を新規公開！ 全16タイプで設計スタイル・強み・相性が分かります！

『コーディング性格診断』を新規公開！ 32問で精密採点＆8軸レーダーチャートで分析！

参考文献・出典

このサイトの運営者

シェアしてね

AIが迷わない疎結合設計へリファクタ！
コードレビュー道場 #2：凝集度/結合度

#2 凝集度/結合度：疎結合へリファクタ！目次

2. 次のコードは、複数の外部サービスに直接依存しています。結合度の観点から、どのような問題がありますか？

`function notifyUser(user, message) { Slack.post('#alerts', message); Email.send(user.email, message); Sms.send(user.phone, message); }`

5. 次のコードは、ドメインロジックがUIの具体的なIDに依存しています。結合度の観点から、最も問題となる点はどれでしょうか？

`function updateUserName(user) { const input = document.getElementById('user-name-input'); user.name = input.value; }`

6. 次の2つのクラスは、互いにお互いを new しています。結合度の観点から、どのような問題がありますか？

`class A { constructor() { this.b = new B(this); } } class B { constructor(a) { this.a = a; } }`

9. 次のコードは、内部データ構造に強く依存した取り方をしています。結合度の観点から、最も問題となる点はどれでしょうか？

`function getFirstItemPrice(cart) { return cart.items[0].price; }`

10. 次のコードでは、テストしづらい結合が生まれています。結合度の観点から、どこが問題でしょうか？

`function calculateDiscount(user, now = new Date()) { if (user.isPremium && now.getDay() === 5) { return 0.2; } return 0; }`

『AI時代のエンジニアタイプ診断』を新規公開！
全16タイプで設計スタイル・強み・相性が分かります！

『コーディング性格診断』を新規公開！
32問で精密採点＆8軸レーダーチャートで分析！