Tôi có một đoạn mã có thể được biểu diễn dưới dạng:
public class ItemService {
public void DeleteItems(IEnumerable<Item> items)
{
// Save us from possible NullReferenceException below.
if(items == null)
return;
foreach(var item in items)
{
// For the purpose of this example, lets say I have to iterate over them.
// Go to database and delete them.
}
}
}Bây giờ tôi tự hỏi liệu đây là cách tiếp cận đúng hay tôi nên ném ngoại lệ. Tôi có thể tránh ngoại lệ, bởi vì trả về sẽ giống như lặp đi lặp lại một bộ sưu tập trống, có nghĩa là, không có mã quan trọng nào được thực thi, nhưng mặt khác tôi có thể ẩn các vấn đề ở đâu đó trong mã, bởi vì tại sao mọi người muốn gọi DeleteItemsvới nulltham số nào? Điều này có thể chỉ ra rằng có một vấn đề ở đâu đó trong mã.
Đây là một vấn đề tôi thường gặp phải với các phương thức trong dịch vụ, vì hầu hết chúng đều làm gì đó và không trả về kết quả, vì vậy nếu ai đó truyền thông tin không hợp lệ thì dịch vụ sẽ không có gì để làm, vì vậy nó sẽ trả về.
2
Đây chỉ là ý kiến cá nhân của tôi, nhưng tôi luôn thấy nó có ý nghĩa nhất là một phương pháp nên đưa ra một ngoại lệ khi nó làm điều gì đó ngoài ý muốn (ngoại lệ). Trong trường hợp của bạn, tôi sẽ ném một UnlimitedOperationException nếu ai đó cố xóa các mục null / 0.
—
Falgantil
@gnat Câu hỏi của tôi là cụ thể về các dịch vụ, vì cách chúng được sử dụng và mục đích của chúng. "Bản sao" này chỉ nói về các trường hợp chung và câu trả lời chính thậm chí mâu thuẫn với chính nó trong bối cảnh phương pháp chính xác của tôi.
—
FCin
Có thể cho vô số để trống thay vì null? Bạn sẽ xử lý khác nhau?
—
JAD
@Facheantil Tôi có thể thấy null xứng đáng là một ngoại lệ, nhưng tôi nghĩ thật vô lý khi ném một ngoại lệ vào một danh sách trống.
—
Kevin