在一个类中增加方法时应十分克制,鼓励添加而不是修改,体现开闭原则。
如果不能快速理直接清楚方法之间的关系那么方法关系就太复杂了
包可见也便于单元测试
set([^(]++)\([^.]++\.get\1\(
- private 不便于单元测试。
- 体现对拓展开放,否则装饰者继承无法使用 private 的内容,就不得不复制代码。
- Sonar 也是提示改成 protected 而不是 private。
String a;
if ... {
a = ...
} else if {
a = ...
} else if {
...
抽取成:
String foo() {
if (...) {
return ...
} else if {
return ...
} else if {
...
while 预定义除外
public protected private abstract default static final transient volatile synchronized native strictfp
https://google.github.io/styleguide/javaguide.html#s4.8.7-modifiers
https://github.com/google/styleguide/blob/gh-pages/javaguide.html