Rails モデル同士で紐づけをしたのにメソッドがビューで使えません

Question

発生している問題・エラーメッセージ

・MicropostモデルとUserモデルを紐づけたが、userメソッドが使えない
↓がエラーメッセージです。

NoMethodError in Microposts#index
Showing /home/ubuntu/environment/rails_app/app/views/microposts/_micropost.html.erb where line #3 raised:

undefined method `user' for #<Class:0x00007fab78d47418>
Did you mean?  super

app/views/microposts/_micropost.html.erb:3:in `_app_views_microposts__micropost_html_erb__4579145426954223205_70304152209200'
app/views/microposts/index.html.erb:2:in `block in _app_views_microposts_index_html_erb___1913828141426248439_70304152043480'
app/views/microposts/index.html.erb:1:in `_app_views_microposts_index_html_erb___1913828141426248439_70304152043480'

ActionView::Template::Error (undefined method `user' for #<Class:0x0000565271879460>
Did you mean?  super):
    1: <% micropost = Micropost %>
    2: <li id="micropost-<%= micropost.ids %>">
    3:   <%= link_to gravatar_for(micropost.user, size: 50), micropost.user %>
    4:   <span class="user"><%= link_to micropost.user.name, micropost.user %></span>
    5:   <span class="content"><%= micropost.content %></span>
    6:   <span class="timestamp">
  
app/views/microposts/_micropost.html.erb:3:in `_app_views_microposts__micropost_html_erb__4579145426954223205_70304152209200'
app/views/microposts/index.html.erb:2:in `block in _app_views_microposts_index_html_erb___1913828141426248439_70304152043480'
app/views/microposts/index.html.erb:1:in `_app_views_microposts_index_html_erb___1913828141426248439_70304152043480'
  Rendering /home/ubuntu/.rvm/gems/ruby-2.6.3/gems/actionpack-5.0.7.2/lib/action_dispatch/middleware/templates/rescues/template_error.html.erb within rescues/layout
  Rendering /home/ubuntu/.rvm/gems/ruby-2.6.3/gems/actionpack-5.0.7.2/lib/action_dispatch/middleware/templates/rescues/_source.html.erb
  Rendered /home/ubuntu/.rvm/gems/ruby-2.6.3/gems/actionpack-5.0.7.2/lib/action_dispatch/middleware/templates/rescues/_source.html.erb (3.8ms)
  Rendering /home/ubuntu/.rvm/gems/ruby-2.6.3/gems/actionpack-5.0.7.2/lib/action_dispatch/middleware/templates/rescues/_trace.html.erb
  Rendered /home/ubuntu/.rvm/gems/ruby-2.6.3/gems/actionpack-5.0.7.2/lib/action_dispatch/middleware/templates/rescues/_trace.html.erb (1.6ms)
  Rendering /home/ubuntu/.rvm/gems/ruby-2.6.3/gems/actionpack-5.0.7.2/lib/action_dispatch/middleware/templates/rescues/_request_and_response.html.erb
  Rendered /home/ubuntu/.rvm/gems/ruby-2.6.3/gems/actionpack-5.0.7.2/lib/action_dispatch/middleware/templates/rescues/_request_and_response.html.erb (0.6ms)
  Rendered /home/ubuntu/.rvm/gems/ruby-2.6.3/gems/actionpack-5.0.7.2/lib/action_dispatch/middleware/templates/rescues/template_error.html.erb within rescues/layout (17.7ms)

該当のソースコード

app/model/User.rb

class User < ApplicationRecord
 has_many :microposts, dependent: :destroy
|
|
end

app/model/Micropost.rb

class Micropost < ApplicationRecord
belongs_to :user
|
|
end

class MicropostsController < ApplicationController

 def index
   @microposts=Micropost.all
 end
end

/app/view/microposts/index.html.erb

<[e-mail] do |post|%>
  <%= render "microposts/micropost" %>
<%end%>
 <%= paginate @microposts %> 

/app/view/microposts/_micropost.html.erb

<% micropost = Micropost %>
<li id="micropost-<%= micropost.id %>">
  <%= link_to gravatar_for(micropost.user, size: 50), micropost.user %>
  <span class="user"><%= link_to micropost.user.name, micropost.user %></span>
  <span class="content"><%= micropost.content %></span>
  <span class="timestamp">
    <%= time_ago_in_words(micropost.created_at)%> 前
  </span>
</li>

試したこと

binding.pryを使ってコンソール内でいくつか検証してみました。
Micropost.first.user⇒ユーザー参照できました。
micropost.first.userも同様です。

補足情報（FW/ツールのバージョンなど）

cloud9使用
Rails 5.0.7.2

助けていただけるとありがたいです。お願いします。

Answer 1

<% micropost = Micropost %>
...
<%= link_to ..., micropost.user %>

このように書いてありますがMicropostのクラスメソッドとしてuserがあるのですか？

module MicropostsHelper
  def user
    return User.find_by(id: self.user_id)    
  end
end

としてありますが、ここでのselfはMicropostクラスでもクラスインスタンスでもないのは理解していますか？

多分これで動くかな？(未検証)

<ul>
<%= render @microposts %>
</ul>
<%= paginate @microposts %> 

<li id="micropost-<%= micropost.id %>">
  <%= link_to gravatar_for(micropost.user, size: 50), micropost.user %>
  <span class="user"><%= link_to micropost.user.name, micropost.user %></span>
  <span class="content"><%= micropost.content %></span>
  <span class="timestamp">
    <%= time_ago_in_words(micropost.created_at)%> 前
  </span>
</li>

Comments

[k_yusuke]

>このように書いてありますがMicropostのクラスメソッドとしてuserがあるのですか？
すみません、理解不足でした。userメソッドはbelong_to user で定義されたMicropostのメソッドだったようです...
>としてありますが、ここでのselfはMicropostクラスでもクラスインスタンスでもないのは理解していますか？
理解できていないと思います。
selfはメソッドのレシーバーに代入されるのかなという認識でした。
この場合のselfは何を指しているのでしょうか？もしかして空っぽだったりしますか？

[rhiroe]

> この場合のselfは何を指しているのでしょうか？

実際に出力してみれば早い話ですが
#<#<Class:0x0000...9c0>:0x0000...da0>
こんな感じになっています。
あんまりModule単体でのselfを考えないのでようわかりませんが
Module自体はクラスではないがサブクラスにClassクラスを持つので
そこが呼ばれているのかな...と言う感じです。

[k_yusuke]

回答ありがとうございます。selfが何を指すのか深く考えれていなかったように思います。
クラスやサブクラスについて理解が足りないので恥ずかしいです。ちゃんと勉強します。

[rhiroe]

あとは
<% micropost = Micropost %>
としているので
Micropostクラスインスタンスであるべきmicropost変数が
Micropostクラスそのものになっています。

> userメソッドはbelong_to user で定義されたMicropostのメソッドだったようです...

とおっしゃっているuserというメソッドはMicropostのインスタンスメソッドなので
Micropostクラスインスタンスに対して使用する必要があります。
ちなみにMicropostクラスそのものに対して使われるメソッドはクラスメソッドです。

クラスインスタンスは例えば Micropost.new で生成されるオブジェクトです。
クラスそのものとは、そのままMicropostそのものです。

Micropost.new.method で使われるようなものがインスタンスメソッド
Micropost.method で使われるようなものがクラスメソッドです。

[k_yusuke]

なるほど！めちゃくちゃわかりやすいです！
今回の場合<% micropost = Micropost %>と書いてしまうと、micropost.userという記述のuserメソッドのレシーバーはMicropostクラスそのものになってしまうんですね...!

Answer 2

そのエラーがどのファイルのどの行で起きたかという情報が有ったはずです、それが判るようにして頂けると、確実性が上がります。

まず確認ですが、
class　Micropost に 　belongs_to user が定義されていますね？
＞　Micropost.first.user⇒ユーザー参照できました。
＞　micropost.first.userも同様です。
ここのuserはその定義で作成された、Micopostのmethodです。
示されたviewのcodeはみなこのuserを指しているように読めるので、ここでエラーがでる？　のが不思議です。

NoMethodError in Microposts#index　とありますので、エラーは　app/controllers/microposts_controller.rbにあるのではないでしょうか

あと、勘違いされているかも、なので追記
app/helpers/micropost_helper.rbに定義するmethodは Misropostのmethodではないです。viewのmethodです。
ですので
return User.find_by(id: self.user_id)
のselfはMicropostではありません。

selfをmicropostとして、User.find_by(id: self.user_id)　と同じ結果を得るには
micropost.user でよいです。

Comments

[k_yusuke]

ご回答ありがとうございます！
まず、お役に立てるかわかりませんが質問文に記載してあるmicropostパーシャルの転送前を記載しておきます
```Indexhrml
<[e-mail] do |post|%>
<%= render "microposts/micropost" %>
<%end%>
<%= paginate @microposts %>
```
MicropostControllerでは、indexアクション内で`@microposts=Micropost.all`と記載しています。

エラーがあったのは下記場所です。３行目はgravatarの行なのですがgarvatarを削除しても次のuserでエラーが出てしまうため、gravatarのエラーである可能性はなさそうです。
```
Showing /home/ubuntu/environment/rails_app/app/views/microposts/_micropost.html.erb where line #3 raised:
```
MicropostClassのbelong_to,UserClassのhas_manyは記載されています。
またMicropostのメソッドについて理解不足だったのでググってきました。僕が実現したいのはMicropostのメソッドを用いてmicropost.user⇒記事に紐づいたユーザー参照、という動きなんだなと理解しました
なので、わざわざヘルパーでメソッドを定義する必要はなさそうですね...

すみません、長くなってしまいましたが引き続きご教授お願いします

[winterboum]

文章ではなく、コードで相談の本文に載せて頂けませんか。取捨選択なしで。
質問者と回答者では着眼点が大抵の場合違いますから、元情報がないと難しいです

現時点で何が問題なのか、とそれに関わる コード、エラーメッセージを載せて整理しなおして頂けますか

[k_yusuke]

本当にすみません..追記しなおしました！
足りないコードがあれば言ってください！

[winterboum]

エラーを全文。
ファイルと行番号のある長いのが着いてるとおもうのですが

[winterboum]

app/view/microposts/_micropost.html.erb　に
<li id="micropost-<%= micropost.id %>">
がありますが、microlost が未定義にみえるのですが、そのエラーがない？
これはどこに定義してありますか

[k_yusuke]

返信遅くなりました！
エラーメッセージ記載しました。
micropostなんですがapp/view/microposts/_micropost.html.erbの１行目が抜けてしまっていました。
最初micropostが未定義だといわれたので、原因を探るためにビューの１行目にとりあえず置いておいたものです。