Sẽ hiệu quả hơn khi sử dụng if-return-return hoặc if-other-return?

Giả sử tôi có một iftuyên bố với a return. Từ góc độ hiệu quả, tôi nên sử dụng

if(A > B):
    return A+1
return A-1

hoặc là

if(A > B):
    return A+1
else:
    return A-1

Tôi nên thích cái này hay cái khác khi sử dụng ngôn ngữ được biên dịch (C) hoặc ngôn ngữ được viết theo kịch bản (Python)?

Vì returncâu lệnh chấm dứt việc thực thi hàm hiện tại, hai dạng tương đương (mặc dù dạng thứ hai có thể đọc được nhiều hơn so với dạng thứ nhất).

Hiệu quả của cả hai hình thức là tương đương nhau, mã máy bên dưới phải thực hiện bước nhảy nếu ifđiều kiện là sai.

Lưu ý rằng Python hỗ trợ cú pháp cho phép bạn chỉ sử dụng một returncâu lệnh trong trường hợp của mình:

return A+1 if A > B else A-1

Từ hướng dẫn về kiểu dáng của Chromium :

Không sử dụng khác sau khi trở về:

# Bad
if (foo)
  return 1
else
  return 2

# Good
if (foo)
  return 1
return 2

return 1 if foo else 2

Về phong cách mã hóa:

Hầu hết các tiêu chuẩn mã hóa cho dù ngôn ngữ cấm nhiều câu lệnh trả về từ một chức năng là thực tiễn xấu.

(Mặc dù cá nhân tôi sẽ nói rằng có một số trường hợp trong đó nhiều câu lệnh trả về có ý nghĩa: trình phân tích cú pháp giao thức văn bản / dữ liệu, các hàm có xử lý lỗi mở rộng, v.v.)

Sự đồng thuận từ tất cả các tiêu chuẩn mã hóa công nghiệp là biểu thức nên được viết là:

int result;

if(A > B)
{
  result = A+1;
}
else
{
  result = A-1;
}
return result;

Về hiệu quả:

Ví dụ trên và hai ví dụ trong câu hỏi hoàn toàn tương đương về hiệu quả. Mã máy trong tất cả các trường hợp này phải so sánh A> B, sau đó phân nhánh với phép tính A + 1 hoặc A-1, sau đó lưu kết quả của điều đó vào một thanh ghi CPU hoặc trên ngăn xếp.

BIÊN TẬP :

Nguồn:

• MISRA-C: 2004 quy tắc 14.7, lần lượt trích dẫn ...:
• IEC 61508-3. Phần 3, bảng B.9.
• IEC 61508-7. C.2.9.

Với bất kỳ trình biên dịch hợp lý, bạn sẽ quan sát không có sự khác biệt; chúng nên được biên dịch thành mã máy giống hệt nhau vì chúng tương đương nhau.

Đây là một câu hỏi về phong cách (hoặc sở thích) vì thông dịch viên không quan tâm. Cá nhân tôi sẽ cố gắng không đưa ra tuyên bố cuối cùng của hàm trả về giá trị ở mức thụt lề khác với cơ sở hàm. Cái khác trong ví dụ 1 che khuất, nếu chỉ một chút, nơi kết thúc của hàm là.

Theo sở thích tôi sử dụng:

return A+1 if (A > B) else A-1

Vì nó tuân theo cả quy ước tốt là có một câu trả về duy nhất là câu lệnh cuối cùng trong hàm (như đã đề cập) và mô hình lập trình chức năng tốt để tránh kết quả trung gian kiểu bắt buộc.

Đối với các hàm phức tạp hơn, tôi thích chia hàm thành nhiều hàm phụ để tránh trả về sớm nếu có thể. Mặt khác, tôi trở lại sử dụng một biến kiểu bắt buộc được gọi là rval. Tôi cố gắng không sử dụng nhiều câu lệnh return trừ khi hàm này không quan trọng hoặc câu lệnh return trước khi kết thúc là kết quả của một lỗi. Trở về sớm làm nổi bật thực tế là bạn không thể tiếp tục. Đối với các hàm phức tạp được thiết kế để phân nhánh thành nhiều hàm con, tôi cố gắng mã hóa chúng dưới dạng các câu lệnh tình huống (được điều khiển bởi một dict chẳng hạn).

Một số áp phích đã đề cập đến tốc độ hoạt động. Tốc độ thời gian chạy là thứ yếu đối với tôi vì nếu bạn cần tốc độ thực thi thì Python không phải là ngôn ngữ tốt nhất để sử dụng. Tôi sử dụng Python vì hiệu quả của mã hóa (tức là viết mã không có lỗi) quan trọng với tôi.

Cá nhân tôi tránh elsecác khối khi có thể. Xem Chiến dịch chống nếu

Ngoài ra, họ không tính phí 'thêm' cho dòng, bạn biết: p

"Đơn giản là tốt hơn phức tạp" & "Dễ đọc là vua"

delta = 1 if (A > B) else -1
return A + delta

Phiên bản A đơn giản hơn và đó là lý do tại sao tôi sẽ sử dụng nó.

Và nếu bạn bật tất cả các cảnh báo của trình biên dịch trong Java, bạn sẽ nhận được cảnh báo trên Phiên bản thứ hai vì nó không cần thiết và làm tăng độ phức tạp của mã.

Tôi biết câu hỏi được gắn thẻ python, nhưng nó đề cập đến các ngôn ngữ động vì vậy tôi nghĩ rằng tôi nên đề cập rằng trong ruby, câu lệnh if thực sự có kiểu trả về để bạn có thể làm gì đó như

def foo
  rv = if (A > B)
         A+1
       else
         A-1
       end
  return rv 
end

Hoặc bởi vì nó cũng có sự trở lại ngầm

def foo 
  if (A>B)
    A+1
  else 
    A-1
  end
end

mà xung quanh vấn đề phong cách của việc không có nhiều lợi nhuận khá độc đáo.