getThis().getEvilAdvocate().setDead(getTrue())

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日的时候,让我们放开身心,勇敢地被日以及日吧!“
  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值