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

4.0 던전 정보 추가 #44

Merged
merged 7 commits into from
Jul 7, 2017
Merged

4.0 던전 정보 추가 #44

merged 7 commits into from
Jul 7, 2017

Conversation

Jaehyuk-Lee
Copy link
Contributor

@Jaehyuk-Lee Jaehyuk-Lee commented Jul 3, 2017

글로벌 서버 신규 던전에 맞춰 던전 정보를 추가했습니다.
DFAssist를 켜놓고 신규 던전에 참가 신청을 하면, 던전 ID가 DFAssist에 표기된다는 점을 이용해서 추가했습니다.

한국서버와 충돌은 없었습니다.

GitHub를 처음써서 잘 모르니.. GitHub에 올릴 때는 어떻게 해주면 좋겠다는 코멘트 남겨주시면 감사하겠습니다.

17.07.04 - 오메가 레이드 델타편 던전 정보를 추가했습니다.

글로벌 서버 신규 던전에 맞춰 던전 정보를 추가했습니다.
DFAssist를 켜놓고 신규 던전에 참가 신청을 하면, 던전 ID가 DFAssist에 표기된다는 점을 이용해서 추가했습니다.
@Jaehyuk-Lee
Copy link
Contributor Author

이 프로그램이 글로벌 서버 클라이언트 사용시, 무작위 임무 처리를 못하는걸 왜 그런지 알아봤는데
무작위 임무 신청과 특정 던전 선택 신청을 구분하던 부분이 4.0부터는 동일하게 나옵니다.

3.x에서는 Network.Analyser.cs 에서 opcode == 0x0076 확인으로 무작위 임무를 확인하고 data[192] 부분을 확인해서 어떤 임무인지 확인하던데

4.0에서 분석해보니 무작위 임무 매칭하면, opcode가 0x0079이 나옵니다. 특정 던전을 선택해서 신청할 때도 이렇게 나오죠. 그래서 구분을 못해주고 있습니다. 그리고 data[192]는 아예 존재가지 않습니다.
opcode == 0x0079를 무작위 임무로 인식하라고 한 뒤 data[192]를 받아와 보라고 했는데, 아예 없습니다. data.length가 수치는 기억이 안나는데 특정 던전을 신청했을 때와 동일했습니다.

이 부분을 고치려면 글로벌 서버 선택 옵션이 필요할 것 같아 일단 한국 서버를 최우선으로 지원한다는 목적으로 아예 글로벌 서버 무작위 임무는 지원하지 않는 것이 좋을 것 같습니다. 제가 더 깊게 알아보지 않은 것도 있지만요...

앞서 업데이트한 던전 정보 추가는 한국 서버 이용에 아무런 영향을 끼치지 않으니 반영해주셨으면 좋겠다 싶어 올렸습니다.

@devunt devunt merged commit 73d3f3d into devunt:master Jul 7, 2017
@devunt
Copy link
Owner

devunt commented Jul 7, 2017

안녕하세요, 먼저 기여에 감사드립니다.

보내주신 메일도 잘 받아보았으나 주중에는 바빠 답장을 못 드려 죄송합니다.
xml 데이터에 3.5 및 4.0 컨텐츠 추가하신 부분은 문제 없는 것 같습니다. 감사합니다.

글로벌 서버 지원은 한참 전부터 생각하던건데 제가 글섭을 안 하는 관계로 우선순위가 밀리고 있던 작업입니다.
아마 글로벌 서버 옵션이나 혹은 클라이언트 프로세스명 등으로 한섭과 글섭을 구분할 수 있지 않을까 생각합니다.

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