Bảo trì khôn ngoan, là 'khác trong khi' mà không can thiệp niềng răng được coi là an toàn?

Là else whilekhông can thiệp niềng răng được coi là bảo trì "an toàn" khôn ngoan?

Viết if-elsemã mà không cần niềng răng như dưới đây ...

if (blah)
    foo();
else
    bar();

... Rủi ro vì thiếu niềng răng khiến việc thay đổi ý nghĩa của mã vô tình rất dễ dàng.

Tuy nhiên, dưới đây cũng có rủi ro?

if (blah)
{
    ...
}
else while (!bloop())
{
    bar();
}

Hoặc là else whilekhông can thiệp niềng răng được coi là "an toàn"?

Điều đó nhắc nhở tôi mã này:

if ( ... ) try {
..
} catch (Exception e) {
..
} else {
...
}

Mỗi khi bạn kết hợp hai loại khối, quên niềng răng và không tăng thụt, bạn đang tạo mã rất khó hiểu và duy trì.

Có lẽ đó là vì tôi đã học được giao dịch của mình (từ khi trở về) bằng cách sử dụng phương pháp Sơ đồ cấu trúc thực thể Jackson , nhưng tôi đăng ký với quan điểm rằng thuật ngữ không liên kết chính xác duy nhất sau một ifhoặc một elselà tiếp theo if(nghĩa là cho phép một else ifbậc thang)

Bất cứ điều gì khác (không có ý định chơi chữ) đều tiềm ẩn những hiểu lầm và / hoặc vấn đề bảo trì. Đó là khía cạnh trung tâm của ý tưởng OP là "không an toàn".

Tôi cũng sẽ rất thận trọng trong việc bao gồm while()cùng một dòng với else- dù có giằng hay không. Nó không đọc đúng với tôi ... việc thiếu bất kỳ mặt nạ thụt đầu dòng nào mà nó là elsemệnh đề. Và thiếu rõ ràng làm cho sự hiểu lầm (xem ở trên).

Vì vậy, trong ví dụ tôi rất muốn khuyên / giới thiệu (và nhấn mạnh, trong nhóm của tôi) về:

if ( blah )
{
    ...
}
else
{
    while ( !bloop() )
    {
        bar();
    }
}

Tất nhiên, tôi cũng mong đợi để xem bình luận phù hợp quá.

-- Chỉnh sửa --

Gần đây, Apple đã bị một lỗ hổng SSL, gây ra bởi một bản sửa lỗi bảo trì kém - đã thêm một dòng thứ hai vào một dòng duy nhất không có dấu ngoặc. Vì vậy, hãy đặt ra ý tưởng rằng các dòng đơn không có dấu là OK?

