đôi nếu điều kiện, bất kỳ cách nào để làm cho nó ngắn

Question

tôi thường gặp trên phải làm một cái gì đó như thế này,

if(condition1) { 
     if {condition2) { //this condition is repeated again below 
      //dosomething here 1 code 1 
     } 
     else{ 
      //dosomething here 2 code 2 
     } 
    } 
    else { 
     if {condition2) { //same if condition as above 
      //dosomething here 3 code 3 
     } 
     else{ 
      //dosomething here 4 code 4 
     } 
    } 

Về cơ bản, chỉ kiểm tra if (condition2) được lặp lại trong cả hai trường hợp, và DoSomething là khác nhau ở tất cả 4 vị trí tức là code1, code2, code3, code4 là tất cả các mã khác nhau.

Vì vậy, có anyway để làm cho nó nhỏ gọn và dễ đọc Hoặc điều này là tốt?

Cảm ơn.

Sau khi trải qua các chỉnh sửa và xem câu trả lời, tôi tự hỏi bây giờ nếu câu hỏi này có ý nghĩa gì cả. Bây giờ tôi cảm thấy ngu ngốc.

Answer 1

Đối với trường hợp cụ thể của bạn, thay thế tốt nhất là:

không còn áp dụng - giá trị trong trường hợp DoSomething đây 1 là như nhau với DoSomething đây 3

if (c2) 
{ 
} 
else if (c1) 
{ 
} 
else 
{ 
} 

Đối với trường hợp ít đơn giản , bạn có thể nhóm các điều kiện lại với nhau:

if (c1 && c2) 
{ 
} 
else if (c1 && !c2) 
{ 
} 
else if (!c1 && c2) 
{ 
} 
else if (!c1 && !c2) 
{ 
} 

mặc dù tôi không biết điều này có dễ đọc hơn hay không.

Nếu có nhiều điều kiện cần thiết, tôi đã nhìn thấy mã như thế này:

do { 
    if (c1) 
    { 
     //.... 
     break; 
    } 
    if (c2) 
    { 
     //.... 
     break; 
    } 
    //..... 
} while (false); 

Answer 2

ĐÁP MỚI AS HỎI đã được chỉnh sửa

if (condition1 && condition2) 
{ 
    //dosomething here 1 code 1 
} 
else if (condition1 && !condition2) 
{ 
    //dosomething here 2 code 2 
} 
else if (!condition1 && condition2) 
{ 
    //dosomething here 3 code 3 
} 
else 
{ 
    //dosomething here 4 code 4 
} 

mã sau không còn giá trị AS CÂU HỎI CÓ THAY ĐỔI!

tôi sẽ làm:

if(condition2) 
{ 
    if(condition1) 
    { 
    //dosomething here 
    } 
    else 
    { 
    //dosomething here 2, 
    //which might be different than what we are doing above 
    } 
} 

Đó là cho câu hỏi cụ thể của bạn, cho các vấn đề phức tạp hơn các giải pháp có thể khác nhau