Java에서 sql 쿼리를 써야할 때 이렇게 코딩하는게 맞나요

Question

프론트 개발 후 이직한 곳에서 기존 코드를 보고 있는데 제가 db쪽은 처음이기도하고 경력이 그닥 길지 않아 가입하고 여쭤보게 되었습니다.현재 회사에서 sql쿼리를 모듈화하지않고 아래와 같은 방식으로 버퍼에서 append로 이어붙여 하드코딩을 해가지고 java와 sql쿼리를 연동하더라고요..sql.append(“SELECT * FROM TABLE”);sql.append(“WHERE @@@@“);... 중략…n*100줄....sql.append(“END”);//쿼리 하나당 함수 하나에 위와 같이 하드코딩으로 짜여져 함수가 수백개있음이런 쿼리랑 java코드가 같은 한 코드에 모여 1만줄가량 쌓여있습니다근데 이렇게 다 구현을 해서 값이 바뀌거나 로직 수정이 필요하면 현업에서 요청이 올 때마다 개발자가 코드를 일일이 수정해주는 상황이고 모듈화도 안되서 코드가 기본 5천-1만줄인 상황입니다Append와 버퍼의 필요성도 알고 있으나 그게 이렇게 하드코딩을 하라는 말은 아닌거로 아는데 보통 다른 기업에서도 이렇게 짜나요..?제 생각엔 쿼리만 따로 sql파일에 넣어두고 import하거나 불러오는게 맞다고 생각이 되서 제가 짠 코드는 따로 sql에 모듈화 했는데 협업 하시는 분들은 다 이렇게 짜고 있는데 이게 맞나요

겸손하되당당하게 · Accepted Answer

상당한 레거시 시스템을 사용하고 있나 보네요.모든 기업이 다 그런건 아니지만 아직도 종종 보이곤 합니다...작성자님 말대로 쿼리를 따로 불러와서 쓰는 경우가 대다수이죠.

동도인 · Answer

ㄷㄷㄷ...저렇게 append 로 String 객체로 쿼리 사용해도 sql인젝션 같은 보안 에 안걸리나요?

더미 · Answer

호랑이가 담배피던 시절에 쓰던 방식이죠.

도니코니 · Answer

엄청난 초 레거시의 레거시 시스템이네요..미친..다른 사람들은 기존에 이렇게 되어있었으니까 똑같이 해야지~ 라는 마인드일거에요. 특히 오래된 레거시의 경우 뭘 바꿀때 상당히 조심스러울듯

전뚱이인데요 · Answer

아 완전 네이트브 쿼리이네요.. ㄷㄷ

B급 감성으로 · Answer

그래도 + "" + 보단 낫네요.

yeori · Answer

최소 10년 이상 된 시스템같습니다.

망밍뭉멩몽 · Answer

12년전 프로젝트때 봤었던 코드 ㅋㅋ 코드가 정말 중구난방이었던 기억이 나네요..따로따로 개발되어 있었던....

HJOW · Answer

좀 프로젝트가 커지면 그렇게 안합니다.(연계모듈 같은, 작은 프로젝트들은 아직 이 방법 많이 씁니다.)가장 보편적인 방법이 MyBatis 를 이용하는 건데쿼리는 xml 파일로 분리할 수 있습니다.더 최신의 방법은 JPA 라고SQL문을 짜지 않고, 자바 코드로 데이터를 요청하면, JPA 라이브러리가 자동으로 SQL문을 만듭니다.

xml개발자 · Answer

이야.. 언제적 시스템이야...20년쯤은 되었을듯

MuDaeng · Answer

전 회사코드에 기대하지 않아요.신기술은 제 개인프로젝트에다가 쓰고 회사코드는 신경안씁니다

출출해 · Answer

문자열연결만보고 sql인젝션 얘기하는애들은 하수다 prepared statement 사용하면 문제없음

정말이쁜 · Answer

지금 구조에서 큰 이슈가 발생하지 않는 이상 그대로 작업하자고 할 가능성이 큽니다.생각하시는 문제점은 단순 기우일거에요 저도 20년째 운영중인 jsp 유지보수중인데.. 큰문제 없더라구요 . 보안은 보안장비랑 보안 툴 팔아먹는 업체가 다 해줍니다. 스스로 한번 테스트해보세요 바로 보안회사서 연락올겁니다. 아피 자동 차단되고 .. 맘애 안들면 다른 회사로... 가시면 됩니다.

mom dokha · Answer

레거시에는 정답이 없습니다. 상상 초월 그 자체이니 이해하려해봤자 머리아파와요.

권절현도 · Answer

은행에서 statement 오히려 안씁니다. 값이 변경없는 쿼리는 캐싱도 분석기도 제대로 반응하지 않습니다. mybatis등의 솔루션은 사용자의 역량에 의해 설정을 잘못하면 안쓰니만 못하긴 합니다. 그러나, 테이블고정되고 조건절의 필드와 값이 고정된 쿼리가 아니라 빌더객체를 쓰는거 가체가 자바의 JDBC표준을 모르는 사람이 구현한 것인데 저걸 둔다? 인건비싸게 후려치거나 한거라 고객은 벼르고 있을 겁니다. 기간계에 저렇게 했다? ... 말줄입니다

권절현도 · Answer

필자분은 당장 살돈이 없는게 아니면 빨리 이직하시는 게 좋습니다. 저거 어디회사인지 알면 전 그회사가 발주한 프로젝트 혹은 수주한 프로젝트는 안가는게 답이네요

Java에서 sql 쿼리를 써야할 때 이렇게 코딩하는게 맞나요

16개의 답변

#sql 목록