Пакеты

[xmafee]

Всем привет, отсылаю пакет при открытие гуи на сервер с целью получить информацию с базы данных, но почему-то в итоге оно мне ничего не возвращает.
По моей логике это должно работать так:
CLIENT --> SERVER (запрос на получение данных)
SERVER --> CLIENT (отсылает информацию на клиент)

    @SubscribeEvent
    public void onGuiOpened(GuiOpenEvent event) {
        if (event.getGui() instanceof ListScreen) {
            System.out.println("Gui opened");
            new CPacketInfo().sendToServer();
        }
    }

@ElegantPacket
@Value
public class CPacketInfo implements ClientToServerPacket {

    @Override
    public void onReceive(EntityPlayerMP player) {
        System.out.println("[CPacketInfo] Packet here");
        new SPacketInfo().sendToPlayer(player);
    }
}

@ElegantPacket
@Value
public class SPacketInfo implements ServerToClientPacket {

    public String getName(EntityPlayer player) {
        String getNameS = new String();
        try {
            getNameS = SqlHelper.sqlHelperInstance.getName(player.getDisplayNameString());
        } catch (SQLException e) {
            e.printStackTrace();
        }
        return getNameS ;
    }

    @Override
    public void onReceive(Minecraft mc) {
        System.out.println("[SPacketInfo] Packet here");
        System.out.println("Name: " + getClanName(mc.player));
    }
}

Писал этот код после того, как выпил кваса, поэтому не факт, что где-то сильно затупил, не надо бить палками, спасибо!)
Возможно где-то в логике затупил, что скорее всего и есть, если есть предложения для иной реализации - буду рад услышать и буду благодарен за совет!

 

[will0376]

Пакет лучше всего слать в момент срабатывания бинда на гуи. И в момент получения данных - открывать саму гуишку

 

[xmafee]

Пакет лучше всего слать в момент срабатывания бинда на гуи. И в момент получения данных - открывать саму гуишку

Так дело в том, что по сути и так должно работать, хотябы в консоль выводить, а ничего не происходит

И кстати, почему именно так будет лучше сделать?

 

[will0376]

И кстати, почему именно так будет лучше сделать?

99% уверенность, что данные дойдут до того, как откроется гуишка.

public void onGuiOpened

у тебя регистрация класса через аннотацию? там статик метод тогда нужен.

 

[xmafee]

у тебя регистрация класса через аннотацию? там статик метод тогда нужен.

нет, в clientproxy

 

[will0376]

clientproxy

хм. проверь выполнение кода

new CPacketInfo().sendToServer();

 

[xmafee]

хм. проверь выполнение кода

в плане?
в консоль выводиться текст при открытии гуи, если ты об этом

 

[will0376]

@hohserg

 

[will0376]

И кстати, getName перетащи куда-нибудь и передавай через стринг в SPacketInfo. А то получается, что сейчас ты на стороне клиента в момент выполнения кода тянешь данные с скл.

 

[xmafee]

куда-нибудь

например куда?

 

[hohserg]

Наверное, он имел ввиду, что данные из бд нужно записывать в пакет до отправки, а не получать их(откуда?) при получении

 

[xmafee]

записывать в пакет

а где их будет лучше всего записывать?

 

[hohserg]

Просто поле добавь
Оно само запишется

 

[xmafee]

Просто поле добавь

Это понятно

Оно само запишется

И это понятно

Я о том, что где его лучше всего записать, тк там в любом случае игрока надо получать? Можно было бы например в каком-то событие на стороне сервера, но это ведь не то немного

 

[hohserg]

Ну типо там, где ты создаешь экземпляр пакета перед отправкой
CPacketInfo#onReceive

 

[xmafee]

@ElegantPacket
@Value
public class CPacketInfo implements ClientToServerPacket {

    @Override
    public void onReceive(EntityPlayerMP player) {
        String name = new String();

        try {
            name = SqlHelper.sqlHelperInstance.getName(player.getDisplayNameString());
        } catch (SQLException e) {
            e.printStackTrace();
        }

        System.out.println("packet here");
        new SPacketClanInfo(name).sendToPlayer(player);
    }
}

@ElegantPacket
@Value
public class SPacketInfo implements ServerToClientPacket {

    String name;

    @Override
    public void onReceive(Minecraft mc) {
        System.out.println("name: " + name);
    }
}

Где туплю?

 

[will0376]

Где туплю?

Внимательнее вчитайся в названия интерфейсов.
Получать данные из sql на стороне клиента - наитупейшая идея.

 

[xmafee]

Внимательнее вчитайся в названия интерфейсов.
Получать данные из sql на стороне клиента - наитупейшая идея.

@ElegantPacket
@Value
public class SPacketInfo implements ServerToClientPacket {
    
    @Override
    public void onReceive(Minecraft mc) {
        String name = new String();
        
        try {
            name = SqlHelper.sqlHelperInstance.getName(mc.player.getDisplayNameString());
        } catch (SQLException e) {
            e.printStackTrace();
        }
        System.out.println("name: " + name);
    }
}

абсолютно то-же самое

 

[xmafee]

Пробовал ещё пару вариантов, но всё безуспешно

 

[will0376]

2 примера из мода, который я недавно писал:
На клиент: hastebin
На сервер: hastebin
Для ясности: в билдере задаются все аргументы, а потом только вызывается метод билда класса и метод отправки пакета на таргет.