스타벅스 App 을 클론한 프로젝트 입니다.
MSA 아키텍처 컨셉을 차용하여 하나의 애플리케이션에 각각의 Service 는 독립적으로 구현되어 있으며 비즈니스 로직은 WebClient 를 통해서 소통하고 있습니다.
`logCardRepository.updateAmount(cardId, balance);` 로직에서 `Cucurrency` 문제가 발생할 가능성이 높습니다.
낙관론적락 vs 비낙관론적락을 적용하여 DB에서 Lock 적용 혹은 Version을 검토해 주세요.
JPA Framework에서 락을 지원하는 방법으로 컨셉을 이해하셔도 좋겠습니다.
Originally posted by @flab-true in #19 (comment)
webClient는 `non-blocking I/O`를 지원합니다. 그렇다면 `비동기`와 `non-blocking` 개념을 설명해 주실 수 있을까요?
Originally posted by @flab-true in #20 (comment)
Service Layer에서 유효한 `Member`만 리턴하시고, 값이 없는 경우는 `예외`를 적용해 주세요.
Controller 로직이 비대해지는 것 같습니다.
Member memberInfo = memberService.login(id, password);유효한 Member를 Controller argument로 받는 방법이 있습니다. spring argument resolver 란 무엇이고, 동작 방식을 남겨주세요. 향후 argument resolver로 개선되면 좋겠습니다.
Originally posted by @flab-true in #1 (comment)
1. HTTP Request `/mycard`를 호출한 결과값과, doPay 메서드에서 `confirmMyCard` 호출한 결과 값이 다릅니다.
왜 그런지 설명을 바랍니다.
2. 비지니스 로직을 재사용하려면 Controller Layer의 confirmMyCard 호출 보다는 @Service의 로직을 재사용하는 방법으로 개선해 주세요.
3. ArgumentResolve 이용하여 파라미터를 조금 더 유연하게 가져가는 방향은 어떠세요?
public PayResponse doPay(@RequestBody PayRequest payRequest, Card cardInfo ) {
return payService.runPay(payRequest, cardInfo);
}Originally posted by @flab-true in #19 (comment)
회원 가입 시 2명이 동시에 동일한 ID에 대한 중복된 아이디를 체크한다고 가정합시다.
이후 어떤 문제점이 발생 될 수 있을까요? 이를 해결할 방법은 있을까요?
Originally posted by @flab-true in #6 (comment)
memberService.findById 에서 유효한 유저가 없는 경우에 대한 적절한 오류 메시지 및 상태 코드 반환이 안되고 있습니다.
리턴 타입을 Member로 명시해주세요.
Originally posted by @flab-true in #1 (comment)
시퀀스 다이어그램 PR로 올렸습니다 ~ !
MessageDigest를 사용하여 암호화 하고 있는데요. `공객 키` 방식에서 `MessageDigest`는 어떤 목적으로 사용되고 있을까요?
작성 로직은 동일한 텍스트에 대하여 항상 값은 해시 값을 만들고 있을까요? 그렇다면 무차별 대입 공격에 안전할까요? 어떤 대안이 있을까요? 조사해 주세요.
암호학에 대한 부분인데요. 공객키 방식과 해싱에 대한 공부는 프로젝트 이후라도 반드시 숙지 하시길 바랍니다.
Originally posted by @flab-true in #1 (comment)
`SHA256Util.encryptSHA256(beforePw);` 로직들은 도메인 로직이라고 생각됩니다.
응용 서비스에 도메인 로직들이 포함되어 코드 품질이 좋지 않다고 생각합니다.
1. 리포지터리에서 member 정보 조회, member = findById(....)
2. member에 암호를 변경하기 전에 기존 암호가 올바른지 확인하는 것은 도메인 로직으로 이관
3. 예를 들어 비밀번호 체크 member..matches(beforePw), 비밀 번호를 설정 member.setPassword(afterPw) 등
4. 이후 리포지터리 save
5. findByIdAndPassword -> 도메인 클래스 로직 수행 -> updatePassword 이런 방식을 사용하셔도 됩니다.
Originally posted by @flab-true in #1 (comment)
`LocalDateTime`은 `Timezone`을 사용하는 `타입`일까요?
저장되는 시간의 기준은 무엇을 고려하셨나요?
Originally posted by @flab-true in #6 (comment)
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
An Open Source Machine Learning Framework for Everyone
Django
The Web framework for perfectionists with deadlines.
Laravel
A PHP framework for web artisans
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
Some thing interesting about web. New door for the world.
A server is a program made to process requests and deliver data to clients.
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
Some thing interesting about visualization, use data art
Some thing interesting about game, make everyone happy.
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
Microsoft
Open source projects and samples from Microsoft.
Google
Google ❤️ Open Source for everyone.
Alibaba
Alibaba Open Source for everyone
D3
Data-Driven Documents codes.
Tencent
China tencent open source team.