2월 14일 TIL

Refactoring

• 오늘은 DodGe.GG의 자잘한 오류 및 개선을 하기위해 코드를 켰다
• 근데 왜인걸.. 생각보다 알수없고 많은 오류가 많이 확인되었다
  • 그전엔 그러지않았는데..
• 이 오류들을 확인하기 위해 코드를 살펴보았는데..
  • 아무래도 오픈소스 API이다 보니 그전과 요청 url이나 데이터 안에 값이 변경된것들이 확인 되었다

라이엇의 실수..?

• user의 Tier정보 및 리그의 정보를 확인 할 수있는 API Response 안에 queueType이라는 정보가 있다
  • 이정보는 이 리그의 정보가 어떤 리그의 정보인지 담고있는것
    • 예를들면 솔로랭크 게임의 티어인지, 자유랭크 게임의 티어인지를 담고있음
  • 예전에는 RANK_SOLO_5X5 이런식의 값이 String으로 넘어왔었다
• 근데 이것이 문제라는걸 알기 전까지는 왜 넘어오지 않는지 두시간 이상 찾았던것 같다
  • 403은 아니고.. 데이터도 200으로 분명 네트워크 요청이 넘어가고 URL도 틀린게 없는데 왜?인지 모르겠었다
• 다시 코드를 거슬러 올라가보았다
  • 밑 사진은 오류의 정체를 알고 고친후의 코드임
    
• 확인해보니.. 솔로랭크와 자유랭크를 구분하기위해 필터를 하고있었다
  • 근데 사진에서 보이겠지만 솔로랭크의 string값이
  • RANKED_TFT_DOUBLE_UP이라고 되어있다
    • 해당부분은 롤토체스의 더블업 모드의 게임타입을 말하는것
• 잘못된 값이 할당되어 있어서 200 요청은 정상적으로 들어가서 값이 넘어왔지만 filter되는 과정에서 문제가 있었던것
• 이 코드에서 더이상 손대기는 어렵다는 판단에 추후 변경될 예정이라는 주석을 달고 우선 마무리를 지었다..

• 처음겪는일이라 너무 당황스러웠지만 Open API를 사용하는 입장으로써 화를 내거나 할 필요는 없다고 느꼈다
• 사람이기 때문에 실수할 수 있다고 생각하고 추후에 변경되면 한번더 변경 하도록 하겠다!