코드리뷰

[self-review] 로직의 논리오류

사실 로직을 분석하고 구현할 때 단순한 문법오류는 굉장히 훌륭해진 IDE(이클립스를 위시한)들이 잡아준다. 다만, 교묘한 논리적인 오류들이 개발자의 머리를 복잡하게 만든다. 문제)

int count = 5;

List<App> appList = appBO.getAppList();

if (appList.size() < count) {

List<BestApp> bestAppList = bestAppBO.getAppList(count - appList.size());

for (BestApp app : bestAppList) {

if (isNotContains(appList, app) {

appList.add(convert(app));

}

위 로직은… 어떤 리스트를 구성하는데, 갯수가 부족하면 베스트앱으로 채워넣는 로직이다. 위 로직에서 어떤 부분이 문제일까. 기존에 별 문제가… 더 보기 »[self-review] 로직의 논리오류