代码里直接返回对象造成的可读性、扩展性以及性能问题

错误严重度

不会引起较大故障，但对代码可读性、扩展性以及性能会产生一定影响

错误案例

代码里直接返回值，例如获得一个boolean值，例如：

……
return true；
或者
return false；
……

错误分析

一般来说，类可以提供一个公有的静态工厂方法，所谓静态工厂方法，实际上只是一个简单的静态方法，它返回类的一个实例。类可以为它的客户提供一些静态工厂方法，来替代构造方法，当然同时也提供一些构造方法。可以参考JDK的源码String.Valueof()或者某些类的getInstanc()方法。

使用静态工厂方法来代替构造函数或直接返回值的优点在于： 1、静态工厂方法具有名字，可以更好的描述被返回的对象。如果一个构造方法的参数没有确切的描述被返回的对象，那么选用适当名字的静态工厂方法可以使一个类更易于使用，并且相应的客户代码更易于阅读。

2、静态工厂方法在每次被调用的时候，通常并不强制要求必须创建一个新的对象。这使得一些非可变类可以使用一个预先构造好的实例，或者把已经构造好的实例缓存起来，以后再把这些实例分发给客户。减少了创建、销毁对象的开销

3、静态工厂方法可以返回一个原返回类型的子类对象。使用这样的静态工厂方法，可以强迫客户通过接口来引用被返回的对象，而不是通过实现类来引用被返回的对象，这是一个符合OO规范的良好习惯。

静态工厂方法并没有明显的缺点，当然这也不意味着它可以放之四海而皆准。如果构造函数可以满足要求，就没有必要多次一举。

静态工厂方法的最大缺点在于，它与其他的静态方法没有任何区别。在API文档中不能像构造方法那样被明确标示出来，常常会被开发人员忽视。我们通常拿到一个类，都会去看看它的构造函数是如何的，但却往往会忽视它到底有哪些静态工厂方法（因为没有明显标识）

正确用法

……
public static Boolean valueOf(boolean b){
return (b?Boolean.TRUE:Boolean.FALSE);
}
……