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

Update version 1.0.5 #15

Closed
tvd12 opened this issue Aug 8, 2021 · 8 comments
Closed

Update version 1.0.5 #15

tvd12 opened this issue Aug 8, 2021 · 8 comments

Comments

@tvd12
Copy link
Member

tvd12 commented Aug 8, 2021

[ x ] 1. Thêm hàm maxPlayer vào MMORoom.Builder
[ x ] 2. Thêm trường master vào MMORoom
[ ] 3. playerManager, roomManager cho MMOVirtualWorld --> globalPlayerManager, globalRoomManager --> Issue #20
[ x ] 4. Sửa lại MMORoomGroup.sleep phải tính khoảng sleep time còn lại thay vì sleep đúng bằng time tick. Vì nếu việc tính toán chậm sẽ làm delay tích tụ dần
[ x ] 5. Bổ sung clientTimeTick vào MMOPlayer
[ ] 6. Xem xét xem có cách nào để làm logic tìm room master của LocatedRoom đơn giản hơn không --> Issue #21
[ x ] 7. Bổ sung các vector đơn vị Vec3.left, Vec3.right, Vec3.up, Vec3.down
[ x ] 8. final private List nearbyPlayerNamesBuffer

@tvd12
Copy link
Member Author

tvd12 commented Aug 8, 2021

final private List nearbyPlayerNamesBuffer

@vu-luong
Copy link
Member

vu-luong commented Aug 8, 2021

final private List nearbyPlayerNamesBuffer

Cái này là sao anh nhỉ?

@tvd12
Copy link
Member Author

tvd12 commented Aug 8, 2021

final private List nearbyPlayerNamesBuffer

Cái này là sao anh nhỉ?

Là do mình quên ko để final, trong file này em ạ: https://github.com/youngmonkeys/game-box/blob/master/src/main/java/com/tvd12/gamebox/handler/SyncPositionRoomUpdatedHandler.java, biến đó ko phép được khởi tạo từ bên ngoài em ạ, nhưng do mình đang để @AllArgsContructor nên nó bị include vào hàm tạo

@vu-luong
Copy link
Member

vu-luong commented Aug 9, 2021

@tvd12 Em thấy phải có 1 cái playerManager cho MMOVirtualWorld. Ví dụ khi user thoát app, dev chỉ cần gọi mmoVirtualWorld.remove(player), nó sẽ tự tìm phòng tương ứng và xóa player khỏi phòng đó

@tvd12
Copy link
Member Author

tvd12 commented Aug 9, 2021

@tvd12 Em thấy phải có 1 cái playerManager cho MMOVirtualWorld. Ví dụ khi user thoát app, dev chỉ cần gọi mmoVirtualWorld.remove(player), nó sẽ tự tìm phòng tương ứng và xóa player khỏi phòng đó

anh thấy hợp lý em ạ

@vu-luong
Copy link
Member

vu-luong commented Aug 9, 2021

@tvd12 Em nghĩ sau khi mình thảo luận chốt feature anh nên edit cái list ở comment đầu tiên, sau xem lại cho dễ đỡ phải lướt comments anh ạ.

@tvd12
Copy link
Member Author

tvd12 commented Aug 9, 2021

@tvd12 Em nghĩ sau khi mình thảo luận chốt feature anh nên edit cái list ở comment đầu tiên, sau xem lại cho dễ đỡ phải lướt comments anh ạ.

ok em, anh vừa update rồi nhé

@vu-luong
Copy link
Member

Mình phải thêm roomManager cho MMOVirtualWorld nữa anh ạ. Em muốn lấy list các room của MMOVirtualWorld mà hiện tại phải dựa vào cái globalRoomManager

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

No branches or pull requests

2 participants