プロローグ — なぜアーキテクチャパターンを学ぶのか

class OrdersController < ApplicationController
  def create
    # バリデーション
    if params[:user_id].blank?
      render json: { error: "user_id is required" }, status: 400
      return
    end
 
    user = User.find(params[:user_id])
    if user.nil?
      render json: { error: "User not found" }, status: 404
      return
    end
 
    # 在庫チェック
    params[:items].each do |item|
      product = Product.find(item[:product_id])
      if product.stock < item[:quantity]
        render json: { error: "#{product.name}の在庫が不足しています" }, status: 422
        return
      end
    end
 
    # 合計金額計算
    total = 0
    params[:items].each do |item|
      product = Product.find(item[:product_id])
      total += product.price * item[:quantity]
    end
 
    # クーポン適用
    if params[:coupon_code].present?
      coupon = Coupon.find_by(code: params[:coupon_code])
      if coupon && coupon.valid? && coupon.expires_at > Time.current
        if coupon.discount_type == "percentage"
          total = total * (1 - coupon.discount_value / 100.0)
        else
          total = total - coupon.discount_value
        end
      end
    end
 
    # 注文作成
    order = Order.new(
      user_id: user.id,
      total_amount: total,
      status: "pending"
    )
 
    params[:items].each do |item|
      product = Product.find(item[:product_id])
      order.order_items.build(
        product_id: product.id,
        quantity: item[:quantity],
        price: product.price
      )
    end
 
    if order.save
      # メール送信
      OrderMailer.confirmation(order).deliver_now
 
      # 在庫を減らす
      params[:items].each do |item|
        product = Product.find(item[:product_id])
        product.update!(stock: product.stock - item[:quantity])
      end
 
      # ポイント付与
      user.update!(points: user.points + (total * 0.01).floor)
 
      # Slack通知
      SlackNotifier.notify("#orders", "新しい注文が入りました: ##{order.id}")
 
      render json: order, status: 201
    else
      render json: { errors: order.errors.full_messages }, status: 422
    end
  end
end

# コードスメルの例：同じProductクエリが3箇所に散らばっている
# controllers/orders_controller.rb
product = Product.where(active: true).find(params[:product_id])
 
# services/cart_service.rb
product = Product.where(active: true).find(product_id)
 
# jobs/inventory_check_job.rb
products = Product.where(active: true).where("stock < ?", threshold)

# bin/code_health_check.rb
# 実行: ruby bin/code_health_check.rb
 
require 'find'
 
class CodeHealthCheck
  THRESHOLDS = {
    controller_lines: 100,   # コントローラーの行数上限
    model_lines: 200,        # モデルの行数上限
    method_lines: 20,        # メソッドの行数上限
  }.freeze
 
  def run
    puts "=== コード健康チェック ===\n\n"
 
    check_fat_controllers
    check_fat_models
    check_scattered_queries
    check_mixed_concerns
  end
 
  private
 
  def check_fat_controllers
    puts "【コントローラーサイズ】"
    fat = []
 
    Dir.glob("app/controllers/**/*.rb").each do |path|
      lines = File.readlines(path).count
      fat << { path: path.sub("app/controllers/", ""), lines: lines } if lines > THRESHOLDS[:controller_lines]
    end
 
    if fat.empty?
      puts "  ✓ 全コントローラーが#{THRESHOLDS[:controller_lines]}行以下"
    else
      puts "  ! ファットコントローラー検出:"
      fat.each { |f| puts "    #{f[:path]}: #{f[:lines]}行" }
    end
    puts
  end
 
  def check_scattered_queries
    puts "【クエリの散在チェック】"
    model_names = Dir.glob("app/models/**/*.rb").map { |f|
      File.basename(f, ".rb").camelize
    }
 
    scattered = {}
    model_names.each do |model|
      files = `grep -rn "#{model}\.where\|#{model}\.find\|#{model}\.joins" app/ --include="*.rb" -l 2>/dev/null`.split("\n")
      files = files.reject { |f| f.include?("models/") || f.include?("spec/") }
      scattered[model] = files if files.size > 2
    end
 
    if scattered.empty?
      puts "  ✓ クエリの散在なし"
    else
      puts "  ! クエリが散在しているモデル:"
      scattered.each { |m, files| puts "    #{m}: #{files.size}箇所" }
    end
    puts
  end
 
  def check_mixed_concerns
    puts "【コントローラーの責任混在チェック】"
    issues = []
 
    Dir.glob("app/controllers/**/*.rb").each do |path|
      content = File.read(path)
      controller = File.basename(path, ".rb")
 
      # メール送信がコントローラー内にある
      if content.match?(/Mailer\.\w+.*deliver/)
        issues << "#{controller}: メール送信がコントローラーにある"
      end
 
      # DBアクセスが直接ある（belongs_to等でなく）
      if content.match?(/\.where\(|\.find_by\(|\.joins\(/)
        issues << "#{controller}: 複雑なDBクエリがコントローラーにある"
      end
    end
 
    if issues.empty?
      puts "  ✓ 関心の混在なし"
    else
      puts "  ! 改善が必要:"
      issues.each { |i| puts "    #{i}" }
    end
  end
end
 
CodeHealthCheck.new.run

# db/seeds/tech_debt_register.rb
# 技術的負債をDBで管理する例
 
TechDebt.create!([
  {
    title: "OrdersControllerの分割",
    severity: :critical,  # critical / high / medium / low
    estimated_hours: 8,
    affected_files: ["app/controllers/orders_controller.rb"],
    pattern_to_apply: "layered_architecture",
    business_impact: "デプロイ時に毎回800行のファイルを確認する必要がある",
    created_at: 1.month.ago
  },
  {
    title: "Orderモデルのバリデーション整理",
    severity: :high,
    estimated_hours: 3,
    affected_files: ["app/models/order.rb"],
    pattern_to_apply: "domain_model",
    business_impact: "バリデーションエラーが本番で週3件発生",
    created_at: 2.weeks.ago
  },
  {
    title: "クエリのN+1問題",
    severity: :high,
    estimated_hours: 4,
    affected_files: ["app/models/order.rb", "app/controllers/orders_controller.rb"],
    pattern_to_apply: "repository_pattern",
    business_impact: "注文一覧ページが3秒かかる",
    created_at: 1.week.ago
  }
])