NilClass:Class 的未定义方法“model_name”

Question

Ruby 3.1 Rails 1.9.2 绝对新手

当我尝试在应用程序中编辑现有广告时，问题就开始了。

我收到“NilClass:Class 的未定义方法 `model_name'”错误。以下信息可能会提供有关问题所在的线索。我花了几个小时试图弄清楚它无济于事。

Routes.rb

match 'ads/:id/edit' => 'ads#edit'
match 'ads/:id/update' => 'ads#update'
match 'ads/' => 'ads#index'
match 'ads/new' => 'ads#new'
match 'ads/create' => 'ads#create

控制器中的条目

def edit
  @ad=Ad.find(params[:id])
end

def update
        @ad=Ad.find(params[:id])
        @ad.update_attributes(params[:id])
        redirect_to "/ads/#{@ad.id}"
end

查看：

<%= form_for(@ad,:url=>{:action=>'update'}) do |f| %>
<p><b>Name</b><br /><%= f.text_field :name %></p>
<p><b>Description</b><br /><%= f.text_area :description, rows: 10 %></p> 
<p><b>Price</b><br /><%= f.text_field :price %></p>
<p><b>Seller</b><br /><%= f.text_field :seller_id %></p>
<p><b>E-mail</b><br /><%= f.text_field :email %></p>
<p><b>Img url</b><br /><%= f.text_field :img_url %></p>
<p><%= f.submit "Update" %></p>
<% end %>

---

任何帮助将不胜感激。 谢谢。

Answer 1

有几点：

@mu-is-too-short 的更新方法示例是正确的。您应该将属性哈希 (params[:ad]) 发送到 @ad 对象，而不是参数 params[:id]。

明确写出这样的路线可能会变得非常混乱。您可以将您在routes.rb 文件中写入的所有内容替换为：

resources :ads

这将使您能够使用诸如

之类的东西

redirect_to ad_path(@ad)

代替

redirect_to "/ads/#{@ad.id}"

此外，您的表单将更易于管理且不易出错，因为您可以说：

<%= form_for @ad do |f| %>
  <p><b>Name</b><br /><%= f.text_field :name %></p>
  <p><b>Description</b><br /><%= f.text_area :description, rows: 10 %></p> 
  <p><b>Price</b><br /><%= f.text_field :price %></p>
  <p><b>Seller</b><br /><%= f.text_field :seller_id %></p>
  <p><b>E-mail</b><br /><%= f.text_field :email %></p>
  <p><b>Img url</b><br /><%= f.text_field :img_url %></p>
  <p><%= f.submit "Update" %></p>
<% end %>

顺便说一句，我建议对 HTML 语义进行一些研究。看起来更像这样的表单更易于维护和设置样式，并且更易于访问：

<%= form_for @ad do |f| %>
  <fieldset>
    <%= f.label :name %>
    <%= f.text_field :name %>
  </fieldset>

  <fieldset>
    <%= f.label :description %>
    <%= f.text_area :description, :rows => 10 %>
  </fieldset>

  <!-- etc -->

  <fieldset>
    <%= f.submit 'Update' %>
  </fieldset>
<% end %>

如果您还没有阅读过，我强烈建议您阅读所有 Rails Guides，尤其是 guide on routing。

Answer 2

我认为您的 update 控制器方法很混乱。将 ID 传递给 update_attributes 不会让您到达您想去的任何地方，您想将一些属性传递给该方法，并且这些属性应该在 params[:ad] 中，一些错误处理也可能很好：

def update
    @ad = Ad.find(params[:id])
    if(@ad.update_attributes(params[:ad]))
        # It worked, redirect to the show action.
    else
        # It didn't work, make them do it again.
    end
end