控制器结构不当致多动作异常：加减分按钮功能错误排查

嘿，我来帮你排查下这个按钮逻辑不符的问题～从你给出的代码和描述来看，大概率是按钮的请求指向错了或者负分方法的逻辑写反了，下面给你具体的排查和解决步骤：

1. 先检查视图里的负分按钮是不是连错了接口 #

这是最常见的原因——你可能把负分按钮的请求指向了add_positive这个方法，而不是对应的负分方法（比如add_negative）。

举个例子，如果你用的是Rails的button_to，错误的写法可能是这样：

# 错误：负分按钮却调用了add_positive接口
<%= button_to "减分", add_positive_path(@user), method: :patch %>

正确的应该是指向add_negative的路由：

# 正确：负分按钮调用add_negative接口
<%= button_to "减分", add_negative_path(@user), method: :patch %>

同时要确认routes.rb里有没有配置add_negative的路由，比如：

resources :users do
  patch 'add_positive', on: :member
  patch 'add_negative', on: :member # 这个路由配置必须要有
end

2. 检查add_negative方法的逻辑是否写反了 #

如果你已经写了add_negative方法，可能不小心在方法里操作了正分字段，比如：

# 错误：本来要加负分，却给正分加了1
def add_negative
  @user = User.find(params[:id])
  @recordid = Record.find_by(user_id: @user.id)
  @recordid.positivescore +=1 # 这里写错了！
  @recordid.save
end

根据你的业务逻辑，应该改成操作负分对应的字段，比如：

# 正确：给负分字段加1
def add_negative
  @user = User.find(params[:id])
  @recordid = Record.find_by(user_id: @user.id)
  # 如果你的Record表有negativescore字段，就操作它
  @recordid.negativescore +=1
  # 或者如果是总分=正分-负分，需要扣减正分的话，就写@recordid.positivescore -=1
  @recordid.save! # 用save!可以在控制台看到报错信息，方便调试
end

3. 额外的小优化建议 #

• 用find_or_create_by替代find_by，避免用户没有Record记录时出现nil报错：

  def add_positive
    @user = User.find(params[:id])
    @recordid = Record.find_or_create_by(user_id: @user.id) # 没有记录就自动创建
    @recordid.positivescore +=1
    @recordid.save!
  end
  

• 可以在方法里加日志，方便排查触发的是哪个方法：

  def add_positive
    logger.info "触发了【添加正分】操作，用户ID：#{params[:id]}"
    # 其他代码...
  end
  
  def add_negative
    logger.info "触发了【添加负分】操作，用户ID：#{params[:id]}"
    # 其他代码...
  end
  

这样你点击按钮后，看控制台的日志就能知道实际触发的是哪个方法了。

内容的提问来源于stack exchange，提问作者Jack Smith