今天在qq的java群里面，有同学贴了一道代码评审题

这块看上去写得还可以的代码，我说说个人的看法

问题1：最大的问题就是方法名，这个函数实际上是有数据操作的，不应该叫getXXX，给人一种人畜无害的感觉。调用者不会想到里面把数据改了。

问题2: 【可能】存在的空指针异常。order可能为空，是否必须要加空判断呢？我的意见是不一定。具体原因见此贴：我的编码习惯 - 异常处理

问题3: 魔法数字0和1

问题4：没有日志，我个人建议，数据操作还是应该带上日志，方便跟踪。如果真的只是查询数据，不加日志没有关系，参见此贴：我的编程习惯 - 日志建议

问题5: 这里的字符串拼接不需要自己使用 StringBuffer。这个严格来讲不算是问题，只是很多人不知道这个知识点。当然你这样写不是问题，如果你本来没有使用StringBuffer拼接字符，评审的是说要改成StringBuffer来拼接字符串，那就不应该了。这应该都是很老的编程规范了！

简单说明一下，jdk实际上已经给大家做了这一步了，看代码

/**
 * 使用 javap -c StringDemo.class 反编译看字节码
 * @author 晓风轻
 *
 */
public class StringDemo {

  private String name;

  public String getName() {
    return name;
  }

  public void setName(String name) {
    this.name = name;
  }

  public static void main(String[] args) {
    StringDemo demo = new StringDemo();

    demo.setName("晓风轻");

    String s = "我的名字是[" + demo.getName() + "].";
    System.out.println(s);
  }

}

使用 javap -c StringDemo.class 反编译看字节码:

可以看出，字节码里面已经自动使用StringBuilder拼接字符串了。