ActionView::Template::Error (undefined method `strftime' for nil:NilClass):

Question

前提・実現したいこと

作成したアプリケーションをherokuにデプロイし、本番環境でエラーがでるページがあります。具体的には、current_sign_in_at.strftimeを書いているページです（current_sign_in_at.strftimeの記述を消せばエラーは出ない）。

しかし、最終ログインの表示をどうしてもさせたいのでcurrent_sign_in_at.strftimeを使いたいのですが、どのようにすればよろしいでしょうか。ログは以下に載せておきます。

#####heroku logs コマンド実行後のログ

#####herokuにデプロイ後のログ


![]

該当のソースコード

【エラーが出るページ】

<% @users.each do |user|%>
    <div class="hidden wid list1-1">
        <% if user.grade == 1 && user.class_number == 1%>
        <%= user.student_number%>番
        <%=link_to new_user_diary_path(user) do %>
        <span class="user-name"><%= user.name%></span>
        <% end %>
            <% @login = user.current_sign_in_at.strftime('%m月%d日 %H:%M') if user && user.created_at%>
            <%if @login < Date.current.ago(6.days || 7.days || 8.days)%>
            <span class="login-day">５日間以上</span>
            <%elsif @login < Date.current.ago(5.days)%>
            <span class="login-day">５日前</span>
            <%elsif @login < Date.current.ago(4.days)%>
            <span class="login-day">４日前</span>
            <%elsif @login < Date.current.ago(3.days)%>
            <span class="login-day">３日前</span>
            <%elsif @login < Date.current.ago(2.days)%>
            <span class="login-day">２日前</span>
            <%elsif @login < Date.current.ago(1.days)%>
            <span class="login-day">１日前</span>
            <%elsif @login = Date.today%>
            <span class="login-day-today"></span>
            <%end%>
        <% end %>
    </div>
    <% end %>

user.rbにtrackableは追加しています。

class User < ApplicationRecord

  devise :database_authenticatable, :registerable,
         :trackable, :recoverable, :rememberable, :validatable

省略

end

usersのマイグレーションファイルでtrackableをコメントアウト解除しています。

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

省略

      # 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

試したこと

heroku run rails db:migrate

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

Rails 6.0

Answer 1

まだ一度もloginしていないUserがあるのではないですか？
とすると
<% @login = user.current_sign_in_at.strftime('%m月%d日 %H:%M') if user && user.created_at%>
のところ
<% @login = user.current_sign_in_at.strftime('%m月%d日 %H:%M') if ser.current_sign_in_at%>
でしょう。
userインスタンスがあるということは必ずcreated_atがありますから、そのチェックは意味無いです。
また
@users.each do |user| の @users はDBからの検索でしょうから、nilがはいっては居ないでしょうから user のチェックも意味無いです

Comments

[moto12]

仰る通りでした。あれば表示させて、無ければ表示させないというだけの問題ですね。大変助かりました！