Tôi luôn được dạy giữ mọi thứ trong niềng răng, thụt lề và bình luận. Tôi tin rằng nó dễ dàng hơn nhiều để đọc và phát hiện lỗi. Cá nhân tôi cảm thấy tính mô đun là chính vì vậy tôi sẽ luôn viết mã như thế này:

    if(blah)
    {
     ....
    }
    else
    {
       while(!bloop()
       {
        bar;
       }
    }

Tôi sẽ trích xuất phương pháp và làm cho nó

if ( blah )
{
    ...
}
else
{
   newMethod();
}

Xem cách tiếp cận "phương pháp trích xuất" được giải thích tại trang Danh mục tái cấu trúc :

Bạn có một đoạn mã có thể được nhóm lại với nhau.

Biến đoạn này thành một phương thức có tên giải thích mục đích của phương thức.

void printOwing() {
    printBanner();

    //print details
    System.out.println ("name:    " + _name);
    System.out.println ("amount    " + getOutstanding());
}

                                                                                                        

void printOwing() {
    printBanner();
    printDetails(getOutstanding());
}

void printDetails (double outstanding) {
    System.out.println ("name:    " + _name);
    System.out.println ("amount    " + outstanding);
}

Tôi sẽ coi đó là một thói quen xấu của tiền mã hóa. Đôi khi nó hoạt động hoàn toàn tốt và bạn sẽ không gặp vấn đề gì trong việc biên dịch và chạy mã. Vào những thời điểm khác, nó có thể gây ra lỗi nghiêm trọng và cuối cùng bạn sẽ mất hàng giờ để sửa lỗi đó.

Nó luôn luôn được khuyến nghị để mô đun hóa mã của bạn. Nếu bạn phải đặt một vòng lặp while trong phần khác, hãy đặt nó vào một khối để những người khác, khi làm việc với mã của bạn có thể dễ dàng hiểu được logic.

if ( blah )
{
    ...
}
else
{
    while ( !bloop() )
    {
        bar();
    }
}

Đó là một thực hành tốt để đặt mã trong các khối. Nó làm cho nó đơn giản và dễ hiểu hơn và gỡ lỗi là tốt.

Có thể sẽ ổn nếu nó vẫn đơn giản như vậy mặc dù cá nhân tôi không thích nó và sở thích của tôi là sử dụng niềng răng ngay cả đối với các khối mã if / other đơn giản nhất. Nó chỉ gọn gàng hơn đối với tôi.

Tuy nhiên, nơi điều này trở nên lộn xộn là khi bạn đã lồng vào nhau nếu / khác lặp một số vòng có một số không có. Một vòng lặp While ở giữa tất cả các spaghetti! Tôi đã làm việc với mã xấu và đó là một cơn ác mộng để gỡ lỗi và hiểu. Điều này tiếp nối từ điểm đầu tiên, nó ổn khi nó đơn giản và bạn hài lòng với nó, nhưng sau đó các lập trình viên khác xuất hiện và thêm mọi thứ vào đó, có thể là một thứ khác trong vòng lặp while. Nếu nó được viết sạch ở nơi đầu tiên thì sẽ ít có cơ hội điều này xảy ra.

Vấn đề là làm cho mọi thứ rõ ràng để các lập trình viên khác có thể thấy trong nháy mắt những gì đúng và sai. Viết mã để mô hình trông đúng và không bình.

Mặt trái của điều này là có lẽ tôi có thể thấy một số người tuyên bố rằng trong một số trường hợp nhất định, điều này đọc tốt. Nếu tôi có tài nguyên tôi cần thực hiện việc xử lý này trong khi tôi đang chờ đợi một cái gì đó làm điều này. Ngay cả với tôi vẫn không có sự khác biệt nào trong việc lồng vòng lặp while bên trong khối khác.

Chà, mặc dù mọi người tranh cãi về việc sử dụng niềng răng và dường như yêu thích chúng, tôi thực sự thích điều ngược lại. Tôi chỉ sử dụng niềng răng khi tôi phải bởi vì tôi thấy nó dễ đọc và súc tích hơn mà không có.

if (...)
   foo();
else while(...)
   bar();

... Tôi thực sự tìm thấy điều này " else while(...)" đáng chú ý có thể đọc được! Nó thậm chí đọc như tiếng Anh đơn giản! Nhưng tôi đoán tất cả mọi người sẽ thấy lạ vì ít nhất là không bình thường.

Cuối cùng, tất cả chúng ta đều có xu hướng làm cho nó trở nên ngớ ngẩn với niềng răng mặc dù ... bởi vì, bạn biết đấy.

Tôi luôn đặt niềng răng, chỉ vì bạn có thể thêm một dòng khác khi vội vàng, thụt nó, quên niềng răng và gãi đầu những gì đang xảy ra. Tôi giữ khung mở trên cùng một dòng, nhưng nó không thành vấn đề. Tôi cả hai trường hợp tốt hơn là có chúng.

if(blah) {
    foo();
}
else {
    bar();
}

Có gì sai với niềng răng mà rất nhiều người đang cố gắng tránh viết chúng?

Vấn đề gì làm chính xác else while giải quyết ?

Niềng răng là rẻ và tốt, và họ làm cho ý định mã rõ ràng và rõ ràng trái ngược với thông minh và dí dỏm.

Trích dẫn triết lý Unix:

Quy tắc rõ ràng: Rõ ràng tốt hơn thông minh.

Bởi vì bảo trì rất quan trọng và tốn kém, hãy viết các chương trình như thể giao tiếp quan trọng nhất họ làm không phải là máy tính thực thi chúng mà là con người sẽ đọc và duy trì mã nguồn trong tương lai (bao gồm cả chính bạn).

Không có gì sai với

if (blah)
    foo();
else
    bar();

giống như không có gì sai với

if (blah) foo(); else bar();

trong trường hợp phù hợp (hoàn cảnh của bạn có thể khác nhau, nhưng phong cách cụ thể đó được sử dụng tốt nhất khi bạn có nhiều mã lặp đi lặp lại - thì chế độ xem của mỗi dòng quan trọng hơn thụt lề kiểu cách)

Nhưng nếu bạn chọn một cách, hãy giữ vững - sự nhất quán là vua. Vì vậy, ví dụ thứ hai của bạn là rất xấu, vì biểu thức if có dấu ngoặc cho câu lệnh của nó, câu lệnh while phải nằm trong dấu ngoặc cho mệnh đề khác, không nằm ngoài nó.

Ngoài ra, tôi đã thấy mã này trước đây:

if (x) foo()
{
  bar();
}

và nó được viết bởi chính tiêu chuẩn mã hóa nazi (người khăng khăng đòi ngoặc cho mọi thứ).

Các IDE hiện đại có thể dễ dàng được cấu hình để định dạng lại (bao gồm loại bỏ hoặc thêm các dấu ngoặc không cần thiết) và / hoặc giới thiệu lại mã khi lưu tệp. Do đó, ví dụ của bạn sẽ tự động trông giống như ví dụ

if (blah) {
  blub();
} else
  while (!bloop()) {
    bar();
  }

Các vết lõm luôn làm cho việc làm tổ dễ dàng nhìn thấy, ngay cả khi không có niềng răng không cần thiết. Vì vậy, nếu bạn quản lý để thực thi các cài đặt IDE như vậy, tôi thấy không có rủi ro.

Cá nhân, tôi thấy mã bỏ qua các dấu ngoặc nhọn và ngắt dòng dễ đọc hơn nhiều vì nó tránh được sự lộn xộn:

if (blah) blub();
else while (!bloop()) bar();

Nhưng tất nhiên, nhiều nhà phát triển rất vui khi tranh luận về những điều như vậy mãi mãi và một ngày.