자바 if else 로직 관련 질문입니다.

Question

안녕하세요, 선배님들.비즈니스 로직 작성 중에 문득 궁금한 게 생겨 질문을 올려봅니다.. 코드는 다음과 같은데요.결제 시에 사용할 수 있는 결제 수단으로는기타(현금, 카드 등), 포인트, 정액권까지 총 세 개가 있습니다. if (params.getUs_ticket_amt() > 0 && params.getUs_pnt_amt() > 0 && params.getUs_other_amt() > 0) { // 정액권, 포인트, 현금을 모두 사용한 경우 params.setType(KakaoTemplate.ALL_USE); } else { // 정액권을 사용한 경우 if (params.getUs_ticket_amt() > 0) { if (params.getUs_pnt_amt() > 0) { // 포인트를 함께 사용한 경우 params.setType(KakaoTemplate.TICKET_POINT_USE); // 정액권 & 포인트 결제 } else if (params.getUs_other_amt() > 0) { // 현금을 함꼐 사용한 경우 params.setType(KakaoTemplate.TICKET_OTHER_USE); // 정액권 & 현금 결제 } else { params.setType(KakaoTemplate.TICKET_USE); // 정액권만 사용한 경우 } } else if (params.getUs_pnt_amt() > 0) { // 포인트를 사용한 경우 if (params.getUs_other_amt() > 0) { params.setType(KakaoTemplate.POINT_OTHER_USE); // 포인트 & 현금 결제 } else { params.setType(KakaoTemplate.POINT_USE); // 포인트만 사용한 경우 } } else if (params.getUs_other_amt() > 0) { // 현금을 사용한 경우 params.setType(KakaoTemplate.POINT_SAVE); } // end of first if } 여기서 문제가 되는게..기타, 포인트, 정액권 각각으로 결제할 수도 있고,세 가지를 합쳐서 결제할 수도 있고,원하는 결제 수단을 선택해서 결제할 수 있습니다.즉, 여러 가지 케이스가 생기는 상황입니다..(ex: 카드만 사용, 정액권만 사용, 포인트만 사용 카드와 정액권을 사용, 카드와 포인트를 사용 카드와 정액권과 포인트 모두 사용) 해당 로직을 if else를 사용하지 않거나,지금보다 깔끔하게 작성할 수 있는 방법이 있을까요?선배님들의 답변 미리 감사드립니다.코로나, 감기 조심하세요. ^^

지붕뚫고높이차 · Answer

재밌는 질문이네요.퇴근중이라집에서 한번 쉽게 만들어 볼께요.

친슈 · Answer

흠 한가지는 if else 블럭들을 바깥으로 빼는 방법이 있겠네요public void setPaymentType(Params params){ String type; boolean usedTicket &#61; (params.getUs_ticket_amt() > 0); boolean usedPoint &#61; (params.getUs_pnt_amt() > 0); boolean usedOther &#61; (params.getUs_other_amt() > 0); if (usedTicket && usedPoint && usedOther) { // 정액권, 포인트, 현금을 모두 사용한 경우 type &#61; KakaoTemplate.ALL_USE } else { // 정액권을 사용한 경우 if (usedTicket) { type &#61; this.getTicketType(usedPoint, usedOther); } else if (usedPoint) { // 포인트를 사용한 경우 type &#61; this.getTicketType(usedOther) } else if (usedOther) { // 현금을 사용한 경우 type &#61; KakaoTemplate.POINT_SAVE; } // end of first if } params.setType(type); } private String getTicketType (boolean usedPoint, boolean usedOther){ String type; if(usedPoint){ type &#61; KakaoTemplate.TICKET_POINT_USE; } else if(usedOther){ type &#61; KakaoTemplate.TICKET_OTHER_USE } else { type &#61; KakaoTemplate.TICKET_USE } return type; } private String getPointType (boolean hasUsedOther){ String type; if (usedOther) { type &#61; KakaoTemplate.POINT_OTHER_USE; } else { type &#61; KakaoTemplate.POINT_USE; } return type; }

포포핀 · Answer

Enum 계산식을 활용한 방법 3.데이터그룹관리 부터 보시면 될듯합니다. https://woowabros.github.io/tools/2017/07/10/java-enum-uses.html

도뎡 · Answer

지붕뚫고높이차 님 답변 감사드립니다.요즘들어 기본에 충실한 게 최고라는 생각이 드네요 ^^..

도뎡 · Answer

친슈 님 답변 감사드립니다.참고해서 한번 작성해 볼게요. ^^~

도뎡 · Answer

포포핀 님, 링크 걸어주신 ENUM 활용기 포스팅은 몇번이고 보았지만..제 내공으로는 이해가 쉽지가 않네요. ^^..답변 주셔서 감사드립니다!

엡실론 · Answer

binary네요. boolean ticket &#61; params.getUs_ticket_amt() > 0;...if (ticket && point && other) { return ALL_USE; } if (ticket && point) { return TICKET_POINT_USE; }이런식으로 nesting을 없앨수도 있고, true를 1, false를 0으로 해서.payment &#61; [NONE_USED, POINT_SAVE, POINT_USE, POINT_OTHER_USE, ...] return payment[ticket * 4 &#43; point * 2 &#43; other]할 수도 있으나, 가독성면에서 별로 좋진 않은 것 같네요.

지붕뚫고높이차 · Answer

제가 작성한 첫번째 코드는 setType 메소드를 만들어조건에 대한 int 변수를 조합if return 형태로 사용한 코드입니다.- 금액이 없을 경우를 위한 방어코드가 있고- 3개의 조건이면 2^3 &#61; 8개 에 대한 코드가 있어야 하는데 실수로 누락되면 오류가 발생하도록 했지만- 보는 사람에 따라서 가독성이 저하되는 문제가 있기는 하겠네요. String type &#61; setType(params);params.setType(type); private String setType(Params params) { int 티켓사용 &#61; 0; int 포인트사용 &#61; 0; int 기타사용 &#61; 0; if (params.getUs_ticket_amt() > 0) { 티켓사용 &#61; 100; } if (params.getUs_pnt_amt() > 0) { 포인트사용 &#61; 10; } if (params.getUs_other_amt() > 0) { 기타사용 &#61; 1; } int 구분 &#61; 1000 &#43; 티켓사용 &#43; 포인트사용 &#43; 기타사용; if (구분 &#61;&#61; 1000) { // 사용할 금액이 없음 throw new RuntimeException(); } if (구분 &#61;&#61; 1111) { return KakaoTemplate.ALL_USE; } if (구분 &#61;&#61; 1100) { return KakaoTemplate.TICKET_USE; } if (구분 &#61;&#61; 1010) { return KakaoTemplate.POINT_USE; } if (구분 &#61;&#61; 1001) { return KakaoTemplate.POINT_SAVE; } if (구분 &#61;&#61; 1110) { return KakaoTemplate.TICKET_POINT_USE; } if (구분 &#61;&#61; 1101) { return KakaoTemplate.TICKET_OTHER_USE; } if (구분 &#61;&#61; 1011) { return KakaoTemplate.POINT_OTHER_USE; } // 여기까지 오면 조건이 누락되었음 throw new RuntimeException(); }

지붕뚫고높이차 · Answer

두번째 방법은 사람에게 익숙한 문자열과 컬랙션 객체인 Map 을 사용하는 방법입니다.- 개인적으로 자주 사용하고 있구요. 추가조건이 있어도 수정이 쉽습니다.- 컬랙션 객체에 민감하신 분은 사용을 안하기도 하죠. public String setType2(Params params) { String 티켓사용 = "티켓X"; String 포인트사용 = "포인트X"; String 기타사용 = "기타X"; if (params.getUs_ticket_amt() > 0) { 티켓사용 = "티켓O"; } if (params.getUs_pnt_amt() > 0) { 포인트사용 = "포인트O"; } if (params.getUs_other_amt() > 0) { 기타사용 = "기타O"; } String x = 티켓사용 + 포인트사용 + 기타사용; HashMap hm = new HashMap(); hm.put("티켓O포인트O기타O", KakaoTemplate.ALL_USE); hm.put("티켓O포인트X기타X", KakaoTemplate.TICKET_USE); hm.put("티켓X포인트O기타X", KakaoTemplate.POINT_USE); hm.put("티켓X포인트X기타O", KakaoTemplate.POINT_SAVE); hm.put("티켓O포인트O기타X", KakaoTemplate.TICKET_POINT_USE); hm.put("티켓O포인트X기타O", KakaoTemplate.TICKET_OTHER_USE); hm.put("티켓X포인트O기타O", KakaoTemplate.POINT_OTHER_USE); String x2 = hm.get(x); if (x2 == null) { // 조건이 누락 throw new RuntimeException(); } return x2; }

ppppqqqq · Answer

4중 if ㄷㄷㄷ 다른 고급 언어였다면 걍 튜플로 패턴매칭을 돌렸겠죠. hashmap이나 bitmasking같은걸 써서 어찌저찌하는거 그렇게 스마트한 방법 아닙니다. 읽기도 어렵고요. enum도 나쁜 방법은 아닌데 그닥.. 너무 복잡하게 생각하지 말고 그냥 읽기 쉽고 리팩토링하기 쉽게 작성하세요. if (params.getUs_ticket_amt() > 0 && params.getUs_pnt_amt() > 0 && params.getUs_other_amt() > 0) { params.setType(KakaoTemplate.ALL_USE); } else if (params.getUs_ticket_amt() > 0 && params.getUs_pnt_amt() > 0 && params.getUs_other_amt() <= 0) { params.setType(KakaoTemplate.TICKET_POINT_USE); } else if (params.getUs_ticket_amt() > 0 && params.getUs_pnt_amt() <= 0 && params.getUs_other_amt() > 0) { params.setType(KakaoTemplate.TICKET_OTHER_USE); } else if (params.getUs_ticket_amt() > 0 && params.getUs_pnt_amt() <= 0 && params.getUs_other_amt() <= 0) { params.setType(KakaoTemplate.TICKET_USE); } else if (params.getUs_ticket_amt() <= 0 && params.getUs_pnt_amt() > 0 && params.getUs_other_amt() > 0) { params.setType(KakaoTemplate.POINT_OTHER_USE); } else if (params.getUs_ticket_amt() <= 0 && params.getUs_pnt_amt() > 0 && params.getUs_other_amt() <= 0) { params.setType(KakaoTemplate.POINT_USE); } else if (params.getUs_ticket_amt() <= 0 && params.getUs_pnt_amt() <= 0 && params.getUs_other_amt() > 0) { params.setType(KakaoTemplate.POINT_SAVE); } else if (params.getUs_ticket_amt() <= 0 && params.getUs_pnt_amt() > 0 && params.getUs_other_amt() > 0) { }

도뎡 · Answer

엡실론님 답변 감사드립니다. ^^참고해서 작성해 볼게요!

도뎡 · Answer

지붕뚫고높이차님 답변 감사드립니다!작성하신 코드 참고해 보도록 할게요. ^^

도뎡 · Answer

ppppqqqq님 답변 감사드립니다...4중 if에서 터져버렸네요 ㅋㅋㅋ...코드 참고해서 자성해 볼게요. ^^

도뎡 · Answer

지붕뚫고높이차님의 두 번째 답변으로 결정했습니다!저도 평소에 Map을 자주 이용하는 편이기에..(도메인 객체 제외)익숙한 걸 사용하는 게 맞다는 생각이 드네요. ^^;답변 주신 모든 분들 감사드립니다! String useTicket = (payInfo.getUs_ticket_amt() > 0) ? "T1" : "T0"; String usePoint = (payInfo.getUs_pnt_amt() > 0) ? "P1" : "P0"; String useOther = (payInfo.getUs_other_amt() > 0) ? "O1" : "O0"; String code = useTicket + "-" + usePoint + "-" + useOther; HashMap codes = new HashMap<>(); codes.put("T1-P1-O1", KakaoTemplate.ALL_USE); codes.put("T1-P1-O0", KakaoTemplate.TICKET_POINT_USE); codes.put("T1-P0-O1", KakaoTemplate.TICKET_OTHER_USE); codes.put("T1-P0-O0", KakaoTemplate.TICKET_USE); codes.put("T0-P1-O1", KakaoTemplate.POINT_OTHER_USE); codes.put("T0-P1-O0", KakaoTemplate.POINT_USE); codes.put("T0-P0-O1", KakaoTemplate.POINT_SAVE); log.info("code : {}", codes.get(code));

박종복 · Answer

final int ticket = params.getUs_ticket_amt() > 0 ? 1 << 2: 0; final int point = params.getUs_pnt_amt() > 0 ? 1 << 1: 0; final int other = params.getUs_other_amt() > 0 ? 1 : 0; int paymentType = ticket | point | other; public final class KakaoTemplate{ public final static int ALL_USE = 7; public final static int TICKET_POINT_USE = 6; public final static int TICKET_OTHER_USE = 5; public final static int TICKET_USE = 4; public final static int POINT_OTHER_USE = 3; public final static int POINT_USE = 2; public final static int POINT_SAVE = 1; }

도뎡 · Answer

박종복님 답변 감사드립니다!시프트 연산자를 사용하는 것도 한번 고려해 보아야겠습니다. ^^좋은 하루 되세요~

자바 if else 로직 관련 질문입니다.

16개의 답변

Q&A 목록