일반적으로 다음 줄 중 어느 코드 형태가 더 나은(?) 코드 인지

가끔, return은 한 곳에서 되어야한다를 고집하시는 오래된 분들이 계십니다.
저도 이유를 알 고 싶긴한데, 혹시 최적화를 못하는 오래된 컴파일러가
스택을 정리하는 코드가 return이 들어 있는 곳마다 일어나는 것때문에 생기는 낭비스런 모습이 아닐까를 생각해봤는데...
다른 상상은 안되더군요.

확실한 기억은 아니지만,
제 기억에 예전 컴파일러는 return 밑으로 코드가 남아있으면,
경고문을 냈습니다.

2005년 11월 16일.

함수의 기능과 역할, 처리 방식에 따라 그때 그때 다를 수 있지만, 궂이 하나로 묶을 필요는 없다고 생각합니다. 함수를 끝내는 곳이 하나여야만 가독성을 높인다고 보는 것도 힘들 것 같습니다.

제 경우에는, 아래 foo() 보다는 bar()와 같은 식의 코드를 선호합니다:

void foo(void)
{
   if (!error) {
      ...
  }
}

void bar(void)
{
  if (error)
    return;
  ...
}

그렇다고 꼭 종료할 필요가 있을 때, 바로 종료하는 것도 좋지 않을 것 같습니다. 특히 아래와 같은 함수가 쓰일 필요가 있을 경우에는...

void foo(void)
{
  FILE *fp;
  char *s;
  int *ptr;

  fp = fopen(...);
  if (!fp)
    goto end;

  ptr = malloc(sizeof(int) * 10);
  if (!ptr)
    goto freef;

  s = strdup("asdf");
  if (!s)
    goto freei;
  
  /* ... */

 frees:
  free(s);
  
 freei:
  free(pts);

 freef:
  fclose(fp);

 end:
  return;
}

결론을 말하자면, 함수의 동작 방식과 역할에 따라 그때그때 다를 수 있다에 한 표~ :wink: