railsでダイレクトメッセージの投稿者を表示させたい

Question

お世話になります。
投稿者の名前を表示させたいのですが、下記のようなエラーが出てしまい困っています。
https://gyazo.com/c374adda3994fd11c9b9123bc48c9019

<%= m.user.name%>が問題でそれを消せば問題なく表示されるので、
userの情報をうまく持ってこればよいということはわかるのですが持ってくる方法がわからないです。
###エラー

NoMethodError in Dmrooms#show
Showing /home/ec2-user/environment/game-friends6/app/views/dmrooms/show.html.erb where line #14 raised:
undefined method `name' for nil:NilClass

メッセージ自体はDB側で保存されているのは確認済みです

#logger.debug @dmroom.inspectでの結果
#<ActiveRecord::Associations::CollectionProxy [#<Dm id: 1, from_id: 2, dmroom_id: 58, content: "efafakjl;", created_at: "2019-10-31 12:02:02", updated_at: "2019-10-31 12:02:02">, #<Dm id: 2, from_id: 2, dmroom_id: 58, content: "efafafa", created_at: "2019-10-31 12:02:30", updated_at: "2019-10-31 12:02:30">, #<Dm id: 3, from_id: 2, dmroom_id: 58, content: "fjkal;fjaf;a", created_at: "2019-10-31 12:06:05", updated_at: "2019-10-31 12:06:05">]>

###該当箇所のモデルです

#dm.rb
class Dm < ApplicationRecord
  belongs_to :user, optional: true
  belongs_to :dmroom, optional: true
end


#dmroom.rb
class Dmroom < ApplicationRecord
  has_many :dms, dependent: :destroy
  has_many :entries, dependent: :destroy    
end

#entry.rb
class Entry < ApplicationRecord
  belongs_to :user
  belongs_to :dmroom
end

#user.rb
class User < ApplicationRecord
  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :validatable
  validates :name, presence: true
  has_many :dms, dependent: :destroy
  has_many :entries, dependent: :destroy
  
end

###該当箇所のコントローラーです

#dmrooms_controller
class DmroomsController < ApplicationController
  before_action :authenticate_user!
  def create
    @dmroom = Dmroom.create!
    logger.debug '+++++++++++++'
    logger.debug @dmroom.inspect
    logger.debug '+++++++++++++'
    @entry1 = Entry.create(dmroom_id: @dmroom.id, user_id: current_user.id)
    @entry2 = Entry.create(params.require(:entry).permit(:user_id, :dmroom_id).merge(dmroom_id: @dmroom.id))

    redirect_to "/dmrooms/#{@dmroom.id}"
  end

  def show
    @dmroom = Dmroom.find(params[:id])
    
    if Entry.where(:user_id => current_user.id, :dmroom_id => @dmroom.id).present?
      @dmessages = @dmroom.dms
      @dm = Dm.new
        logger.debug '+++++++++++++'
        logger.debug @dmessages.inspect
        logger.debug '+++++++++++++'
      @entries = @dmroom.entries
    
    else
      redirect_back(fallback_location: root_path)
    end
  end    
end

#dms_controller
class DmsController < ApplicationController
  before_action :authenticate_user!, :only => [:create]
  def create
    if Entry.where(:user_id => current_user.id, :dmroom_id => params[:dm][:dmroom_id]).present?
      @dm = Dm.create(params.require(:dm).permit(:from_id, :content, :dmroom_id).merge(:from_id => current_user.id))
      redirect_to "/dmrooms/#{@dm.dmroom_id}"
    else
      redirect_back(fallback_location: root_path)
    end
  end
end

###app/views/dmrooms/show.html.erb（一部）

<% if @dmessages.present? %>
  <% @dmessages.each do |m| %>
    <strong><%= m.content %></strong>
    <small>by <strong><a href="/users/<%= m.from_id %>"><%= m.user.name%>さん</a></strong></small>
    <hr>
  <% end %>
<% else %>
  <h3 class="text-center">メッセージはまだありません</h3>
<% end %>

<%= form_for @dm do |f| %>
  <%= f.text_field :content, :placeholder => "メッセージを入力して下さい" , :size => 70 %>
  <%= f.hidden_field :dmroom_id, :value => @dmroom.id %>
  <br>
  <%= f.submit "投稿する" %>
<% end %>

申し訳ありませんが、よろしくおねがいします

Answer 1

該当する dm.user_id に値が入っていないか、もしくはそのuserが作成できていないかのどちらかかと思います
belongs_to :user, optional: true　
となっているのでそういうデータの存在が出来てしまいます。
ユーザのないDmが有っても良いなら
<%= m.user.try(:name) %>
ありえないなら　optional: true　を削除して、userを必須にして下さい

Comments

[p-plus]

ありがとうございます、とりあえずおまじないのようにoptionalつけていましたが
あまり良くないようですね・・・

Answer 2

直感的にこんな感じですかね?

<%= m.user.name %>
↓
<%= User.find(m.from_id).name %>

Comments

[p-plus]

ありがとうございます、こちらで解決いたしました！
ベストアンサーとさせていただきます