더 많은 반환 문장들과 많은 들여 쓰기

Question

다음의 두 가지 코드 블럭 중 어느 것이 전반적으로 더 좋은가?

하나의 return 문, 더 들여 쓰기 코드 :

struct dnode *dllist_push_front(struct dllist *dllist, void *data) 
{  
     struct dnode *dnode = malloc(sizeof(struct dnode)); 
     if (dnode) { 
       dnode->data = data; 
       dnode->next = dllist->sentinel->next; 
       dnode->prev = dlllist->sentinel; 
       dnode->next->prev = dnode; 
       dllist->sentinel->next = dnode; 
       dllist->size++; 
     } 
     return dnode; 
} 

또는

두 반환 문에서도 덜 들여 쓰기 코드 :

struct dnode *dllist_push_front(struct dllist *dllist, void *data) 
{ 
     struct dnode *dnode = malloc (sizeof(struct dnode)); 
     if (!dnode) 
       return NULL; 
     dnode->data = data; 
     dnode->next = dllist->sentinel->next; 
     dnode->prev = dlllist->sentinel; 
     dnode->next->prev = dnode; 
     dllist->sentinel->next = dnode; 
     dllist->size++; 
     return dnode; 
} 

Answer 1

내가 치우쳐 것 "빠르다"는 말은, 당신이 어떤 상태에 빠졌을 때 필요한 것보다 오래 기다리지 않는다는 의미입니다. 프로그램을 어기십시오. 그렇지 않으면 코드가 중첩 된 if 절의 다루기 힘든 혼란으로 바뀔 수 있습니다.

참조 : 나는 즉시 "실패"상태를 호출하기 때문에 후자를 선호하는 경향이 있지만 당신은 실제로 어느 길을 갈 수 Should a function have only one return statement?

Answer 2

. 내 개인적인 취향은 해당 조건 더욱를 호출 이후 처음 return 다음 빈 줄을 가지고하는 것입니다 (나는 의견의 큰 팬이에요) :

struct dnode *dllist_push_front(struct dllist *dllist, void *data) { 
    // Get new node, exit immediately if not possible. 

    struct dnode *dnode = malloc (sizeof (struct dnode)); 
    if (!dnode) return NULL; 

    // Node exists here, populate data and pointers then return it to caller. 

    dnode->data = data; 
    : 
    return dnode; 
} 

명심 여러 반환 지점 수하지만 항상 그렇지는 않습니다.

여러 지침이 잘못되어있는 이유와 그 이유가 때때로 이해하기 어려운 코드로 이어질 수 있기 때문에 기억해야합니다.

흐름을 쉽게 볼 수 있다면 (편집기 창이 13 라인 이하가 아니라면 두 가지 경우 모두 가능합니다.) 복수의 리턴 포인트가 좋습니다.

Answer 3

명확한 흐름과 디버깅을 위해 각 기능에는 정확히 하나의 종료점이 있습니다. goto의 훈련을 활용

는

쉬운 일을한다 : goto를 사용 themselfs을 제한하는 사람들을 위해

#include <stdio.h> 
#include <stdlib.h> 

... 

typedef enum 
{ 
    Error_Uninitialised = -1, 
    Error_Ok = 0, 
    Error_1, 
    Error_2, 
    Error_3, 
    Error_4, 
    Error_5, 
    ... 
} ErrorCode_t; 

#ifdef DEBUG 
#define LOG(fmt, ...) fprintf(stderr, fmt, __VA_ARGS__) 
#else 
#define LOG(fmt, ...) 
#endif 

int func(int a, char b, void * p, ...) 
{ 
    LOG("entering: '%s'\n", __func__); 

    int result = Error_NoError; 

    if (42 == i) 
    { 
    result = Error_1; 
    goto lblExit; 
    } 

    if ('y' == c) 
    { 
    result = Error_2; 
    goto lblExit; 
    } 

    if (!p) 
    { 
    result = Error_3; 
    goto lblExit; 
    } 

    void * ptmp = realloc(p, i); 
    if (!ptmp) 
    { 
    result = Error_4; 
    goto lblExt; 
    } 

    ... 

    lblExit: 

    LOG("'%s' returns %d\n", __func__, result); 

    return result; 
} 

은 다음 버전은 도움이 될 수 있습니다 : 나중에 프로젝트 중

int func(int a, char b, void * p, ...) 
{ 
    LOG("entering: '%s'\n", __func__); 

    int result = Error_NoError; 

    do 
    { 
    if (42 == i) 
    { 
     result = Error_1; 
     break; 
    } 

    if ('y' == c) 
    { 
     result = Error_2; 
     break; 
    } 

    if (!p) 
    { 
     result = Error_3; 
     break; 
    } 

    void * ptmp = realloc(p, i); 
    if (!ptmp) 
    { 
     result = Error_4; 
     break; 
    } 

    ... 

    } while (0); 

    LOG("'%s' returns %d\n", __func__, result); 

    return result; 
} 

다음의 경우, 때를 최적화를 위해 이 실제로 인 경우이 접근법에서 벗어날 수 있습니다.