imageカラムが参照できない

Question

Actioncable + devise でリアルタイムチャットアプリを作成しています。

チャット画面で投稿する際に以下のエラーがでてしまい、画面リロードをしないと文字が表示されなくなりました。

Finished "/cable/" [WebSocket] for 192.168.33.1 at 2019-11-29 04:22:12 +0100
RoomChannel stopped streaming from room_channel_
Started GET "/cable" for 192.168.33.1 at 2019-11-29 04:22:12 +0100
Cannot render console from 192.168.33.1! Allowed networks: 127.0.0.0/127.255.255.255, ::1
Started GET "/cable/" [WebSocket] for 192.168.33.1 at 2019-11-29 04:22:12 +0100
Successfully upgraded to WebSocket (REQUEST_METHOD: GET, HTTP_CONNECTION: Upgrade, HTTP_UPGRADE: websocket)
User Load (0.2ms) SELECT "users".* FROM "users" WHERE "users"."id" = ? ORDER BY "users"."id" ASC LIMIT ? [["id", 1], ["LIMIT", 1]]
↳ app/channels/application_cable/connection.rb:12:in find_verified_user' User Load (0.2ms) SELECT "users".* FROM "users" WHERE "users"."id" = ? LIMIT ? [["id", 1], ["LIMIT", 1]] ↳ app/channels/application_cable/connection.rb:12:in find_verified_user'
Registered connection (Z2lkOi8vY2hhdGFwcC9Vc2VyLzE)
RoomChannel is transmitting the subscription confirmation
RoomChannel is streaming from room_channel_1
RoomChannel#speak({"message"=>"i have no idea"})
(0.1ms) begin transaction
↳ app/channels/room_channel.rb:12:in speak' User Load (0.1ms) SELECT "users".* FROM "users" WHERE "users"."id" = ? LIMIT ? [["id", 1], ["LIMIT", 1]] ↳ app/channels/room_channel.rb:12:in speak'
Room Load (0.1ms) SELECT "rooms".* FROM "rooms" WHERE "rooms"."id" = ? LIMIT ? [["id", 1], ["LIMIT", 1]]
↳ app/channels/room_channel.rb:12:in speak' Message Create (2.8ms) INSERT INTO "messages" ("content", "created_at", "updated_at", "user_id", "room_id") VALUES (?, ?, ?, ?, ?) [["content", "i have no idea"], ["created_at", "2019-11-29 03:22:23.549954"], ["updated_at", "2019-11-29 03:22:23.549954"], ["user_id", 1], ["room_id", 1]] ↳ app/channels/room_channel.rb:12:in speak'
(7.4ms) commit transaction
↳ app/channels/room_channel.rb:12:in speak' [ActiveJob] Enqueued MessageBroadcastJob (Job ID: 57f40449-6e0e-41c7-8dbc-efd67e142aac) to Async(default) with arguments: #<GlobalID:0x00007f2910d615b0 @uri=#<URI::GID gid://chatapp/Message/59>> Message Load (0.2ms) SELECT "messages".* FROM "messages" WHERE "messages"."id" = ? LIMIT ? [["id", 59], ["LIMIT", 1]] [ActiveJob] [MessageBroadcastJob] [57f40449-6e0e-41c7-8dbc-efd67e142aac] Performing MessageBroadcastJob (Job ID: 57f40449-6e0e-41c7-8dbc-efd67e142aac) from Async(default) enqueued at 2019-11-29T03:22:23Z with arguments: #<GlobalID:0x00007f2928034708 @uri=#<URI::GID gid://chatapp/Message/59>> [ActiveJob] [MessageBroadcastJob] [57f40449-6e0e-41c7-8dbc-efd67e142aac] Rendered messages/_message.html.erb (Duration: 6.9ms | Allocations: 1487) [ActiveJob] [MessageBroadcastJob] [57f40449-6e0e-41c7-8dbc-efd67e142aac] Error performing MessageBroadcastJob (Job ID: 57f40449-6e0e-41c7-8dbc-efd67e142aac) from Async(default) in 8.56ms: ActionView::Template::Error (undefined method image' for nil:NilClass):
/home/vagrant/environment/chatapp/app/views/messages/_message.html.erb:2:in _app_views_messages__message_html_erb__1902079675368871319_69907370882620' /home/vagrant/.rbenv/versions/2.6.4/lib/ruby/gems/2.6.0/gems/actionview-6.0.1/lib/action_view/base.rb:274:in _run'

ネットで調べ、controllerを変更したり、deviseの仕様についても調べましたが,解決しませんでした。

以下が関係していると感じているコードです。

rooms_controller.rb ↓

class RoomsController < ApplicationController

  def index   
    @user = current_user.id
    @rooms = Room.all.order(:id)
  end

  def show
    @user = User.find(params[:id])
    @room = Room.find(params[:id])
    @messages = @room.messages
  end
end

</code>

コード

