エピローグ — 3Rが身につくとき

# 3ヶ月前: 変更頻度が高いファイルTOP5
git log --format=format: --name-only | sort | uniq -c | sort -rg | head -5
# 89 app/models/order.rb             ← Fat Model（1247行）
# 67 app/services/discount_service.rb ← 重複コードの源
# 45 app/controllers/orders_controller.rb ← ビジネスロジックが漏れ出ている
# 38 app/models/user.rb              ← 別のFat Model
# 31 app/helpers/application_helper.rb ← 雑多なメソッド置き場
 
# 3ヶ月後（同じコマンド）
git log --format=format: --name-only | sort | uniq -c | sort -rg | head -5
# 28 spec/services/order_creation_service_spec.rb ← テストが増えた！
# 23 app/services/order_creation_service.rb       ← 分割・整理済み
# 18 app/models/concerns/user/loyalty_program.rb  ← Concern
# 15 spec/models/concerns/user/loyalty_program_spec.rb
# 12 app/services/discount_service.rb

# テストカバレッジの計測（SimpleCov）
COVERAGE=true bundle exec rspec
 
# Coverage report:
# +----------+--------+----------+--------+
# | Group    | Lines  | Relevant | %      |
# +----------+--------+----------+--------+
# | Services | 1,234  | 1,156    | 87.2%  |
# | Models   | 892    | 845      | 91.1%  |
# | Concerns | 456    | 432      | 94.7%  |
# +----------+--------+----------+--------+
# | Total    | 3,847  | 3,201    | 83.2%  |
# +----------+--------+----------+--------+

# RuboCopでコード品質を定量化
bundle exec rubocop --format json | jq '
  {
    total_offenses: .summary.offense_count,
    files_inspected: .summary.inspected_file_count,
    avg_per_file: (.summary.offense_count / .summary.inspected_file_count | floor)
  }
'
 
# 3ヶ月前:
# { "total_offenses": 847, "files_inspected": 124, "avg_per_file": 6 }
 
# 3ヶ月後:
# { "total_offenses": 89, "files_inspected": 156, "avg_per_file": 0 }

# コードを書く前に問いかける質問
 
# 変数名を付けるとき:
# 「3ヶ月後の自分がこの変数を見たとき、何のための変数かわかるか？」
 
# メソッドを作るとき:
# 「このメソッドは1つのことだけしているか？名前がその1つを表しているか？」
 
# クラスを作るとき:
# 「このクラスが変更される理由は何か？複数あるなら分割すべきか？」
 
# コードを書いた後に確認する:
# 「初めてこのコードを見る人が5分で理解できるか？」

# PRセルフレビューチェックリスト
 
## このPRは何をしているか？
- [ ] 1文で説明できるか（できなければ分割を検討）
- [ ] タイトルが変更内容を正確に表しているか
 
## R1チェック（Readability）
- [ ] 変数名・メソッド名を読むだけで意図が伝わるか
- [ ] マジックナンバーが定数化されているか
- [ ] メソッドが15行以内か
- [ ] コメントが「なぜ」だけを説明しているか（「何をする」ではない）
 
## R2チェック（Reusability）
- [ ] 重複コードを書いていないか（既存のService/Concernを使えているか）
- [ ] 新しいビジネスロジックが適切な場所にあるか（コントローラに漏れていないか）
 
## R3チェック（Refactorability）
- [ ] 新しいテストを書いたか
- [ ] テストがグリーンになっているか（CIの確認）
- [ ] 変更の影響範囲が限定されているか（変えた場所だけが変わっているか）
- [ ] 外部サービスへの依存が注入可能になっているか
 
## 将来の変更者へ
- [ ] なぜこの実装にしたかがコードまたはコメントから読み取れるか
- [ ] 将来変わりそうな数値・文字列が定数になっているか

# 毎回のPRで「少し良くする」を積み重ねる
 
# 新機能を追加するついでに:
# - 通り道のメソッド名を改善する（変数 x → order）
# - 見かけたWETなコードをDRYにする（割引ロジックの重複をServiceに抽出）
# - カバレッジが0のメソッドにテストを追加する
 
# 大規模なリファクタリングスプリントより
# 継続的な小さな改善の方が持続可能
 
# なぜなら:
# 大規模リファクタリング → 長期間コードが不安定 → チームが嫌がる → 実行されない
# 継続的な小改善 → 常に安定 → チームに受け入れられる → 継続できる

