코드작성법에 대해 질문드립니다

Question

방법론적으로 질문드리고 싶은게 있는데요사용하고있는게 java spring입니다글을 등록하거나 수정 혹은 어떤 액션을 처리할때로그인한 회원의 seq를 가지고 대상을 지정하고 있는데요이 로그인한 회원의 seq를 가져오는 곳을controller에서 가져와서 각 service단으로 넘겨주는방식으로 하고있었는데사수분이 seq는 service에서 가지고 오는게 맞고옵셔널한 값만 service로 보내야 한다는데seq를 꼭 service에만 가져와야하나요?아니면 어디서든 가져와도 되나요?

장독깨기 · Answer

회원 번호를 어디서 가져오시는지?

qkrtjddn**** · Answer

jwt토근에 담겨있습니다

장독깨기 · Answer

조금 말씀드려 보겠습니다.서비스 레이어를 두고, 서비스 메서드를 만들었잖아요.그럼, 그 서비스는 특정 기능이 완전하게 돌아가야겠죠.특정 기능이 완전하게 돌아가기 위해서는, 입력 값과 반환 값이 정의 되어 있어야 한다는 얘기가 되구요,근데, 회원번호를 서비스에서 가져오려면, 서비스 인자로 JWT 든 세션이든 인자로 넘겨야 한다는 얘기가 됩니다.그럼, 서비스 메서드 스키마만 보면, 이 서비스가 왜 JWT 를 인자로 받는지 알 수가 있을까요. 모르겠죠.또는, 공통함수를 사용한다 했는데요, 서비스 내에서 이 공통함수를 호출해서 회원번호를 가져온다면,우리가 이 서비스만 보고 회원번호가 필요한지 아닌지 알 수가 없겠죠.컨트롤러에서 회원번호를 가져와 서비스 인자로 넘기면,아, 이 서비스는 특정 기능을 동작하기 위해 회원번호가 필요하구나, 이걸 알 수 있다는 얘깁니다.개념적으로 이렇게 생각하면 무난합니다.그니까, 컨트롤러에서 서비스 호출에 필요한 값을 다 정의해서 서비스 호출 시 인자로 넘기면 됩니다.그런데, 이런 걸로 싸우지 마세요, ㅎ어짜피, 그 서비스 재 활용하는 것도 아닐테고, 어디서 회원 번호를 구하든 요구 기능은 동작할 거잖아요.그냥 개념적으로 저렇게 생각만 하고 있으면 됩니다.그리고, 누가 물어보면 저런 식으로 근거나 이유를 얘기하면 될 거 같습니다.아무 근거나 이유 없이, 이렇게 하는게 좋을거 같아, 이런 접근 방식은 개발자로서 바람직한 자세가 아닙니다.열심히 하세요. :)

바냐파노 · Answer

둘 다 써봤는데, 개인적으로는 controller에서 seq를 뽑아 service 에는 해당 값만을 전달하는 게 나은 것 같습니다.그럴 일은 없겠지만, 만일 인증방식을 토큰이 아닌 세션으로 변경한다면? 그러면 그때는 HttpSession 을 파라미터로 다 넘기는 작업을 해 줘야 합니다. 하지만 값만을 넘기는 경우 controller 단만 수정하면 되겠지요.

데브런 · Answer

seq가 무엇인지를 파악하는 게 중요합니다. 글로만봐서는 seq가 무엇인지 명확히 보이지는 않는데요. seq를 가져오는 로직 자체가 domain 즉 service에 해당하는지 controller에 해당하는지를 파악하면 좋을 것 같습니다. 토큰에 저장된 seq를 가져온다면, spring security를 이용할 때 securitycontext에 저장된 authentication을 통해서 조회하는 것으로 예상됩니다. 이 기준으로 얘기한다면 클라이언트와 interface하는 토큰 형태가 아닌 내부의 인증 및 권한을 관리하기 위한 도메인 객체인 authentication에 대한 seq 조회는 controller 보다는 service 성향이 강합니다. 따라서 객체는 service 계층에서 재활용 가능한 객체로서 활용하는 게 좋아보입니다.반면에 단순 토큰을 매 controller에서 받아서 authentication에 대응하는 객체를 생성하는 거라면 controller가 적합합니다.그게 아니라 단순히 domain 객체인 예를 들어 user에 있는 시스템 성향의 id 값 혹은 도메인에서 정의한 일렬번호라면 어떤 경우든지 controller로 가면 안 되고 서비스 계층에서 해결해야 합니다.

qkrtjddn**** · Answer

다들 답변 감사드립니다

오후 · Answer

꼭 Service 단에서 가져올필요는없고 내부 규칙(스타일)에 맞춰 작업하면됩니다.

82savage · Answer

서비스 인터페이스가 여러 곳에서 쓰인다면다른 동료들도 컨트롤러에서 회원 SEQ 가지고 와서 넘겨줘야겠죠?반복된 코드들이 생길 겁니다.서비스 계층으로 넘길때, 도메인 객체에 컨버팅을 한 후 계층간 이동은 도메인으로 해보세요.도메인에 빌더 시, 회원 SEQ를 셋팅 하는 로직을 넣어둔다면사용성이 뛰어날거 같네요

rarity · Answer

하나도 안중요합니다상화에 따라서 까세요.

tnalswh · Answer

서로 역할을 아시면 어디에 둘지 판단 될듯요.controller : 클라이언트 요청을 받아서 조건에 따라 servic에 위임하고 결과를 클라인트에 응답 처리service : 비지니스(업무)로직을 처리(DB조회, 다른 업무 service 호출 등)

코드작성법에 대해 질문드립니다

10개의 답변

Q&A 목록