質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.92%

メッセージ投稿がsaveされない

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 214

Slimeda

score 2

解決したいこと

メッセージを投稿したいのですが、エラーは出ずにelseの方が読み込まれてしまい、ifの方に読み込まれないです。
group,calendar,messageとゆう順番にrouteはネストされています。
calendarのindexでの下のeach文で@calendarsに@groupの値を入れないと表示がされなく、そのため、messagecontrollerで@calendars = @groupとしているのですが、これのせいでcreateでの@messageのあとの@calendarが@groupのidになっているのでしょうか?
私なりにいろいろ試してみたのですがelseの方が読み込まれるか、エラーが出てNo methodとでるので全然解決しないのでよければ教えていただきたいです!

Messagescontroller

class MessagesController < ApplicationController
  before_action :set_group
  before_action :set_calendar

  def index
    @message = Message.new
    @messages = @calendar.messages.includes(:user)
    @calendars = @group
  end

  def create
    @calendars = @group
    @message = @calendar.messages.new(message_params)
    if @message.save
      redirect_to group_calendar_messages_path(@calendar, @calendar)
    else
      @messages = @calendar.messages.includes(:user)
      flash.now[:alert] = 'メッセージを入力してください。'
      render :index
    end
  end

  private

  def message_params
    params.require(:message).permit(:content, :image).merge(user_id: current_user.id)
  end

  def set_group
    @group = Group.find(params[:group_id])
  end

  def set_calendar
    @calendar = Calendar.find(params[:calendar_id])
  end
end

messageのリンクにとぶためのcalendar#index

.wrapper
  .hoom
    .center
      .center__top
        %h2.center__top__name
          = current_user.name
      .center__naka
        %h5.center__naka__date
          Schedule
        .center__naka__new
          = link_to new_group_calendar_path do
            = icon('fas', 'calendar-plus', class:'new')
      .center__bottom
        - @calendars.calendars.each do |calendar|
          .center__bottom__list
            = link_to group_calendar_messages_path(calendar.group, calendar), class:"a" do
              .center__bottom__list__name
                = calendar.date

message#index

.hoom
  .light
    .main
      .main__footer
        = form_with model: [@group, @calendar, @message], html: {class: "form"}, local: true do |f|
          .input-box
            = f.text_field :content, class: 'input-box__text', placeholder: 'message'
            = f.label :image, class: 'input-box__image' do
              = icon('fas', 'camera', class: 'input-box__icon')
              = f.file_field :image, class: 'input-box__image__file'
          = f.submit 'Go', class: 'submit__btn'
      .main__header
        %h4.main__header__group
          = @group.name
        .main__header__edit
          = link_to edit_group_path(@group) do
            = icon('fas', 'cog', class:'edit')
      .main__calendar
        .main__calendar__content
          = render @messages

message#indexのrenderの先のリンク

.box
  .box__username
    = message.user.name
  .box__message
    - if message.content.present?
      %p.box__message__content
        = message.content
    = image_tag message.image.url, class: 'box__message__image' if message.image.present?

message#model

class Message < ApplicationRecord
  belongs_to :user
  belongs_to :group
  belongs_to :calendar

  validates :content, presence: true, unless: :image?

  mount_uploader :image, ImageUploader
end

message#migrate

class CreateMessages < ActiveRecord::Migration[6.0]
  def change
    create_table :messages, id: :integer do |t|
      t.string :content
      t.string :image
      t.integer :group_id, foreign_key: true
      t.integer :calendar_id, foreign_key: true
      t.integer :user_id, foreign_key: true
      t.timestamps
    end

  end
end

group#model

class Group < ApplicationRecord
  has_many :group_users
  has_many :users, through: :group_users
  has_many :calendars
  has_many :messages

  validates :name, presence: true, uniqueness: true
end

group_user#model

class GroupUser < ApplicationRecord
  belongs_to :group
  belongs_to :user
end

calendar#model

class Calendar < ApplicationRecord
  belongs_to :user
  belongs_to :group
  has_many :messages

  validates :date, presence: true
end

user#model

