Skip to content

Conversation

@BuDum-Tss
Copy link
Collaborator

No description provided.

@BuDum-Tss BuDum-Tss requested a review from Foxtezy March 28, 2023 09:09
@Foxtezy Foxtezy changed the base branch from develop to nik_connecting March 28, 2023 10:52
true,id, RandomSystem.random.nextInt());
Entity weapon = Factory.spawnWeapon(null,position,wid,player);

MCClient.getClientState().getIdHashTable().put(id,player);
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ты пока без спавна сделала?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

пока да

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ну тоесть она есть - закомментирована но работает или нет - хз

@BuDum-Tss BuDum-Tss requested a review from Foxtezy March 28, 2023 11:55
@BuDum-Tss
Copy link
Collaborator Author

я тут отдебажила throw/take(не вылетают exeptions) но они не работают

@BuDum-Tss
Copy link
Collaborator Author

и смену оружия

@BuDum-Tss
Copy link
Collaborator Author

смена оружия работает

@BuDum-Tss
Copy link
Collaborator Author

но вот с take/throw вопросы

@BuDum-Tss
Copy link
Collaborator Author

BuDum-Tss commented Apr 1, 2023

Короче вроде добавила более менее нормальную систему классов JSON-ов, в соответствии с этим подправила мультиплеер - по идее исправила костыль нахождением заспавненого объекта по координатам. Но теперь для каждого Action нужно генерить ID(String) - выдает создатель вместе со своим ID в формате : "1:123234" 1 игрок, 123234 действие.
+ нужно каким то образом раздать ID каждому клиенту. В это уже не лезла - просто оставила в GameAction статик функцию как заглушку.

@BuDum-Tss
Copy link
Collaborator Author

Текущая структура примерно такая: GameStatePacket состоит из update данных(List) и из ActionPacket в котором хранятся все листы с каждым Action(в последствии - пока только Spawn и немного TakeWeapon).
есть вспомогательный класс ActionQueues который по идее должен принимать в себя новые Actions и передавать их в ActionPacket

@BuDum-Tss
Copy link
Collaborator Author

Еще у каждого Action есть метод do() который автоматически все данные использует и спавнит(и добавляет Entity в мапу)

@BuDum-Tss
Copy link
Collaborator Author

Села писать удаление объектов и поняла что у нас есть проблема: в данный момент мы на сервере просто подтверждаем данные - и все. Правильнее хранить ActionPacket с активными действиями и уже их обновлять инфой из пакетов и передавать.
Иначе у нас данные из пакетов будут теряться при большем кол-ве игроков

BuDum-Tss and others added 30 commits April 11, 2023 13:33
… bdtss-data-management-for-connection

# Conflicts:
#	src/main/java/ru/nsu/fit/dib/projectdib/newMultiplayer/data/ActionPacket.java
#	src/main/java/ru/nsu/fit/dib/projectdib/newMultiplayer/data/ActionStatus.java
#	src/main/java/ru/nsu/fit/dib/projectdib/newMultiplayer/data/EntityState.java
#	src/main/java/ru/nsu/fit/dib/projectdib/newMultiplayer/data/actions/GameAction.java
#	src/main/java/ru/nsu/fit/dib/projectdib/newMultiplayer/data/actions/NewEntity.java
#	src/main/java/ru/nsu/fit/dib/projectdib/newMultiplayer/threads/ClientThread.java
#	src/main/java/ru/nsu/fit/dib/projectdib/newMultiplayer/threads/ServerThread.java
…ta-management-for-connection

# Conflicts:
#	src/main/java/ru/nsu/fit/dib/projectdib/data/ProjectConfig.java
#	src/main/java/ru/nsu/fit/dib/projectdib/ui/MainMenu.java
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.

4 participants