parameterの値をint型に変換して保存したい

Question

概要

パラメータで受け取ったstring型の値をinteger型のカラムに変換してデータベースに保存したいですがどこでどのように値を変換すればいいかわからないです。

###ソースコード

    <%= form_for @upload, :url => {:action => :create} do |f| %>
      <%= f.label :file %>
      <%= f.file_field :file %></br>
      <%= f.select :lesson_id, @lessons.map {|l| l.id} %>
      <%= f.label :レッスン番号 %><br />
      <%= f.select :genre, @genres.map {|g| [g.key, g.id.to_i]} %>
      <%= f.label :割り当て %><br />
      <%= f.submit "Upload" %>
    <% end %>

class UploadersController < ApplicationController
  
  def index
  end

  def new
    @upload = UploadFile.new
    @lessons = Lesson.all
    @genres = Genre.all
  end

  def create
    @upload_file = UploadFile.new(file: file_params[:file], lesson_id: file_params[:lesson_id], genre: file_params[:genre])
    if @upload_file.save
      redirect_to teacher_uploaders_path, notice: "送信に成功"
    else
      flash.now[:error] = '送信に失敗しました'
      render :new
    end
  end

  
  private 
  
    def file_params
      params.require(:upload_file).permit(:file, :lesson_id, :genre)
    end
end

起こるエラー

ActiveRecord::AssociationTypeMismatch in TeachersUploadersController#create
Genre(#70072727747660) expected, got "1" which is an instance of String(#15875560)

試したこと

コントローラのcreateアクションでfile_paramsでto_iメソッドを使って変換

def create
    @upload_file = UploadFile.new(file: file_params[:file], lesson_id: file_params[:lesson_id], genre: file_params[:genre].to_i)
    if @upload_file.save
      redirect_to teacher_uploaders_path, notice: "送信に成功"
    else
      flash.now[:error] = '送信に失敗しました'
      render :new
    end
  end

エラーが出た

ActiveRecord::AssociationTypeMismatch in UploadersController#create
Genre(#39949360) expected, got 1 which is an instance of Integer(#15858780)

genreの値が"1"から1に変わっているのでto_iメソッドはちゃんと動いているのに先ほどど同じエラーが出たのは何故ですかね？
1は整数型ではないのかと疑問。

どなたかご教示いただけると幸いです。

バージョン

ruby 2.6.0
Rails 5.2.2

Answer 1

整理したいのですが、
maisumakunさんが指摘されていた時のupload_filesテーブルのmigrationファイルのコードの中身はどうだったのでしょうか？

create_table "upload_files", options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t|
t.binary "file"
t.integer "lesson_id"
t.integer "genre"
t.datetime "created_at", null: false
t.datetime "updated_at", null: false
end

これだったのでしょうか？
これだと整数を型とするgenreという名前のカラムであり、Genreクラスがexpectedされることは無いと思うのですが。

Answer 2

genre_id: file_params[:genre]

で行けるはず

Comments

[rou_7201]

upload_fileのカラムの値がgenre_idでなくgenreですのでダメでした。

create_table "upload_files", options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t|
t.binary "file"
t.integer "lesson_id"
t.integer "genre"
t.datetime "created_at", null: false
t.datetime "updated_at", null: false
end

Answer 3

to_iメソッドはちゃんと動いているのに先ほどど同じエラーが出たのは何故ですかね？

エラーメッセージをよく読みましょう。Genre(#39949360) expectedとなっているので、ここで渡すべきものは文字列でも整数でもなく、Genreのインスタンスです。

Comments

[rou_7201]

解答ありがとうございます。エラーメッセージちゃんと理解していませんでした。
インスタンスを渡せるようにコード変えてみたのですが同じエラーが出てしまいました。
```
def create
file_genre = Genre.find(file_params[:genre])
@upload_file = UploadFile.new(file: file_params[:file], lesson_id: file_params[:lesson_id], genre: @file_genre.id, to_group_id: file_params[:to_group_id])
if @upload_file.save
redirect_to teacher_uploaders_path, notice: "ファイルを送信しました"
else
flash.now[:error] = '送信に失敗しました'
render :new
end
end
```
どこが間違ってるか分かりますかね、、？
インスタンスを渡すにはどうすればよろしいのでしょうか。