Skip to content

feat: 아키텍처 및 프로세스 개선사항 분석 (ASIS) #7

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 1 commit into
base: rolroralra
Choose a base branch
from
Open

feat: 아키텍처 및 프로세스 개선사항 분석 (ASIS) #7

wants to merge 1 commit into from

Conversation

rolroralra
Copy link

  • 레거시 코드 유지보수성 관련 프로세스
  • 외부 도메인 데이터 동기화 관련 프로세스
  • 개발 진행 시 문서화와 테스트 코드 관련 프로세스

@rolroralra
feat: 아키텍처 및 프로세스 개선사항 분석 (ASIS)
c8d1a84 
- 레거시 코드 유지보수성 관련 프로세스
- 외부 도메인 데이터 동기화 관련 프로세스
- 개발 진행 시 문서화와 테스트 코드 관련 프로세스
titlet
titlet requested changes Dec 10, 2023
View reviewed changes
Copy link
Contributor

@titlet titlet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

크게 변경할 내용은 없습니다.
다만 선택 하기가 쉽지 않은 과제들이라 tobe를 선택할 때 많이 고민 될 듯 하네요.
수치같은것이 있으면 좋을 것 같아서 해당 부분 의견 드립니다.

asis/ARCHITETURE.md
```

## 외부 도메인 데이터 동기화 관련 프로세스
- 외부 도메인 데이터에 대한 데이터 동기화를 수동으로 처리하지 않고 자동화할 수는 없을까?
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수동으로 처리하는 시간이 얼마나 소요되는지 측정할 수 있을까요?

asis/ARCHITETURE.md
## 레거시 코드 유지보수성 관련 프로세스
- 도메인 모델링이 제대로 되어 있지 않아서, 기능을 추가하거나 수정할 때 마다 코드 분석에 많은 시간이 소요된다.
- 기능을 추가하거나 수정할 때 마다, 수많은 if 분기문과 하드 코딩으로 인해 코드가 점점 더 커지고 복잡해진다.
- 코드 리팩터링을 통해 도메인 모델링을 제대로 하고, 기능을 추가하거나 수정할 때 마다 코드 분석에 소요되는 시간을 최소화하고 싶다.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

전체 구성원들의 참여가 필요해 보입니다.
리펙토링 이후에도 지속적으로 clean code를 하는 개발문화 정착도 필요해 보이네요.
리더를 하고 있기에 개인적인 개선과제를 이런 개발 문화 정착을 위한 것으로 잡아봐도 좋을것 같긴한데 과정기간동안에 진행 하기는 어려울 것 같긴 합니다.
개선 과제를 선정할때 이런 부분에 대한 의견도 고민해 주시면 좋을 것 같아요.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@titlet titlet titlet requested changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@rolroralra @titlet