class User < ApplicationRecord
  # Include default devise modules. Others available are:
  # :confirmable, :lockable, :timeoutable, :trackable and :omniauthable
  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :validatable

  has_many :group_users
  has_many :groups, through: :group_users
  has_many :messages
  has_many :calendars
end

group#migate

class CreateGroups < ActiveRecord::Migration[6.0]
  def change
    create_table :groups do |t|
      t.string :name, null: false
      t.index :name, unique: true
      t.timestamps
    end
  end
end

user#migrate

class DeviseCreateUsers < ActiveRecord::Migration[6.0]
  def change
    create_table :users do |t|
      ## Database authenticatable
      t.string :name,               null: false
      t.string :email,              null: false, default: ""
      t.string :encrypted_password, null: false, default: ""

      ## Recoverable
      t.string   :reset_password_token
      t.datetime :reset_password_sent_at

      ## Rememberable
      t.datetime :remember_created_at

      ## Trackable
      # t.integer  :sign_in_count, default: 0, null: false
      # t.datetime :current_sign_in_at
      # t.datetime :last_sign_in_at
      # t.string   :current_sign_in_ip
      # t.string   :last_sign_in_ip

      ## Confirmable
      # t.string   :confirmation_token
      # t.datetime :confirmed_at
      # t.datetime :confirmation_sent_at
      # t.string   :unconfirmed_email # Only if using reconfirmable

      ## Lockable
      # t.integer  :failed_attempts, default: 0, null: false # Only if lock strategy is :failed_attempts
      # t.string   :unlock_token # Only if unlock strategy is :email or :both
      # t.datetime :locked_at


      t.timestamps null: false
    end

    add_index :users, :name, unique: true
    add_index :users, :email,                unique: true
    add_index :users, :reset_password_token, unique: true
    # add_index :users, :confirmation_token,   unique: true
    # add_index :users, :unlock_token,         unique: true
  end
end

calendar#migrate

class CreateCalendars < ActiveRecord::Migration[6.0]
  def change
    create_table :calendars do |t|
      t.string :date
      t.references :group, foreign_key: true
      t.references :user, foreign_key: true
      t.timestamps
    end
  end
end

補足

何かお気付きの箇所などありましたらご助言いただけると助かります。他のデータも必要でしたら載せますのでおっしゃってください。
お願いいたします!
```

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

0

変数名は見てなにだかわかるような名前にしてくれませんか

  1. @message = Message.new
  2. @messages = @calendar
  3. @messages = @calendar.messages.includes(:user)
  4. @calendars = @group

1: @messageはMessageのインスタンスだというのですからこれは普通
2: @messagesはCalendarのインスタンス?なの?
3: @messagesはCalendarに紐付いたmessage復数 これはわかる
4: 復数のCalendarはGroupのインスタンスである???
Groupってなに?

でないとcodeが読み解けません

追記
class Message < ApplicationRecord   belongs_to :user   belongs_to :group   belongs_to :calendar
とあるので、 user_id、group_id、calendar_id に実在するそれらのidが入っている必要があります。
しかし message_params には user_id しかありません。
group_id、calendar_id も入れるようにしてください

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/07/30 20:55

    komasan1さんご回答ありがとうございます!
    save1にしたところバリデーションに失敗しました:Groupを入力してくださいとゆうエラーが発生しました。
    何か載せるものがあったら載せさせていただくのでお願いします!

    キャンセル

  • 2020/07/31 12:41

    > しかし message_params には user_id しかありません。
    > group_id、calendar_id も入れるようにしてください

    エラーでもgroup_idがないようなので、回答者の方がおっしゃっているようにgroup_idを入れて保存すればいいです。

    キャンセル

  • 2020/08/13 10:54

    解決いたしました!
    winterboumさんとkomasan1さんありがとうございます!
    referenceだけの新しいmigrationファイルを作りその中にcalendarとuserをmessageに紐付け、groupはアソシエーションを外すことによって解決いたしました!
    長い時間かかりましたが1番最初に回答してくださったwinterboumさんをベストアンサーにさせていただきました!

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.92%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る