trả lại cho vòng lặp hoặc vòng lặp ngoài

Question

Hôm nay, một người nào đó đã tham dự tôi sử dụng sai từ khóa return trong Java. Tôi đã viết một vòng lặp đơn giản for để xác thực điều gì đó nằm trong một mảng. Giả array là một mảng có độ dài n này là mã của tôi:

for(int i=0; i<array.length; ++i){ 
    if(array[i]==valueToFind) return true; 
} 
return false; 

Bây giờ ai đó nói với tôi rằng đây không phải là chương trình rất tốt vì tôi sử dụng câu lệnh return bên trong một vòng lặp và điều này sẽ gây ra thu gom rác thải bị trục trặc. Do đó mã tốt hơn sẽ là:

int i = 0; 
while(i<array.length && array[i] != valueToFind) ++i; 
return i != array.length; 

Vấn đề là tôi không thể tìm ra lý do tại sao vòng lặp đầu tiên không phải là một thực hành tốt. Ai đó có thể cho tôi một sự thôi thúc không?

Answer 1

Bây giờ ai đó nói với tôi rằng đây không phải là chương trình rất tốt vì tôi sử dụng câu lệnh return bên trong một vòng lặp và điều này sẽ gây ra thu gom rác thải bị trục trặc.

Điều đó không đúng và gợi ý bạn nên đối xử với lời khuyên khác từ người đó với mức hoài nghi.

Câu thần chú "chỉ có một câu trả lời" (hoặc thường chỉ có một điểm thoát) là quan trọng bằng các ngôn ngữ mà bạn phải tự quản lý tất cả tài nguyên - theo cách đó bạn có thể đảm bảo mã ở một nơi.

Nó ít hữu ích hơn nhiều trong Java: ngay sau khi bạn biết rằng bạn nên quay trở lại (và giá trị trả về sẽ là gì), chỉ cần trả về. Bằng cách đó, việc đọc sách đơn giản hơn - bạn không phải thực hiện bất kỳ phần còn lại nào của phương pháp để tìm ra điều gì khác sẽ xảy ra (ngoài các khối finally).

Answer 2

Bây giờ ai đó nói với tôi rằng đây không phải là chương trình rất tốt bởi vì tôi sử dụng câu lệnh return bên trong một vòng lặp và điều này sẽ gây ra rác bộ sưu tập để trục trặc.

Đó là một đống rác. Tất cả mọi thứ bên trong phương pháp sẽ được làm sạch trừ khi có các tham chiếu khác cho nó trong lớp hoặc ở nơi khác (một lý do tại sao đóng gói là quan trọng). Theo quy tắc chung, tốt hơn hết nên sử dụng một câu lệnh trả về đơn giản vì dễ dàng tìm ra phương thức sẽ thoát.

Cá nhân, tôi sẽ viết:

Boolean retVal = false; 
for(int i=0; i<array.length; ++i){ 
    if(array[i]==valueToFind) { 
     retVal = true; 
     break; //Break immediately helps if you are looking through a big array 
    } 
} 
return retVal; 

Answer 3

Đã có phương pháp trong tất cả các ngôn ngữ ủng hộ cho việc sử dụng một câu lệnh return duy nhất trong bất kỳ chức năng. Tuy nhiên không thể nó có thể ở trong một số mã, một số người cố gắng làm điều đó, tuy nhiên, nó có thể làm cho mã của bạn trở nên phức tạp hơn (như trong nhiều dòng mã), nhưng mặt khác, dễ theo dõi hơn (như trong logic lưu lượng).

Điều này sẽ không làm hỏng bộ sưu tập rác theo bất kỳ cách nào !!

Cách tốt nhất để làm điều đó là đặt giá trị boolean, nếu bạn muốn nghe anh ấy.

boolean flag = false; 
for(int i=0; i<array.length; ++i){ 
    if(array[i] == valueToFind) { 
     flag = true; 
     break; 
    } 
} 
return flag; 

Answer 4

Một số người cho rằng phương pháp nên có một điểm thoát duy nhất (ví dụ: chỉ một return).Cá nhân, tôi nghĩ rằng cố gắng gắn bó với quy tắc đó tạo ra mã khó đọc hơn. Trong ví dụ của bạn, ngay sau khi bạn tìm thấy những gì bạn đang tìm kiếm, hãy trả lại ngay lập tức, nó rõ ràng và hiệu quả.

Quoting the C2 wiki:

Ý nghĩa ban đầu của việc có một mục duy nhất và thoát duy nhất cho một chức năng là nó là một phần của định nghĩa ban đầu của StructuredProgramming như trái ngược với SpaghettiCode goto vô kỷ luật, và cho phép phân tích toán học sạch trên cơ sở đó. Bây giờ, lập trình có cấu trúc từ lâu đã giành được ngày, không ai đặc biệt quan tâm đến điều đó nữa, và phần còn lại của trang chủ yếu là về thực tiễn tốt nhất và tính thẩm mỹ và không, về phân tích toán học của các cấu trúc lập trình có cấu trúc.

Answer 5

Mã hợp lệ (nghĩa là, sẽ biên dịch và thực thi) trong cả hai trường hợp.

Một trong những giảng viên của tôi tại Uni nói với chúng tôi rằng nó không phải là mong muốn có continue, return tuyên bố trong bất kỳ vòng lặp - for hoặc while. Lý do cho điều này là khi kiểm tra mã, nó không phải là không ngay lập tức rõ ràng cho dù chiều dài đầy đủ của vòng lặp sẽ được thực hiện hoặc return hoặc continue sẽ có hiệu lực.

Xem Why is continue inside a loop a bad idea? để biết ví dụ. Điều quan trọng cần ghi nhớ là đối với các tình huống đơn giản như thế này không phải vấn đề (IMO) nhưng khi bạn có logic phức tạp xác định giá trị trả về, thì mã "thường" dễ đọc hơn nếu bạn có một báo cáo trả về thay vì một vài câu lệnh.

Liên quan đến Bộ sưu tập rác - Tôi không biết tại sao điều này lại là vấn đề.