유지 보수 현명한, 중괄호없이 '다른 동안'안전하다고 간주됩니까?

가 else while현명 "안전한"유지 보수로 간주 중괄호 개입하지 않고?

if-else아래와 같이 중괄호없이 코드 작성 ...

if (blah)
    foo();
else
    bar();

... 중괄호가 없으면 코드의 의미를 실수로 쉽게 변경하기 때문에 위험이 따릅니다.

그러나 아래도 위험합니까?

if (blah)
{
    ...
}
else while (!bloop())
{
    bar();
}

아니면 else while중괄호없이 "안전한"것으로 간주됩니까?

이 코드를 생각 나게합니다.

if ( ... ) try {
..
} catch (Exception e) {
..
} else {
...
}

괄호를 잊어 버리고 들여 쓰기를 늘리지 않고 두 가지 유형의 블록을 결합 할 때마다 이해하고 유지하기가 매우 어려운 코드를 작성합니다.

아마도 내가 사용하여 내 무역 (웨이 백) 배운 때문이다 잭슨 법인 구조 다이어그램 방법을,하지만 난 유일한한다는 견해에 가입 올바른 애프터 비지지 용어 if또는이 else후속이다 if(즉 수 있도록 else if사다리를)

찌르지 않은 물건이 있으면 오해 및 / 또는 유지 관리 문제가 발생할 가능성이 있습니다. 이것이 "안전하지 않은"OPs 아이디어의 핵심입니다.

또한 중괄호 여부에 관계없이 while()같은 줄 에 포함시키는 것에 대해 매우 겁이납니다 else. 그것은 나에게 올바르게 읽히지 않습니다 ... else절 이라는 추가 들여 쓰기 마스크가 부족합니다 . 명확성이 부족하면 오해가 발생합니다 (위 참조).

따라서이 예에서 나는 다음에 대해 강력하게 조언 / 권장하고 (팀에서 주장한다)

if ( blah )
{
    ...
}
else
{
    while ( !bloop() )
    {
        bar();
    }
}

물론 적절한 의견도 기대할 것입니다.

-- 편집하다 --

최근 애플은 유지 보수 수정 문제로 인해 SSL 취약점으로 어려움을 겪었습니다. 괄호가없는 단일 행은 괜찮다는 생각을 자고 자합시다.

