Question

我试图让这个嵌套的if语句工作，但我的语法错了，我无法理解。我在Rails 4应用程序上有一个搜索框和一个排序下拉列表。在搜索结果页面上，我想根据用户在排序下拉列表中选择的内容对产品列表进行排序。如果用户没有输入搜索词，我想要显示一条消息。这是我的控制器代码。

如果我删除条件并只显示默认排序，搜索结果页面显示正常，因此错误在if语句语法中。

def search
    if params[:search].present?

      if params[:sort] == "- Price - Low to High"
        @listings = ...
      elsif params[:sort] == "- Price - High to Low"
        @listings = ...
      elsif params[:sort] == "- New Arrivals"
        @listings = ...
      elsif params[:sort] == "- Random Shuffle"
        @listings = ...
      else
        @listings = ...
      end

    else
      flash[:notice] = "Please enter one or more search terms e.g. blue shirt."
    end

  end

Answer 1

这里你想要的是case语句，来自JavaScript和C等其他语言的开关：

def search
  if params[:search].present?
    @listings =
      case (params[:sort])
      when  "- Price - Low to High"
        ...
      when "- Price - High to Low"
        ...
      when "- New Arrivals"
        ...
      when "- Random Shuffle"
        ...
      else
        ...
      end
  else
    flash[:notice] = "Please enter one or more search terms e.g. blue shirt."
  end
end

在Ruby中，case语句的结果可用于分配给变量，因此消除了大量@listings =次重复。同样适用于if。

你在这里比较的东西看起来异常具体。如果您有一个用于选择排序顺序的下拉列表，那么它们应该在内部使用更简洁的值，例如plh代表＆＃34;价格 - 低到高＆＃34;甚至数字代码。这意味着如果更改了措辞，您的代码仍然有效。

Answer 2

我会将您的下拉列表更改为price asc，price desc，arrival_date asc（不确定随机随机播放）等值，然后您可以使用。

def search
  if params[:search].present?
    sort_listings(params[:sort])
  else
    flash[:notice] = "Please enter one or more search terms e.g. blue shirt."
  end
end

def sort_listings(sort)
  @listings ||= ...
  @listing.order(sort)
end

Ruby - 嵌套的IF语句语法

2 个答案: