source commit
All checks were successful
kiz-shop-pipeline/pipeline/head This commit looks good

This commit is contained in:
2023-12-26 17:50:18 +09:00
parent 12a6043205
commit 7e8d344888
8 changed files with 61 additions and 37 deletions

View File

@@ -4,7 +4,7 @@ import org.springframework.stereotype.Component;
import com.kiz.application.member.port.output.CreateMemberOutput; import com.kiz.application.member.port.output.CreateMemberOutput;
import com.kiz.domain.member.model.Member; import com.kiz.domain.member.model.Member;
import com.kiz.domain.member.repository.MemberRepository; import com.kiz.domain.member.repository.MemberCommand;
import lombok.RequiredArgsConstructor; import lombok.RequiredArgsConstructor;
@@ -12,11 +12,11 @@ import lombok.RequiredArgsConstructor;
@RequiredArgsConstructor @RequiredArgsConstructor
public class CreateMemberPersistence implements CreateMemberOutput { public class CreateMemberPersistence implements CreateMemberOutput {
private final MemberRepository memberRepository; private final MemberCommand memberCommand;
@Override @Override
public void createMember(final Member member) { public void createMember(final Member member) {
memberRepository.createMember(member); memberCommand.createMember(member);
} }
} }

View File

@@ -6,7 +6,7 @@ import org.springframework.stereotype.Component;
import com.kiz.application.member.port.output.FindMemberOutput; import com.kiz.application.member.port.output.FindMemberOutput;
import com.kiz.domain.member.model.Member; import com.kiz.domain.member.model.Member;
import com.kiz.domain.member.repository.MemberRepository; import com.kiz.domain.member.repository.MemberQuery;
import com.kiz.domain.member.vo.MemberId; import com.kiz.domain.member.vo.MemberId;
import lombok.RequiredArgsConstructor; import lombok.RequiredArgsConstructor;
@@ -15,11 +15,11 @@ import lombok.RequiredArgsConstructor;
@RequiredArgsConstructor @RequiredArgsConstructor
public class FindMemberPersistence implements FindMemberOutput { public class FindMemberPersistence implements FindMemberOutput {
private final MemberRepository memberRepository; private final MemberQuery memberQuery;
@Override @Override
public Optional<Member> getMemberById(String memberId) { public Optional<Member> getMemberById(String memberId) {
return memberRepository.getMemberById(MemberId.from(memberId)); return memberQuery.getMemberById(MemberId.from(memberId));
} }
} }

View File

@@ -0,0 +1,32 @@
package com.kiz.adapter.member.persistence.repository;
import org.springframework.stereotype.Component;
import com.kiz.adapter.member.persistence.jpa.mapper.MemberMapper;
import com.kiz.adapter.member.persistence.jpa.repository.MemberJpaRepository;
import com.kiz.domain.member.model.Member;
import com.kiz.domain.member.repository.MemberCommand;
import com.kiz.domain.member.vo.MemberPassword;
import lombok.RequiredArgsConstructor;
@Component
@RequiredArgsConstructor
public class MemberCommandRepository implements MemberCommand {
private final MemberJpaRepository repository;
@Override
public void createMember(Member member) {
repository.save(MemberMapper.toEntity(member));
}
@Override
public void updateMember(Member member) {
}
@Override
public void changeMemberPassword(MemberPassword memberPassword) {
}
}

View File

