Rubyでクラスとインスタンスを用いてコードを書き換えたいです。

Question

学校の課題で、クラスとインスタンスを使って現在のコードを書き換えたいのですが、途中まで考えてわからなかったので教えてほしいです。

def post_review(a_posts)

    post = {}
    puts "ジャンルを入力してください"
    post[:genre] = gets.chomp
    puts "タイトルを入力してください"
    post[:title] = gets.chomp
    puts "感想を入力してください"
    post[:review] = gets.chomp
    line = "-------------------------------"

    puts "ジャンル : #{post[:genre]}\n#{line}"
    puts "タイトル : #{post[:title]}\n#{line}"
    puts "感想 : \n#{post[:review]}\n#{line}"


    a_posts << post

    return a_posts
  end

def read_reviews(a_posts)

  number = 0
  a_posts.each do |post|
    puts "[#{number}] : #{post[:title]}のレビュー"
    number += 1
  end

  puts "\nみたいレビューの番号を入力してください :"
  input = gets.to_i


  post = a_posts[input]


  line = "-------------------------------"
  puts "ジャンル : #{post[:genre]}\n#{line}"
  puts "タイトル : #{post[:title]}\n#{line}"
  puts "感想 : \n #{post[:review]}\n#{line}"
end

def end_program
  exit
end

def exception
  puts "入力された値は無効な値です"
end

posts = []

while true do
      
    puts "レビュー数 : #{posts.length}"
    puts "[0]レビューを書く"
    puts "[1]レビューを読む"
    puts "[2]アプリを終了する"
    input = gets.to_i

    if input == 0
        posts = post_review(posts)
    elsif input == 1
        read_reviews(posts)
    elsif input == 2
        end_program
    else
        exception
    end
end

途中まで考えたコードはこちらです。

class Review
  attr_accessor :genre, :title, :voice
  @@post = []

  def self.input
    puts "ジャンルを入力してください"
    genre = gets.chomp
    puts "タイトルを入力してください"
    title = gets.chomp
    puts "感想を入力してください"
    voice = gets.chomp
    line = "-------------------------------"

    review = Review.new(genre, title, voice)
    @@post << review
  end

  def initialize(genre, title, voice)
    @genre = genre
    @title = title
    @voice = voice
  end
end

Review.input

Rubyについてあまり理解できていないため、正解のコードを知って逆から理解していきたいとおもっています。
よろしくおねがいします。

Answer 1

賛否両論あるかもしれませんが、書き直してみました

class Review
  attr_accessor :genre, :title, :review
  def initialize( genre:, title:, review:)
    @genre  = genre
    @title  = title
    @review = review
  end
end

module ReviewInterface
  class << self
    attr_accessor :reviews
    def reviews
      @reviews ||= []
    end

    def post_review
      puts "ジャンルを入力してください"
      genre = gets.chomp
      puts "タイトルを入力してください"
      title = gets.chomp
      puts "感想を入力してください"
      review = gets.chomp

      review = Review.new(genre: genre, title: title, review: review)

      show_review(review)

      reviews << review
    end

    def read_review
      if reviews.empty?
        puts "まだレビューがありません"
        return
      end

      show_reviews

      puts "\nみたいレビューの番号を入力してください :"
      index = Integer(gets)

      show_review(reviews[index])
    rescue
      puts "不正な番号が指定されました"
    end

    def show_reviews
      reviews.each_with_index do |review, i|
        puts "[#{i}] : #{review.title}のレビュー"
      end
    end

    def show_review(review)
      line = "-------------------------------"
      puts "ジャンル : #{review.genre}\n#{line}"
      puts "タイトル : #{review.title}\n#{line}"
      puts "感想 : \n #{review.review}\n#{line}"
    end 

    def start
      loop do
        show_menu

        input = Integer(gets) rescue -1

        act_menu(input)
      end
    end

    def show_menu
      puts "レビュー数 : #{reviews.length}"
      puts "[0]レビューを書く"
      puts "[1]レビューを読む"
      puts "[2]アプリを終了する"
    end

    def act_menu(input)
      if input == 0
        post_review
      elsif input == 1
        read_review
      elsif input == 2
        exit
      else
        puts "入力された値は無効な値です"
      end
    end
  end
end

ReviewInterface.start

※エラー処理を少し追加してます

Comments

[退会済みユーザー]

0のレビューを書くを実行すると下記のエラーが出てしまいます・・・。
27:in `post_review': undefined method `show_review' for ReviewInterface:Module (NoMethodError)
これは27行目で48行目のインスタンスメソッドを呼び出しているから起こるものでしょうか・・・？

[satoshih]

show_reviewのメソッドが抜けてました。すいません。
ソースは編集したので、再度コピーしてもらうと動くと思います

[退会済みユーザー]

ありあとうございます。
私の知らない表現がたくさんでてきていて、勉強になります！
satoshihさんが作っていただいたコードを分析して、理解できるように励みます。

ありがとうございました！