参数错误：参数个数不对（应为2个，但只提供了1个）

Question

我很新手Rails、MVC和CRUD，想要使用update方法来改变一篇文章的投票数量。在我的Posts Controller update方法中，有以下代码：

def update
    @post = Post.find(params[:id])

    if params[:vote] == 'up'
        @post.update_column(:ups => @post[:ups] + 1)
    elsif params[:vote] == 'down'
        @post.update_column(:downs => @post[:downs] + 1)
    end

    flash[:notice] = "Thanks for voting! This helps us determine important issues in our schools."

    redirect_to 'Posts#index'
end

我在我的routes.rb文件中有以下代码：

OpenMCJC::Application.routes.draw do
  root :to => 'posts#index'
  resources :posts
  match '/posts/:id/:vote', :to => 'posts#update'
end

在导航到“/posts/3/up”后，它会抛出以下错误：

ArgumentError in PostsController#update

wrong number of arguments (1 for 2)

根据页面的要求，请求参数如下：

{"id"=>"3",
"vote"=>"up"}

你能帮我弄清楚出了什么问题吗？

- airplaneman19

Answer 1

update_column方法需要两个参数，但您只传递了一个参数。

替代以下代码：

@post.update_column(:ups => @post[:ups] + 1)

尝试：

@post.update_column(:ups, @post[:ups] + 1)

这可能看起来像是两个参数：

:ups => @post[:ups] + 1

但实际上它只是一个哈希值。

使用更常见的update_attributes，你可以传递一个哈希值：

@post.update_attributes(:ups => @post[:ups] + 1)

- Mischa

---

这曾经让我吃过苦头。可悲的是。 - courtsimas

Answer 2

正如Mischa所指出的那样，update_column需要两个参数。然而，我不建议您使用这种方法。首先，它会跳过验证，这可能不是您想要的。其次，Rails已经内置了用于增加或减少值的方法。在您的情况下，您可以将控制器方法更改为以下内容：

if params[:vote] == 'up'
  @post.increment(:ups)
elsif params[:vote] == 'down'
  @post.increment(:downs)
end

- Peter Brown

---

是的，他应该使用这些方法：+1 - Mischa