Simplifing một lồng NẾU tuyên bố

Question

Tôi có một chức năng đó là như sau

string Foo(bool A, bool B) 
{ 
    if(A) 
    { 
     if(B) 
     { 
      return "W"; 
     } 
     else 
     { 
      return "X"; 
     } 
    } 
    else 
    { 
     if(B) 
     { 
      return "Y"; 
     } 
     else 
     { 
      return "Z"; 
     } 
    } 
} 

Đó đôi làm tổ chỉ cảm thấy không đúng với tôi. Có cách nào tốt hơn để triển khai mẫu này không?

---

Cảm ơn tất cả mọi người đã trợ giúp, tôi sẽ kết thúc với lộ trình ba chiều. Nó giúp tắt chức năng này:

if (female) 
{ 
    if (nutered) 
    { 
     destRow["TargetSex"] = "FS"; 
    } 
    else 
    { 
     destRow["TargetSex"] = "F"; 
    } 
} 
else 
{ 
    if (nutered) 
    { 
     destRow["TargetSex"] = "MN"; 
    } 
    else 
    { 
     destRow["TargetSex"] = "M"; 
    } 
} 

trong này

destRow["TargetSex"] = female ? (nutered ? "FS" : "F") 
           : (nutered ? "MN" : "M"); 

Answer 1

if (A) 
{ 
    return B ? "W" : "X"; 
} 
return B ? "Y" : "Z"; 

Hoặc thậm chí ngắn gọn hơn:

return A ? (B ? "W" : "X") 
     : (B ? "Y" : "Z"); 

Nếu bạn đi với điều kiện độc quyền unnested:

if (A && B) return "W"; 
if (A && !B) return "X"; 
return B ? "Y" : "Z"; 

Answer 2

Bạn có bốn trạng thái có thể. Bản trình bày ngắn hơn (mặc dù không nhất thiết phải dễ bảo trì hơn) sẽ là

if (A && B) { 
    return "W"; 
} else if (A && !B) { 
    return "X"; 
} else if (!A && B) { 
    return "Y"; 
else return "Z"; 

Answer 3

Hợp lý, không. Bạn có 4 điều kiện riêng biệt cho hai biến.

Bạn có thể làm cho nhiều mã súc tích, mặc dù:

string Foo(bool A, bool B) 
{ 
    return A ? 
     B ? "W" : "X" 
     : 
     B ? "Y" : "Z"; 

} 

Hoặc nếu bạn đang cảm thấy đặc biệt ác, đặt nó trên cùng một dòng không có dấu ngoặc !:

return A?B?"W":"X":B?"Y":"Z"; 

Answer 4

tôi chỉ cần ném điều này vào cho vui:

string Foo(bool A, bool B) 
    { 
     var labels = new[]{"W", "X", "Y", "Z"}; 
     return labels[(A ? 0 : 2) + (B ? 0 : 1)]; 
    }