新しいアカウントでNoMethodErrorが発生するステートメント

Question

私はユーザーが1日1回投稿を作成できるようにするプロジェクトに取り組んでいます。ユーザーがその日に既に投稿をしている場合は、表示しない投稿を送信するためのUI要素が必要です。これを行うには、私は使用しました：

<% if current_user.posts.last.date.past? %> 

私のhtml.erbファイル。しかし、この問題は、ユーザが投稿をしたことがない場合、nilの未定義のメソッドdate（Nilクラスエラー）が発生することです。私はこれを解決するために何をすべきか分からないのですか？

Answer 1

最速1は、Ruby 2.3.0で提示safe navigation演算子、使用することです：あなたはルビー< 2.3を使っているのであれば、あなたはtryに行くことができます

<% if current_user.posts.last&.date&.past? %> 

を：

<% if current_user.posts.last.try(:date).try(:past?) %> 

投稿したユーザーにのみこの条件を使用するようにしたい場合：

class User 
    def post_date_past? 
    return false unless posts.any? 
    posts.last.date.past? 
    end 
end 

ビューでそれを使用する：最後のポストのための

if current_user.post_date_past? 

Answer 2

チェックを日付をチェックする前に。

<% if current_user.posts.last.nil? || current_user.posts.last.date.past? %>

それが真の解決の最後のポストがない場合は、それ以外の場合は、過去の日付をチェックします。

Answer 3

これはトラックでLaw of Demeterを繰り返し実行するようなものです。そのチェーンのどこかに、ユーザーが投稿を持っているかどうかを実際に確認する必要があります。

tryまたはsafe navigation演算子を使用できますが、コードにはいくつかのデカップリングが有効です。

<% if current_user.posts.any? %> 
<% current_user.posts.last.tap do |post| %> 
    <% if post.date.past? %> 
    <% # ... %> 
    <% end %> 
<% end %> 
<% end %> 

一部activesupportのの涼しさでこれを行うための別の方法は、ブロックと.tryを使用することです。

<% current_user.posts.try(:last) do |post| %> 
    <% if post.date.past? %> 
    <% # ... %> 
    <% end %> 
<% end %> 

ブロックは、.try(:last)がnilを返さない場合にのみ呼び出されます。

​​

Answer 4

をごUserモデルクラス内部：ビューで次に

def has_past_post? 
    past_post = posts.last and past_post.date.past? 
    !!past_post 
end 

：あなたがそうのようにそれを行うことができますので、また

.any?は、ブロックを取る

<% if current_user.has_past_post? %> 

Answer 5

誰だので、ここに私のことを示唆している：

class User 
    # Runs the supplied block with user's last post. If user 
    # doesn't have a last post, then the block won't run. 
    def with_last_post 
    raise ArgumentError unless block_given? 
    posts.last && yield posts.last 
    end 

    # Runs the supplied block only if the user has posts 
    def having_posts 
    raise ArgumentError unless block_given? 
    yield if posts.any? 
    end 

    # Runs the supplied block if the user has no posts 
    def without_posts 
    raise ArgumentError unless block_given? 
    yield if posts.any? 
    end 
end 

<% current_user.with_last_post do |last_post| %> 
    <%= "last post in past!" if last_post.date.past? %> 
<% end %> 

<% current_user.having_posts do %> 
    <%= "last post in past" if current_user.posts.last.date.past? %> 
<% end %> 

<% current_user.without_posts do %> 
    You haven't posted anything! 
<% end %> 

それを行う意図正しい方法のような何かを行うことによって、コントローラ内のビューで必要な情報を収集するために、次のようになります。

# FooController: 
def show 
    @last_post = current_user.posts.last 
end 

# views/foo/show.html.erb : 
<%= render 'last_post', last_post: @last_post %> 

# views/foo/_last_post.html.erb : 
<% if @last_post %> 
    Last post: <%= @last_post.date %> 
<% else %> 
    You haven't posted anything ever. 
<% end %> 

またはヘルパーを使用して：

# app/helpers/foo_helper.rb 
module FooHelper 
    def user_last_post_date 
    last_post = current_user.posts.last 
    if last_post 
     last_post.date.past? "in the past" : "in the future(??!)" 
    else 
     "never" 
    end 
    end 
end 

# app/views/foo/show.html.erb 
Last post date: <%= user_last_post_date %>