Nó không phải là một thực hành tốt để xử lý các ngoại lệ thời gian chạy trong mã?

Tôi đang làm việc trên một ứng dụng Java và tôi thấy rằng các ngoại lệ thời gian chạy được xử lý ở nhiều nơi. Ví dụ,

try {
    // do something
} catch(NullPointerException e) {
    return null;
}

Câu hỏi của tôi là, khi nào thì nên xử lý ngoại lệ Runtime? Khi nào nên để lại ngoại lệ?

Nó phụ thuộc.

Ví dụ: Integer#parseIntném NumberFormatException(là RTE) nếu chuỗi được cung cấp không thể được phân tích cú pháp. Nhưng chắc chắn bạn không muốn ứng dụng của mình bị sập chỉ vì người dùng đã viết "x" vào trường văn bản dành cho số nguyên? Và làm thế nào để bạn biết liệu chuỗi có thể được phân tích cú pháp, trừ khi bạn cố gắng phân tích cú pháp trước? Vì vậy, trong trường hợp này, RTE chỉ là một tín hiệu lỗi sẽ gây ra một số loại thông báo lỗi. Người ta có thể lập luận rằng đó phải là một ngoại lệ được kiểm tra, nhưng bạn có thể làm gì - không phải vậy.

NullPulumExceptions thường là dấu hiệu của một kiểm tra null bị thiếu. Vì vậy, thay vì bắt nó như thế này, bạn nên thêm kiểm tra null phù hợp để đảm bảo không ném ngoại lệ.

Nhưng đôi khi, nó là thích hợp để xử lý RunTimeExceptions. Ví dụ: khi bạn không thể sửa đổi mã để thêm kiểm tra null tại địa điểm thích hợp hoặc khi ngoại lệ là một thứ gì đó không phải là NullPulumException.

Ví dụ của bạn về xử lý ngoại lệ là khủng khiếp. Làm như vậy, bạn mất dấu vết ngăn xếp và thông tin chính xác về vấn đề. Và bạn thực sự không giải quyết nó vì có thể bạn sẽ kích hoạt một NullPulumException khác ở một nơi khác và nhận thông tin sai lệch về những gì đã xảy ra và cách giải quyết nó.

Tôi xử lý các trường hợp ngoại lệ dự kiến ​​nơi tôi mong đợi chúng. (Giống như lỗi đọc / ghi DB). Trường hợp ngoại lệ bất ngờ tôi bong bóng lên. Một nơi nào khác có thể đang mong đợi ngoại lệ và có logic cho nó.

Ngoại lệ chỉ nên là .. ngoại lệ. Thực hành tốt nhất khi sử dụng các ngoại lệ là sử dụng chúng để bao quát tình huống trong đó điều gì đó trái ngược với những gì bạn sẽ xảy ra. Ví dụ kinh điển là FileNotFoundException sẽ bị ném khi một tệp đơn giản là không có ở đó. Nếu bạn đang kiểm tra sự tồn tại của tệp, thì bạn sử dụng File.exists () vì bạn chỉ đơn giản là dùng một cây gậy 10 chân để xem bạn có đánh thứ gì không.

Về mặt kỹ thuật, bạn có thể hoàn thành các kết quả tương tự bằng cách bao quanh nó trong một lần thử và sử dụng tệp như thể nó tồn tại, nhưng các ngoại lệ A) thường tốn kém về tài nguyên và các lập trình viên sẽ cho rằng bạn có nghĩa là tệp tồn tại nếu nó tồn tại trong một thử bắt, làm tăng thêm sự nhầm lẫn chung của một chương trình.

Có nhiều tình huống trong đó tôi sẽ viết một phương thức lấy một số giá trị từ cơ sở dữ liệu. Một ngàn điều có thể sai, và xem làm thế nào tôi chỉ cần một mẩu thông tin nhỏ, thật bất tiện khi bao quanh cuộc gọi với một danh sách thử bắt có 5 trường hợp ngoại lệ khác nhau. Vì vậy, tôi sẽ bắt ngoại lệ trong phương thức tìm nạp. Nếu có lỗi xảy ra, tôi thực hiện bất kỳ hành động thích hợp nào để đóng kết nối cơ sở dữ liệu hoặc không có gì trong mệnh đề cuối cùng và trả về null. Đây là cách thực hành tốt không chỉ vì nó đơn giản hóa mã của bạn mà còn bởi vì "null" gửi cùng một thông điệp mà bạn có thể nhận được từ một ngoại lệ .. rằng một cái gì đó đã không đi theo kế hoạch. Quản lý các chi tiết cụ thể ngoại lệ trong phương thức tìm nạp, nhưng quản lý những việc cần làm khi mọi thứ không '

Ví dụ:

Integer getUserCount() {
   Integer result = null;
   try {
      // Attempt to open database and retrieve data
   } catch (TimeoutException e) {
      logger.error("Got a watch?");
   } catch (MissingDatabaseException e) {
      logger.error("What are you smoking?");
   } catch (PermissionsToReadException e) {
      logger.error("Did you *really* think you were getting away with that?");
   } catch (PressedSendButtonToHardException e) {
      logger.error("Seriously.. just back away from the computer... slowly..");
   } catch (WTFException e) {
      logger.error("You're on your own with this one.. I don't even know what happened..");
   } finally {
      // Close connections and whatnot
   }
   return result;
}

void doStuff() {
   Integer result = getUserCount();
   if(result != null) {
       // Went as planned..
   }
}

vâng, bạn đang xử lý chính xác các ngoại lệ về thời gian chạy không phải là một cách thực hành tốt. Lý do là nó được coi là tốn kém / tốn bộ nhớ.