Coolang25 / SpringCoreExample

0 stars 0 forks source link

Empty catch block #2

Open vzzederzv opened 4 months ago

vzzederzv commented 4 months ago

https://github.com/Coolang25/SpringCoreExample/blob/9350e06f81a270cb7c36280c45ca3dd6ec3bd373/src/main/java/com/trinity/service/AuthorService.java#L39

@Coolang25 Trong mọi trường hợp, chúng ta không nên bỏ trống khối lệnh trong mệnh đề catch, logic căn bản của việc ném ra ngoại lệ (Exception) trong java là khi đoạn code của chúng ta gặp phải 1 trường hợp không thể hoàn thành được thì đoạn code sẽ ném ra 1 ngoại lệ để thông báo cho phần logic bậc cao hơn đang bao bên ngoài để xử lý (ở đây là catch block). Nếu như bỏ trống thì ngoại lệ được ném ra là hoàn toàn vô nghĩa, ít nhất hãy log lại thông tin lỗi để có thể dễ dàng theo dõi.

vzzederzv commented 4 months ago

https://github.com/Coolang25/SpringCoreExample/blob/9350e06f81a270cb7c36280c45ca3dd6ec3bd373/src/main/java/com/trinity/service/BookService.java#L55

Tương tự với trường hợp này