论坛首页 Java企业应用论坛

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

浏览 7540 次
精华帖 (14) :: 良好帖 (0) :: 新手帖 (0) :: 隐藏帖 (0)
作者 正文
   发表时间:2008-08-14  
code review过程中,看到这样的代码:
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日的时候,让我们放开身心,勇敢地被日以及日吧!“


   发表时间:2008-08-14  
不是被get set,日,而是,人,日了get set,get set是无辜的。
0 请登录后投票
   发表时间:2008-08-14  
用map代替的话
IDE又不能返回想要的KEY
如果想把get set去的的话
只能在ide上想办法了。

难不成要运行时生成指定的代码?
0 请登录后投票
   发表时间:2008-08-14  
等Java7的Property declaration语法糖出来会好一点:
http://www.iteye.com/news/598
0 请登录后投票
   发表时间:2008-08-14  
引用
它不是public,也没有工具要用java bean api来读它,有必要一本正经地搞getFoo(), getBar()吗?直接就两个foo,bar field,简单点不好?

加get set,propterty语法糖都不需要。简单简单,写java就非得将自己变白痴吗!
0 请登录后投票
   发表时间:2008-08-14  
nihongye 写道
引用
它不是public,也没有工具要用java bean api来读它,有必要一本正经地搞getFoo(), getBar()吗?直接就两个foo,bar field,简单点不好?

加get set,propterty语法糖都不需要。简单简单,写java就非得将自己变白痴吗!

因为其他lib都是按照这种"白痴"的方式来读取javabean的属性,比如常见的struts, ognl,如果你直接用public field,和这些lib就很难配合,还得自己额外写一些转化的工具类。

其实这和整个编程语言的习惯有关系,比如在Java中,我们通常用isAdmin做方法名,而在ruby中,就会改用admin?做方法名,整个业界都是用getter/setter,哪怕它很白痴,我们也得随大流。Java语言想做大的改变,很难,毕竟要考虑到那么多历史兼容问题。
0 请登录后投票
   发表时间:2008-08-14  
抛出异常的爱 写道
用map代替的话
IDE又不能返回想要的KEY
如果想把get set去的的话
只能在ide上想办法了。

难不成要运行时生成指定的代码?


我的办法就是:
final class FooAndBar {
  final Foo foo;
  final Bar bar;

  FooAndBar(Foo foo, Bar bar) {...}
}

用C++的术语,struct。
0 请登录后投票
   发表时间:2008-08-14  
Quake Wang 写道
nihongye 写道
引用
它不是public,也没有工具要用java bean api来读它,有必要一本正经地搞getFoo(), getBar()吗?直接就两个foo,bar field,简单点不好?

加get set,propterty语法糖都不需要。简单简单,写java就非得将自己变白痴吗!

因为其他lib都是按照这种"白痴"的方式来读取javabean的属性,比如常见的struts, ognl,如果你直接用public field,和这些lib就很难配合,还得自己额外写一些转化的工具类。

其实这和整个编程语言的习惯有关系,比如在Java中,我们通常用isAdmin做方法名,而在ruby中,就会改用admin?做方法名,整个业界都是用getter/setter,哪怕它很白痴,我们也得随大流。Java语言想做大的改变,很难,毕竟要考虑到那么多历史兼容问题。

不用写工具类。如果最终需要和这些白痴工具集成,我就refactor一下好了,多大点事?

事实上,这个类在后续的重构中又消失了,这些额外的getter/setter所付出的代价完全没有得到任何回报。
0 请登录后投票
   发表时间:2008-08-14  
ajoo 写道

我的办法就是:
final class FooAndBar {
  final Foo foo;
  final Bar bar;

  FooAndBar(Foo foo, Bar bar) {...}
}

用C++的术语,struct。


这个FooAndBar只能包内可见,那没法传递给客户端啦?

再,如果碰到了FooAndBarAndOther咋办?

0 请登录后投票
   发表时间:2008-08-14  
sorphi 写道
ajoo 写道

我的办法就是:
final class FooAndBar {
  final Foo foo;
  final Bar bar;

  FooAndBar(Foo foo, Bar bar) {...}
}

用C++的术语,struct。


这个FooAndBar只能包内可见,那没法传递给客户端啦?

再,如果碰到了FooAndBarAndOther咋办?


本来就是包内的辅助数据结构,所谓的algebraic type。不需要对外公开。何况即使需要公开,把一切都变成public就是了,毕竟是项目自己用的,又不是jdk。

如果碰到Other,就再加一个field呗。
0 请登录后投票
论坛首页 Java企业应用版

跳转论坛:
Global site tag (gtag.js) - Google Analytics