skyhills13 / PhotoMosaic

2014-3-실전 프로젝트 레파지토리
3 stars 1 forks source link

코드리뷰 확인하세요 #157

Open crongro opened 9 years ago

crongro commented 9 years ago

04d68c05dedc1adc98b69da6ae92bdaa7b159b27

skyhills13 commented 9 years ago

수정하지 않은(을) 피드백에 대한 변

+* SQL 문을 소스코드에서 나중에는 분리할 계획이 있는지? 혹시 계획이 아예없다면 나중에 분리하는 게 좋겠지? +* 다른 클래스도 마찬가지고..

-> my-batis라는 프레임워크로 분리할 계획이었으나, 학습 없이 분리하다가 지난 토요일 하루를 몽땅 날리고, 이번 프로젝트에서는 db 프레임워크를 적용치 않기로 결정하였습니다. 따라서 mysql 분리는 없을 것 같습니다.마는 Constants처럼 따로 sql만 빼는 것은 검색해보고 괜찮은 방식을 찾아 바꿔보겠습니다.

//아직 완성품이 아닌 듯? 매개변수에 선언된 건 많은데 메서드에서 사용하는 건 별로 없어보이길래. ->현재 DTO에서 사용하는 모든 생성자는 모두 사용하고 있는 것들입니다. DAO에서 select문 사용시 필요로합니다.

// 아래 getter setter는 분명히 필요해서 추가한 것들이겠지? -> 쓰지 않아도 getter setter는 모두 생성하는 것으로 알고있었습니다. 좀 더 학습한 후 수정하겠습니다.