deviseで作成したモデルと元々作っていたモデルの紐づけがうまくいかない

Question

タイトルの通りbelongs_toとhas_oneを使用してモデルの紐づけを行おうとしているのですがうまく動いてくれません

・やろうとしていること
rails g devise Account
で作成したAccountモデルとマイクロポストの内容を保存していたPostモデルを紐づけてAccountモデルからPostモデルの内容を呼び出せるようにしたい

account.rb

class Account < ApplicationRecord
  belongs_to :post
  accepts_nested_attributes_for :post
  # Include default devise modules. Others available are:
  # :confirmable, :lockable, :timeoutable, :trackable and :omniauthable
  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :validatable
  
  #Emailバリデーション
  VALID_EMAIL_REGEX = /\A[\w+\-.]+@[a-z\d\-.]+.[a-z]+\z/i
  validates :email, presence: true, uniqueness: true, format: { with: VALID_EMAIL_REGEX }
  #Passwordバリデーション
  VALID_PASSWORD_REGEX = /\A(?=.*?[a-z])(?=.*?\d)[a-z\d]+\z/i
  validates :password, presence: true, length: { minimum: 7 }, format: { with: VALID_PASSWORD_REGEX }
end

post.rb

class Post < ApplicationRecord
    has_one :account
end

上記のように記載してAccount.find(1).Postと入力してもNo method errorとなってしまいPostが見つからないと言われてしまいます
うまくいかない理由や別のやり方について教えていただけると幸いです

20201109075604_create_posts.rb

class CreatePosts < ActiveRecord::Migration[5.1]
  def change
    create_table :posts do |t|
      t.text :content

      t.timestamps
    end
  end
end

Answer 1

恐らくですが、逆になっているように思います。

class Account < ApplicationRecord
  has_one :post
〜省略〜
end

class Post < ApplicationRecord
    belongs_to :account
end

如何でしょうか？

Comments

[morimoriservice]

回答ありがとうございます。
試してみましたがやっぱりNo method errorになりますね

[tomtomtomtom]

Account.find(1).Post→Account.find(1).post
では如何でしょうか？

[morimoriservice]

また別のエラーらしきものが出ましたね
>> Account.find(1).post
Account Load (0.1ms) SELECT "accounts".* FROM "accounts" WHERE "accounts"."id" = ? LIMIT ? [["id", 1], ["LIMIT", 1]]
Post Load (0.5ms) SELECT "posts".* FROM "posts" WHERE "posts"."account_id" = ? LIMIT ? [["account_id", 1], ["LIMIT", 1]]
Traceback (most recent call last):
2: from (irb):6
1: from (irb):6:in `rescue in irb_binding'
ActiveRecord::StatementInvalid (SQLite3::SQLException: no such column: posts.account_id: SELECT "posts".* FROM "posts" WHERE "posts"."account_id" = ? LIMIT ?)

[tomtomtomtom]

なるほどですね。
post側のマイグレーションファイルに
t.references :account, foreign_key: true
などの記述があり、
postsテーブルにaccount_idカラムはありますか？

[morimoriservice]

マイグレーションファイルはcreate_table以外特に記載しておりません
マイグレーションファイルを追記いたしましたのでご確認お願い致します。
postsテーブルはfind(1)で見ましたがaccount_idカラムはないですね

[tomtomtomtom]

なるほどですね。
簡単に説明しますと、
Account.find(1)
でaccounts table からid=1のレコードを取得します。
これに紐づく、postレコードを取得したいのですが、
post tableにaccount_idカラムが無いので
紐付けが出来ない状態だと思います。
間違っていたらすいません。
ですので、まずは、カラムを追加して下さい。
そのためには、
```
t.references :account, foreign_key: true
t.text :content
```
などと記述して、カラムの追加をお願いします。
また、カラムが出来ましたら、
もちろんレコードが必要です。
レコードを登録して再度実行してみて下さい。
如何でしょうか？

[tomtomtomtom]

それと
mysqlをご使用でしょうか？
GUIはsequel pro でしょうか？
GUIを使用すると簡単にデータベースが確認できます^^

[morimoriservice]

ご丁寧な説明ありがとうございます
一度やってみますので結果がわかりましたらまた報告いたします

dbについてはsqlite3を使用しており私の環境だとCUIでの確認しかできないそうなのでそれで確認してみます

[morimoriservice]

やってみましたがやっぱり駄目ですね…
調べてみたところadd_index :posts, [:user_id, :created_at]というのも追加するらしくやってみたのですが、結果は変わりませんでした。

[tomtomtomtom]

そうですか
お力になれずに申し訳有りません。
githubのurlなど分かればこちらのlocalにダウンロードして検証してみたいところですが、、、
add_indexはテーブルにカラムを追加するメソッドです。
https://railsdoc.com/page/add_index
やはりpostsテーブルにuser_idのカラムを追加することは間違いないみたいですね

[morimoriservice]

いえいえ、色々とありがとうございました。
一応githubのurlを下記に記載しておきますので確認していただけるとありがたいです。
https://github.com/k1o0b1a9/myapp01

[tomtomtomtom]

只今検証中ですが、
気づいた点
・add_index :posts, [:user_id, :created_at]
↑これは
・add_index :posts, [:account_id, :created_at]
だと思います。morimoriさんはuserではなくaccountモデルなので

[tomtomtomtom]

それとcreated_atは紐付けには関係ないように思います^^

[morimoriservice]

ありがとうございます!
指摘箇所については修正いたします

[tomtomtomtom]

さて、こちらでは以下の方法で実現できました。

まずpostのマイグレートファイル
```
class CreatePosts < ActiveRecord::Migration[5.1]
def change
create_table :posts do |t|
t.text :content
t.references :account, foreign_key: true
t.timestamps
end
end
end

```
↑これにして、rails db:migrate:resetを実行
※dbがからになるので注意。

postコントローラー
```
Post.create(content: params[:text],account_id:current_account.id)

```
これで、account_idも一緒に保存しないと保存できません。

最後にaccountモデル。
```
has_one :post,dependent: :destroy
```
↑postにforeign_keyを設定したのでdependent: :destroyをしてやらないと消せません。

恐らくこれで出来上がると思います。
如何でしょうか？

[morimoriservice]

今やってみたのですが無事紐づいてくれました!!
本当にありがとうございます!

ただ、アカウント毎の投稿内容をtopページで表示させようとすると最初にサインインしたアカウント以外でエラーになってしまっていてなかなか解決できないので自分で解決できなそうでしたらまた質問させていただくと思います

今回の件は調べても正解がなかったので本当に助かりました、ありがとうございます。

[tomtomtomtom]

解決できてホッとしました^^
またよろしくおねがいします^^
※コントローラーではストロングパラメーターを使ったほうがよいですよ^^