code review过程中,看到这样的代码:
[code]
Pair<String, String> getFooAndBar() {
...
}
[/code]
恶魔说,弄个Pair来表达FooAndBar不太容易理解,要不干脆弄个类把foo和bar封装起来,返回出来吧。
天使:“同意”。
下一版代码:
[code]
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;
}
}
[/code]
嗯。恶魔不爽了。这个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日的时候,让我们放开身心,勇敢地被日以及日吧!“
[code]
Pair<String, String> getFooAndBar() {
...
}
[/code]
恶魔说,弄个Pair来表达FooAndBar不太容易理解,要不干脆弄个类把foo和bar封装起来,返回出来吧。
天使:“同意”。
下一版代码:
[code]
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;
}
}
[/code]
嗯。恶魔不爽了。这个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日的时候,让我们放开身心,勇敢地被日以及日吧!“