未定义的方法“to_key”对于#<类 ：0x17a6408>-rails-3

Question

我在多态上传表单上遇到未定义方法“to_key”的问题。

这是部分形式：

<%= form_for [@parent, Upload], :html => { :multipart => true } do |f|  %>

  <div class="field">
    <%= f.label :document %><br />
    <%= f.file_field :document %>
  </div>

  <div class="actions">
    <%= f.submit "Upload"%>
  </div>
<% end %>

这是控制器：

class UploadsController < ApplicationController
  before_filter :find_parent

  respond_to :html, :js

  def index
    @uploads = @parent.uploads.all unless @uploads.blank?
    respond_with([@parent, @uploads])
  end

  def new
    @upload = @parent.uploads.new unless @uploads.blank?
  end

  def show
    @upload = @parent.upload.find(params[:upload_id])
  end

  def create
    # Associate the correct MIME type for the file since Flash will change it
    if  params[:Filedata]
      @upload.document = params[:Filedata]
      @upload.content_type = MIME::Types.type_for(@upload.original_filename).to_s
      @upload = @parent.uploads.build(params[:upload])
      if @upload.save
        flash[:notice] = "suceessfully saved upload"
        redirect_to [@parent, :uploads]
      else
        render :action => 'new'
      end
    end
  end

  def edit
    @upload = Upload.where(params[:id])
  end
  private


  def find_parent
    classes ||= []
    params.each do |name ,value|
      if name =~ /(.*?)_id/
        @parent =  classes << $1.pluralize.classify.constantize.find(value)
      end
    end
    return unless classes.blank?
  end
end

如果我更改

<%= form_for [@parent, Upload], :html => { :multipart => true } do |f| %>

为

<%= form_for [parent, Upload], :html => { :multipart => true } do |f| %>

，我会收到一个新错误：#<#:0x21a30e0> 的未定义局部变量或方法“parent”

这是错误跟踪：

