설정 플래그 전에 작업

Question

을 수행 한 후

은 가끔 자신이 다음과 같은 일을 찾을 수 있습니다 : 그것은 종종 함께 해킹 느낌

if (some_condition) 
    set_flag(true) 

do_some_work(); 
// more work ... 

if (some_condition) 
    set_flag(false) 

. 내가 이것을 리팩토링 할 수있는 방법이있어 청소기가 되었습니까?
1
2 추가 추상화/플래그를 확인하기보다는 다시 더 효율적이어야한다을 통해 상태를 확인 : 내 opion에서

Answer 1

, 내가 제안 사이비 코드가 여러 가지 이유로 잘이라고 생각 리팩토링은 길 아래에 두통의 이상을 야기하고 더 나은 다른 곳에서 사용할 수 비 부가가치 일 것으로 보인다
3. 플래그를 사용하여 그들이 의미있는 이름을 경우 일반적으로 읽을 수있는 (또는 할 수있다 메소드 이름 이 경우)