복수형을 간결하게 표현한 경우

Question

아래의 if 문을 좀 더 간결하게 구성 할 수 있습니까?

if (abciden != null) { 
    PERFORM SSS 
} 

if (abciden == null 
     && subject 
       .equalsIgnoreCase(IOAConstants.DDD_REPORT_SUBJECT_ert)) { 
    //PERFORM CCC 
} 

if (abciden == null 
     && subject 
       .equalsIgnoreCase(IOAConstants.DDD_REPORT_SUBJECT_fgt)) { 
    //PERFORM BBB 
} 

이 조건을 압축하는 방법에 대한 조언을 구하려면 속기 조건부 삼항 연산자를 사용하려면 바꾸시겠습니까? :

if (abciden != null) { 
    // PERFORM SSS 
} 
else { 
    if (subject.equalsIgnoreCase(IOAConstants.DDD_REPORT_SUBJECT_ert)) { 
     //PERFORM CCC 
    } 
    if (subject.equalsIgnoreCase(IOAConstants.DDD_REPORT_SUBJECT_fgt)) { 
     //PERFORM BBB 
    } 
} 

을 또는 내가 subject 있으리라 믿고있어 같은 양의 상수 일치하지 않을 수 있습니다 :

는

Answer 1

당신은 else를 사용하여 편집 다시

if (abciden != null) { 
    // PERFORM SSS 
} 
else if (subject.equalsIgnoreCase(IOAConstants.DDD_REPORT_SUBJECT_ert)) { 
    //PERFORM CCC 
} 
else if (subject.equalsIgnoreCase(IOAConstants.DDD_REPORT_SUBJECT_fgt)) { 
    //PERFORM BBB 
} 

---

:

원하는을 if를 속기 조건부로 대체한다. 3 차 연산자? : 특별히는 삼항 원하는 경우

좋아, 다음이 "수행"단계는 무엇이든, 그들은 값을 반환하는지 확인합니다. 그렇다면이 괴물을 사용할 수 있습니다 :

rv = abciden != null ? performSSS() : subject.equalsIgnoreCase(IOAConstants.DDD_REPORT_SUBJECT_ert) ? performCCC() : subject.equalsIgnoreCase(IOAConstants.DDD_REPORT_SUBJECT_fgt) ? performCCC() : didNothingValue; 

I. Do. 아니. 권하다. 그것. :-) 읽기 어렵고, 디버깅하기 어렵고, 유지하기가 어렵습니다.

Answer 2

if (abciden != null) { 
    // PERFORM SSS 
} 
else { 
    if (subject.equalsIgnoreCase(IOAConstants.DDD_REPORT_SUBJECT_ert)) { 
     //PERFORM CCC 
    } 
    else if (subject.equalsIgnoreCase(IOAConstants.DDD_REPORT_SUBJECT_fgt)) { 
     //PERFORM BBB 
    } 
} 

약간 변형 된 @ T.J. code : p