控制器专用方法不起作用

Question

我正在构建一个用户可以发帖的应用程序。每个帖子都会被点赞和反对。在显示帖子的地方，我有一个允许用户过滤帖子的小表单。它只是将一个名为params[:p ost_filter]（localhost：3000/somepage？post_filter=value）的参数传递给url。现在，这工作得很好，除了我编写的修改查询的私有方法不起作用。

这是我的查询：

def room
            @posts = Post.where('lesson_id = ?', params[:id]).order(post_filter_params).page(params[:page]).per(30)
    end

这是我的私人方法：

private
def post_filter_params
    chosen_option = params[:post_filter].to_i == 1 or 2 or 3 ? params[:post_filter] : '1'
    case chosen_option
    when 1
        'created_at DESC'
    when 2
        'upvotes DESC'
    when 3
        'downvotes DESC'
    end
end

现在，每当我将 .order（） 值替换为私有方法中的一个字符串时，一切都按计划进行。但是，将私有方法名称放在 .order（） 值中不起作用。对正在发生的事情有什么想法吗？

编辑

为了确保所有值都是相同的数据类型，我这样做了，但它仍然不起作用：

def post_filter_params
    param_option = params[:post_filter].to_i
    chosen_option = param_option == 1 or 2 or 3 ? param_option : 1
    case chosen_option
    when 1
        'created_at DESC'
    when 2
        '(upvotes - downvotes) DESC'
    when 3
        'downvotes DESC'
    end
end

Answer 1

我不认为私有方法会做你认为的那样。

怎么样？

def post_filter_params
  case params[:post_filter].to_i
  when 1
    'created_at DESC'
  when 2
    'upvotes DESC'
  when 3
    'downvotes DESC'
  else
    'created_at DESC'
  end
end

Answer 2

尝试将数字与数字进行比较，或将字符串与字符串进行比较。

报价 1、2 和 3

when '1'
...

或将chosen_options转换为数字

Answer 3

我想

解释一下@shioyama的意义。首先，让我们关注以下代码：

param_option = params[:post_filter].to_i
chosen_option = param_option == 1 or 2 or 3 ? param_option : 1

在IRB会话中。

irb(main):005:0> param_option = 1
=> 1
irb(main):006:0> chosen_option = param_option == 1 or 2 or 3 ? param_option : 1
=> true
irb(main):007:0> chosen_option
=> true
irb(main):008:0> param_option = 2
=> 2
irb(main):009:0> chosen_option = param_option == 1 or 2 or 3 ? param_option : 1
=> 2
irb(main):010:0> chosen_option
=> false

因此，chosen_option = param_option == 1 or 2 or 3 ? param_option : 1行与chosen_option = (param_option == 1) or (2) or (3 ? param_option : 1)相同，即true或false。

你想要的可能是：chosen_option = param_option == 1 || param_option == 2 || param_option == 3 ? param_option : 1.

irb(main):036:0> chosen_option = param_option == 1 || param_option == 2 || param_option == 3 ? param_option : 1
=> 3
irb(main):037:0> chosen_option
=> 3

一个常见的快捷方式是 chosen_option = [1, 2, 3].include?(param_option) ? param_option : 1 .但在这种情况下，最好的是@shioyama建议的：

def post_filter_params
  case params[:post_filter].to_i
  when 2
    'upvotes DESC'
  when 3
    'downvotes DESC'
  # you can just fallback to else
  # when 1
  #   'created_at DESC'
  else
    'created_at DESC'
  end
end

最后一个提示：考虑使用CONSTANTS而不是幻数，它将帮助其他人在（不久的）未来:)阅读您的代码和您自己