두 번 발생하는 조건문을 어떻게 리팩터링 할 수 있습니까?

Question

과정이 가득 차서 및 가독성을 향상시킬 수 있는지 확인하기 위해 두 번째 조건을 제거 할 수 있습니까?

...  
if (course.isFull()) { 
    attemptStudentCapacityIncrease(course); 
    if (course.isFull()) { 
    // could not increase course student capacity 
    return; 
    } 
} 
// course is not full 
... 

Answer 1

실패시 false를 반환하고 성공시 true를 반환하도록 함수를 만듭니다.)

if (course.isFull()) { 
    try { 
     attemptStudentCapacityIncrease(course); 
    } catch (Exception ex) { 
     // could not increase course student capacity 
     return; 
    } 
} 

을하지만, 예외적 인 상황에 대한 예외를 사용하는 기억 :

if (course.isFull()) { 
    if (!attemptStudentCapacityIncrease(course)) { 
     // could not increase course student capacity 
     return; 
    } 
} 

당신은 또한 다음과 같이 처리하는, 실패시 예외를 던질 기능을 수정하는 것이 좋습니다.

Answer 2

아니면 API의 변경 수없는 경우 :

int attempts = 3; 
while (attempts-- && course.isFull()) 
    attemptStudentCapacityIncrease(course); 

if (attempts == 0) 
    return; 

Answer 3

또한 사용자 정의 예외를 던질 수있다.

if (course.isFull()) { 
    try { 
     attemptStudentCapacityIncrease(course); 
    } catch (IncreaseCourseException ice) { 
     return; 
    } 
} 

나는 그것이 attemptStudentCapacityIncrease에서 부울 vlaue을 반환하는 것보다 더 좋은 것을 생각하지만, 덜 효율적이다, 그래서에 의존하는 방법은