【发布时间】:2014-01-12 09:27:28
【问题描述】:
draw 和 hand 变量是列表。我想将最上面的牌移出平局并将其添加到手牌中。这是函数:
def pick_up_card(draw, hand) do
[new_card | new_draw ] = draw
{hand ++ [new_card], new_draw}
end
我可以通过包含 [head |参数列表中的tail]部分:
def pick_up_card([new_card | new_draw], hand) do
{hand ++ [new_card], new_draw}
end
那会是更惯用的 Elixir 吗?它会损害可读性吗?我想如果函数定义具有传入列表的名称而不是 Head|Tail 调用,那么以后代码会更容易阅读。
这是个人喜好问题,还是我最好保存那行代码并将 Head|Tail 调用粘贴在函数定义中?
【问题讨论】:
-
new_card后面需要加hand吗?如果不是更有效地做{[new_card|hand], new_draw}。如果draw不是列表,则您的两个替代结果会产生基本相同的代码,只是不同的错误。 -
是的,为了实现这个功能,我将这张牌从 draw 列表中取出,并将其添加到 hand 列表中。 (不过,它是添加到 hand 列表的前面还是末尾并不重要。)在现实世界中,我正在捡起一张牌并将其添加到我的手上。
-
如果您不在乎将其添加到
hand列表的哪个位置,那么罗伯特的建议会更好,因为添加到列表顶部是 O(1),同时添加到尾部是 O(n)。 -
谢谢,莎莎。这是关于 O(1) 的一个很好的观点。基于这一点,我可能会重写的不仅仅是这个函数。谢谢!
标签: function parameters elixir