ActionView::Template::Error (undefined method `to_key' for #<Class:0x2205e88>):
1: <%= render :partial => "uploads/uploadify" %>
2: 
3: <%= form_for [@parent, Upload], :html => { :multipart => true } do |f|  %>
4: 
5: 
6:  <div class="field">

“uploads/uploadify”部分在此要点中： https://gist.github.com /911635

任何指针都会有帮助。谢谢

Answer 1

据我所知，您的 form_for 应该类似于

<%= form_for [@parent, @upload], :html => { :multipart => true } do |f| %>

我假设您的上传对象嵌套在另一个对象中，类似于以下内容：

resources :posts do
  resources :uploads
end

当传递这样的数组时，form_for 会执行什么操作是根据给定对象的类以及它们是否是新记录来构造相关路径。

在您的情况下，您在控制器的新操作中创建一个新的上传对象，因此 form_for 将检查数组，获取 @parent 的类和 id，然后获取 @upload 的类和 id。但是，因为@upload没有id，所以它会POST到/parent_class/parent_id/upload，而不是PUTting到parent_class/parent_id/upload/upload_id。

如果这不起作用，请告诉我，我们会进一步解决:)

- 编辑 - 评论后 -

这意味着 @parent 或 @upload 之一为零。要进行检查，您可以将以下内容放入您的视图中

<%= debug @parent %>

，对于 @upload 也是如此，看看哪个是 nil。但是，我猜测 @upload 为零，因为控制器中的这一行：

# UploadsController#new
@upload = @parent.uploads.new unless @uploads.blank?

特别是 unless @uploads.blank? 部分。除非你在ApplicationController中初始化它，否则@uploads总是nil，这意味着@uploads.blank？永远是 true，这又意味着 @upload 永远不会被初始化。更改读取的行

@upload = @parent.uploads.new

，问题有望得到解决。您使用的其他方法也是如此，除非@uploads.blank？。

在一个半相关的说明中，在 UploadsController#find_parent 中，您有这一行

classes ||= []

，因为该变量是 find_parent 方法的本地变量，您可以放心它没有初始化，而应该编写classes = []。

有这行代码

return unless classes.blank?

另外，在方法结束之前 。您是否添加了它，以便在 @parent 初始化后从该方法返回？如果是这样，该行应该位于每个块内。

此外，由于类不在方法之外使用，为什么还要定义它呢？代码可以如下所示，并且仍然具有相同的行为

def find_parent
  params.each do |name ,value|
    @parent = $1.pluralize.classify.constantize.find(value) if name =~ /(.*?)_id/
    return if @parent
  end
end

除其他事项外，您将看到这做了一些事情：

1. 避免初始化不需要的变量。
2. 内联 if 语句，这有助于提高单行条件的可读性
3. 将 unlessvariable.blank 的使用更改为 ifvariable。除非你的变量是布尔值，否则这会完成同样的事情，但会减少认知负担，因为前者本质上是你的大脑必须解析的双重否定。

-- 编辑 - 来自有关该问题的电子邮件交换 --

您是正确的 - 如果父级已初始化，if @parent 将返回 true。然而，正如我在 SO 中提到的，例外情况是 @parent 已初始化并设置为 false。本质上它的意思是在 Ruby 中，除了 nil 和 false 之外的所有值都被认为是 true。当实例变量尚未初始化时，它的默认值为 nil，这就是该行代码起作用的原因。这有道理吗？

就在 UsersController 中呈现表单的每个操作中设置 @parent 而言，以下哪一个是在索引操作上执行此操作的正确方法。我已经尝试了所有 3 个但出现错误

请记住，@parent 和 @upload 都必须是 ActiveRecord (AR) 对象的实例。在第一种情况下，您将@parent设置为User.all，这是一个AR对象数组，这是行不通的。另外，您尝试在 @parent 初始化之前调用 @parent.uploads ，这将给出 no method 错误。但是，即使您要交换这两行，当parent是一个数组时，您也会调用@parent.uploads。请记住，uploads 方法是在单个 AR 对象上定义的，而不是在它们的数组上定义的。由于索引的所有三个实现都执行类似的操作，因此上述注意事项以各种形式适用于所有这些实现。

users_controller.rb

定义索引
@upload = @parent.uploads
@parent = @user = User.all
结束

<前><代码>或

定义索引
# @user = @parent.user.all
@parent = @user = User.all
结束

<前><代码>或

定义索引
@parent = @upload = @parent.uploads
@users = User.all
结束

快速引导您完成我所做的更改。在开始之前，我应该解释一下，这

<%= render "partial_name", :variable1 => a_variable, :variable2 => another_variable %>

与执行此操作等效

<%= render :partial => "partial_name", :locals => {:variable1 => a_variable, :variable2 => another_variable} %>

，只是一种更短（并且更干净）的渲染方式。同样，在控制器中，您可以执行

render "new"

以下

render :action => "new"

操作 ： .org/layouts_and_rendering.html 现在来看代码。

#app/views/users/_form.html.erb
<%= render :partial => "uploads/uploadify" %>

<%= form_for [parent, upload], :html => { :multipart => true } do |f|  %>


 <div class="field">
    <%= f.label :document %><br />
    <%= f.file_field :document %>
  </div>

  <div class="actions">
    <%= f.submit "Upload"%>
  </div>
<%end%>

在上传表单上，您会看到我将@parent 和@upload 更改为parent 和upload。这意味着您需要在渲染表单时传递变量，而不是让表单查找控制器设置的实例变量。您将看到这允许我们执行以下操作：

#app/views/users/index.html.erb
<h1>Users</h1>
<table>
  <% @users.each do |user| %>
    <tr>
      <td><%= link_to user.email %></td>
      <td><%= render 'uploads/form', :parent => user, :upload => user.uploads.new %></td>
    </tr>
  <% end %>
</table>

在 UsersController#index 中为每个用户添加上传表单。您会注意到，因为我们现在显式传入父级并上传，所以我们可以在同一页面上有多个上传表单。这是一种更干净、更可扩展的嵌入部分的方法，因为父级和上传的设置立即变得显而易见。使用实例变量方法，不熟悉代码库的人可能很难确定@parent和@upload的设置位置等。

#app/views/users/show.html.erb
<div>
  <% @user.email %>
  <h3 id="photos_count"><%= pluralize(@user.uploads.size, "Photo")%></h3>
  <div id="uploads">
    <%= image_tag @user.upload.document.url(:small)%>
    <em>on <%= @user.upload.created_at.strftime('%b %d, %Y at %H:%M') %></em>
  </div>

  <h3>Upload a Photo</h3>
  <%= render "upload/form", :parent => @user, :upload => user.uploads.new %>
</div>

这与上面的更改类似，我们传入父对象和上传对象。

 #config/routes.rb
 Uploader::Application.routes.draw do
  resources :users do
    resources :uploads
  end

  devise_for :users

  resources :posts do
    resources :uploads
  end

  root :to => 'users#index'
end

您会看到我删除了作为路由中顶级资源的上传。这是因为上传需要某种父级，因此不能是顶级的。

#app/views/uploads/new.html.erb
<%= render 'form', :parent => @parent, :upload => @upload %>

我做了与上面相同的更改，显式传递父级并上传。显然，无论您在何处呈现表单，都需要执行此操作。

#app/controllers/users_controller.rb
class UsersController < ApplicationController
 respond_to :html, :js

  def index
    @users =  User.all
  end

  def show
    @user = User.find(params[:id])
  end

  def new
    @user = User.new
  end

  def create
    @user = User.new(params[:user])
    if @user.save
      redirect_to users_path
    else
      render :action => 'new'
    end
  end

  def update
    @user = User.find_by_id(params[:id])
    @user.update_attributes(params[:user])
    respond_with(@user)
  end

  def destroy
    @user = User.find_by_id(params[:id])
    @user.destroy
    respond_with(@user)
  end
end

我已经从用户控制器中删除了对 @parent 的任何提及，因为我们显式地传递了它。

希望这一切都有道理。您可以从这些示例中进行推断，并在您想要呈现上传表单的任何位置传递父对象和上传对象。

Answer 2

[@parent，上传] => [@parent, :upload]

<%= form_for [@parent, :upload], :html => { :multipart => true } do |f|  %>

UPD

您应该更改位置 :upload 和 @parent

<%= form_for [:upload, @parent], :html => { :multipart => true } do |f|  %>