나는 항상 모든 것을 중괄호 안에 넣고 들여 쓰기하고 주석을달라고 배웠습니다. 오류를 읽고 발견하는 것이 훨씬 쉽다고 생각합니다. 개인적으로 모듈성이 중요하다고 생각하므로 항상 다음과 같은 코드를 작성합니다.

    if(blah)
    {
     ....
    }
    else
    {
       while(!bloop()
       {
        bar;
       }
    }

나는 방법 을 추출 하고 만들 것입니다

if ( blah )
{
    ...
}
else
{
   newMethod();
}

리팩토링 카탈로그 사이트 에서 설명한 "추출 방법"접근 방식을 참조하십시오 .

함께 그룹화 할 수있는 코드 조각이 있습니다.

프래그먼트를 이름이 메소드의 목적을 설명하는 메소드로 변환하십시오.

void printOwing() {
    printBanner();

    //print details
    System.out.println ("name:    " + _name);
    System.out.println ("amount    " + getOutstanding());
}

                                                                                                        

void printOwing() {
    printBanner();
    printDetails(getOutstanding());
}

void printDetails (double outstanding) {
    System.out.println ("name:    " + _name);
    System.out.println ("amount    " + outstanding);
}

나는 그것을 나쁜 코딩 습관으로 생각할 것입니다. 때로는 완벽하게 작동하며 코드를 컴파일하고 실행하는 데 아무런 문제가 없습니다. 다른 경우 심각한 버그가 발생할 수 있으며 해당 버그를 수정하는 데 몇 시간을 소비하게됩니다.

항상 코드를 모듈화하는 것이 좋습니다. 다른 부분에 while 루프를 넣어야하는 경우 다른 사람이 코드를 작업 할 때 논리를 쉽게 이해할 수 있도록 블록에 넣으십시오.

if ( blah )
{
    ...
}
else
{
    while ( !bloop() )
    {
        bar();
    }
}

코드를 블록에 넣는 것이 좋습니다. 또한 이해하기 쉽고 디버그하기도 쉽습니다.

개인적으로 마음에 들지 않지만 가장 선호하는 if / else 코드 블록에도 중괄호를 사용하는 것이 선호되지만 간단하게 유지하는 것이 좋습니다. 그것은 단지 더 깔끔합니다.

그러나 이것이 혼란스러운 곳은 if / else가 중괄호로 일부를 루프하지 않고 중첩 한 경우입니다. 그 스파게티 중간에 While 루프! 나는 그 나쁜 코드로 작업했으며 디버그하고 이해하는 것은 악몽입니다. 이것은 처음부터 간단하게 유지되고 만족할 때 좋습니다. 그러나 다른 프로그래머가 와서 무언가를 추가 할 수 있습니다. 어쩌면 while 루프 내에 다른 것이 있습니다. 처음부터 깨끗하게 쓰여지면 이런 일이 발생할 가능성이 줄어 듭니다.

요점은 다른 프로그래머가 옳고 그른 것을 한 눈에 볼 수 있도록 물건을 명확하게하는 것입니다. 패턴이 올바르게 표시되고 방해되지 않도록 코드를 작성하십시오.

이것의 반대 측면은 아마도 어떤 사람들은 어떤 경우에는 이것이 잘 읽힌다 고 주장하는 사람들을 볼 수 있다는 것입니다. 리소스가 있다면 뭔가를 기다리는 동안 다른 처리를해야합니다. 그럼에도 불구하고 else 블록 안에 while 루프를 중첩시키는 데 여전히 차이가 없습니다.

글쎄, 모두가 중괄호 사용에 대해 논쟁하고 그것을 좋아하는 것처럼 보이지만, 나는 실제로 반대를 선호합니다. 나는 더 읽기 쉽고 간결하지 않기 때문에 필요할 때만 중괄호를 사용합니다.

if (...)
   foo();
else while(...)
   bar();

... 실제로이 " else while(...)"를 읽을 수 있습니다. 심지어 평범한 영어처럼 읽습니다! 그러나 나는 사람들이 모든 것을 이상하게 생각할 것이라고 생각합니다.

결국, 우리 모두는 중괄호로 바보 증거를 만드는 경향이 있습니다.

나는 항상 중괄호를 넣습니다. 서두를 때 다른 줄을 추가하고, 들여 쓰기하고, 중괄호를 잊어 버리고, 일어나고있는 일을 머리를 긁을 수 있기 때문입니다. 오프닝 브래킷을 같은 줄에 유지하지만 중요하지 않습니다. 나는 두 가지 경우 모두 더 낫다.

if(blah) {
    foo();
}
else {
    bar();
}

너무 많은 사람들이 괄호를 쓰려고하지 않는 중괄호의 문제점은 무엇입니까?

정확히 어떤 문제가 else while해결됩니까?

중괄호는 싸고 좋으며 영리하고 재치있는 것이 아니라 코드 의도를 분명하고 명확하게 만듭니다.

인용하는 유닉스 철학 :

선명도의 규칙 : 선명도는 영리함보다 낫습니다.

유지 관리는 매우 중요하고 비용이 많이 들기 때문에 프로그램을 작성하는 가장 중요한 통신은 실행하는 컴퓨터가 아니라 향후 소스 코드를 읽고 유지 관리 할 사람 (자신 포함)에게 보내는 것처럼 프로그램을 작성하십시오.

아무 문제 없어

if (blah)
    foo();
else
    bar();

아무 문제가없는 것처럼

if (blah) foo(); else bar();

올바른 상황에서 (여러 상황은 다를 수 있지만 반복되는 코드가 많을 때 특정 스타일이 가장 잘 사용됩니다. 그러면 각 줄의보기가 문체 들여 쓰기보다 중요합니다)

그러나 한 가지 방법을 선택하면 일관성을 유지하십시오. 두 번째 예제는 if 표현식에 명령문에 대괄호가 있으므로 while 문은 else 절의 대괄호 안에 있어야합니다.

게다가, 나는이 코드를 전에 보았다 :

if (x) foo()
{
  bar();
}

그리고 그것은 코딩 표준 나치 자신 (모든 것에 대한 대괄호를 주장한)에 의해 작성되었습니다.

최신 IDE는 파일을 저장할 때 코드를 다시 포맷하거나 (불필요한 괄호 제거 또는 추가 포함) 코드를 다시 들여 쓰도록 쉽게 구성 할 수 있습니다. 따라서 예를 들어 자동으로

if (blah) {
  blub();
} else
  while (!bloop()) {
    bar();
  }

들여 쓰기는 불필요한 괄호가 없어도 항상 중첩을 쉽게 볼 수 있습니다. 따라서 그러한 IDE 설정을 적용하면 위험이 없습니다.

개인적으로, 나는 중괄호와 줄 바꿈을 생략하여 코드를 더 쉽게 읽을 수 있기 때문에 훨씬 더 읽기 쉬운 코드를 찾습니다.

if (blah) blub();
else while (!bloop()) bar();

그러나 물론 많은 개발자들은 그러한 것들에 대해 영원히 하루 종일 논쟁하는 것을 매우 기쁘게 생각합니다.