Question

我是Rails，MVC和CRUD的新手，我正在尝试使用更新方法来更改帖子的投票数量。我的帖子控制器更新方法中有以下代码：

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"}

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

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)

Answer 2

正如米沙指出的，update_column有两个论点。但是，我不鼓励你使用这种方法。首先，它会跳过可能不是您想要的验证。其次，Rails有incrementing or decrementing values的内置方法。在您的情况下，您可以将控制器方法更改为以下内容：

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

ArgumentError：参数数量错误（1表示2）

2 个答案: