我有一个场景,我正在使用一个Query对象和一个StringBuffer
,我将构建一个SQL查询来执行.
这里是:
countQueryBuf.append("Select count(e) From " + clz.getSimpleName() + " e");
在这里,我的大四学生告诉我,使用字符串文字是不好的做法"Select count(e) from "
.我们可以做到:
string public final static selectCount="Select count(e) From "`
在一个接口中,然后在字符串缓冲区中使用它.但是在看到这个与java中的静态字段的接口共享"常量"问题之后,这说明这是不好的做法,我很困惑.谁能告诉我什么是最好的方法来证明我的情景?
首先,应该强调的是,你质疑的先决条件是有缺陷的.Java中的字符串文字不会生成多个String
实例,因此定义指向这些String
实例的常量不会更改实例数.但是,很可能,你的大四学生并不打算用这个建议讨论实例的数量.
字符串文字可以被视为常量值,如123
或44.1f
.当这些值出现在代码中的某个位置时,它们通常被称为"魔术文字",因为它们看起来是虚假的,没有可识别的来源.在这些情况下,应该首选使用名称解释其来源的命名常量.例如
static final float COMPACT_DISC_FREQ_KHZ = 44.1f;
告诉你一些事情.
相比之下,常数就像
static final int ONE = 1;
不要告诉你任何事情,也没有改进,只是试图伪造更好的编码风格.我认为是一个常数
static final String selectCount="Select count(e) From ";
具有相同的荒谬类别,因为它的名称并没有告诉我任何我从常数值中看不到的东西.
无论是将命名常量放入interface
普通常量还是普通常量中class
都没有太大区别.但是在过去,常量被放置在接口中,目的是实现接口以基本导入这些常量,以便能够通过简单的名称引用它们.它不是将常量放置到界面中,这使得它成为一种糟糕的编码风格,而是实际上没有意义的类型继承关系,它只是为了节省源代码中的输入而存在.从Java 5开始,您可以使用import static
将常量放入所需的任何类型,并通过简单的名称引用它们,而不会产生可疑的继承关系.所以在大多数情况下,你不想使用interface
它.
正如其他人已经指出的那样,您的代码还存在其他问题.在大多数用例中StringBuffer
已被取代StringBuilder
,此外,将String
串联与任一用法StringBuffer
或StringBuilder
用法混合起来没有多大意义.
使用
countQueryBuf.append("Select count(e) From ").append(clz.getSimpleName()).append(" e");
一致的,如果countQueryBuf
需要现有的,即是否有其他片段要追加.如果查询只包含这三个片段,那么代码就像
String query = "Select count(e) From "+clz.getSimpleName()+" e";
是优选的.在Java 5之前,它使用了一个StringBuffer
底层,从Java 5开始,它将被编译为使用一个StringBuilder
并从Java 9开始,它将被编译为使用内置的String concat工厂.换句话说,这个简单的表达式将在(重新)编译时自动获得未来改进的好处,而处理StringBuffer
或StringBuilder
手动需要维护并有时需要重写代码以赶上这样的开发.
如果查询的片段代表值,则应始终使用a PreparedStatement
而不是每次组合新的查询字符串...