エピローグ — ロードマップの先へ

# 入社1年目のコード
class UsersController < ApplicationController
  def create
    user = User.new(params.require(:user).permit(:name, :email, :password))
    if User.where(email: params[:user][:email]).exists?
      render json: { error: "このメールは既に使われています" }, status: 422
      return
    end
    if params[:user][:password].length < 8
      render json: { error: "パスワードは8文字以上" }, status: 422
      return
    end
    if user.save
      UserMailer.welcome_email(user).deliver_now
      slack_client = Slack::Web::Client.new
      slack_client.chat_postMessage(channel: "#new-users", text: "新規ユーザー登録: #{user.email}")
      user.points.create!(amount: 100, reason: "signup_bonus")
      render json: { user: user }, status: 201
    else
      render json: { errors: user.errors }, status: 422
    end
  end
end
 
# 3年目の同じ機能
class UsersController < ApplicationController
  def create
    result = UserRegistrationService.new(registration_params).call
 
    if result.success?
      render json: UserSerializer.new(result.user).as_json, status: :created
    else
      render json: { errors: result.errors }, status: :unprocessable_entity
    end
  end
 
  private
 
  def registration_params
    params.require(:user).permit(:name, :email, :password)
  end
end

# 副作用のないコードを書く練習
# ローカルでOSSにコントリビュートする
# 技術ブログで学んだことを言語化する
# 社内勉強会で発表する
 
# 「人に説明できるか？」が理解度のリトマス紙

# サイドプロジェクトのアイデア
# - 自分のRails APIをECSにデプロイする
# - 分散キューをRedisで自作してみる
# - テストを書かないとマージできないCI環境を作る
# - モノリスを段階的にサービス分割する練習

# コードレビューのお願い例
# 「動くコードをレビューしてほしい」ではなく
# 「この設計のここが迷っているので意見を聞かせてほしい」
# と具体的な問いを持ってお願いする

# 良いエンジニアの思考プロセス
 
# Step 1: Make it work
def calculate_discount(user, price)
  # とにかく動かす
  price * 0.9
end
 
# Step 2: Make it right
def calculate_discount(user, price)
  # テストを書きながら正しくする
  return price unless user.premium?
  (price * (1 - user.discount_rate)).ceil
end
 
# Step 3: Make it fast (必要なら)
def calculate_discount(user, price)
  # プロファイリングでボトルネックを特定してから最適化
  Rails.cache.fetch("discount_rate/#{user.id}", expires_in: 5.minutes) do
    user.discount_rate
  end.then { |rate| (price * (1 - rate)).ceil }
end

# ジュニアのコード
class OrderController < ApplicationController
  def create
    order = Order.new(params.require(:order).permit(:user_id, :total))
    if order.save
      # ウェルカムメール送信
      OrderMailer.confirmation(order).deliver_now
      # Slackに通知
      slack = Slack::Web::Client.new
      slack.chat_postMessage(channel: "#orders", text: "新規注文: #{order.id}")
      render json: { order: order }, status: 201
    else
      render json: { errors: order.errors }, status: 422
    end
  end
end