我有一个场景,我使用Query对象和StringBuffer,我将在其中构建一个要执行的SQL查询.
这里是:
countQueryBuf.append("Select count(e) From " + clz.getSimpleName()
+ " e");
在这里,我的大四学生告诉我,使用像“从中选择计数(e)”这样的字符串文字是不好的做法.我们可以做到:
string public final static selectCount="Select count(e) From "`
在一个接口中,然后在字符串缓冲区中使用它.但在看到这个Interfaces with static fields in java for sharing ‘constants’问题之后,我觉得这很糟糕.谁能告诉我什么是最好的方法来证明我的情景?
解决方法:
首先,应该强调的是,你质疑的先决条件是有缺陷的. 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 ";
具有相同的荒谬类别,因为它的名称并没有告诉我任何我从常数值中看不到的东西.
无论是将命名常量放入接口还是普通类都没有太大区别.但是在过去,常量被放置在接口中,目的是实现接口以基本导入这些常量,以便能够通过简单的名称引用它们.它不是将常量放置到界面中,这使得它成为一种糟糕的编码风格,而是实际上没有意义的类型继承关系,它只是为了节省源代码中的输入而存在.从Java 5开始,您可以使用import static将常量放入您希望的任何类型,并通过简单的名称引用它们,而不会创建可疑的继承关系.因此,在大多数情况下,您不希望使用接口.
正如其他人已经指出的那样,您的代码还存在其他问题.对于大多数用例,StringBuilder已被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则需要维护并有时重写代码以赶上这样的开发.
如果查询的片段代表值,则应始终使用PreparedStatement而不是每次组合新的查询字符串…
标签:java,string
来源: https://codeday.me/bug/20190715/1465281.html