checkboxで取得した複数の値を、一つのフィールドに保存する方法を教えてください

Question

rails初心者です。
DBはmySQLを使っています。

チェックボックスで複数選択をしても、最後の値しかデータベースに入っていない状況です。
例えばsat4,sun4という形で表示されて欲しいのにデータベースにはsun4しか表示されていません。

checkboxで取得した複数の値を、一つのフィールドに保存する方法を教えてください。

teachers/edit.html.erb

            <table>
              <tr>
                <td>曜日/時間</td>
                <td>月曜</td>
                <td>火曜</td>
                <td>水曜</td>
                <td>木曜</td>
                <td>金曜</td>
                <td>土曜</td>
                <td>日曜</td>
              </tr>
              <tr>
                <td>午前中</td>
                <td><input type="checkbox" name="time[]" value="mon1"></td>
                <td><input type="checkbox" name="time[]" value="tue1"></td>
                <td><input type="checkbox" name="time[]" value="wed1"></td>
                <td><input type="checkbox" name="time[]" value="thu1"></td>
                <td><input type="checkbox" name="time[]" value="fri1"></td>
                <td><input type="checkbox" name="time[]" value="sat1"></td>
                <td><input type="checkbox" name="time[]" value="sun1"></td>
              </tr>
              <tr>
                <td>13:00~16:00</td>
                <td><input type="checkbox" name="time[]" value="mon2"></td>
                <td><input type="checkbox" name="time[]" value="tue2"></td>
                <td><input type="checkbox" name="time[]" value="wed2"></td>
                <td><input type="checkbox" name="time[]" value="thu2"></td>
                <td><input type="checkbox" name="time[]" value="fri2"></td>
                <td><input type="checkbox" name="time[]" value="sat2"></td>
                <td><input type="checkbox" name="time[]" value="sun2"></td>
              </tr>
              <tr>
                <td>16:00~19:00</td>
                <td><input type="checkbox" name="time[]" value="mon3"></td>
                <td><input type="checkbox" name="time[]" value="tue3"></td>
                <td><input type="checkbox" name="time[]" value="wed3"></td>
                <td><input type="checkbox" name="time[]" value="thu3"></td>
                <td><input type="checkbox" name="time[]" value="fri3"></td>
                <td><input type="checkbox" name="time[]" value="sat3"></td>
                <td><input type="checkbox" name="time[]" value="sun3"></td>
              </tr>
              <tr>
                <td>19:00~22:00</td>
                <td><input type="checkbox" name="time[]" value="mon4"></td>
                <td><input type="checkbox" name="time[]" value="tue4"></td>
                <td><input type="checkbox" name="time[]" value="wed4"></td>
                <td><input type="checkbox" name="time[]" value="thu4"></td>
                <td><input type="checkbox" name="time[]" value="fri4"></td>
                <td><input type="checkbox" name="time[]" value="sat4"></td>
                <td><input type="checkbox" name="time[]" value="sun4"></td>
              </tr>
            </table>

def new
    @teacher = Teacher.new
  end
  
  def show
    @teacher = Teacher.find_by(id: params[:id])
  end
  
  def create
    @teacher = Teacher.new(teacher_params)
    if @teacher.save
      flash[:notice] = "家庭教師登録が完了しました"
      redirect_to("/teachers/#{@teacher.id}")
    else
      render("teachers/new")
    end
  end
  
  def edit
    @teacher = Teacher.find_by(id: params[:id])
  end
  
  # updateアクションを作成してください
  def update
    @teacher = Teacher.find_by(id: params[:id])
    
    if params[:image]
      @teacher.img = "#{@teacher.id}.jpg"
      image = params[:image]
      File.binwrite("public/teacher_images/#{@teacher.img}", image.read)
    end
    
    if @teacher.update(teacher_params)
      flash[:notice] = "ユーザー情報を編集しました"
      redirect_to("/teachers/#{@teacher.id}")
    else
      flash[:notice] = "編集できませんでした"
      render("teachers/edit")
    end
  end
  
  def login_form
  end
  
  def login
    @teacher = Teacher.find_by(email: params[:email])
    if @teacher && @teacher.authenticate(params[:password])
      session[:teacher_id] = @teacher.id
      flash[:notice] = "ログインしました"
      redirect_to("/")
    else
      @error_message = "メールアドレスまたはパスワードが間違っています"
      @email = params[:email]
      @password = params[:password]
      
      render("teachers/login_form")
    end
  end
  
  def logout
    session[:teacher_id] = nil
    flash[:notice] = "ログアウトしました"
    redirect_to("/login")
  end
  
  # ensure_correct_userを定義してください
  def ensure_correct_teacher
    if @current_teacher.id != params[:id].to_i
      flash[:notice] = "権限がありません"
      redirect_to("/")
    end
  end
  
  private
  def teacher_params
    params.permit(:name, :email, :password, :nickname, :gender, :university, :wage, :trafficwage, :time, :prefecture, :region, :profile, :message)
  end
end

Answer 1

ちょっと全体的にぼやっとしててわかりにくいんですが、controllerでどんな処理をしているのか？
paramsの中身に何が入っているのか？等、やったことをもう少し詳しく書いていただけるとわかりやすいかなと思います。

本題ですが、chekboxのnameがすべて同じになっているので、最後にチェックした値しかパラメータとしてcontrollerには飛んでないのではないでしょうか？？

viewからcontrollerへチェックしたもの全て渡したいのであれば、
nameを一意にして上げれば飛ぶはずです。

追記

すみません、説明が悪かったです。
一つ例として<td><input type="checkbox" name="time[day_of_the_week][]" value="mon1"></td>のような感じで、全てのinputでnameをこの形に変更してください。

そして、コントローラのdef teacher_params を以下のようにします。

  def teacher_params
    params.permit(:name, :email, :password, :nickname, :gender, :university, :wage, :trafficwage, time: [], :prefecture, :region, :profile, :message)
  end

すると、 teache_paramsのtime内にチェックした値全てが入るかと思います

Comments

[morimi714]

すみません、情報不足だったため、コントローラーの中身を追加しました。おっしゃっているのは、:nameだけではいけないということでしょうか？？

[mot93]

コードを書いたので上記に追記しております
確認してみてくださいー

[morimi714]

回答ありがとうございます！試してみたのですがコントローラ内の
time: []　ですとsyntax errorとなったので:time[]と直したのですが、それだとargument errorとなり引数に何も入っていない状態になってしまうそうです・・・(T . T)

でもちゃんとリクエストに"time"=>{"day_of_the_week"=>["sat4", "sun4"]},と入っています

[mot93]

すみません。
permit時にtime: []を一番最後にする必要がありました。

[morimi714]

解決しました＾＾ありがとうございます！