0

我正在将邮箱 gem 实现到我的应用程序中，并认为这应该可以工作，但出现上述错误，我认为这与 ||= 运算符有关

我得到这个

    conversations_controller.rb:16: formal argument cannot be an instance variable def     

trash_folder @trash ||= current_user.mailbox.trash.all end ^    

/home/action/booklist/app/controllers/conversations_controller.rb:16: syntax error,      

unexpected tOP_ASGN, expecting ';' or '\n' def trash_folder @trash ||=  

current_user.mailbox.trash.all end ^ 

/home/action/booklist/app/controllers/conversations_controller.rb:18: syntax error,  unexpected '.', expecting ';' or '\n' def trash conversation.move_to_trash(current_user) 

... ^ /home/action/booklist/app/controllers/conversations_controller.rb:18: syntax  

error, unexpected tIDENTIFIER, expecting end-of-input ...rash(current_user) redirect_to 

:conversations end ... ^

对话控制器：

class ConversationsController < ApplicationController

helper_method :mailbox, :conversation


def index
@conversations ||= current_user.mailbox.inbox.all
end


 def reply
  current_user.reply_to_conversation(conversation, *message_params(:body, :subject))
 redirect_to conversation
 end

def trash_folder     @trash ||= current_user.mailbox.trash.all   end

def trash  conversation.move_to_trash(current_user)  redirect_to :conversations end 

def untrash  conversation.untrash(current_user)  redirect_to :back end

def empty_trash   current_user.mailbox.trash.each do |conversation|       conversation.receipts_for(current_user).update_all(:deleted => true)
 end
redirect_to :conversations
end


private

def mailbox
@mailbox ||= current_user.mailbox
end

def conversation
 @conversation ||= mailbox.conversations.find(params[:id])
end

def conversation_params(*keys)
 fetch_params(:conversation, *keys)
end

def message_params(*keys)
 fetch_params(:message, *keys)
end

def fetch_params(key, *subkeys)
 params[key].instance_eval do
   case subkeys.size
 when 0 then self
 when 1 then self[subkeys.first]
 else subkeys.map{|k| self[k] }
     end

end

end

对话视图索引：

<% @conversations.each do |conversation| %>

<% if participant != current_user %>
 <%= participant.name, participant %>
<% end %>

<%= link_to conversation.subject, conversation %>
<%= conversation.updated_at.strftime("%a, %m/%e/%Y %I:%M %p") %>
<%= link_to "Move to Trash", {:controller => "conversations", :action => "trash", :id => conversation.id}, :title=> "Move to Trash", :method=>'post' %>
<% end %>

并链接到 current_user_session 路径中的收件箱

<%= link_to "inbox", conversations_path %>

我有其他观点，但我认为问题出在对话控制器中。我不确定这些错误是怎么回事，它应该可以工作

4

2 回答 2

2

def如果不使用分号，则不能将方法的内容与其所在的行放在同一行。

如果您希望您的方法位于一行，请将它们重构为如下所示：

def trash_folder; @trash ||= current_user.mailbox.trash.all; end

编辑

我的回答并不完全正确。正如 Jörg 在评论中所说，完全可以在没有分号的一行上定义一个方法。Ruby 只需要知道参数列表的结束位置和方法体的开始位置。这可以通过使用换行符、分号或空参数列表来实现。

于 2014-04-14T21:15:29.413 回答
0

将您的方法定义放在多行上，如下所示：

def trash_folder
  @trash ||= current_user.mailbox.trash.all
end

当您将其全部放在一行上时，您的@trash变量将被解释为方法参数。我真的建议不要使用任何单行方法，由于 ruby​​ 的可选括号规则，它们很难阅读并且可能会令人困惑。

于 2014-04-14T21:16:01.940 回答