# app/services/quality_report.rb
class QualityReport
  def self.generate_and_notify
    report = new.generate
    Slack::Notifier.new(ENV['QUALITY_SLACK_WEBHOOK']).ping(
      attachments: [report.to_slack_attachment]
    )
  end
 
  def generate
    QualityMetrics.new(
      coverage: current_test_coverage,
      rubocop_offenses: rubocop_offense_count,
      fat_models: fat_model_count,
      average_method_length: average_method_length,
      measured_at: Time.current
    )
  end
 
  private
 
  def current_test_coverage
    json = JSON.parse(File.read('coverage/.last_run.json'))
    json.dig('result', 'line').round(1)
  rescue StandardError
    nil
  end
 
  def rubocop_offense_count
    output = `bundle exec rubocop --format json 2>/dev/null`
    JSON.parse(output).dig('summary', 'offense_count')
  rescue StandardError
    nil
  end
 
  def fat_model_count
    Dir['app/models/**/*.rb'].count do |f|
      File.read(f).lines.count > 200
    end
  end
 
  def average_method_length
    total_methods = 0
    total_length = 0
 
    Dir['app/**/*.rb'].each do |file|
      content = File.read(file)
      methods = content.scan(/^\s+def .+?(?=\s+(?:def |end\s*$))/m)
      methods.each do |method|
        total_methods += 1
        total_length += method.lines.count
      end
    end
 
    return 0 if total_methods.zero?
    (total_length.to_f / total_methods).round(1)
  end
end
 
QualityMetrics = Data.define(
  :coverage,
  :rubocop_offenses,
  :fat_models,
  :average_method_length,
  :measured_at
) do
  def to_slack_attachment
    {
      title: "週次コード品質レポート（#{measured_at.strftime('%Y/%m/%d')}）",
      color: overall_health_color,
      fields: [
        { title: "テストカバレッジ", value: "#{coverage}%", short: true },
        { title: "RuboCop違反", value: "#{rubocop_offenses}件", short: true },
        { title: "Fat Model数", value: "#{fat_models}個", short: true },
        { title: "平均メソッド長", value: "#{average_method_length}行", short: true }
      ]
    }
  end
 
  private
 
  def overall_health_color
    return 'good' if coverage >= 80 && rubocop_offenses <= 50 && fat_models == 0
    return 'warning' if coverage >= 60
    'danger'
  end
end

# config/schedule.rb（whenever gemでcron設定）
# 毎週月曜9時にレポートをSlackに送信
every :week, at: '9:00am', roles: [:app] do
  runner "QualityReport.generate_and_notify"
end

# プロローグで見たコード（3Rの旅の出発点）
def p(o, u, c)
  x = Order.find(o)
  if x.st == 1
    if u.pr == true
      x.tt = x.tt * 0.8
    end
    # ...
  end
end
 
# 9ヶ月後（3Rの旅の到達点）
class OrderCompletionService
  PREMIUM_DISCOUNT_RATE = 0.20
 
  def initialize(order, user, coupon_code: nil)
    @order = order
    @user = user
    @coupon_code = coupon_code
  end
 
  def call
    return ServiceResult.failure('注文が保留中ではありません') unless @order.pending?
 
    ActiveRecord::Base.transaction do
      apply_premium_discount
      apply_coupon_if_present
      complete_order
    end
 
    ServiceResult.success(@order)
  end
 
  private
 
  def apply_premium_discount
    return unless @user.premium?
 
    discount = (@order.total * PREMIUM_DISCOUNT_RATE).round(2)
    @order.discount_amount = discount
    @order.total -= discount
  end
 
  def apply_coupon_if_present
    return unless @coupon_code.present?
 
    coupon = Coupon.find_by(code: @coupon_code)
    return unless coupon&.redeemable?
 
    coupon.apply_to(@order)
  end
 
  def complete_order
    @order.update!(status: :completed, completed_at: Time.current)
    OrderMailer.confirmation(@order.user, @order).deliver_later
  end
end

# 今日からできる3アクション
 
# アクション1: 一番読みにくいメソッドを1つ選んで名前を改善する
# Before: def p(o, u)
# After:  def process_order(order, user)
 
# アクション2: 重複している処理を1つ見つけてDRYにする
# 3箇所に同じ割引ロジック → DiscountService に集約
 
# アクション3: テストがないメソッドを1つ選んでテストを書く
# describe OrderCompletionService do
#   it '注文を完了させる' do
#     ...
#   end
# end

## R1: Readability チェックリスト
- [ ] 変数名が意図を表している（x → order, d → discount_amount）
- [ ] メソッド名が1つの動作を表している（process → process_order）
- [ ] ブール値メソッドが ? で終わる（premium?, active?, expired?）
- [ ] コレクションが複数形（item → items, order → orders）
- [ ] メソッドが15行以内（例外的に長い場合は分割を検討）
- [ ] ガード節で入れ子を減らしている
- [ ] コメントが「なぜ」だけを説明している（「何をする」はコードが説明する）
 
## R2: Reusability チェックリスト
- [ ] 同じロジックが3箇所以上あれば抽出（3回ルール）
- [ ] ビジネスロジックがコントローラに漏れていない
- [ ] 複数モデルに共通するロジックはConcernに切り出している
- [ ] チーム内/複数プロジェクトで共通のコードはGem化を検討
- [ ] マジックナンバーが定数になっている
 
## R3: Refactorability チェックリスト
- [ ] テストカバレッジが80%以上
- [ ] 1クラス1責任（SRP）
- [ ] 新機能追加で既存コードを変えずに済む（OCP）
- [ ] 依存性注入でテストが書ける（DIP）
- [ ] インフラはTerraform/CloudFormationで管理（IaC）
- [ ] CIでテスト・Lintが自動実行される