개선 시스템 아키텍처 작성 #6
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
boorownie
requested changes
Dec 9, 2023
|
성현님 안녕하세요, 팀이나 회사 측면에서 위 사항은 현실적으로 어렵다 하여 주제 다시 결정해서 PR 진행하도록 하겠습니다. |
boorownie
requested changes
Dec 13, 2023
| ### 개선포인트 분석 | ||
| - 취소 기능에 대한 레거시 코드에서 계속해서 취소 기능에 대한 개발이 이루어져 현재 유지보수가 불가한 상태 | ||
| - 이 상태에서도 추가적인 개발이 일어나면 취소 기능은 어쩔 수 없이 기능 개발이 일어나고 있는 상태 | ||
| - 그로인해서 취소 기능과 관련된 여러 장애가 발생해도 걷잡을 수 없는 상태 |
There was a problem hiding this comment.
레거시 코드가 원흉일 순 있지만 어떠한 이유로 어떤 장애가 발생하는지 설명을 해주시면 이해하는데 조금은 도움이 될 것 같습니다.
| - 결제 취소 기능 마이그레이션 | ||
|
|
||
| ### 기대효과 분석 | ||
| - JSP와 JDBC Template을 이용한 코드에서 스프링 부트와 MyBatis, JPA 적용 |
There was a problem hiding this comment.
새로운 기술의 도입이 모든 문제를 해결할 수 있는 것이 아니기 때문에 문제 상황에 대한 공유가 더 필요해 보입니다!
Comment on lines
+9
to
+10
| - 메서드 하나에 존재하는 수백 라인의 코드 단축으로 인한 가시성 향상 | ||
| - 취소 기능에 대한 유지 보수성 증가 |
There was a problem hiding this comment.
가독성 이슈로 휴먼 폴트가 발생하는 것으로 유추할 수 있는데 맞나요?
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
안녕하세요! 개선 시스템 아키텍처 작성하여 PR 진행합니다.
실무 개선 관련하여 굉장히 많은 것을 하고 싶지만 팀, 회사, 개인의 실력 측면에서 여러가지로 봤을 때 현실적으로 골라봤습니다. 주제에 안맞거나 변경해야 할 경우 말씀 해주시면 수정하도록 하겠습니다.