getThis().getEvilAdvocate().setDead(getTrue())
程序员文章站
2022-07-14 14:44:12
...
code review过程中,看到这样的代码:
恶魔说,弄个Pair来表达FooAndBar不太容易理解,要不干脆弄个类把foo和bar封装起来,返回出来吧。
天使:“同意”。
下一版代码:
嗯。恶魔不爽了。这个FooAndBar纯粹就是为了从函数里面返回几个有逻辑关系的值的组合。它不是public,也没有工具要用java bean api来读它,有必要一本正经地搞getFoo(), getBar()吗?直接就两个foo,bar field,简单点不好?
天使:”反正就是IDE点几下,又不费事。“
恶魔:”可是,它占行数啊。我看着没用的代码眼晕啊“
天使:”getter可以控制只读“
恶魔:”不是有final呢嘛?“
天使:”用方法可以implement interface“
恶魔:”可是这里没有interface啊。就是一个纯值对象“
天使:”可以在getter()里面封装其它的逻辑,直接操作field没有这个灵活性“
恶魔:”搜索整个代码库,几乎95%的情况下,getter/setter都是简单的get/set。有必要为这个旷世罕见的灵活性增加复杂度么?“
天使:”getter是标准。sun推荐的。大家都这么干的“
恶魔:(音乐起,微笑)”紧张的生活会造成心跳加快,内分泌失调,皮肤老化,头发枯黄。所以当可以get日和set日的时候,让我们放开身心,勇敢地被日以及日吧!“
Pair<String, String> getFooAndBar() { ... }
恶魔说,弄个Pair来表达FooAndBar不太容易理解,要不干脆弄个类把foo和bar封装起来,返回出来吧。
天使:“同意”。
下一版代码:
class FooAndBar { private final String foo; private final String bar; FooAndBar(String foo, String bar) {...} public String getFoo() { return foo; } public String getBar() { return bar; } }
嗯。恶魔不爽了。这个FooAndBar纯粹就是为了从函数里面返回几个有逻辑关系的值的组合。它不是public,也没有工具要用java bean api来读它,有必要一本正经地搞getFoo(), getBar()吗?直接就两个foo,bar field,简单点不好?
天使:”反正就是IDE点几下,又不费事。“
恶魔:”可是,它占行数啊。我看着没用的代码眼晕啊“
天使:”getter可以控制只读“
恶魔:”不是有final呢嘛?“
天使:”用方法可以implement interface“
恶魔:”可是这里没有interface啊。就是一个纯值对象“
天使:”可以在getter()里面封装其它的逻辑,直接操作field没有这个灵活性“
恶魔:”搜索整个代码库,几乎95%的情况下,getter/setter都是简单的get/set。有必要为这个旷世罕见的灵活性增加复杂度么?“
天使:”getter是标准。sun推荐的。大家都这么干的“
恶魔:(音乐起,微笑)”紧张的生活会造成心跳加快,内分泌失调,皮肤老化,头发枯黄。所以当可以get日和set日的时候,让我们放开身心,勇敢地被日以及日吧!“
上一篇: 【hive】hive的简单使用
下一篇: Hive部署