当函数有太多参数时该怎么办?

时间:2014-03-17 16:16:08

标签: parameters clojure refactoring

我在Clojure中开发了一个系统。我尽量把它写成功能。其中一个问题是引用透明度(对于相同的参数,函数应始终返回相同的结果)。

问题是某些功能(尤其是我的视图和部分控制器)会产生太多的args(例如5-6,将来可能更多)。

你会如何解决这个问题?

我看到几种可能的解决方案各有利弊。

1)创建ParameterMap - 将参数放到Map并使用模式(https://github.com/prismatic/schema)进行验证 - 这是首选解决方案,但此解决方案的问题是我得到运行时错误而不是编译时间错误(现在,当我忘记重构应用程序的某些部分时,应用程序拒绝编译因为错误的arity)

2)使用闭包 - 闭包可以捕获命名变量并等待其余的。它检查编译时间。这个解决方案很简单但是让我创建闭包只是为了将更多的参数分成更多的函数,我认为这些函数可以用来进行贫血设计。这个解决方案类似于使用monad,似乎没有人看到在步骤中添加参数作为不清洁的东西,我不明白为什么。

3)将参数转换为命名参数 - 我仍然会有大量参数,但它们不再清楚,因为我可以看到第一眼看到的含义。我在应用程序的其他部分使用了这种方法并且它可以工作。不幸的是,这在运行时很容易出现NullPointerException,并且在编译时没有检查。

2 个答案:

答案 0 :(得分:3)

嗯,答案很大程度上取决于你的代码库,但可能是所有这些事情的组合。

这不是选择其中一个的问题,而是选择最适合各种特定功能的方法。例如,你应该使用闭包有意义的闭包,在那些有意义的地方命名为params,在有意义的时候映射。

如果您有一堆参数并且您将这些常见参数传递给众多函数,那么使用映射来包含它们并使参数实际更清楚

答案 1 :(得分:2)

延伸评论:

Paul Graham wrote以下关于一个名为Rtml的DSL(其中程序称为模板):

  

Rtml在很大程度上取决于关键字参数,直到那时我都有   一直被认为是Common Lisp的一个更可疑的功能。   ......

     

如果我想为其中一个的行为添加另一个维度   运营商,我可以添加一个新的关键字参数,以及每个人   现有模板将继续有效。一些Rtml   运营商没有采取关键字参数,因为我没想到我   我需要改变它们,几乎每一个我最终都会踢它们   我自己稍后。如果我可以回头从头开始,   我改变的一件事就是我会制作每一个Rtml   运算符采用关键字参数。