Có nên sử dụng khác không nên sử dụng trong các tình huống mà luồng điều khiển làm cho nó bị dư thừa?

Đôi khi tôi vấp phải mã tương tự như ví dụ sau (chức năng này thực hiện chính xác nằm ngoài phạm vi của câu hỏi này):

function doSomething(value) {
  if (check1(value)) {
    return -1;
  }
  else if (check2(value)) {
    return value;
  }
  else {
    return false;
  }
}

Như bạn có thể thấy, if, else ifvà elsebáo cáo được sử dụng kết hợp với returntuyên bố. Điều này có vẻ khá trực quan đối với một người quan sát thông thường, nhưng tôi nghĩ nó sẽ thanh lịch hơn (theo quan điểm của một nhà phát triển phần mềm) để bỏ các else-s và đơn giản hóa mã như thế này:

function doSomething(value) {
  if (check1(value)) {
    return -1;
  }
  if (check2(value)) {
    return value;
  }
  return false;
}

Điều này có ý nghĩa, vì mọi thứ tuân theo một returncâu lệnh (trong cùng phạm vi) sẽ không bao giờ được thực thi, làm cho mã ở trên về mặt ngữ nghĩa bằng với ví dụ đầu tiên.

Điều nào ở trên phù hợp với thực tiễn mã hóa tốt hơn? Có bất kỳ nhược điểm nào đối với một trong hai phương pháp liên quan đến khả năng đọc mã không?

Chỉnh sửa: Một đề xuất trùng lặp đã được đưa ra với câu hỏi này được cung cấp làm tài liệu tham khảo. Tôi tin rằng câu hỏi của tôi chạm vào một chủ đề khác, vì tôi không hỏi về việc tránh các câu trùng lặp như được trình bày trong câu hỏi khác. Cả hai câu hỏi đều tìm cách giảm sự lặp lại, mặc dù theo những cách hơi khác nhau.

Tôi thích cái không có elsevà đây là lý do:

function doSomething(value) {
  //if (check1(value)) {
  //  return -1;
  //}
  if (check2(value)) {
    return value;
  }
  return false;
}

Bởi vì làm điều đó đã không phá vỡ bất cứ điều gì nó không được phép.

Ghét sự phụ thuộc lẫn nhau trong tất cả các hình thức của nó (bao gồm cả việc đặt tên một chức năng check2()). Cô lập tất cả những gì có thể được phân lập. Đôi khi bạn cần elsenhưng tôi không thấy điều đó ở đây.

Tôi nghĩ rằng nó phụ thuộc vào ngữ nghĩa của mã. Nếu ba trường hợp của bạn phụ thuộc vào nhau, hãy nói rõ ràng. Điều này làm tăng khả năng đọc mã và giúp mọi người dễ hiểu hơn. Thí dụ:

if (x < 0) {
    return false;
} else if (x > 0) {
    return true;
} else {
    throw new IllegalArgumentException();
}

Ở đây bạn rõ ràng tùy thuộc vào giá trị của xtrong cả ba trường hợp. Nếu bạn bỏ qua lần cuối else, điều này sẽ không rõ ràng.

Nếu các trường hợp của bạn không phụ thuộc trực tiếp vào nhau, hãy bỏ qua nó:

if (x < 0) {
    return false;
}
if (y < 0) {
    return true;
}
return false;

Thật khó để thể hiện điều này trong một ví dụ đơn giản không có ngữ cảnh, nhưng tôi hy vọng bạn hiểu ý tôi.

Tôi thích tùy chọn thứ hai (tách biệt ifsvới không else ifvà sớm return) NHƯNG đó là miễn là các khối mã ngắn .

Nếu các khối mã dài thì sử dụng tốt hơn else if, bởi vì những người khác bạn sẽ không hiểu rõ về một số điểm thoát mà mã có.

Ví dụ:

function doSomething(value) {
  if (check1(value)) {
    // ...
    // imagine 200 lines of code
    // ...
    return -1;
  }
  if (check2(value)) {
    // ...
    // imagine 150 lines of code
    // ...
    return value;
  }
  return false;
}

Trong trường hợp đó, nó được sử dụng tốt hơn else if, lưu trữ mã trả về trong một biến và chỉ có một câu trả về ở cuối.

function doSomething(value) {
  retVal=0;
  if (check1(value)) {
    // ...
    // imagine 200 lines of code
    // ...
    retVal=-1;
  } else if (check2(value)) {
    // ...
    // imagne 150 lines of code
    retVal=value;
  }
  return retVal;
}

