【Ruby】メソッドを作成するときに考えていること(例のコードもあり)

class BooksController < ApplicationController
  before_action :set_book, only: [:show, :edit, :update, :destroy]

  # GET /books
  # GET /books.json
  def index
    # 別の処理

    # 書籍のフィルタリングロジック
    @books = Book.all
    @books = books.where(genre: params[:genre]) if params[:genre].present?
    @books = books.where(author: params[:author]) if params[:author].present?
    @books = books.where("published_year >= ?", params[:start_year]) if params[:start_year].present?
    @books = books.where("published_year <= ?", params[:end_year]) if params[:end_year].present?
    @books

    # 別の処理
  end

  # 以下のアクションは省略（show, new, edit, create, update, destroy）

  private

  # Use callbacks to share common setup or constraints between actions.
  def set_book
    @book = Book.find(params[:id])
  end
end

class BooksController < ApplicationController
  before_action :set_book, only: [:show, :edit, :update, :destroy]

  # GET /books
  # GET /books.json
  def index
    # 別の処理
    @books = filtered_books(params)
    # 別の処理
  end

  # 以下のアクションは省略（show, new, edit, create, update, destroy）

  private

  # 書籍のフィルタリングロジックをカプセル化
  def filtered_books(params)
    books = Book.all
    books = books.where(genre: params[:genre]) if params[:genre].present?
    books = books.where(author: params[:author]) if params[:author].present?
    books = books.where("published_year >= ?", params[:start_year]) if params[:start_year].present?
    books = books.where("published_year <= ?", params[:end_year]) if params[:end_year].present?
    books
  end

  # Use callbacks to share common setup or constraints between actions.
  def set_book
    @book = Book.find(params[:id])
  end
end

def full_name
  first_name + last_name
end

def age_from_birthday
  Time.current.year - birthday_date.year
end

# 情報的凝集度の例
def calculate_age_and_time_until_retirement(birthday_date)
  # 年齢を計算
  age = Time.current.year - birthday_date.year

  # 定年の年齢 - 年齢
  RETIREMENT_AGE - age
end

# 機能的凝集度のメソッド 2 つに分けられる
def age(birthday_date)
  age = Time.current.year - birthday_date.year
end

def time_until_retirement(age)
  RETIREMENT_AGE - age
end

class Application
  def initialize
    # アプリケーションの初期設定
  end

  def start
    # データベース接続の設定
    # ここで直接データベースに接続するコードを記述
    # ...

    # ログファイルの初期化
    # ここでログファイルを開くか初期化するコードを記述
    # ...

    # 設定ファイルの読み込み
    # ここで設定ファイルを読み込むコードを記述
    # ...

    # セキュリティチェック
    # ここでセキュリティチェックを行うコードを記述
    # ...

    # システムの状態をチェック
    # ここでシステムの状態をチェックするコードを記述
    # ...

    # アプリケーションのメイン機能を開始
    # ここでアプリケーションの主要な機能を開始するコードを記述
    # ...
  end
end

class Application
  def initialize
    # アプリケーションの初期設定
  end

  def start
    # データベース接続の設定
    connect_to_database
    # ログファイルの初期化
    initialize_log_file
    # 設定ファイルの読み込み
    load_configuration
    # セキュリティチェック
    perform_security_checks
    # システムの状態をチェック
    check_system_status
    # アプリケーションのメイン機能を開始
    launch_main_features
  end

  # ここに各タスクの実装が続く...
end

# 悪い例
def modify_value(input_val)
  input_val += 10
  # 他の処理...
  return input_val
end

# いい例
def modify_value(input_val)
  result = input_val + 10
  # 他の処理...
  return result
end