routes.rb　↓

  namespace :admin do
    resources :users
  end
  
  devise_for :users, :controllers => {
    registrations: 'users/registrations'}


  root to: 'top#show'

  resources :rooms, :only => [:index, :show]

  devise_scope :user do
    get 'users/sign_out', :to => 'users/sessions#destroy' 
  end

  resources :users, :only => [:index, :show, :edit, :destroy]

end
コード

users_controller.rb　↓

    before_action :authenticate_user! 

    def show
      @user = current_user.id
    end

    def index
      @user = User.find(params[:id])
    end

    def edit
      @user = current_user.id
    end

    def update
    end

    def destroy
      @user = User.find(params[:id])
      if @user.destroy
      redirect_to admin_users_url
      flash[:notice] = "ユーザー#{@user.username}を削除しました。"    
      end
    end

end
コード

rooms show.html.erb　↓

  <h1>Chat room</h1>
  <div id='messages' data-room_id="<%= @room.id %>">
    <%= render @messages %>
  </div>
</div>

<div class="send-box">
  <%= text_field_tag :content, nil, data: { behavior: 'room_speaker' } %>
  <%= label_tag :content, 'Say something' %>
</div>

コード

_messages.erb　↓

  <% if @user.image.present? %>
    <%= image_tag(@user.image.thumb100.url) %>
  <% else %>
    <%= image_tag('no-image-icon-23500.jpg', :size => '100x100') %>
  <% end %>
  <p><%= "#{message.user.email}: #{message.content}" %></p>
</div>
コード

schema.rb ↓

    t.string "email", default: "", null: false
    t.string "encrypted_password", default: "", null: false
    t.string "reset_password_token"
    t.datetime "reset_password_sent_at"
    t.datetime "remember_created_at"
    t.datetime "created_at", precision: 6, null: false
    t.datetime "updated_at", precision: 6, null: false
    t.string "username"
    t.boolean "admin", default: false, null: false
    t.text "profile"
    t.string "image"
    t.index ["email"], name: "index_users_on_email", unique: true
    t.index ["reset_password_token"], name: "index_users_on_reset_password_token", unique: true
  end

コード

rooms index.html.erb ↓

<h1><%= link_to "#{current_user.username}さんのページ", user_path(@user) %></h1>
  <h3>Chatroom一覧</h3>
   <div>
     <ul>
       <% @rooms.each do |room| %>
        <li><%= link_to "ROOM#{room.id}", room_path(room.id, @user.id) %></li>
       <%  end %>
     </ul>
   </div>
  <h3>お気に入り</h3>
  <% if current_user.try(:admin?) %>
    <%= link_to "管理者画面", admin_users_path, class: 'btn' %>
  <% end %>
</div>
コード

schemaファイルにはimageカラムがあるのに何故エラーが出ているのでしょうか？

ご助力お願いします。

Comments

[winterboum]

codeが読みにくいので、<cdoe>でくくっていただけますか
エラーメッセージはその前後の情報も意味を持ちますので、それらも載せて下さい。

Answer 1

<% if @user.image.present? %>　の　@user がnilだと言われてます
def show
@user = User.find(params[:id])
@room = Room.find(params[:id])
と同じparams[:id]でuser,roomを取り出してますが、これがおかしいですね。
paramsにuserとroomと両方のidが渡るようにして下さい

Comments

[tagomaru025]

回答ありがとうございます。

申し訳ございませんでした。２つのidを渡すとの事だったので、idをpathに追加したり、whereやfind_byを調べ、試しました。しかし、以前と変わらず、imageが取得出来ませんでした。

大変恐縮なのですが、idを２つ渡す方法を教えて頂けないでしょうか？
お手を煩わせてしまい申し訳ございません。

[winterboum]

そのviewはどれでしょう？

[tagomaru025]

indexのviewを追記しましたので、ご確認お願いします。

[winterboum]

room_path(room.id, @user.id)　を　 room_path(room.id, user_id: @user.id) として、
@user = User.find(params[:id])　を　@user = User.find(params[:user_id])　に

[tagomaru025]

すみません。

教えて頂いた方法を試したのですが、以下のエラーが出てしまいました。

undefined method `id' for 1:Integer
Did you mean? i

<ul>
<% @rooms.each do |room| %>
<li><%= link_to "ROOM#{room.id}", room_path(room.id, user_id: @user.id) %></li>
<% end %>
</ul>
</div>

見にくくなってしまい申し訳ありません。<li>から始まる列です。
idが取れておらず、整数クラスに関係しているところまでは調べてわかったのですが、それ以上の情報が出てきませんでした。

何か解決策はあるでしょうか？

[winterboum]

もしかして、@userはUserのインスタンスではなく、そのidをいれてますか？
でしたら、　@user.id　ではなく　@user　で。
変数名は重要です、名前を見て勘違いしないような命名を心がけて下さい

[tagomaru025]

申し訳ございません。今後は変数名にも気を配ります。

@userで実装することが出来ました。
何度も教えて頂きまして本当にありがとうございました。