今天在聊天的时候，大囧说一本书里提到TDD开发，说道：类里的方法大都4-5行是不是很优雅？TDD的说法是如果一个方法超过4-5行就要考虑重构。我在看本TDD的书，例子很优雅
我的观点：我觉得那是到了境界了..我要是4-5行，就需要刻意的去处理，这样就工作效率低下了.于是我发了一段自己的代码
@Override
public <T> PreparedStatement createStatement(Connection connection,
String sql, Class<T>... entity) throws SQLException {
Field[] fields = entity[0].getDeclaredFields();
String pkFieldName = "";
for (Field field : fields) {
// 只有非static和非final类型的变量才能作为主键
if (!Modifier.isStatic(field.getModifiers())
&& !Modifier.isFinal(field.getModifiers())) {
pkFieldName = StringUtils.underscoreName(field.getName());
break;
}
}
logger.debug("pk field name:" + pkFieldName);
if (StringUtils.isNullOrEmpty(pkFieldName)) {
throw new DBException("can not get pk field name from " + entity[0]);
}
return connection.prepareStatement(sql, new String[] { pkFieldName });
}
大囧：第一行，getfields，第二行，调方法，是还能拆，可读性更好一点。
教主：可以无限拆，让你一年写一个方法。。这样比较好。。
好吧。上面的不是重点，kk提了我代码的几个问题
1、entity 只用到 [0
2、entity 没有判断nul
3、pkFieldName初始化时多余
4、logger没有加 enabled判
5、isNullOrEmpty直接换成 == null就
仔细看了下。确实有问题。我解释
1.这个是之前设计的时候，为了解决多个数据库差异写的方言类（实际我也不明白当时咋想的。。。用不定参数
2.这个我在上层调用已经判断过了。所以不是什么大问题，但是也要注意参数检
3.kk指的是初始化pkFieldName=null就可以了，pkFieldName=“”浪费，嗯。这个是我的问
4.先说第5个.isNullOrEmpty只需要使用==null判断就可以了..因为字段的名字不可能为“”啊，这样的话只要循环里赋值了，就不会为 “”，没赋值就是 null.我真是太2了.有些事一些参数的验证。得好好想想参数的来
5.关于isDebug的问题，我的体会没有很深
我：isDebug的影响到底有多大..我没太深的体会.
kk：不大，有一个区别，你的debug里有一个字符串相加的操作，优先于方法调用，如果加上enabled判断，这个 string操作可以省
解决了我一直的一个疑问,一直听说isDebug是为了性能，但是不清楚具体的什么性能.
修改后的代码
```java
@Override
public <T> PreparedStatement createStatement(Connection connection,
String sql, Class<T> entity) throws SQLException {
Field[] fields = entity.getDeclaredFields();
String pkFieldName = null;
for (Field field : fields) {
// 只有非static和非final类型的变量才能作为主键
if (!Modifier.isStatic(field.getModifiers())
&& !Modifier.isFinal(field.getModifiers())) {
pkFieldName = StringUtils.underscoreName(field.getName());
break;
}
}
if (logger.isDebugEnabled()) {
logger.debug("pk field name:" + pkFieldName);
}
if (pkFieldName == null) {
throw new DBException("can not get pk field name from " + entity);
}
return connection.prepareStatement(sql, new String[] { pkFieldName });
}
```