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

function createReport(params) {
  // params = {
  //   userId, userName, userAge,
  //   projectId, projectName,
  //   startDate, endDate,
  //   includePrivate, includeDeleted,
  //   ...
  // }
  // 複雑なレポート生成処理
}

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

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

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

function onClickSaveButton() {
  const user = collectFormData();

  // ドメインロジックを直接 new
  const service = new UserService();
  service.save(user);

  alert('保存しました');
}

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

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

class UserManager {
  constructor(db) {
    this.db = db;
  }

  async findById(id) {
    return this.db.query('SELECT * FROM users WHERE id = ?', [id]);
  }

  async changeEmail(id, newEmail) {
    const user = await this.findById(id);
    // ドメインルールのチェック（省略）
    await this.db.execute('UPDATE users SET email = ? WHERE id = ?', [newEmail, id]);
  }
}

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

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

class UserProfileScreen {
  constructor(apiClient) {
    this.apiClient = apiClient;
  }

  async loadUser(userId) {
    const user = await this.apiClient.get(`/users/${userId}`);
    this.render(user);
  }

  async updateUser(user) {
    await this.apiClient.put(`/users/${user.id}`, user);
    this.render(user);
  }

  render(user) {
    // DOMを書き換えてプロフィール画面を描画
  }
}