[Model Condition] 상품 정렬 및 필터링 로직 최적화 – 중복 조건 제거와 효율적인 SQL 조건 설정 🚀

☑️ 기존 문제점 : 불필요한 중복 조건 처리

• 기존 코드 : 두 가지 조건을 동시에 검사 → 맞다면 SQL 조건을 설정하는 방법

• 카테고리 조건과 정렬 조건이 반복적으로 사용되면서 코드 중복이 많아지고 유지보수성이 떨어지는 문제

→ 아래 코드 블록과 같이 동일한 정렬 조건을 나열함

if ("ORDERPOPULAR".equals(CUProductOrderCondition) && "ALLPRODUCT".equals(CUProductCategory)) {
			productSingleDTO.setCondition("SELECTALL_3");
		} 
        else if ("ORDERHIGHPRICES".equals(CUProductOrderCondition) && "ALLPRODUCT".equals(CUProductCategory)) {
			productSingleDTO.setCondition("SELECTALL_4");
		} 
        else if ("ORDERLOWPRICES".equals(CUProductOrderCondition) && "ALLPRODUCT".equals(CUProductCategory)) {
			productSingleDTO.setCondition("SELECTALL_5");
        }

• 모든 카테고리와 모든 정렬 조건을 동시에 비교함

 

🔄 코드 개선: 중복 제거 및 구조화된 설정

• 중복되는 로직을 제거, 보다 구조화된 방식으로 카테고리와 정렬 조건을 설정
• 카테고리와 정렬 방식을 각각 변수로 처리하여 한 번만 조건을 검사하도록 개선

 

✅ 개선된 코드 핵심 로직

if ("ORDERPOPULAR".equals(CUProductOrderCondition) && "ALLPRODUCT".equals(CUProductCategory)) {
		    productSingleDTO.setCondition("SELECTALL_3");
		} else if ("ORDERHIGHPRICES".equals(CUProductOrderCondition) && "ALLPRODUCT".equals(CUProductCategory)) {
		    productSingleDTO.setCondition("SELECTALL_4");
		} else if ("ORDERLOWPRICES".equals(CUProductOrderCondition) && "ALLPRODUCT".equals(CUProductCategory)) {
		    productSingleDTO.setCondition("SELECTALL_5");
		} else if ("DAILYSUPPLIESPRODUCT".equals(CUProductCategory)) {
		    productSingleDTO.setProductSingleCategory("DAILYSUPPLIESPRODUCT");
		    if ("ORDERPOPULAR".equals(CUProductOrderCondition)) {
		        productSingleDTO.setCondition("SELECTALL_6");
		    } else if ("ORDERHIGHPRICES".equals(CUProductOrderCondition)) {
		        productSingleDTO.setCondition("SELECTALL_7");
		    } else if ("ORDERLOWPRICES".equals(CUProductOrderCondition)) {
		        productSingleDTO.setCondition("SELECTALL_8");
		    }
		} else if ("BEVERAGEPRODUCT".equals(CUProductCategory)) {
		    productSingleDTO.setProductSingleCategory("BEVERAGEPRODUCT");
		    if ("ORDERPOPULAR".equals(CUProductOrderCondition)) {
		        productSingleDTO.setCondition("SELECTALL_6");
		    } else if ("ORDERHIGHPRICES".equals(CUProductOrderCondition)) {
		        productSingleDTO.setCondition("SELECTALL_7");
		    } else if ("ORDERLOWPRICES".equals(CUProductOrderCondition)) {
		        productSingleDTO.setCondition("SELECTALL_8");
		    }
		} else if ("FOODPRODUCT".equals(CUProductCategory)) {
		    productSingleDTO.setProductSingleCategory("FOODPRODUCT");
		    if ("ORDERPOPULAR".equals(CUProductOrderCondition)) {
		        productSingleDTO.setCondition("SELECTALL_6");
		    } else if ("ORDERHIGHPRICES".equals(CUProductOrderCondition)) {
		        productSingleDTO.setCondition("SELECTALL_7");
		    } else if ("ORDERLOWPRICES".equals(CUProductOrderCondition)) {
		        productSingleDTO.setCondition("SELECTALL_8");
		    }
		}

 

✨ 개선 후 기대 효과

✅ 코드 중복 제거로 유지보수성 향상

• 기존 코드에서는 각 카테고리별로 정렬 조건을 따로 설정 → 단 한 번의 조건 검사로 설정이 가능

• 새로운 카테고리를 추가할 때도 중복 코드를 추가하지 않아도 됨

✅ 가독성 개선

• 구조적으로 정렬 조건과 카테고리 조건을 명확하게 구분 → 가독성이 향상

• 불필요한 if-else 블록을 줄이고, 보다 직관적인 조건 분리

✅ 실행 속도 최적화

• 기존 방식에서는 여러 if-else 문이 중복, 불필요한 연산이 발생 → 최소한의 연산으로 필요한 조건을 설정

 

🚀 결론: 더 깔끔하고 유지보수하기 쉬운 코드로 개선

조건 검사 로직이 단순화되고 중복 코드가 제거
코드의 유지보수성을 고려하여 최적화할 수 있는 방안을 고민