Skip to content
This repository was archived by the owner on Oct 7, 2024. It is now read-only.

Conversation

@highright1234
Copy link
Contributor

저만 PlayerUseUnknownEntityEvent 몰랐었나요..?

TMI:

  • attack했을때 같은경우 기존 엔티티의 경우 ItemEntity, ExperienceOrb, Arrow의
    공격당하는 패킷이 들어올시
    킥당하지만 굳이 추가할 필요를 느끼지못하여 추가하지 않음

@monun
Copy link
Owner

monun commented Apr 13, 2023

헐 이거 뭐에요

@ghost
Copy link

ghost commented Apr 13, 2023

PlayerUseUnknownEntityEvent
서버에서 클라가 보낸 entityId에 대한 엔티티 정보가 없을 시에 호출됨
-> 이거 FakeServer에 대입하고 Fake 관련 이벤트 제작 가능
== 구현 잘만하면 지금 서버 기획하신거 코드도 줄고 여러 컨텐츠 개발 방면에서 유용히 쓰일듯합니다
== 제발 머지좀

@highright1234
Copy link
Contributor Author

아 생각해보니 살짝 취약한점 있네요

@highright1234
Copy link
Contributor Author

highright1234 commented Apr 13, 2023

image
TMI부분은 제 언어능력 상실로인해 이미지도 올립니다 ㅎㅎ;;;
기능 다시보니 TMI부분 기능 추가해야할것도 있는것같아 수정중
(다시 확인해보니 굳이 할필요는 없었던것같기도 하고)

@monun
Copy link
Owner

monun commented Apr 13, 2023

이벤트를 발생시키는것보다 consumer 방식 구현이 좋지 않을까요?

@highright1234
Copy link
Contributor Author

highright1234 commented Apr 13, 2023

이벤트를 발생시키는것보다 consumer 방식 구현이 좋지 않을까요?

확실히 그래도 좋긴하겠네유
기존 버킷 시스템따라서 1차적으로 이벤트로 한거긴 해유

@ghost
Copy link

ghost commented Apr 15, 2023

Interact, Damage 관련 두가지 추가하고 이외에 더 추가할게 있으려나요

@highright1234
Copy link
Contributor Author

highright1234 commented Apr 15, 2023

Interact, Damage 관련 두가지 추가하고 이외에 더 추가할게 있으려나요

isAttack도 추가해놨습니다
몇 데미지인지 확인하는것 관련도 일단 할수있는지 봐보겠습니다 ( 이부분은 nms 써야할것같기도 하고 )

@highright1234
Copy link
Contributor Author

highright1234 commented Apr 22, 2023

필요가 없었을지는 모르겠지만
DamageCalculator 추가로 isAttack에서 DamageCaculator 연계로
더 확장성 있게 쓸수있게 해봤습니다

@ghost
Copy link

ghost commented Jun 12, 2023

@highright1234
TODO:

  • Event -> Consumer 구현 방식으로 변경?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants