Thực hành không tốt là viết một phương thức không làm gì ngoại trừ việc ném một ngoại lệ?

Question

Tiêu đề khá nhiều nói nhưng đây là một số nền:

Tôi có ứng dụng ASP.Net MVC nơi tôi cần kiểm tra danh sách đường dẫn tệp để tồn tại. Nếu bất kỳ đường dẫn nào không tồn tại thì một lỗi sẽ được trả về.

Hiện tại, tôi có bộ điều khiển cơ sở nơi sự kiện OnException được triển khai. Ở đây, bất kỳ ngoại lệ không được xử lý nào được xử lý và trang lỗi được trả lại cho người dùng bằng thông báo của ngoại lệ. Cách đơn giản nhất để tôi thực hiện kiểm tra ở trên là viết một phương thức kiểm tra từng đường dẫn để tồn tại và nếu bất kỳ đường dẫn nào bị lỗi, tôi chỉ cần ném (và đăng nhập) một ngoại lệ. Ngoại lệ này sau đó được xử lý bởi bộ điều khiển cơ sở và thông điệp thích hợp được trả về cho người dùng.

Vấn đề của tôi là làm điều này giống như thực hành không tốt. Tôi viết một phương thức trả về khoảng trống và mục đích duy nhất của nó là ném một ngoại lệ trong trường hợp hiếm hoi mà một trong các đường dẫn không tồn tại, trong hầu hết các trường hợp nó không làm gì cả. Đây có phải là một ý tưởng tồi không?

Answer 1

Không có gì sai với điều đó.

Khuôn khổ .NET cũng thực hiện điều này: ví dụ: CancellationToken có phương thức ThrowIfCancellationRequested không có gì ngoài việc ném hoặc không ném tùy thuộc vào một số điều kiện.

Ví dụ khác: Dispatcher Phương thức VerifyAccess, kiểm tra xem người gọi có nằm trên cùng một luồng với điều khiển được cho phép truy cập và ném nếu không.

Answer 2

Trên .net tồn tại tùy chọn để thực hiện thao tác ném mới notimplementedexception, vì vậy bạn có thể tạo các phương thức và triển khai chúng sau này. Vì vậy, nó không phải là một thực tế xấu, thực hành xấu là để lại cho họ ở đó khi phát hành ứng dụng để sản xuất. Đưa ra lỗi không có lý do là thực hành không tốt.

Đối với TDD (phát triển theo hướng thử nghiệm) có thể rất hữu ích, bạn tạo phương pháp và sau đó kiểm tra đơn vị không thành công với ngoại lệ chưa được thực hiện và cuối cùng bạn triển khai phương pháp để vượt qua bài kiểm tra.

Btw đang trả lời tiêu đề câu hỏi của bạn, nhưng bạn nên đổi tên câu hỏi vì phương pháp của bạn làm điều gì đó. Nếu nó làm một cái gì đó và luôn luôn ném một ngoại lệ là thực hành xấu, trường hợp ngoại lệ là tốn kém, bạn nên đăng nhập lỗi và di chuyển trên, mà không có ngoại lệ. Bạn nên làm một hàm PathExists trả về một Boolean, đó là một giải pháp tốt hơn. (ngay cả khi ai đó bỏ phiếu cho tôi -1 vì không có lý do ... Heheh)

Answer 3

Một số người có thể nói rằng đó là một ý tưởng tồi nhưng đôi khi, không có sự thay thế lành mạnh. Nếu bạn muốn đưa ra một ngoại lệ để truyền đạt một lỗi trở lại cho một số người gọi, (có thể bị cô lập khỏi trình kích hoạt bằng mã của bên thứ ba đục), sau đó làm điều đó. Trọng tài cuối cùng - 'nó có hoạt động không?'