Tôi có lạm dụng "cứu hộ" để kiểm tra không?

Question

Tôi sử dụng rescue cho mọi thứ, không chỉ cho các trường hợp ngoại lệ "cứu". Ý tôi là, tôi chỉ thích cách nó kiểm tra xác minh của tôi và kiểm tra lại dữ liệu.

Ví dụ: giả sử tôi có mô hình Item những gì có thể có hoặc không có User. Sau đó, khi tôi muốn để có được tên của chủ sở hữu của mặt hàng đó tôi viết:

item.user.name rescue "" 

thay vì một cái gì đó giống như

item.user.nil? ? "" : item.user.name 

Nó làm cho những suy nghĩ tương tự, vì nil.name kích hoạt một ngoại lệ mà tôi cứu với "" , nhưng tôi không chắc chắn đây là thực hành tốt. Nó làm cho những gì tôi muốn, và nó làm cho nó với ít mã, nhưng ... Tôi không biết, tất cả những gì rescue từ đây và ở đó làm cho tôi cảm thấy không an toàn.

Thực tiễn không tốt hoặc lạm dụng hợp lệ từ khóa rescue có hợp lệ không?

Answer 1

Tôi nghĩ bạn đang lạm dụng cứu trợ một chút, mặc dù trong Rails, có một phương pháp cụ thể cho những vấn đề này: try. Documentation

Trong trường hợp của bạn, item.user.try(:name) có thể là một cách tiếp cận đẹp hơn.

Answer 2

Giống như hầu hết các ngôn ngữ khác, việc tự kiểm tra sẽ chạy nhanh hơn việc sử dụng cứu hộ.

Answer 3

Tôi cho rằng đây không thực sự là một thói quen tốt để tham gia. Tôi đã không bao giờ thực sự sử dụng tính năng này trong Ruby vì nó cảm thấy như tôi chỉ đang ẩn các trường hợp lỗi. Cũng cần lưu ý rằng bạn đang cứu bất kỳ và tất cả ngoại lệ mà không chỉ định bất kỳ loại lỗi dự kiến ​​nào. Nó luôn luôn trông giống như một cái gì đó mà sẽ làm cho gỡ lỗi xuống đường khó hơn nó cần phải được, mặc dù, như tôi đã nói, tôi đã không bao giờ bận tâm để sử dụng nó bản thân mình.

Answer 4

Thay thế cho việc lạm dụng rescue của bạn, hãy xem đá quý andand. Nó tương tự như try một bài khác được đề xuất, nhưng đẹp hơn. andand cho phép bạn nói:

item.user.andand.name 

Khái niệm sẽ nil nếu item.user là nil.