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

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

  user.completedTaskCount++;

  project.lastActivityAt = new Date();
}

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

function connectDb() {
  const client = new DbClient({
    host: 'db.example.internal',
    port: 5432,
    user: 'app_user',
    password: 'secret'
  });
  return client;
}

function placeOrder(order, db, mailer) {
  // DBへ保存
  db.insert('orders', order);

  // ユーザーにメール送信
  const message = `ご注文ありがとうございます：${order.id}`;
  mailer.send(order.userEmail, message);

  // 画面用メッセージを返す
  return {
    message: '注文完了しました',
    orderId: order.id
  };
}

function showMyPage() {
  if (!window.currentUser) {
    throw new Error('ログインしていません');
  }
  // 画面描画
}

function showSettings() {
  if (!window.currentUser) {
    throw new Error('ログインしていません');
  }
  // 設定画面描画
}

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

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

function getCustomerCity(order) {
  return order.customer.address.city;
}

function formatUserAddress(user) {
  return `${user.postalCode} ${user.pref} ${user.city} ${user.street}`;
}

function formatShippingLabel(order) {
  const user = order.user;
  return `${user.postalCode} ${user.pref} ${user.city} ${user.street}`;
}

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

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