추상클래스 활용한 소스 조언 부탁드립니다.

Question

springboot 프로젝트에서 두개의 db에 대한 ddl은 jdbc를 활용하여 제어하려고 합니다.공통적인 부분은 추상클래스에 정의하고각각 처리할 부분, 접속 부분을 상속받는 class에 정의하여 구현하려고 하는데문제는 jdbctemplate에 datasource를 set 하는 부분으로 인해 중복을 벗어나기 힘듭니다. Abstract Dao(){&#64;AutowiredJdbcTemplate jdbcTemplate; public void rename() { String query &#61; &#34;RENAME TABLE name1 to name2&#34;; jdbcTemplate.execute (query); } } 와같이 이름을 변경하는 메서드가 공통이지만 각각 다른 datasource를 set해야하기 때문에상속받은 클래스에서 &#64;Qualifier(&#34;datasource1&#34;)&#64;AutowiredDatasource datasource;와 같이 설정한 datasource를 받아 jdbcTemplate.setDatasource(datasource);와같이 해줘야하는데 이부분을 어떻게 나누는게 맞는지 판단이 잘 되지 않습니다.공통작업이 있는 다중 접속 jdbc dao클래스를 어떻게 코딩하면 될지 조언을 구해봅니다.

2년마다매너리즘 · Answer

지금 답변드리면 늦을려나요?

밤과라임 · Answer

&#64;2년마다매너리즘저는 각 repository를 다루는 datasource를 두개 두고 각각 참조하는 식으로 두 db연결을 제어했습니다.어떤 방법이 있는지 궁금하여 답변 부탁드립니다.

2년마다매너리즘 · Answer

각 repository를 다루는 datasource를 두개 두고 각각 참조하는 식으로 두 db연결을 제어했다는 건상속을 사용하지 않고 처리했다는 말씀인가요? &#64;AutowiredJdbcTemplate jdbcTemplate; public void rename() { String query &#61; &#34;RENAME TABLE name1 to name2&#34;; jdbcTemplate.execute (query);}DB만 다를뿐 사용하는 쿼리는 동일하니깐이게 공통 부분이라고 생각하신거고 추상클래스에 넣고 싶은 구현이고 &#64;Qualifier(&#34;datasource1&#34;)&#64;AutowiredDatasource datasource;와 같이 설정한 datasource를 받아 jdbcTemplate.setDatasource(datasource);하위 클래스에서 데이터소스를 지정해야 하는건가요?그리고 하위클래스에 넣을만한 로직이 더 있을까요?

밤과라임 · Answer

&#64;Qualifier(&#34;devHotelDataSource&#34;)&#64;Autowiredprivate DataSource DevdataSource; &#64;Qualifier(&#34;prodHotelDataSource&#34;)&#64;Autowired private DataSource ProddataSource; 와 같이 Datasource 두 datasource를 만들고 jdbcTemplate.setDataSource (DevdataSource);와 같이대상을 특정하여 풀었습니다.

2년마다매너리즘 · Answer

그렇 지금까지 이해한 내용으로 한번 작성해볼게요, public void rename() {}과 같은 쿼리메소드들이 공통 구현내용이고 변하는 구현은 멀까요?쿼리가 실행될 리소스가 다르다. 라는 걸 변하는 구현이라고 하겠습니다. 그래서 일단 상위클래스를Abstract Dao(){ ... public void rename() { String query &#61; &#34;RENAME TABLE name1 to name2&#34;; jdbcTemplate.setDatasource(getDataSource()); jdbcTemplate.execute (query); } protected abstract DataSource getDataSource(); //하위클래스에서 실행될 데이터 리소스 리턴}이렇게 만듭니다. 그렇다면 하위클래스에서 dataSource를 반환해야겠지요하위클래스명 { &#64;Override protected DataSource getDataSource() { return &#34;devHotelDataSource&#34;의 객체 }} 그럼 하위클래스에서 devHotelDataSource를 어떻게 구하느냐만약 추상-자식클래스 구조로 만든다고 하신다면 자식클래스가 생성되고, bean으로 등록된다는거겠죠.(devHotelDataSource, prodHotelDataSource) 그래서 &#64;Autowired 사용합니다. 그럼 하위클래스명{ &#64;Qualifier(&#34;devHotelDataSource&#34;) &#64;Autowired private DataSource dataSource; &#64;Override protected DataSource getDataSource() { return dataSource; }} 자. 그런데 devHotelDataSource, prodHotelDataSource 를 가지는 하위클래스의 코드를 보면결국 달라지는건 &#64;Qualifier 안에 (&#34;데이터소스ID&#34;) 일 겁니다.데이터소스 ID로 데이터 소스를 구하는것도 중복된다는거겠죠.그래서 이것도 부모클래스에 넣어보겠습니다. Abstract Dao(){ ... public void rename() { String query &#61; &#34;RENAME TABLE name1 to name2&#34;; String dataSourceNanme &#61; getDataSourceName(); DataSource source &#61; //데이터소스명으로 데이터 소스 구하는 코드 ex)context.getBean(dataSourceNanme ) jdbcTemplate.setDatasource(source ); jdbcTemplate.execute (query); } protected abstract String getDataSourceName(); //하위클래스에서 실행될 데이터 리소스 리턴} 하위클래스명{ protected String getDataSourceName() { return &#34;devHotelDataSource&#34;; }} 이제, 하위클래스에서 데이터소스명만 리턴하면 부모클래스에서 스트링문자열로 컨텍스트에서 데이터소스를 찾아와서 쿼리를 실행할겁니다.굳이 추상클래스와 상속을 써야겠다면 위 방법보다 심플한 코드는 없을거에요. 그런데 지금 이상황에서 추상클래스와 상속을 사용한다는 건 하책입니다. 상속을 사용하지 않는 것이상책입니다. 이류를 생각나는대로 한번 적어볼게요. 1.하위클래스명을 멀로 지으시겠습니까 ? 작명을 하면서 하위클래스의 기능.역할에 대해서 고민해 보면이 클래스가 하는일은 이미 스프링프레임워크에 다 있을겁니다. 2.바뀌는 부분이 있고, 바뀌지 않는 부분이 있다고 해서 상속을 사용해야하는 건 아닙니다. 3.동일한 테이블명을 가지는 DB가 추가되면, 단지 하위클래스를 추가하는 것만하면되지만. 이거는 상속을 사용하지 않는 경우와 비교해보면 불필요한 작업 4.그런데 DB가 추가되는것보다 쿼리가 추가되는게 많을겁니다. 고로 자식클래스 수정보다 부모클래스가 수정되는 일이 더 많을 겁니다. 5.그리고 devHotelDataSource, prodHotelDataSource 빈이 2개 생긴다면 선택적으로 특정db만 쿼리를 실행해야한다면 실행할 데이터소스를 선택해줘야 하는 건 위 상속관계에서 해결할수가 없습니다. 예 ) devHotelDataSource.rename(); 같은 6.상속을 사용하지 않는것보다 복잡함 (아래참고) 7.만약 테이블명이 다른 예외가 생긴다면? 8.만약 상급자가 상속을 시러한다면? ㅎㅎㅎ 저라면 상속사용하지 않고 그냥 만들거 같습니다. 그리고 spring의 기능을 사용합니다. class DdlQueryHelper(){ private JdbcTeamplate template; public static void rename(DataSource... soruce) { String query &#61; &#34;RENAME TABLE name1 to name2&#34;; jdbcTemplate.setDataSource(soruce) jdbcTemplate.execute (query);}그래서 호출할때 ddlQueryHelper.rename(devHotelDataSource) 이렇게 사용하는거죠 자. 그런데 상속을 사용한 경우와 다르게 문제가 하나 생기는데바로 스레드세이프하지 않는 겁니다. (jdbcTemplate을 클래스변수로 가지고 있음)ddl쿼리니깐 호출은 충분히 제어가능하겠지만. 어떻게 될지 모르겠죠.따라서 안전하게 하려면 파라미터로 template도 받아야 합니다.class DdlQueryHelper() { private JdbcTeamplate template; public static void rename(JdbcTeamplate template, DataSource... soruce) { String query &#61; &#34;RENAME TABLE name1 to name2&#34;; jdbcTemplate.setDataSource(soruce) jdbcTemplate.execute (query);} 그러면 호출하는데서 너무 많은일을 해야하니깐 귀찮겠죠. 그래서 DDLQueryHelper에 기능을 넣습니다.그래서 저라며 아래처럼 만들겠습니다. 그냥 방법만 제시하는겁니다. &#64;beanclass DdlQueryHelper() implements InitialBean?{ enum DB { DEV_HOTEL_DB ( &#34;devHotelJdbcTemplate&#34;); PROD_HOTEL_DB(&#34;prodHotelJdbcTemplate&#34;) } //enum만들어서 String 잘못입력되는 상황방지 private Map<String, JdbcTeamplate> templateMap; public void renameAll() { //templateMap에 등록된모든 db 리네임} public void rename(DDLQueryHelper.DB db) { //선택 db만 리네임, enum으로 받기 때문에 선택한 db가 map에 없는 경우는 없음 }} 사용할때는 ddlQueryHelper,rename(DDLQueryHelper.DB.DEV_HOTEL_DB )ddlQueryHelper.renameAll(); 그리고 마지막으로 최상책입니다. 저라면 이렇게 합니다.DDL은... 그냥 쿼리브라우저로 하십시오. 계속 관리하는것도 일입니다. ㅎ 제가 제대로 이해하rh 쓴건지 모르겠습니다.아무튼 지금 만드신 class도 스레드세이프하지 않은것 같은데.. 크게 문제 되지 않을것 같다면 그냥 쓰셔도 될거 같습니다.

밤과라임 · Answer

&#64;2년마다매너리즘java를 사용한지 꽤 되었는데도 아직 깊이 부족하여 작업할때라도 조금씩 익히려 하고 있습니다. 최근 좋은쿼리란 중복을 피하고 한 메소드의 코드수가 많지 않아야 한다는 등 외관적인 모습에 신경을 쓰려하다보니본질을 놓치고 복잡하게 돌아갔습니다.db 툴을 통해 해결할 수 있는 일을 수고스럽게 작업하게 된 배경은 첫째론 도전이며,두번째로는 통일과 규칙의 함정에 빠져서입니다.유연한 사고를 늘 생각하려해도..이번에도 빠져버렸네요.더 편리한 방법, 그리고 그렇게 작업하려는 이유. 많은 것을 다시 배웠습니다.감사합니다.

2년마다매너리즘 · Answer

통일과 규칙에 빠지지 않도록 계속 도전을 멈추지 마세요.ㅎㅎ누구든 그 과정을 거치니깐요.위와 같은 문제는 답이 없다고 생각합니다. 검증하면서 더 나은 답을 찾아가는 과정만 있을 뿐.. ㅎㅎ그럼 수고하세요.

추상클래스 활용한 소스 조언 부탁드립니다.

7개의 답변

#abstract 목록