[BE] 5.03 주식 리스트 가져오기 API 구현 #93
Merged
Conversation
This file contains 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
…ampwm-2024/web16-JuGa into feature/api/stockList-#57
uuuo3o
approved these changes
Nov 12, 2024
| @@ -0,0 +1,46 @@ | |||
| import FinanceDataReader as fdr | |||
There was a problem hiding this comment.
🟢 요 라이브러리는 파이썬에만 있는건가요?! 갑자기 파이썬 코드 보니 뭔가 신기하네요 ㅎㅎ
There was a problem hiding this comment.
파이썬에 있는 라이브러리더라구요! 아직 검색이 부족했는지 잘 모르겠지만.. nestjs에선 못찾겠네요
| }); | ||
| } | ||
| if (params.code) { | ||
| queryBuilder.andWhere('code LIKE :code', { code: `%${params.code}%` }); |
There was a problem hiding this comment.
🟢 이거 %{params.code}% 조건을 이렇게 하는 것보다 {params.code}% 이게 더 구현 의도와 맞지 않을까요?
지금 이렇게 조건이 되어있으면
'삼성' 을 쳤을 때
ㅇㅇ삼성 삼성ㅇㅇ 이렇게 삼성이 포함된 모든 단어가 다 출력되니까 어색하게 느껴져요.
삼성ㅇㅇ 처럼 입력한 단어로 시작하는 단어만 출력되는게 더 자연스러울 것 같아요!
There was a problem hiding this comment.
실제로 토스에서 확인 해보았을땐 삼성OO , OO삼성 둘다 나오는 걸로 봐서 이렇게 작성했긴 한데 한번 생각해보겠습니다!
There was a problem hiding this comment.
헐 토스에서 저렇게 하는군요!! 말씀 보자마자 검색하고 왔는데 그렇네요!! 그대로 둬도 괜찮을 거 같아요
sieunie
approved these changes
Nov 12, 2024
| return new StockListResponseDto(stock.code, stock.name, stock.market); | ||
| } | ||
|
|
||
| async findAll() { |
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
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.
✅ 주요 작업
💭 고민과 해결과정
한국 투자 증권 open API에서 전체 주식 목록을 제공하는 API가 따로 존재하지 않다는 것을 알게 되었습니다.
이에 대안으로 공시 정보를 제공하는 DART의 OpenAPI의 사용을 시도 했지만 저희가 원하는 데이터베이스를 구성하기에 부족한 정보를 제공함에 따라 KRX, Naver 증권에서 데이터를 크롤링 해오는 python 라이브러리인 financeDataReader를 알게 되었습니다.
각 회사 정보는 하루에 한번 꼴로 데이터를 가져오면 되기 때문에 API를 구성할 필요성이 적고 해당 패키지를 이용한 python 스크립트를 작성하고 crontab을 이용해 하루에 한번 혹은 여러 번 갱신 할 수 있기 때문에 해당 라이브러리를 사용하는 것도 좋은 대안이 되어 줄 수 있을 것 같습니다.