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

글로벌 서버 무작위 임무 지원 #45

Merged
merged 9 commits into from
Jul 8, 2017
Merged

글로벌 서버 무작위 임무 지원 #45

merged 9 commits into from
Jul 8, 2017

Conversation

Jaehyuk-Lee
Copy link
Contributor

글로벌 서버에서 무작위 임무 신청시 나오는 opcode가 0x00B0 으로 확인되었습니다.
임무 정보는 data[4]에 나타나며, 3.x에서 사용하던 id를 그대로 사용할 수 있습니다. (xml파일에 적용된 id를 말함)
다만, 글로벌 서버에서, 무작위 50레벨 던전이 아니라 무작위 50/60 던전으로 바뀌었는데, 이 부분은 아직 납뒀습니다. (Xml파일에는 한국 서버를 기준으로 하여, 50던전으로 저장되어 있기 때문에 50던전으로 출력되고 있습니다.)
(대충 Xml파일의 Roulette부분을 한국 서버 전용으로 하고 RouletteGS를 새로 만들어서 글로벌 서버용 데이터를 넣은 후에, 글로벌 서버 무작위 던전을 신청했을 때는 RouletteGS를 참고하는 식으로 바꾸면 될 것 같습니다.)
특별한 문제가 있는건 아니지만, 아직 소스 이해도가 부족해 고치지 못했습니다.

  • opcode == 0x00B0이 한국 서버에서 어떻게 쓰이고 있는지 확인하지 못했습니다. 다만, 5분간 이것저것 해본 결과 충돌이 감지되지 않았습니다. 앞으로도 프로그램 사용하며 문제 발생하면 글로벌 서버 선택 또는 감지 옵션을 추가하도록 하겠습니다.
  • 사소한 던전 명칭 개선이 있습니다.

글로벌 서버에서 무작위 임무 신청시 나오는 opcode가 0x00B0 으로 확인되었습니다.
임무 정보는 data[4]에 나타나며, 3.x에서 사용하던 id를 그대로 사용할 수 있습니다. (xml파일에 적용된 id를 말함)
다만, 무작위 50레벨 던전이 아니라, 무작위 50/60 던전으로 바뀌었는데, 이 부분은 아직 납뒀습니다. (Xml파일에는 한국 서버를 기준으로 하여, 50던전으로 저장되어 있기 때문에 50던전으로 출력되고 있습니다.)
(대충 Xml파일의 Roulette부분을 한국 서버 전용으로 하고 RouletteGS를 새로 만들어서 글로벌 서버용 데이터를 넣은 후에, 글로벌 서버 무작위 던전을 신청했을 때는 RouletteGS를 참고하는 식으로 바꾸면 될 것 같습니다.)
특별한 문제가 있는건 아니지만, 아직 소스 이해도가 부족해 고치지 못했습니다.
@Jaehyuk-Lee
Copy link
Contributor Author

Jaehyuk-Lee commented Jul 8, 2017

한국 서버에서 임무 입장할 때 opcode = 0x00B0이 출력됩니다. 글로벌 서버 선택 또는 감지 옵션이 필요하네요.

@Jaehyuk-Lee
Copy link
Contributor Author

Jaehyuk-Lee commented Jul 8, 2017

글로벌 서버에서 opcode = 0x00B0일 때와
한국 서버에서 opcode = 0x00B0일 때,
data.length가 다름을 이용하여 글로벌 서버에서 사용하고 있는지, 한국 서버에서 사용하고 있는지 구분하였습니다.

글로벌 서버 무작위 임무 명칭이 한국 서버와 다른 점도 이번 수정으로 해결하였습니다.

@devunt devunt merged commit 6993386 into devunt:master Jul 8, 2017
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