每天在寫(xiě)Java程序，其實(shí)里面有一些細節大家可能沒(méi)怎么注意，這不，有人總結了一個(gè)我們編程中常見(jiàn)的問(wèn)題。雖然一般沒(méi)有什么大問(wèn)題，但是最好別這樣做。另外這里提到的很多問(wèn)題其實(shí)可以通過(guò)Findbugs( http://findbugs.sourceforge.net/ )來(lái)幫我們進(jìn)行檢查出來(lái)。
字符串連接誤用
錯誤的寫(xiě)法：
String s = '';
for (Person p : persons) {
s = ', ' p.getName();
}
s = s.substring(2); //remove first comma
正確的寫(xiě)法：
StringBuilder sb = new StringBuilder(persons.size() * 16); // well estimated buffer
for (Person p : persons) {
if (sb.length() > 0) sb.append(', ');
sb.append(p.getName);
}
錯誤的使用StringBuffer
錯誤的寫(xiě)法：
StringBuffer sb = new StringBuffer();
sb.append('Name: ');
sb.append(name '\n');
sb.append('!');
...
String s = sb.toString();
問(wèn)題在第三行，append char比String性能要好，另外就是初始化StringBuffer沒(méi)有指定size，導致中間append時(shí)可能重新調整內部數組大小。如果是JDK1.5最好用StringBuilder取代StringBuffer，除非有線(xiàn)程安全的要求。還有一種方式就是可以直接連接字符串。缺點(diǎn)就是無(wú)法初始化時(shí)指定長(cháng)度。
正確的寫(xiě)法：
StringBuilder sb = new StringBuilder(100);
sb.append('Name: ');
sb.append(name);
sb.append('\n!');
String s = sb.toString();
或者這樣寫(xiě)：
String s = 'Name: ' name '\n!';
測試字符串相等性
錯誤的寫(xiě)法：
if (name.compareTo('John') == 0) ...
if (name == 'John') ...
if (name.equals('John')) ...
if (''.equals(name)) ...
上面的代碼沒(méi)有錯，但是不夠好。compareTo不夠簡(jiǎn)潔，==原義是比較兩個(gè)對象是否一樣。另外比較字符是否為空，最好判斷它的長(cháng)度。
正確的寫(xiě)法：
if ('John'.equals(name)) ...
if (name.length() == 0) ...
if (name.isEmpty()) ...
數字轉換成字符串
錯誤的寫(xiě)法：
'' set.size()
new Integer(set.size()).toString()
正確的寫(xiě)法：
String.valueOf(set.size())
利用不可變對象(Immutable)
錯誤的寫(xiě)法：
zero = new Integer(0);
return Boolean.valueOf('true');
正確的寫(xiě)法：
zero = Integer.valueOf(0);
return Boolean.TRUE;
請使用XML解析器
錯誤的寫(xiě)法：
int start = xml.indexOf('<name>') '<name>'.length();
int end = xml.indexOf('</name>');
String name = xml.substring(start, end);
正確的寫(xiě)法：
SAXBuilder builder = new SAXBuilder(false);
Document doc = doc = builder.build(new StringReader(xml));
String name = doc.getRootElement().getChild('name').getText();
請使用JDom組裝XML
錯誤的寫(xiě)法：
String name = ...
String attribute = ...
String xml = '<root>'
'<name att=\'' attribute '\'>' name '</name>'
'</root>';
正確的寫(xiě)法：
Element root = new Element('root');
root.setAttribute('att', attribute);
root.setText(name);
Document doc = new Documet();
doc.setRootElement(root);
XmlOutputter out = new XmlOutputter(Format.getPrettyFormat());
String xml = out.outputString(root);
XML編碼陷阱
錯誤的寫(xiě)法：
String xml = FileUtils.readTextFile('my.xml');
因為xml的編碼在文件中指定的，而在讀文件的時(shí)候必須指定編碼。另外一個(gè)問(wèn)題不能一次就將一個(gè)xml文件用String保存，這樣對內存會(huì )造成不必要的浪費，正確的做法用InputStream來(lái)邊讀取邊處理。為了解決編碼的問(wèn)題, 最好使用XML解析器來(lái)處理。
未指定字符編碼
錯誤的寫(xiě)法：
Reader r = new FileReader(file);
Writer w = new FileWriter(file);
Reader r = new InputStreamReader(inputStream);
Writer w = new OutputStreamWriter(outputStream);
String s = new String(byteArray); // byteArray is a byte[]
byte[] a = string.getBytes();
這樣的代碼主要不具有跨平臺可移植性。因為不同的平臺可能使用的是不同的默認字符編碼。
正確的寫(xiě)法：
Reader r = new InputStreamReader(new FileInputStream(file), 'ISO-8859-1');
Writer w = new OutputStreamWriter(new FileOutputStream(file), 'ISO-8859-1');
Reader r = new InputStreamReader(inputStream, 'UTF-8');
Writer w = new OutputStreamWriter(outputStream, 'UTF-8');
String s = new String(byteArray, 'ASCII');
byte[] a = string.getBytes('ASCII');
未對數據流進(jìn)行緩存
錯誤的寫(xiě)法：
InputStream in = new FileInputStream(file);
int b;
while ((b = in.read()) != -1) {
...
}
上面的代碼是一個(gè)byte一個(gè)byte的讀取，導致頻繁的本地JNI文件系統訪(fǎng)問(wèn)，非常低效，因為調用本地方法是非常耗時(shí)的。最好用BufferedInputStream包裝一下。曾經(jīng)做過(guò)一個(gè)測試，從/dev/zero下讀取1MB，大概花了1s，而用BufferedInputStream包裝之后只需要60ms，性能提高了94%! 這個(gè)也適用于output stream操作以及socket操作。
正確的寫(xiě)法：
InputStream in = new BufferedInputStream(new FileInputStream(file));
無(wú)限使用heap內存
錯誤的寫(xiě)法：
byte[] pdf = toPdf(file);
這里有一個(gè)前提，就是文件大小不能講JVM的heap撐爆。否則就等著(zhù)OOM吧，尤其是在高并發(fā)的服務(wù)器端代碼。最好的做法是采用Stream的方式邊讀取邊存儲(本地文件或database)。
正確的寫(xiě)法：
File pdf = toPdf(file);
另外，對于服務(wù)器端代碼來(lái)說(shuō)，為了系統的安全，至少需要對文件的大小進(jìn)行限制。
不指定超時(shí)時(shí)間
錯誤的代碼：
Socket socket = ...
socket.connect(remote);
InputStream in = socket.getInputStream();
int i = in.read();
這種情況在工作中已經(jīng)碰到不止一次了。個(gè)人經(jīng)驗一般超時(shí)不要超過(guò)20s。這里有一個(gè)問(wèn)題，connect可以指定超時(shí)時(shí)間，但是read無(wú)法指定超時(shí)時(shí)間。但是可以設置阻塞(block)時(shí)間。
正確的寫(xiě)法：
Socket socket = ...
socket.connect(remote, 20000); // fail after 20s
InputStream in = socket.getInputStream();
socket.setSoTimeout(15000);
int i = in.read();
另外，文件的讀取(FileInputStream, FileChannel, FileDescriptor, File)沒(méi)法指定超時(shí)時(shí)間, 而且IO操作均涉及到本地方法調用, 這個(gè)更操作了JVM的控制范圍，在分布式文件系統中，對IO的操作內部實(shí)際上是網(wǎng)絡(luò )調用。一般情況下操作60s的操作都可以認為已經(jīng)超時(shí)了。為了解決這些問(wèn)題，一般采用緩存和異步/消息隊列處理。
頻繁使用計時(shí)器
錯誤代碼：
for (...) {
long t = System.currentTimeMillis();
long t = System.nanoTime();
Date d = new Date();
Calendar c = new GregorianCalendar();
}
每次new一個(gè)Date或Calendar都會(huì )涉及一次本地調用來(lái)獲取當前時(shí)間(盡管這個(gè)本地調用相對其他本地方法調用要快)。
如果對時(shí)間不是特別敏感，這里使用了clone方法來(lái)新建一個(gè)Date實(shí)例。這樣相對直接new要高效一些。
正確的寫(xiě)法：
Date d = new Date();
for (E entity : entities) {
entity.doSomething();
entity.setUpdated((Date) d.clone());
}
如果循環(huán)操作耗時(shí)較長(cháng)(超過(guò)幾ms)，那么可以采用下面的方法，立即創(chuàng )建一個(gè)Timer，然后定期根據當前時(shí)間更新時(shí)間戳，在我的系統上比直接new一個(gè)時(shí)間對象快200倍：
private volatile long time;
Timer timer = new Timer(true);
try {
time = System.currentTimeMillis();
timer.scheduleAtFixedRate(new TimerTask() {
public void run() {
time = System.currentTimeMillis();
}
}, 0L, 10L); // granularity 10ms
for (E entity : entities) {
entity.doSomething();
entity.setUpdated(new Date(time));
}
} finally {
timer.cancel();
}
捕獲所有的異常
錯誤的寫(xiě)法：
Query q = ...
Person p;
try {
p = (Person) q.getSingleResult();
} catch(Exception e) {
p = null;
}
這是EJB3的一個(gè)查詢(xún)操作，可能出現異常的原因是：結果不唯一；沒(méi)有結果；數據庫無(wú)法訪(fǎng)問(wèn)，而捕獲所有的異常，設置為null將掩蓋各種異常情況。
正確的寫(xiě)法：
Query q = ...
Person p;
try {
p = (Person) q.getSingleResult();
} catch(NoResultException e) {
p = null;
}
忽略所有異常
錯誤的寫(xiě)法：
try {
doStuff();
} catch(Exception e) {
log.fatal('Could not do stuff');
}
doMoreStuff();
這個(gè)代碼有兩個(gè)問(wèn)題, 一個(gè)是沒(méi)有告訴調用者, 系統調用出錯了. 第二個(gè)是日志沒(méi)有出錯原因, 很難跟蹤定位問(wèn)題。
正確的寫(xiě)法：
try {
doStuff();
} catch(Exception e) {
throw new MyRuntimeException('Could not do stuff because: ' e.getMessage, e);
}
“圖靈it教育”是今日圖靈it教育培訓機構專(zhuān)門(mén)的信息交流、分享平臺，哈爾濱今日靈圖職業(yè)培訓學(xué)校專(zhuān)注IT培訓15年，秉承著(zhù) “以實(shí)戰為導向，以質(zhì)量為中心”的教學(xué)理念，隨著(zhù)全球IT技術(shù)的不斷進(jìn)步而不斷發(fā)展創(chuàng )新！
