※ 引述《garylee.bbs@bbs.yzu.edu.tw (www.dev.idv.tw)》之銘言:
> 1. 先把while中有關GUI的code全部拿掉。要顯示進度的話,不用每筆都更新吧。
> 一千筆更新一次就行了。我測試過了把progressbar從0跑到610000要花好幾分鐘。
> 2. while loop中的code越精簡越好。
> 3. if (atoi(&NC.....) 這裡有bug。你的程式沒crash掉已經是運氣好了。
> 如果只是要判斷一個字元是不是"0",何不寫成下面這樣呢?
> if (tmpStr[0] == '0' && tmpStr[1] == '0')
> 4. 那個參數有SrcData[i].Caller與SrcData[i].Callee的strcpy是在幹嘛?
> 感覺有點多餘。
> 5. 另外,SrcData也許可以在外面用memset一類的function將他全部清成0吧。這樣
> 在迴圈內就不用為每個element設定初值了。
> 6. 你應該沒有在while loop中做任何配置記憶體的行為吧?千萬別這樣做喔。
感謝...會嘗試看看.....
這樣解釋就比較清楚了....
--
◤◥ Origin: 幽谷˙反地球聯邦組織 aeug.twbbs.org
◣◢ Author: streit 從 anime.animemusic.idv.tw 發表