제어 흐름으로 인해 중복되는 상황에서 "else"를 사용해야합니까?

때로는 다음 예제와 유사한 코드를 우연히 발견했습니다 (이 함수의 기능은이 질문의 범위를 벗어납니다).

function doSomething(value) {
  if (check1(value)) {
    return -1;
  }
  else if (check2(value)) {
    return value;
  }
  else {
    return false;
  }
}

보다시피 if,, else if및 else명령문은 명령문과 함께 사용됩니다 return. 이것은 평범한 관찰자에게는 상당히 직관적 인 것처럼 보이지만 소프트웨어 개발자의 관점에서 else-s 를 삭제하고 다음 과 같이 코드를 단순화하는 것이 더 우아하다고 생각합니다 .

function doSomething(value) {
  if (check1(value)) {
    return -1;
  }
  if (check2(value)) {
    return value;
  }
  return false;
}

이는 return동일한 범위 내에서 명령문 다음에 오는 모든 것이 절대 실행되지 않으므로 위의 코드가 첫 번째 예제와 의미 적으로 동일하므로 의미가 있습니다.

위 중 어느 것이 좋은 코딩 방법에 더 적합합니까? 코드 가독성과 관련하여 두 방법 중 어느 하나의 단점이 있습니까?

편집 : 이 질문에 대한 중복 제안이 참조로 제공되었습니다. 나는 다른 질문에 제시된 것처럼 중복 진술을 피하는 것에 대해 묻지 않기 때문에 내 질문이 다른 주제를 다루고 있다고 생각합니다. 두 가지 질문 모두 약간 다른 방식으로 반복을 줄이려고합니다.

나는없는 사람을 좋아하며 else이유는 다음과 같습니다.

function doSomething(value) {
  //if (check1(value)) {
  //  return -1;
  //}
  if (check2(value)) {
    return value;
  }
  return false;
}

그렇게하는 것이 아무 것도 깨지지 않았기 때문입니다.

함수의 이름을 포함하여 모든 형태의 상호 의존성을 싫어하십시오 check2(). 분리 할 수있는 모든 것을 분리하십시오. 때때로 당신은 필요 else하지만 여기서는 보이지 않습니다.

코드의 의미에 달려 있다고 생각합니다. 세 가지 사례가 서로 의존하는 경우 명시 적으로 명시하십시오. 이렇게하면 코드의 가독성이 향상되고 다른 사람이 더 쉽게 이해할 수 있습니다. 예:

if (x < 0) {
    return false;
} else if (x > 0) {
    return true;
} else {
    throw new IllegalArgumentException();
}

여기 x에서는 세 경우 모두 의 가치에 따라 명확하게 결정 됩니다. last를 생략하면 else덜 명확합니다.

사례가 서로 직접 의존하지 않는 경우 생략하십시오.

if (x < 0) {
    return false;
}
if (y < 0) {
    return true;
}
return false;

문맥없이 간단한 예에서 이것을 보여주기는 어렵지만 내 요점을 알기를 바랍니다.

나는 두 번째 옵션 (분리하지 선호하는 ifs더와 else if조기를 return) 하지만 즉 한 코드 블록이 짧은만큼 .

코드 블록이 길면을 사용하는 것이 좋습니다. 그렇지 않으면 코드 else if의 여러 종료점을 명확하게 이해하지 못하기 때문입니다.

예를 들면 다음과 같습니다.

function doSomething(value) {
  if (check1(value)) {
    // ...
    // imagine 200 lines of code
    // ...
    return -1;
  }
  if (check2(value)) {
    // ...
    // imagine 150 lines of code
    // ...
    return value;
  }
  return false;
}

이 경우 더 나은 사용이 가능 else if하고 리턴 코드를 변수에 저장하고 끝에 하나의 리턴 문장이 있습니다.

function doSomething(value) {
  retVal=0;
  if (check1(value)) {
    // ...
    // imagine 200 lines of code
    // ...
    retVal=-1;
  } else if (check2(value)) {
    // ...
    // imagne 150 lines of code
    retVal=value;
  }
  return retVal;
}

