Whiteship's Note

Report Validator 개선 사항

모하니?/Coding : 2007.03.29 01:42


검색 API를 여러 개 붙이는 것이 생각보다 쉽지 않더군요. 이 말은 확장이 불편하다는 거고 다시 또 이 말은 중복된 코드로 인해 코드 끼리 의존성이 심해졌다는 것입니다.

소스 코드 문제점

현재까지 크게 두 부분이 문제가 됩니다.
public static int search(Sentence sentence) {
        int totalResult = useNaverAPI(sentence, "blog")
            + useNaverAPI(sentence, "webkr")
            + useNaverAPI(sentence, "kin")
            + useDaumAPI(sentence, "blog");
        sentence.setPoint(totalResult);
        return totalResult;
    }

이런식으로 API를 사용하는 메소드를 호출하여 사용하게 되는데 useNaverAPI와 useDaumAPI는 요청을 보내는 문자열만 다를 뿐 하는 일은 완전히 같기 떄문에 엄청난 양의 중복된 코드를 발생시켰습니다.

이것과 더불어 Sentence 객체를 보면 다음과 같이 만들어져 있습니다.
public class Sentence {
    private String line;

    private int copyLebel;

    private String naverBlogURL;

    private String naverWebURL;

    private String naverKnowURL;

    private String daumBlogURL;

어처구니가 없는거죠.ㅋㅋ Map을 쓰면 되는데 API 붙일 때 마다 속성을 하나씩 추가 해야 된다니.. 왜 naverWebURL 속성을 만들 때 미쳐 중복을 제거할 생각을 못했을까요..ㅠ.ㅠ 아니면 생각은 했지만 Map을 쓰는게 익숙하지 않아서 못본척 하고 넘어려는 자아가 공부하려는 자아를 속였을 수 있습니다. 그리고는 마치 몰랐었던 척..하는...나쁜 기선.

화면의 문제점

사용자 삽입 이미지
첫 화면 입니다. 간단한 메뉴얼이 눈에 걸리적 거립니다. 좀 더 이쁘게 보여줄 수 없을런지.ㅠ.ㅠ 전체적인 색은 제가 좋아하는 파란색과 하얀색 계열로 바꿔야 겠습니다.

사용자 삽입 이미지
흠..텍스트 위주의 화면..저도 이쁜 아이콘과 그림으로 꾸미고 싶습니다. 표는 AJAX로 멋지게 "전체 결과 보기" 같은 탭을 열어 주면 쫙~ 펼처지도록 하고 싶...지...만... 31일까진 무리라는거.. CSS는 역시..색을 좀 조절하고.. 테이블 모양도 좀 더 괜찮은 모양으로 바꿔줄까 합니다.

화면 코드 문제점

jsp 파일 두 개에서도 헤더와 푸터가 중복되어 나타나고 있는데요. 그걸 include 인가를 사용해서 제거 해야겠습니다.

그리고 "있다" "없다"를 판단해서 찍어주는 JSTL 코드 부분이 계속 반복해서 나타납니다. 이건 태그파일을 사용해서 코드를 간결하게 줄일 수 있겠습니다.


'모하니? > Coding' 카테고리의 다른 글

Semina Helper v0.5  (0) 2007.04.06
아 이런 바보..ㅠ.ㅠ  (2) 2007.03.31
Report Validator v1.0  (0) 2007.03.29
JAR, WAR 에피소드 해결 방법  (0) 2007.03.29
JAR, WAR 에피소드  (2) 2007.03.29
Report Validator 개선 사항  (0) 2007.03.29
까다로운 녀석..ㅠ.ㅠ  (0) 2007.03.27
Report Validator v0.9  (2) 2007.03.26
Report Validator v0.5  (0) 2007.03.26
아 답답해;;  (2) 2007.03.26
테스트를 꼭 실행 해야 하는 시점  (0) 2007.03.23
top




: 1 : ··· : 272 : 273 : 274 : 275 : 276 : 277 : 278 : 279 : 280 : ··· : 299 :