Boolean return của set.add () trong nếu có điều kiện?

Toán tử add của lớp set trả về một boolean là true nếu phần tử (được thêm vào) chưa có ở đó và ngược lại là false. Đang viết

if (set.add(entry)) {
    //do some more stuff
}

coi phong cách tốt về mặt viết mã sạch? Tôi tự hỏi vì bạn làm hai việc cùng một lúc. 1) thêm phần tử và 2) kiểm tra xem phần tử có tồn tại không.

Vâng, đúng vậy.

Điểm thông thường của việc có một hoạt động trả về giá trị Boolean là bạn có thể sử dụng nó để đưa ra quyết định, tức là trong một ifcấu trúc. Cách khác duy nhất bạn có thể nhận ra tiềm năng này là lưu trữ giá trị trả về trong một biến và sau đó sử dụng lại ngay lập tức if, điều này thật ngớ ngẩn và chắc chắn không phải là cách viết thích hợp hơn if(operation()) { ... }. Vì vậy, cứ tiếp tục và làm điều đó, chúng tôi sẽ không phán xét bạn (vì điều đó).

Tôi sẽ nói rằng nó không sạch nhất có thể, bởi vì nó buộc người bảo trì phải biết hoặc tìm kiếm những gì giá trị trả về biểu thị. Có nghĩa là giá trị đã tồn tại, chưa tồn tại, đã được chèn thành công? Nếu bạn không sử dụng nó nhiều, bạn sẽ không biết, và thậm chí nếu bạn làm thế, thì điều đó còn nặng nề hơn nhiều.

Tôi thích những điều sau đây:

boolean added = set.add(entry);

if (added) {
    //do some more stuff
}

Vâng, dài dòng hơn một chút, nhưng trình biên dịch sẽ tạo ra khá nhiều mã byte chính xác và thậm chí những người không sử dụng bộ Java trong nhiều năm có thể theo logic mà không cần tìm kiếm gì.

Nếu đúng có nghĩa là thành công, thì đó là mã tốt, rõ ràng.

Có một quy ước phổ biến rằng một hàm hoặc phương thức trả về giá trị true (hoặc một cái gì đó đánh giá là đúng) về thành công. Miễn là mã của bạn tuân theo điều đó, tôi nghĩ việc đưa phương thức vào điều kiện là ổn.

Mã như thế này là lộn xộn không cần thiết trong quan điểm của tôi:

boolean frobulate_succeeded = thing.frobulate();

if (frobulate_succeeded) {
    ...
}

Cảm giác như bạn đang lặp lại chính mình.

Tuy nhiên, câu hỏi mơ hồ về ý nghĩa của giá trị trả về. Bạn nói "một boolean cho biết phần tử đã thêm đã tồn tại chưa", điều này có thể ngụ ý rằng true có nghĩa là phần tử tồn tại (và phần bổ sung không xảy ra). Nếu đó là trường hợp, tôi lý tưởng sẽ thay đổi hành vi trả về của phương thức thành thông thường hơn. Nếu điều đó là không thể, tôi sẽ thêm một biến trung gian bổ sung cho phép bạn dán nhãn rõ ràng kết quả trả về trong mã của bạn (như được đề xuất bởi người khác).

Tôi muốn nói nó rất giống C. Hầu hết thời gian tôi muốn có một biến được đặt tên mô tả cho kết quả đột biến và không có đột biến xảy ra trong một ifđiều kiện.

Một trình biên dịch sẽ loại bỏ biến này nếu nó được sử dụng lại ngay lập tức. Một con người sẽ có thời gian dễ dàng hơn để đọc nguồn; Đối với tôi, nó quan trọng hơn.

Nếu ai đó phải mở rộng điều kiện bằng cách thêm một mệnh đề and/ orvào điều kiện if, cuối cùng họ có thể không gọi .add()trong một số trường hợp do đánh giá ngắn mạch. Trừ khi ngắn mạch được dự đoán cụ thể, điều này có thể kết thúc như một lỗi.

Mã của bạn dường như phá vỡ phân tách truy vấn lệnh . Điều này được thảo luận trong sách Clean Code và video Cấu trúc chức năng . Vì vậy, từ góc độ Clean Code tôi nghĩ rằng nó không được coi là phong cách tốt.

Mã sạch Clean là đơn giản và trực tiếp. Mã sạch đọc như văn xuôi được viết tốt. Mã sạch không bao giờ che khuất ý định của nhà thiết kế mà thay vào đó là đầy đủ các khái niệm trừu tượng và các dòng điều khiển đơn giản.

- Grady Booch tác giả của Phân tích và Thiết kế hướng đối tượng với các ứng dụng

Đối với tôi mục đích của mã của bạn là không rõ ràng. Là nếu cả hai thực hiện khi mục nhập được thêm thành công, hoặc cả khi nó đã tồn tại? Điều gì add()trở lại? các mặt hàng? Mã lỗi?