그러나 함수가 짧아지기 위해 노력해야하기 때문에 첫 번째 코드 스 니펫과 같이 짧게 유지하고 일찍 종료한다고 말하고 싶습니다.

나는 다른 상황에서 둘 다 사용합니다. 유효성 검사에서 else를 생략합니다. 제어 흐름에서는 else를 사용합니다.

bool doWork(string name) {
  if(name.empty()) {
    return false;
  }

  //...do work
  return true;
}

vs

bool doWork(int value) {
  if(value % 2 == 0) {
    doWorkWithEvenNumber(value);
  }
  else {
    doWorkWithOddNumber(value);
  }
}

첫 번째 경우는 모든 전제 조건을 먼저 확인하고 방해 요소를 제거한 다음 실행하려는 실제 코드로 진행하는 것처럼 더 많이 읽습니다. 따라서 실제로 실행하려는 수분이 많은 코드는 함수의 범위에 직접 속합니다. 그것은 기능에 관한 것입니다.
두 번째 경우는 두 경로가 어떤 조건에 따라 다른 것과 마찬가지로 기능과 관련이있는 유효한 코드로 실행되는 것처럼 보입니다. 따라서 이들은 서로 비슷한 범위 수준에 속합니다.

내가 본 적이있는 유일한 상황은 다음과 같은 코드입니다.

List<?> toList() {
    if (left != null && right != null) {
        Arrays.asList(merge(left, right));
    }
    if (left != null) {
        return Arrays.asList(left);
    }
    if (right != null) {
        return Arrays.asList(right);
    }
    return Arrays.asList();
}

여기에 분명히 잘못된 것이 있습니다. 문제는 return추가하거나 Arrays.asList제거 해야하는지 확실하지 않다는 것 입니다. 관련 방법을 자세히 검사하지 않으면이 문제를 해결할 수 없습니다. 항상 완전한 if-else 블록을 사용하여 이러한 모호성을 피할 수 있습니다. 이:

List<?> toList() {
    if (left != null && right != null) {
        Arrays.asList(merge(left, right));
    } else if (left != null) {
        return Arrays.asList(left);
    } else if (right != null) {
        return Arrays.asList(right);
    } else {
        return Arrays.asList();
    }
}

먼저 명시 적으로 반환을 추가하지 않으면 (정적으로 검사 된 언어로) 컴파일되지 않습니다 if .

일부 언어는 첫 번째 스타일도 허용하지 않습니다. 나는 엄격하게 명령적인 맥락에서만 또는 긴 방법으로 전제 조건으로 사용하려고합니다.

List<?> toList() {
    if (left == null || right == null) {
        throw new IllegalStateException()
    }
    // ...
    // long method
    // ...
}

이와 같은 함수에서 3 개의 종료가 있으면 코드를 따르고 나쁜 연습을 수행하려고하면 실제로 혼란 스럽습니다.

기능에 대해 하나의 종료 점이 있으면 다른 것이 필요합니다.

var result;
if(check1(value)
{
    result = -1;
}
else if(check2(value))
{
    result = value;
}
else 
{
    result = 0;
}
return result;

실제로 더 나아가겠습니다.

var result = 0;
var c1 = check1(value);
var c2 = check2(value);

if(c1)
{
    result = -1;
}
else if(c2)
{
    result = value;
}

return result;

입력 유효성 검사에 대한 단일 초기 수익을 주장 할 수있을 정도로 공정합니다. 논리를 if에 넣는 경우 전체 대량을 래핑하지 마십시오.

중복 else 문을 생략하는 것이 좋습니다. 코드 검토 또는 리팩토링에서 볼 때마다 작성자가 제어 흐름을 이해하고 코드에 버그가 있는지 궁금합니다.

저자가 else 서술문이 필요하다고 믿고 따라서 return 서술 문의 제어 흐름에 미치는 영향을 이해하지 못했다면 그러한 이해 부족은 버그,이 기능 및 일반적으로 주요한 원인입니다.