▲
dydwls140 2달전 | parent | ★ favorite | on: Show GN: vscode에서 한국 주식을 확인할 수 있는 vscode extension(github.com/marsboy02)

재밋는 확장이네요. AI 로 만드셨겠지만 코드리뷰가 필요해보입니다. 사용해보다가 몇 가지 눈에 띄는 부분이 있어서 이슈 남겨드릴게요.

  • fetchAndUpdate에서 isMarketOpen() 체크 후 분기가 비어있어서 장 외 시간에도 15초마다 API 호출이 계속 됩니다
  • 종목별 개별 HTTP 요청이라 관심 종목 30개면 첫 로드에 최대 90건 요청이 발생합니다
  • tradingValue = volume * price는 실제 거래대금과 차이가 큽니다
  • symbolCache에 TTL/eviction이 없습니다

참고가 되면 좋겠습니다.

주니어를 벗어나려면 자신이 작성한 코드에 대한 이해는 필수입니다.

▲
marsboy02 2달전  [-]

리뷰 감사합니다. 주식 시장에 대한 컨텍스트 외에도 코드 품질을 많이 신경을 써야겠네요.

급하게 네이버의 API가 아니라, yahoo쪽 API로 변경하는 과정에서 품질을 많이 무시했던 것 같습니다.
참고해서 작업해보겠습니다. 추가로 불편하신 점이나 개선 사항이 있으시다면 언제든 피드백 부탁드립니다.

답변달기