This isn't really a method that should be in the controller - I'd probably put it in the work model. You can also make it a little more efficient by using .exists? since we don't need the reading object:

    def marked_for_later?(user)
      Reading.where(work_id: self.id, user_id: user.id, toread: true).exists?
    end