Java編程常見問題彙總(一)

字符串鏈接誤用

錯誤的寫法：

String s = "";  
for (Person p : persons) {  
    s += ", " + p.getName();  
}  
s = s.substring(2); //remove first comma
複製代碼

正確的寫法：

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

錯誤的寫法：

StringBuffer sb = new StringBuffer();  
sb.append("Name: ");  
sb.append(name + '\n');  
sb.append("!");  
...  
String s = sb.toString();
複製代碼

問題在第三行，append char比String性能要好，另外就是初始化StringBuffer沒有指定size，致使中間append時可能從新調整內部數組大小。若是是JDK1.5最好用StringBuilder取代StringBuffer，除非有線程安全的要求。還有一種方式就是能夠直接鏈接字符串。缺點就是沒法初始化時指定長度。

正確的寫法：

StringBuilder sb = new StringBuilder(100);  
sb.append("Name: ");  
sb.append(name);  
sb.append("\n!");  
String s = sb.toString();
複製代碼

或者這樣寫：

String s = "Name: " + name + "\n!";
複製代碼

測試字符串相等性

錯誤的寫法：

if (name.compareTo("John") == 0) ...  
if (name == "John") ...  
if (name.equals("John")) ...  
if ("".equals(name)) ...
複製代碼

上面的代碼沒有錯，可是不夠好。compareTo不夠簡潔，==原義是比較兩個對象是否同樣。另外比較字符是否爲空，最好判斷它的長度。

正確的寫法：

if ("John".equals(name)) ...  
if (name.length() == 0) ...  
if (name.isEmpty()) ...
複製代碼

數字轉換成字符串

錯誤的寫法：

"" + set.size()  
new Integer(set.size()).toString()
複製代碼

正確的寫法：

String.valueOf(set.size())
複製代碼

利用不可變對象(Immutable)

錯誤的寫法：

zero = new Integer(0);  
return Boolean.valueOf("true");
複製代碼

正確的寫法：

zero = Integer.valueOf(0);  
return Boolean.TRUE;
複製代碼

請使用XML解析器

錯誤的寫法：

int start = xml.indexOf("<name>") + "<name>".length();  
int end = xml.indexOf("</name>");  
String name = xml.substring(start, end);
複製代碼

正確的寫法：

SAXBuilder builder = new SAXBuilder(false);  
Document doc = doc = builder.build(new StringReader(xml));  
String name = doc.getRootElement().getChild("name").getText();
複製代碼

請使用JDom組裝XML

錯誤的寫法：

String name = ...
String attribute = ...
String xml = "<root>"
+"<name att=""+ attribute +"">"+ name +"</name>"
+"</root>";
正確的寫法：

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編碼陷阱

錯誤的寫法：

String xml = FileUtils.readTextFile("my.xml");
複製代碼

由於xml的編碼在文件中指定的，而在讀文件的時候必須指定編碼。另一個問題不能一次就將一個xml文件用String保存，這樣對內存會形成沒必要要的浪費，正確的作法用InputStream來邊讀取邊處理。爲了解決編碼的問題, 最好使用XML解析器來處理。

未指定字符編碼

錯誤的寫法：

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();
複製代碼

這樣的代碼主要不具備跨平臺可移植性。由於不一樣的平臺可能使用的是不一樣的默認字符編碼。

正確的寫法：

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");
複製代碼

未對數據流進行緩存

錯誤的寫法：

InputStream in = new FileInputStream(file);   
int b;   
while ((b = in.read()) != -1) {   
...   
}
複製代碼

上面的代碼是一個byte一個byte的讀取，致使頻繁的本地JNI文件系統訪問，很是低效，由於調用本地方法是很是耗時的。最好用BufferedInputStream包裝一下。曾經作過一個測試，從/dev/zero下讀取1MB，大概花了1s，而用BufferedInputStream包裝以後只須要60ms，性能提升了94%! 這個也適用於output stream操做以及socket操做。

正確的寫法：

InputStream in = new BufferedInputStream(new FileInputStream(file));
複製代碼

無限使用heap內存

錯誤的寫法：

byte[] pdf = toPdf(file);
複製代碼

這裏有一個前提，就是文件大小不能講JVM的heap撐爆。不然就等着OOM吧，尤爲是在高併發的服務器端代碼。最好的作法是採用Stream的方式邊讀取邊存儲(本地文件或database)。

正確的寫法：

File pdf = toPdf(file);
複製代碼

另外，對於服務器端代碼來講，爲了系統的安全，至少須要對文件的大小進行限制。

不指定超時時間

錯誤的代碼：

Socket socket = ...   
socket.connect(remote);   
InputStream in = socket.getInputStream();   
int i = in.read();
複製代碼

這種狀況在工做中已經碰到不止一次了。我的經驗通常超時不要超過20s。這裏有一個問題，connect能夠指定超時時間，可是read沒法指定超時時間。可是能夠設置阻塞(block)時間。

正確的寫法：

Socket socket = ...   
socket.connect(remote, 20000); // fail after 20s   
InputStream in = socket.getInputStream();   
socket.setSoTimeout(15000);   
int i = in.read();
複製代碼

另外，文件的讀取(FileInputStream, FileChannel, FileDescriptor, File)無法指定超時時間, 並且IO操做均涉及到本地方法調用, 這個更操做了JVM的控制範圍，在分佈式文件系統中，對IO的操做內部其實是網絡調用。通常狀況下操做60s的操做均可以認爲已經超時了。爲了解決這些問題，通常採用緩存和異步/消息隊列處理。

頻繁使用計時器

錯誤代碼：

for (...) {   
long t = System.currentTimeMillis();   
long t = System.nanoTime();   
Date d = new Date();   
Calendar c = new GregorianCalendar();   
}
複製代碼

每次new一個Date或Calendar都會涉及一次本地調用來獲取當前時間(儘管這個本地調用相對其餘本地方法調用要快)。
若是對時間不是特別敏感，這裏使用了clone方法來新建一個Date實例。這樣相對直接new要高效一些。

正確的寫法：

Date d = new Date();   
for (E entity : entities) {   
entity.doSomething();   
entity.setUpdated((Date) d.clone());   
}
複製代碼

若是循環操做耗時較長(超過幾ms)，那麼能夠採用下面的方法，當即建立一個Timer，而後按期根據當前時間更新時間戳，在個人系統上比直接new一個時間對象快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();   
}
複製代碼

捕獲全部的異常

錯誤的寫法：

Query q = ...   
Person p;   
try {   
p = (Person) q.getSingleResult();   
} catch(Exception e) {   
p = null;   
}
複製代碼

這是EJB3的一個查詢操做，可能出現異常的緣由是：結果不惟一；沒有結果；數據庫沒法訪問，而捕獲全部的異常，設置爲null將掩蓋各類異常狀況。

正確的寫法：

Query q = ...   
Person p;   
try {   
p = (Person) q.getSingleResult();   
} catch(NoResultException e) {   
p = null;   
}
複製代碼

忽略全部異常

錯誤的寫法：

try {   
doStuff();   
} catch(Exception e) {   
log.fatal("Could not do stuff");   
}   
doMoreStuff();
複製代碼

這個代碼有兩個問題, 一個是沒有告訴調用者, 系統調用出錯了. 第二個是日誌沒有出錯緣由, 很難跟蹤定位問題。

正確的寫法：

try {   
doStuff();   
} catch(Exception e) {   
throw new MyRuntimeException("Could not do stuff because: "+ e.getMessage, e);   
}
複製代碼

歡迎加入學習交流羣569772982，你們一塊兒學習交流。