Question

这些params来自erb模板中的html输入（此代码位于主application.rb中），我在检查它们是否已填充之前将其添加到n.requestusers，这将成为数据库条目的一部分。它有效，但感觉就像现在的bash脚本一样。写这样的东西最好的方法是什么？

主要.rb中的路线

if params[:user2].empty? && params[:user3].empty? && params[:user4].empty? && params[:user5].empty?
  n.requestusers = params[:user1]
elsif params[:user3].empty? && params[:user4].empty? && params[:user5].empty?
  n.requestusers = params[:user1], params[:user2]
elsif params[:user4].empty? && params[:user5].empty?
  n.requestusers = params[:user1], params[:user2], params[:user3]
elsif params[:user5].empty?
  n.requestusers = params[:user1], params[:user2], params[:user3], params[:user4]
else
  n.requestusers = params[:user1], params[:user2], params[:user3], params[:user4], params[:user5]
end

Answer 1

您可能对以下内容感兴趣，而不是拥有所有这些条件语句：

n.requestusers = params.select { |key, val| not val.empty? }.values

或@theTinMan建议的更清洁的方式：

n.requestusers = params.reject { |key, val| val.empty? }.values

select允许您获取所有非空参数值并返回它们。 values允许您将这些值作为数组获取。

我对网络框架没有经验，所以我的建议是在黑暗中拍摄。

Answer 2

这没有经过测试，因为没有要测试的样本值，但是，经过一些重构后，我有：

if [:user2, :user3, :user4, :user5].all?{ |s| params[s].empty? }
  n.requestusers = params[:user1]
elsif [:user3, :user4, :user5].all? { |s| params[s].empty? }
  n.requestusers = [:user1, :user2].map{ |s| params[s] }
elsif [:user4, :user5].all? { |s| params[s].empty? }
  n.requestusers = [:user1, :user2, :user3].map{ |s| params[s] }
elsif params[:user5].empty?
  n.requestusers = [:user1, :user2, :user3, :user4].map{ |s| params[s] }
else
  n.requestusers = [:user1, :user2, :user3, :user4, :user5].map{ |s| params[s] }
end

进一步观察，这似乎是明智的：

USER_LIST = [:user1, :user2, :user3, :user4, :user5]
USER_LIST.size.times do |i|
  user_list = USER_LIST
  get_users = user_list.shift(1 + i)
  if user_list.all?{ |s| params[s].empty? }
    n.requestusers = params.values_at(get_users)
    break
  end
end

就像我说的那样，没有经过测试，但我会按照这些方式进行处理。

根据需要调整USER_LIST。

Answer 3

我对Sinatra并不熟悉，但是如果你想为数组添加元素，你可以这样做（假设n.requestusers已经初始化）：

n.requestusers << params[:user1] unless params[:user1].empty?

您可以为每个用户参数执行此操作。

修改：检查参数是否存在可能更好，因为如果找不到，则返回nil - 在{{empty?上调用1}}抛出一个nil。最好这样做：

NoMethodError

这将确保它添加param（如果存在）。

检查Ruby中的多个参数

3 个答案: