Skip to content
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.

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

API-Layer-&-Fetch-Functions 번역 #11

Merged
merged 1 commit into from
Jun 29, 2024
Merged

API-Layer-&-Fetch-Functions 번역 #11

merged 1 commit into from
Jun 29, 2024

Conversation

lurgi
Copy link
Member

@lurgi lurgi commented Jun 10, 2024

간단한 내용이라 피드백 할게 별로 없어보이네요~
이번에 추상화 관련 내용으로 번역 해봤습니다~

Copy link
Member

@brgndyy brgndyy left a comment

Choose a reason for hiding this comment

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

러기~ 고생 많으셨습니다.

조금이나마 의역을 추가해볼수 있는 부분을 댓글로 남겨보았는데 확인 부탁드려요! 🙇
(반영 안하셔도 전혀 상관 없습니다~!)

- 한편으로는 선택의 자유가 있습니다.
- 다른 한편으로 많은 프로젝트들이 맞춤형이고 종종 지저분한 구조로 끝나곤 합니다.

이 글은 소프트웨어 아키텍처와 리액트 앱에 관한 연재의 두 번째 부분으로, 우리는 나쁜 관행이 많은 코드 베이스를 단계별로 리팩토링해 나갈 것입니다.
Copy link
Member

Choose a reason for hiding this comment

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

나쁜 관행이 많은 코드 는 즉 수정할 부분이 많은 코드라고 느껴지는데, 뭔가 조금 더 직관적인 단어로 대체해볼수 있을까요..!?


## 해결책: API 연결 함수를 추출한다.

API에 연결하는 함수를 별도의 장소로 추출하면 API 관련 코드와 UI 코드의 결합을 크게 줄일 수 있습니다.
Copy link
Member

Choose a reason for hiding this comment

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

별도로 추출하면 이나, 별도의 파일로 라고 해도 괜찮을거 같다고 생각이 듭니다 🏃‍♀️


그러나 사실상, 우리는 UI코드와 API 관련 함수를 효과적으로 분리하기 시작했습니다.

반복해서 말하는 것이 지겹지 않습니다: 이제 컴포넌트는 다음과 같은 많은 API 관련 세부사항을 알지 못합니다.
Copy link
Member

Choose a reason for hiding this comment

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

반복해서 말해도 지나치지 않습니다. 라는 표현은 어떠실까요 ..?!

Copy link
Member Author

Choose a reason for hiding this comment

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

건디 말씀도 일리가 있습니다! 신경쓰면서 의역하는 것 보단 일단 직역한 것으로 두겠습니다!

@lurgi lurgi merged commit 6c30a52 into main Jun 29, 2024
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants