중첩을 단순화하는 데 도움이 필요한 경우 블록

Question

I 코드의 다음 블록을 가지고

x는 정수이고

if (x > 5) 
{ 
    if (!DateTime.TryParse(y, out z)) 
     break; 
    if (w.CompareTo(z) == -1) 
     break; 
} 

, Y는 문자열, Z 및 W는 날짜 시간 변수이다. break;의 이유는 전체 블록이 루프 내에 있기 때문입니다.

읽기 쉽게하기 위해이 방법을 단순화 할 수 있습니까?

Answer 1

더 중요한 : w, x, y, z에 대한 설명 변수 이름을 사용 (희망이 이름은 당신의 예를 들어했다) :

또한 운영자 대신 CompareTo보다보다 작거나 큰를 사용할 수 있습니다.

if (x > 5) 
{ 
    bool isValidDate = DateTime.TryParse(y, out z); 
    if (!isValidDate || z > w) 
    { 
     // comment like: stop processing if the current date 
     // is after the reference date, or if there was a parsing error 
     break; 
    } 
} 

Answer 2

if (x > 5) 
{ 
    if(!DateTime.TryParse(y,out z) || w.CompareTo(z) == -1) 
     break; 
} 

두 조건은 동일한 결과를 가지므로 하나의 조건으로 결합 될 수 있습니다.

Answer 3

if ((x > 5) && (!DateTime.TryParse(y, out z) || w.CompareTo(z) == -1)) 
    break; 

Answer 4

if (x > 5){ 
    if (!DateTime.TryParse(y, out z) || w.CompareTo(z) == -1) break; 
} 

Answer 5

'단순화'는 읽기가 쉽지 않음을 의미합니다.

당신은에 의해 코드를 쉽게 읽을 수 (그리고 더 다양한 코딩 규칙의 관련 확보) 할 수있다 : '!'

1) 문 및 alikes가

2) 사용하지 않는 경우 항상 괄호를 사용하여 ('== false'는 훨씬 더 명시적임)

3) 변수가 무엇인지 명시 적으로 나타내는 변수 이름을 사용하십시오.

4) 여러 개의 break 문을 사용하지 마십시오. 대신 while 조건에서 평가되는 플래그를 사용하십시오.

5) 코드가 여전히 읽기 어려운 경우 : 의견을 사용하십시오!

Answer 6

다음은 하나 이상의 버전입니다.

var Break = x > 5 ? ((!DateTime.TryParse(y, out z) || w.CompareTo(z) == -1) ? true : false) : false; 

짧지 만 가독성을 저해합니다. 만, 두 가지 중 하나를 수행 루프를 실행하거나 루프 (하나를 경우 한 다른)을 실행하지 않기 때문에

Answer 7

당신은하지 필요성에게 코드를 실행할 수 multilpe if 블록을한다. 표시된 바와 같이 here은 하나의 부울 표현식을 사용하여 루프 반복을 건너 뛸 지 여부를 나타낼 수 있습니다.

(x > 5) && (!DateTime.TryParse(y, out z) || w.CompareTo(z) == -1) 

이와 같이 루프 내부와 같은 복잡한 조건을 포함하면 가독성이 저하 될 수 있습니다.

while(!done) // or whatever the while loop condition is 
{ 
    if(itemIsValid(x, y, w, out z)) 
    { 
     //the rest of your loop 
    } 
} 

//it may make sense for x, y, w, and possibly z to be wrapped in an object, or that already may be the case. Consider modifying as appropriate. 
//if any of the variables are instance fields they could also be omitted as parameters 
//also don't add z as an out parameter if it's not used outside of this function; I included it because I wasn't sure if it was needed elsewhere 
private bool itemIsValid(int x, string y, DateTime w, out DateTime z) 
{ 
    return (x > 5) 
     && (!DateTime.TryParse(y, out z) || w.CompareTo(z) == -1) 
} 

이 몇 가지 장점이 있습니다 : 루프 이런 식으로 뭔가를 보았다 그래서 개인적으로, 나는 간단하게하는 방법으로이 조건을 추출합니다. 첫째, 주석을 필요없이 코드를 스스로 문서화하는 방법입니다. 루프를 볼 때 "내가 끝내지 않은 상태에서 항목이 유효하면이 모든 작업을 수행합니다"라고 읽을 수 있습니다. 타당도가 정의 된 방법에 관심이있는 경우 메서드를 살펴보고, 그렇지 않으면 건너 뜁니다.메소드의 이름을 "isReservationSlotFree"또는 실제로 표시되는 것과 같은 더 구체적인 이름으로 변경할 수도 있습니다.

유효성 검사 논리가 복잡한 경우 (다소 복잡함) 더 복잡한 루프를 어지럽히 지 않고 설명과 설명을 추가 할 수 있습니다.