@@ -1,4 +1,4 @@
package com.kiz.adapter.member.persistence; package com.kiz.adapter.member.persistence.repository;
import java.util.List; import java.util.List;
import java.util.Optional; import java.util.Optional;
@@ -8,18 +8,17 @@ import org.springframework.stereotype.Component;
import com.kiz.adapter.member.persistence.jpa.mapper.MemberMapper; import com.kiz.adapter.member.persistence.jpa.mapper.MemberMapper;
import com.kiz.adapter.member.persistence.jpa.repository.MemberJpaRepository; import com.kiz.adapter.member.persistence.jpa.repository.MemberJpaRepository;
import com.kiz.domain.member.model.Member; import com.kiz.domain.member.model.Member;
import com.kiz.domain.member.repository.MemberRepository; import com.kiz.domain.member.repository.MemberQuery;
import com.kiz.domain.member.vo.MemberId; import com.kiz.domain.member.vo.MemberId;
import com.kiz.domain.member.vo.MemberPassword;
import lombok.RequiredArgsConstructor; import lombok.RequiredArgsConstructor;
@Component @Component
@RequiredArgsConstructor @RequiredArgsConstructor
public class MemberRepositoryImpl implements MemberRepository { public class MemberQueryRepository implements MemberQuery {
private final MemberJpaRepository repository; private final MemberJpaRepository repository;
@Override @Override
public Optional<Member> getMemberById(MemberId memberId) { public Optional<Member> getMemberById(MemberId memberId) {
return Optional.ofNullable(MemberMapper.toDomain(repository.findByMemberId(memberId.value()))); return Optional.ofNullable(MemberMapper.toDomain(repository.findByMemberId(memberId.value())));
@@ -29,18 +28,5 @@ public class MemberRepositoryImpl implements MemberRepository {
public List<Member> getMembers() { public List<Member> getMembers() {
return null; return null;
} }
@Override
public void createMember(Member member) {
repository.save(MemberMapper.toEntity(member));
}
@Override
public void updateMember(Member member) {
}
@Override
public void changeMemberPassword(MemberPassword memberPassword) {
}
} }

View File

@@ -5,8 +5,8 @@ import org.springframework.web.bind.annotation.RequestBody;
import org.springframework.web.bind.annotation.RequestMapping; import org.springframework.web.bind.annotation.RequestMapping;
import org.springframework.web.bind.annotation.RestController; import org.springframework.web.bind.annotation.RestController;
import com.kiz.application.member.port.usecase.CreateMemberUseCase;
import com.kiz.application.member.port.usecase.CreateMemberUseCase.CreateMemberCommand; import com.kiz.application.member.port.usecase.CreateMemberUseCase.CreateMemberCommand;
import com.kiz.application.member.service.CreateMemberService;
import jakarta.validation.Valid; import jakarta.validation.Valid;
import jakarta.validation.constraints.NotBlank; import jakarta.validation.constraints.NotBlank;
@@ -17,11 +17,11 @@ import lombok.RequiredArgsConstructor;
@RequiredArgsConstructor @RequiredArgsConstructor
public class CreateMemberApi { public class CreateMemberApi {
private final CreateMemberService createMemberService; private final CreateMemberUseCase createMemberUseCase;
@PostMapping @PostMapping
public void createMember(@RequestBody @Valid final CreateMemberRequest request) { public void createMember(@RequestBody @Valid final CreateMemberRequest request) {
createMemberService.createMember(request.toDto()); createMemberUseCase.createMember(request.toDto());
} }
record CreateMemberRequest( record CreateMemberRequest(

View File

@@ -6,8 +6,8 @@ import org.springframework.web.bind.annotation.PathVariable;
import org.springframework.web.bind.annotation.RequestMapping; import org.springframework.web.bind.annotation.RequestMapping;
import org.springframework.web.bind.annotation.RestController; import org.springframework.web.bind.annotation.RestController;
import com.kiz.application.member.port.usecase.FindMemberUseCase;
import com.kiz.application.member.port.usecase.FindMemberUseCase.MemberInfo; import com.kiz.application.member.port.usecase.FindMemberUseCase.MemberInfo;
import com.kiz.application.member.service.FindMemberService;
import lombok.RequiredArgsConstructor; import lombok.RequiredArgsConstructor;
@@ -16,11 +16,11 @@ import lombok.RequiredArgsConstructor;
@RequiredArgsConstructor @RequiredArgsConstructor
public class FindMemberApi { public class FindMemberApi {
private final FindMemberService findMemberService; private final FindMemberUseCase findMemberUseCase;
@GetMapping("/{memberId}") @GetMapping("/{memberId}")
public ResponseEntity<FindMemberResponse> getMember(@PathVariable(name = "memberId") final String memberId) { public ResponseEntity<FindMemberResponse> getMember(@PathVariable(name = "memberId") final String memberId) {
return ResponseEntity.ok(FindMemberResponse.from(findMemberService.getMemberById(memberId))); return ResponseEntity.ok(FindMemberResponse.from(findMemberUseCase.getMemberById(memberId)));
} }
record FindMemberResponse( record FindMemberResponse(

View File

@@ -1,15 +1,9 @@
package com.kiz.domain.member.repository; package com.kiz.domain.member.repository;
import java.util.List;
import java.util.Optional;
import com.kiz.domain.member.model.Member; import com.kiz.domain.member.model.Member;
import com.kiz.domain.member.vo.MemberId;
import com.kiz.domain.member.vo.MemberPassword; import com.kiz.domain.member.vo.MemberPassword;
public interface MemberRepository { public interface MemberCommand {
Optional<Member> getMemberById(MemberId memberId);
List<Member> getMembers();
void createMember(Member member); void createMember(Member member);
void updateMember(Member member); void updateMember(Member member);
void changeMemberPassword(MemberPassword memberPassword); void changeMemberPassword(MemberPassword memberPassword);

View File

@@ -0,0 +1,12 @@
package com.kiz.domain.member.repository;
import java.util.List;
import java.util.Optional;
import com.kiz.domain.member.model.Member;
import com.kiz.domain.member.vo.MemberId;
public interface MemberQuery {
Optional<Member> getMemberById(MemberId memberId);
List<Member> getMembers();
}