Nhưng vì người ta nên cố gắng để các chức năng được ngắn, tôi nói hãy giữ nó ngắn và thoát sớm như trong đoạn mã đầu tiên của bạn.

Tôi sử dụng cả hai trong hoàn cảnh khác nhau. Trong một kiểm tra xác nhận, tôi bỏ qua cái khác. Trong một luồng điều khiển, tôi sử dụng cái khác.

bool doWork(string name) {
  if(name.empty()) {
    return false;
  }

  //...do work
  return true;
}

đấu với

bool doWork(int value) {
  if(value % 2 == 0) {
    doWorkWithEvenNumber(value);
  }
  else {
    doWorkWithOddNumber(value);
  }
}

Trường hợp đầu tiên đọc giống như bạn đang kiểm tra tất cả các điều kiện tiên quyết trước tiên, đưa chúng ra khỏi đường đi, sau đó tiến tới mã thực tế bạn muốn chạy. Như vậy, mã juicy bạn thực sự muốn chạy thuộc phạm vi của hàm trực tiếp; đó là những gì chức năng thực sự là về.
Trường hợp thứ hai đọc giống như cả hai đường dẫn là mã hợp lệ sẽ được chạy có liên quan đến chức năng như các đường dẫn khác dựa trên một số điều kiện. Như vậy, chúng thuộc các cấp phạm vi tương tự với nhau.

Tình huống duy nhất tôi từng thấy nó thực sự quan trọng là mã như thế này:

List<?> toList() {
    if (left != null && right != null) {
        Arrays.asList(merge(left, right));
    }
    if (left != null) {
        return Arrays.asList(left);
    }
    if (right != null) {
        return Arrays.asList(right);
    }
    return Arrays.asList();
}

Rõ ràng có điều gì đó sai ở đây. Vấn đề là, không rõ nếureturn nên thêm hoặc Arrays.asListxóa. Bạn không thể sửa lỗi này mà không kiểm tra sâu hơn các phương pháp liên quan. Bạn có thể tránh sự mơ hồ này bằng cách luôn luôn sử dụng các khối if-if đầy đủ. Điều này:

List<?> toList() {
    if (left != null && right != null) {
        Arrays.asList(merge(left, right));
    } else if (left != null) {
        return Arrays.asList(left);
    } else if (right != null) {
        return Arrays.asList(right);
    } else {
        return Arrays.asList();
    }
}

sẽ không biên dịch (bằng các ngôn ngữ được kiểm tra tĩnh) trừ khi bạn thêm trả lại rõ ràng trước if.

Một số ngôn ngữ thậm chí không cho phép phong cách đầu tiên. Tôi cố gắng chỉ sử dụng nó trong bối cảnh bắt buộc nghiêm ngặt hoặc cho các điều kiện tiên quyết trong các phương pháp dài:

List<?> toList() {
    if (left == null || right == null) {
        throw new IllegalStateException()
    }
    // ...
    // long method
    // ...
}

Có ba lối thoát từ chức năng như vậy thực sự khó hiểu nếu bạn đang cố gắng tuân theo mã thông qua và thực hành xấu.

Nếu bạn có một điểm thoát duy nhất cho hàm thì các elses là bắt buộc.

var result;
if(check1(value)
{
    result = -1;
}
else if(check2(value))
{
    result = value;
}
else 
{
    result = 0;
}
return result;

Trong thực tế cho phép đi xa hơn nữa.

var result = 0;
var c1 = check1(value);
var c2 = check2(value);

if(c1)
{
    result = -1;
}
else if(c2)
{
    result = value;
}

return result;

Đủ công bằng bạn có thể tranh luận cho một lợi nhuận sớm duy nhất về xác nhận đầu vào. Chỉ cần tránh gói toàn bộ số lượng lớn nếu logic của bạn trong một nếu.

Tôi thích bỏ qua các tuyên bố khác dư thừa. Bất cứ khi nào tôi nhìn thấy nó (trong đánh giá mã hoặc tái cấu trúc) tôi tự hỏi liệu tác giả có hiểu luồng điều khiển hay không và có thể có lỗi trong mã.

Nếu tác giả tin rằng tuyên bố khác là cần thiết và do đó không hiểu ý nghĩa của dòng điều khiển của tuyên bố hoàn trả thì chắc chắn sự thiếu hiểu biết như vậy là một nguồn chính của lỗi, trong chức năng này và nói chung.