作者willieliao (Willie Liao)
看板java
標題Re: [問題] 請教在linux跑的java錯誤訊息
時間Thu Dec 27 13:53:23 2007
※ 引述《willieliao (Willie Liao)》之銘言：
: 你的程式有memory leak，也就是說程式跑越久memory就越用越多，
: 而且garbage collection之後也不會下降，最後就是出現outofmemory exception
: 這種情況短期治標是加大memory（-Xmx多少m)，長期治本當然是要看哪裡有memory
: leak，也就是物件越漲越大，reference的物件越來越多，而沒有從memory中releas
: e掉。雖然java 有garbage collection，但是有reference到的物件是不會被collect的
....
: 推 dwi2:推一下這篇!我之前也有遇過這種問題 11/24 20:18
: 推 slalala:不知道可否請您分享 讓新手注意如何建少記體用量的技巧呢? 11/24 22:53
: → slalala:因為我寫的批次處理的程式 都還是會使記憶體用量偏大 11/24 22:55
: → slalala:除非縮視窗才會釋放些空間應該有些地方觀念上寫的不夠嚴謹 11/24 22:56
不好意思回一下有點久的文章，因為我現在才看到有人推文問問題。一般沒有寫過
c++直接寫java的新手因為沒有destroy object的觀念，特別容易造成memory leak
首先我要強力推薦使用profiler，我們公司是用要付費的borland，不過免費的
jconsole（附在jdk1.5以上，到jdk_home/bin找jconsole.exe）更好用，而且有
強大的find thread deadlock功能（大感謝！讓我少死很多腦細胞）。
如果你找不出來哪裡有memory leak的話，讓程式run一段時間用profiler去監視
哪種class的instance數量一直上升不下降，那就有可能是來源。
回到程式本身，一些小技巧如下：
第一，絕對不要寫出無限迴圈！即使是有限迴圈，也要盡量減少跑的次數。
public int getThis() {
return this.
getThis();
}
以上的程式編譯會過，但是一跑保證不到一秒就會出現stackoverflowerror。
第二，盡量不要保留intermediate（過渡）的物件
最耗記憶體的寫法：
public void writeXML(FileWriter writer) {
Vector tempV = new Vector();
tempV.add("<root>");
for (int i = 0; i < 10000; i++) {
tempV.add("<value> + String.valueOf(i) + "</value>");
}
tempV.add("</root>");
for (int j = 0; j < tempV.size(), j++) {
writer.write(tempV.get(j)); //這裡不需要cast
}
writer.close();
}
這個call會暫存10002個string objects..
有點sense的寫法：
public void writeXML(FileWriter writer) {
StringBuffer sb = new StringBuffer();
sb.append("<root>");
for (int i = 0; i < 10000; i++) {
sb.append("<value>" + String.valueOf(i) + "</value>");
}
sb.append("</root>");
writer.write(sb.toString());
writer.close();
}
這個寫法只會有一個stringbuffer的instance，但是這個instance會很大。
最省記憶體的寫法
public void writeXML(FileWriter writer) {
writer.write("<root>");
for (int i = 0; i < 10000; i++) {
writer.write("<value>" + String.valueOf(i) + "</value>");
}
writer.write("</root>");
writer.close();
}
一邊生成一邊寫出最省記憶體..
第三，不需要的物件要清掉，也就是把本來指向這個物件的variable
指向null
private HashMap tempHashMap = new HashMap();
public void doSomething() {
for (int i = 0; i < 100000; i++) {
tempHashMap.put(new Object(), new Object());
}
...
}
這個doSomething()假設你call他1000次，程式還是會執行無誤。但是，你的
tempHashMap就會長大成1000倍，不但變慢還會memory leak。
解決之道要嗎把變數搬進method裡，要嗎像這樣：
public void doSomething() {
for (int i = 0; i < 100000; i++) {
tempHashMap.put(new Object(), new Object());
}
...
tempHashMap.clear();
}
或是
private HashMap tempHashMap;
public void doSomething() {
tempHashMap = new HashMap();
for (int i = 0; i < 100000; i++) {
tempHashMap.put(new Object(), new Object());
}
...
tempHashMap = null;
}
這兩種方法的的效果其實差不多，不過後者更省記憶體一點。壞處是有的時候不小心
null pointer就會跑出來。
最後，盡量避免一些java中memory intensive的api calls。
這個的話真的要靠經驗。比方說，java的regular expression和 JAXP（java API
for xml）裡面都有很多吃memory的怪獸（美國這邊我們叫它memory hog）。碰到的
話也只好用profiler找出來在想辦法避開了。
大概就想到這麼多吧，板上的神人大大門不要客氣盡量鞭...
Willie
--
※ 發信站: 批踢踢實業坊(ptt.cc)
◆ From: 76.111.69.123
→ willieliao:好像錯字很多，各位大大多包含，中文越來越不行了 12/27 14:07
推 PsMonkey:第二點看不太懂.... 12/27 15:01
→ willieliao:改一下好了，其實2跟三差不多 12/27 15:20
→ willieliao:我改一下例子，請大家暫時不要推文 12/27 15:43
※ 編輯: willieliao 來自: 76.111.69.123 (12/27 16:07)
→ willieliao:改好了，版大看看有沒有比較通順點 12/27 16:08
推 kojilin:第一個應該是噴StackOverflowError之類的? 12/27 16:17
→ willieliao:nice catch, 我改一下。 12/27 16:43
※ 編輯: willieliao 來自: 76.111.69.123 (12/27 16:46)
→ willieliao:第一點其實應該要解釋一下outofMemorryError (HEAP) 12/27 17:01
→ willieliao:跟stackoverflowerror（STACK）的區別，不過太晚了明天 12/27 17:02
→ willieliao:在來改好了 12/27 17:03
推 PsMonkey:ㄜ... 還有，第三點的例子... 個人覺得不是很恰當 XD 12/27 18:47
推 qrtt1:第三個例子是很多短命鬼 12/27 21:04
推 linshihpong:有教學總比看到教學再抓錯好...推一個~ 12/27 21:10
→ willieliao:恩 舉第三點的例子是因為我下面的junior programmer 12/27 23:13
→ willieliao:常常cache用完不清掉...我再來想想有沒有更好的例子 12/27 23:14
※ weii:轉錄至看板 SFFamily 12/28 19:39
推 freesamael:我以前都不知道java也會有memory leak耶...@@ 12/28 22:53
→ freesamael:<--- 一直都只會寫 c++ 12/28 22:54
→ freesamael:我只知道c++要擔心leak之外還要擔心double free...XD" 12/28 22:54
推 qrtt1:gc algorithm 沒設計好就會 leak 了 XD 12/28 22:58
→ qrtt1:或是有不受 gc 管理的資源 leak 了 12/28 22:59
推 godfat:該放掉的資源沒放掉就會 leak, 有沒有 gc 都會 leak 12/28 23:17