여러명 의 회원정보를 업데이트 할때의 로직 을 만들었는데 잘못되었을까요?

Question

다음 코드는 배열로 여러명의 회원정보를 받아서 업데이트를 하는 경우의 로직입니다.DTO 는 배열 형태isNext 메서드를 통해서 ValiDation 체크 업데이트 카운트 와 실제로 넘어온 데이터의 길이가 다를 경우 RollBack이러한 질문을 하는 이유는 해당 로직이 잘못된 로직인지 판단해줄수 있는 분이 없어서 입니다..&#64;Transactional(rollbackFor &#61; Exception.class) &#64;Override public int updateMember(MemberDTO[] memberDto) throws SQLException { int applyCount &#61; 0; // applyCount 변수를 초기화하고 지역 변수로 선언 for(int i &#61; 0; i < memberDto.length; i&#43;&#43;) { // 회원 정보 String name &#61; memberDto[i].getSsNm(); String phoneNumber &#61; memberDto[i].getPhoneNumber(); // isNext 메서드가 회원 정보를 기반으로 다음 처리를 할지 결정 boolean next &#61; isNext(name, phoneNumber); if(next) { // updateMember 메서드 호출 후 applyCount 증가 applyCount &#43;&#61; memberMapper.updateMember(name,phoneNumber); } } // memberDto의 모든 요소가 성공적으로 처리되었는지 확인 if(applyCount &#61;&#61; memberDto.length) { return 1; // 성공 시 1 반환 } else { throw new SQLException(&#34;Failed to update all members&#34;); // 실패 시 SQLException 던지기 } }

동도인 · Answer

계정이 두개로 인식 해서 수정을 할수 없는데...일단 테스트 해본 결과 원하는 결과대로 작동은 하긴 합니다..

Clickin · Answer

이정도면 거의 수도코드 수준이라서 로직이 잘못되고 아니고가 있나요...?배열 길이와 applyCount를 비교하는게 맞을까요라는 질문이라면굳이 updateMember 실행하지 않고서도 이걸로 판단할 수 있겠죠하나라도 isNext를 통과하지 못하면 당연히 길이가 다를테니까요// java에서는 보통 array, list는 ~~s로 네이밍합니다. 올바르지 않은 명명규칙입니다. for (MemberDTO member: memberDTO) { if (!isNext(member.getSsNm(), member.getPhoneNumber())) { throw new SQLException(&#34;Failed to update all members&#34;); } }목적이 확실하셔야하는데 All or Nothing 전략을 취하겠다면 트랜잭션으로 처리해야 하는 부분이고사용자에게 N개중 M개는 유효성 통과하지 못해서 재작성하라는 의미라면 isNext만 봐도 될겁니다.PK 중복 등 유효성 검사에서 미처 못 거른걸 알려주고 싶다면 또 다른 방식을 취해야겠죠

캐티 · Answer

어떻게 하여도 자유이어서 뭐라고 못하지 않겠나요.편한 처리 방식이 있다면 INSERT ON DUPLICATE 방식으로 중복된 키값에 대해서 일괄적 업데이트를 치는 방법이 있고, JSON 객체를 전달할 수있다면 UPDATE JOIN 방식으로 일괄적 업데이트를 치는 방법이 있어요.검증은 was단에서 처리하여야 할 것이고요. 이렇게 하면 검증부분만 루프를 돌고 검증이 완료된 데이터는 통째로 db에 넘겨줘요.로직의 간소화와 혹여나 중요한 부분에 롤백을 걸 수 있어서 말이어요 ..

감독님테스트가하고싶어요 · Answer

// memberDto의 모든 요소가 성공적으로 처리되었는지 확인 if(applyCount == memberDto.length) { return 1; // 성공 시 1 반환 } else { throw new SQLException("Failed to update all members"); // 실패 시 SQLException 던지기 }굳이 마지막에 가서 모든 요소가 성공적으로 처리됐는지 확인할 필요가 있을까요?early return 시키는게 더 좋아보입니다.

wharup · Answer

요청만큼 업데이트해야 성공이면isNext()가 false이면 바로 throwmapper.updateMember()가 0이면 바로 throw하는게 심플해보입니다.

여러명 의 회원정보를 업데이트 할때의 로직 을 만들었는데 잘못되었을까요?

5개의 답변

Q&A 목록