[轉錄]關於 memory leak 的幾個 tip

看板SFFamily作者 (迷惑失道)時間16年前 (2007/12/28 19:39), 編輯推噓0(000)
留言0則, 0人參與, 最新討論串1/1
※ [本文轉錄自 java 看板] 作者: 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

12/27 14:07,
好像錯字很多,各位大大多包含,中文越來越不行了
12/27 14:07

12/27 15:01,
第二點看不太懂....
12/27 15:01

12/27 15:20,
改一下好了,其實2跟三差不多
12/27 15:20

12/27 15:43,
我改一下例子,請大家暫時不要推文
12/27 15:43
※ 編輯: willieliao 來自: 76.111.69.123 (12/27 16:07)

12/27 16:08,
改好了,版大看看有沒有比較通順點
12/27 16:08

12/27 16:17,
第一個應該是噴StackOverflowError之類的?
12/27 16:17

12/27 16:43,
nice catch, 我改一下。
12/27 16:43
※ 編輯: willieliao 來自: 76.111.69.123 (12/27 16:46)

12/27 17:01,
第一點其實應該要解釋一下outofMemorryError (HEAP)
12/27 17:01

12/27 17:02,
跟stackoverflowerror(STACK)的區別,不過太晚了明天
12/27 17:02

12/27 17:03,
在來改好了
12/27 17:03

12/27 18:47,
ㄜ... 還有,第三點的例子... 個人覺得不是很恰當 XD
12/27 18:47

12/27 21:04,
第三個例子是很多短命鬼
12/27 21:04

12/27 21:10,
有教學總比看到教學再抓錯好...推一個~
12/27 21:10

12/27 23:13,
恩 舉第三點的例子是因為我下面的junior programmer
12/27 23:13

12/27 23:14,
常常cache用完不清掉...我再來想想有沒有更好的例子
12/27 23:14
-- 三月的柳絮不飛 妳的心如小小的寂寞的城 我達達的馬蹄是美麗的錯誤 我不是歸人 我是馬~ -- ※ 發信站: 批踢踢實業坊(ptt.cc) ◆ From: 220.132.117.169
文章代碼(AID): #17TE1dUn (SFFamily)