3

这些params来自 erb 模板中的 html 输入（此代码在 main 中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
4

3 回答 3

6

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

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

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

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

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

我对 web 框架没有经验，所以我的建议有点摸不着头脑。

于 2013-05-07T21:41:29.397 回答
1

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

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。

于 2013-05-07T23:08:45.713 回答
0

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

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

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

编辑：最好只检查参数是否存在，因为如果找不到，nil则返回 - 调用empty?会nil抛出NoMethodError. 这样做可能会更好：

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

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

于 2013-05-07T21:32:25.103 回答