maily89 / fbdm

Automatically exported from code.google.com/p/fbdm
0 stars 0 forks source link

[Code review] INV Summary Rank #118

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
File: IndividualSummaryRank.cs

- AddRank(IndividualSummaryRanks rank): Hàm này không thấy xài tới -> 
thừa code?

- Hàm AddRank() -> nên bỏ try catch đi?

Original issue reported on code.google.com by shio.se...@gmail.com on 5 Apr 2011 at 1:45

GoogleCodeExporter commented 9 years ago
lỗi thừa code là đúng, còn hàm addrank ko nên bỏ catch, lỡ ko 
kết nối được với csdl hoặc lỗi gì đó có thể tạo nên 
exception, mà thực ra thì như nhau, đằng nào tầng trên cũng bắt, 
nói chung là ko cần bỏ try catch đi :)

Original comment by dragontl...@gmail.com on 9 Apr 2011 at 1:51