ruby-on-rails - これらのコードが期待どおりに機能しないのはなぜですか？

Question

私はcontrolloerを書きます

class NotificationsController < ApplicationController
  before_filter :authenticate_user!

  def index
    @notification = current_user.notification
    if @notification.unread == 0
      redirect_to root_path
    else
      @notification.unread == 0
      @notification.save
    end
  end
end

@notification.unreadインデックスページを表示した後は0になると思いますが、実際には機能しません。 これらのコードを変更して正しく機能させる方法。

あなたが私を助けてくれてありがとうございます:)

Answer 1

@notification.unread = 0代わりに使用してみてください@notification.unread == 0

Answer 2

あなたが何をしようとしているのか完全には理解できませんが、==を2回呼び出しています。これは比較演算子です。2番目のセクションでは値を設定したいので、=のみを使用する必要があります。

このような

class NotificationsController < ApplicationController
  before_filter :authenticate_user!

  def index
    @notification = current_user.notification
    if @notification.unread == 0
      redirect_to root_path
    else
      @notification.unread = 0
      @notification.save
    end
  end
end

Answer 3

   else
      @notification.unread = 0
      @notification.save
    end

@ notification.unread==0は属性の値を変更しません。:)

Answer 4

ビジネスロジックをモデルに移行することは常に良い考えです。したがって、これを次のように書くことができます。

class Notification < ActiveRecord::Base

  def unread?
    unread == 0
  end

end

class NotificationsController < ApplicationController
  before_filter :authenticate_user!

  def index
    @notification = current_user.notification
    @notification.unread? ? (@notification.update_attributes("unread", 0) : (redirect_to root_path))
  end
end