Ruby on Rails で新規投稿機能を実装したがdbに保存できない

Question

新規投稿を保存したいです

• 新規投稿機能を動作し、保存できるようにしたいです。

前提

Ruby on Railsで読書を習慣化できるようにタスク管理できるアプリを作ってます。
新規投稿ページで
タイトルと時間と内容をフォームに記述したら保存できるよ西谷のですができません。

該当のソースコード

app/views/reads/index.html.erb

<%= render "shared/header" %>

    <% if user_signed_in? %>
      <div class="greeting"> 
        <%= "こんにちは、#{current_user.nickname}さん!" %>
       </div>
    <% else %>
    <% end %>

<p id="notice"><%= notice %></p>

<div class="toppage">
<h1 class="top">カレンダー</h1>
<%= link_to 'イベント追加', new_read_path %>


<div class="col-md-8">
<%= month_calendar reads: @reads do |date, reads| %>
  <%= date.day %>

  <% reads.each do |read| %>
    <div>
      <%= link_to read.title, read %>
    </div>
  <% end %>
<% end %>
</div>
</div>
</div>

app/views/reads/new.html.erb

<%= form_with(model: @read, local: true) do |form| %>

  <div class="title">
    <%= form.label :title %>
    <%= form.text_field :title %>
  </div>

    <div class="time">
    <%= form.label :start_time %>
    <%= form.datetime_select :start_time %>
  </div>

  <div class="content">
    <%= form.label :content %>
    <%= form.text_field :content %>
  </div>


  <div class="submit">
    <%= form.submit %>
  </div>

<% end %>

config/routes

Rails.application.routes.draw do
  devise_for :users

  root to: "reads#index"
  resources :reads
end

app/controllers/reads_controller.rb

class ReadsController < ApplicationController

  def index
    @reads = Read.all
    @read = Read.new
  end
  
  def new
    @read = Read.new
  end

  def show
    @read = Read.find(params[:id])
  end

  def create
    @read = Read.new(read_parameter)

    if @read.save
      redirect_to reads_path, notice: '投稿が保存されました'
    else
      render 'new'
    end
  end

  def destroy
    @read = Read.find(params[:id])
    @read.destroy
    redirect_to reads_path, notice:"削除しました"
  end

  def edit
    @read = Read.find(params[:id])
  end

  def update
    @read = Read.find(params[:id])
    if @read.update(read_parameter)
      redirect_to reads_path, notice: "編集しました"
    else
      render 'edit'
    end
  end

  private

  def read_parameter
    params.require(:read).permit(:title, :content, :start_time)
  end

end

app/models/reads

class Read < ApplicationRecord
  belongs_to :user
  
  validates :title,   presence: true
  validates :content,     presence: true
  validates :start_time , presence: true
end

db/migrate/20231125061018_create_reads.rb

class CreateReads < ActiveRecord::Migration[7.0]
  def change
    create_table :reads do |t|
      t.string     :title,      null: false
      t.text       :content,    null: false
      t.datetime   :start_time, null: false
      t.references :user,       null: false


      t.timestamps
    end
  end
end

スペルミスなどは確認しましたが、特に間違ってるところはないです。
すみませんがよろしくお願いいたします。

Answer 1

モデルのsaveメソッドは保存できたかどうかをtrue/falseで返すだけなので何かの問題があっても気付けない事が多いです。

更新系のメソッドはすべて半角ビックリマーク付きのメソッドを使うようにすると良いです。
例: update!、save!
このメソッドは保存に失敗すると例外が発生して画面にも詳細が出るし、エラーログにも内容が出力されるので原因の特定がしやすくなります。
一方でバリデーションのエラーの時は例外を出したくないはずなので、valid?を使ってsave!/update!の前にバリデーションエラーの有無を確認して入力画面を表示するか判定する必要はあります。

  def create
    @read = Read.new(read_parameter)

    # バリデーションエラーがあれば入力画面を表示
    unless @read.valid?
       render :new
       return
    end

　 # save!で保存。何らかのエラーがあると例外が出る
    @read.save!
    redirect_to reads_path, notice: '投稿が保存されました'
  end

  def update
    @read = Read.find(params[:id])
    @read.attributes = read_parameter

    unless @read.valid?
       render :edit
       return
    end

    @read.update!
    redirect_to reads_path, notice: "編集しました"
  end

Comments

[satoukouhei]

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