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

[BE] topfive 코드 리팩터링 #41

Merged
merged 3 commits into from
Nov 7, 2024
Merged

[BE] topfive 코드 리팩터링 #41

merged 3 commits into from
Nov 7, 2024

Conversation

uuuo3o
Copy link
Collaborator

@uuuo3o uuuo3o commented Nov 7, 2024

✅ 주요 작업

  • 환경변수 가지고 오는 방식 통일
  • 공통 함수 분리
  • 주석 추가

💭 고민과 해결과정

  • 전반적으로 통일감을 주기 위해 환경변수를 가지고 오는 방식을 시은님과 동일하게 변경했습니다. 사실 어제 잘 되지 않았던 부분인데 아마 Service class 밖에서 가지고 온 다음에 적용하려고 해서 에러가 났던 것 같습니다. class 내부에서 바로 값을 가져오니 정상 동작합니다.
  • 시은님과 제 코드를 비교하며 공통 함수로 분리할 수 있는 부분을 찾아 공통함수로 분리했습니다. 다음 API를 호출할 때에도 동일하게 사용될 부분이라 편리하게 사용할 수 있을 것 같습니다.
  • 외부 API 호출만 해서 주요 로직이 서비스 계층에 모두 모여있습니다. 코드 읽으실 때 이해를 돕기 위해 JSDocs 주석 방식을 사용해 주석을 추가했습니다.

@uuuo3o uuuo3o requested review from jinddings and sieunie November 7, 2024 09:02
@uuuo3o uuuo3o self-assigned this Nov 7, 2024
return {
'content-type': 'application/json; charset=utf-8',
authorization: `Bearer ${accessToken}`,
appkey: process.env.KOREA_INVESTMENT_APP_KEY,
Copy link
Collaborator

Choose a reason for hiding this comment

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

🟢 해결하셨군뇨 좋습니당ㅎㅎ

@@ -0,0 +1,10 @@
export const getHeader = (accessToken: string, trId: string) => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

🟢 이따 머지하시면 저도 getHeader, getFullURL 사용하도록 변경해보겠습니다~

@uuuo3o uuuo3o merged commit 98b875c into back/main Nov 7, 2024
2 checks passed
@uuuo3o uuuo3o added BE 백엔드 REFACTORING 코드 리팩터링 labels Nov 8, 2024
@uuuo3o uuuo3o deleted the refactor/api/topfive branch November 15, 2024 07:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE 백엔드 REFACTORING 코드 리팩터링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants