배고파서서러워요
1k
2018-09-14 16:02:27
8
1093

예약어는 조심합시다..


.......데이터베이스 좀 손볼일 있어서 봤더니, 컬럼명이 예약어로 도배가 되어있네요...

`처리로 일단 땜빵하긴 했습니다만, 이건 좀 너무한듯....


그리고 왜 비지니스 로직이 모델이랑 같이 있는지 모르겠....

모델은 좀 바인딩만 하고 뒤로 빼....orz


소스보다 화가나네요.....

단순 분기처리해야 할 곳을 일일이 추상클래스 때려박은것도 짜증나는데(...특정 컬럼값이 있냐 없냐를 구분하는 처리입니다... 이걸 클래스로 따로 파서 추상을 박아놨더라고요...) 기본도 못하는 코드 봤더니 참....

2
0
  • 댓글 8

  • 안개짱
    774
    2018-09-14 16:15:33

    EJB 흉내 낸다고 그런건지 개발자 3명에 쬐그만 프로젝트 인데도 service 인터페이스에 serviceimpl로 죄다 빼놓고.. 로직도 controller, service 구분없이 들어가있고.. 가관도 아니죠..


    pom.xml 보면 쓸데 없는것들 죄다 들어가있고.


    딱봐도 어디서 배껴다 쓴 티가 확 나는 프로젝트들 많네요.


    글보니 저도 덩달아 짜증나서 괜시리 투덜대어 봅니다.

    0
  • OrOl
    600
    2018-09-14 16:50:08 작성 2018-09-14 16:50:21 수정됨

    안개짱 개발자 3명에 쬐그만 프로젝트 인데도 service 인터페이스에 serviceimpl로 죄다 빼놓고

    찔리네요... ㅋㅋㅋㅋ 배우기를 그렇게 배워서 그런가 그대로 하고 있는데 작은 프로젝트에선 굳이 분리할 필요 없는건가요??

    1
  • 냉동참치
    230
    2018-09-14 16:54:21

    OrOl

    그 뒷말이 핵심일듯 하네요. 분리하긴 했지만 여기저기 구분이 없는..

    0
  • 안개짱
    774
    2018-09-14 17:11:10

    OrOl //

    제 의견이지만요.

    service를 인터페이스로 만드는 것은

    1. 서비스단만 작업하는 팀이 따로 있을때(비즈니스로직만 담당)

    2. EJB 구성으로 다른 장비에서 호출하기 위한 interface가 필요할때

    가 아니면 굳이 나눌 필요가 없다고 봅니다.


    간혹 고객쪽에서 그렇게 요구하면 그대로 해주고요.


    spring-mvc 프로젝트 기준..

    controller와 service에 어떤 로직을 넣는가도 제 나름의 기준이 있는데요.


    service쪽에 들어가는 로직에는 web관련 로직이 절대 들어가면 안된다는 겁니다.

    service쪽에 구현한 함수에 파라메터로 http 어쩌구 들어가면 안되고요. (request, response, session 등)

    비즈니스를 수행하기위한 데이터 validation과 비즈니스 로직만으로 구성되는게 좋다고 봅니다.

    이외에 UI단 관련된 로직은 controller에서 다 처리 하고요.


    제 생각에 mvc 모델과 Spring 구조를 볼때.

    스프링에서 말하는 controller는 mvc중에 view에 가깝다고 봅니다. UI에 관련된것들로만 구성되니까요.

    또 그래야 한다고 생각하고요.


    mvc모델중 C(controller)는 Spring의 service가 주로 담당한다고 생각되고요.(뭐 내용상 DB connection관련 것들도 포함되겠죠. - 주로 트랜젝션을 service단위로 잡으니...)


    model은 당연히 DB외 VO객체들로 이루어져 있다고 생각 합니다.


    다른 의견들도 있겠지만 제 생각이 그렇다는 겁니다.

    1
  • onimusha
    5k
    2018-09-14 17:19:01

    컨트롤러 소스가 뚱뚱하면 그 당시 개발한 개발자들의 지옥같은 역사가 쭈욱~ 스쳐 지나갑니다..

    (오만가지 로직이 다  컨트롤러에 다 붙어 있는데 서비스 메소드 열어보면 select 한줄 띵!)

    0
  • 배고파서서러워요
    1k
    2018-09-14 17:29:37

    .....으으.. 다른분들의 고통에 공감이 갑니다...ㅠㅠ

    0
  • OrOl
    600
    2018-09-14 18:01:06

    안개짱 저도 다 공감하는 내용이네요. 저도 거의 똑같이 기준 잡고 컨트롤러 다이어트 열심히 시키고 있습니다 ㅋㅋㅋ

    뭔가 좀 컨트롤러가 길다싶으면 멈춰서 좀 생각하고 하다보면 분리할부분이 많더라구요

    0
  • 삼식이
    1k
    2018-09-15 12:24:51

    추상클래스, 모델, 인터페이스를 쓸때 장단점이 뭔지도 모르면서

    업무 성격도 틀린데 따라하기식 해놓고 똥싸놓는 개발자(?)를 보면 죽빵 날리고 싶더군요.

    0
  • 로그인을 하시면 댓글을 등록할 수 있습니다.