嵌套的IF语句语法

我试图让这个嵌套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

你在这里想要的是一个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来表示“价格 - 低到高”,甚至数字代码。 这意味着如果改变措辞你的代码仍然有效。


我会改变你的下拉值,如price ascprice descarrival_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
链接地址: http://www.djcxy.com/p/25653.html

上一篇: Nested IF statement syntax

下一篇: Ruby challange: experienced developers please reply