"буферные" поля в пакетных класах

[Sp1tfire]

зачем в

CharInfo.java

Код:

package l2ft.gameserver.network.l2.s2c;

import l2ft.gameserver.Config;
import l2ft.gameserver.instancemanager.CursedWeaponsManager;
import l2ft.gameserver.instancemanager.ReflectionManager;
import l2ft.gameserver.model.Creature;
import l2ft.gameserver.model.Player;
import l2ft.gameserver.model.base.TeamType;
import l2ft.gameserver.model.instances.DecoyInstance;
import l2ft.gameserver.model.items.Inventory;
import l2ft.gameserver.model.items.PcInventory;
import l2ft.gameserver.model.matching.MatchingRoom;
import l2ft.gameserver.model.pledge.Alliance;
import l2ft.gameserver.model.pledge.Clan;
import l2ft.gameserver.skills.effects.EffectCubic;
import l2ft.gameserver.utils.Location;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;

public class CharInfo extends L2GameServerPacket
{
    private static final Logger _log = LoggerFactory.getLogger(CharInfo.class);

    private int[][] _inv;
    private int _mAtkSpd, _pAtkSpd;
    private int _runSpd, _walkSpd, _swimSpd, _flRunSpd, _flWalkSpd, _flyRunSpd, _flyWalkSpd;
    private Location _loc, _fishLoc;
    private String _name, _title;
    private int _objId, _race, _sex, base_class, pvp_flag, karma, rec_have;
    private double speed_move, speed_atack, col_radius, col_height;
    private int hair_style, hair_color, face, _abnormalEffect, _abnormalEffect2;
    private int clan_id, clan_crest_id, large_clan_crest_id, ally_id, ally_crest_id, class_id;
    private int _sit, _run, _combat, _dead, private_store, _enchant;
    private int _noble, _hero, _fishing, mount_type;
    private int plg_class, pledge_type, clan_rep_score, cw_level, mount_id;
    private int _nameColor, _title_color, _transform, _agathion, _clanBoatObjectId;
    private EffectCubic[] cubics;
    private boolean _isPartyRoomLeader, _isFlying;
    private TeamType _team;

    public CharInfo(Player cha)
    {
        this((Creature) cha);
    }

    public CharInfo(DecoyInstance cha)
    {
        this((Creature) cha);
    }

    public CharInfo(Creature cha)
    {
        if(cha == null)
        {
            System.out.println("CharInfo: cha is null!");
            Thread.dumpStack();
            return;
        }

        if(cha.isInvisible())
            return;

        if(cha.isDeleted())
            return;

        Player player = cha.getPlayer();
        if(player == null)
            return;

        if(player.isInBoat())
        {
            _loc = player.getInBoatPosition();
            if(player.isClanAirShipDriver())
            {
                _clanBoatObjectId = player.getBoat().getObjectId();
            }
        }

        if (_loc == null)
            _loc = cha.getLoc();

        _objId = cha.getObjectId();

        // Проклятое оружие и трансформации для ТВ скрывают имя и все остальные опознавательные знаки
        if(player.getTransformationName() != null || (player.getReflection() == ReflectionManager.GIRAN_HARBOR || player.getReflection() == ReflectionManager.PARNASSUS) && player.getPrivateStoreType() != Player.STORE_PRIVATE_NONE)
        {
            _name = player.getTransformationName() != null ? player.getTransformationName() : player.getName();
            _title = "";
            clan_id = 0;
            clan_crest_id = 0;
            ally_id = 0;
            ally_crest_id = 0;
            large_clan_crest_id = 0;
            if(player.isCursedWeaponEquipped())
                cw_level = CursedWeaponsManager.getInstance().getLevel(player.getCursedWeaponEquippedId());
        }
        else
        {
            _name = player.getName();
            if(player.getPrivateStoreType() != Player.STORE_PRIVATE_NONE)
                _title = "";
            else if(!player.isConnected())
            {
                _title = "NO CARRIER";
                _title_color = 255;
            }
            else
            {
                _title = player.getTitle();
                _title_color = player.getTitleColor();
            }

            Clan clan = player.getClan();
            Alliance alliance = clan == null ? null : clan.getAlliance();
            //
            clan_id = clan == null ? 0 : clan.getClanId();
            clan_crest_id = clan == null ? 0 : clan.getCrestId();
            large_clan_crest_id = clan == null ? 0 : clan.getCrestLargeId();
            //
            ally_id = alliance == null ? 0 : alliance.getAllyId();
            ally_crest_id = alliance == null ? 0 : alliance.getAllyCrestId();

            cw_level = 0;
        }

        if(player.isMounted())
        {
            _enchant = 0;
            mount_id = player.getMountNpcId() + 1000000;
            mount_type = player.getMountType();
        }
        else
        {
            _enchant = player.getEnchantEffect();
            mount_id = 0;
            mount_type = 0;
        }

        _inv = new int[PcInventory.PAPERDOLL_MAX][2];
        for(int PAPERDOLL_ID : PAPERDOLL_ORDER)
        {
            _inv[PAPERDOLL_ID][0] = player.getInventory().getPaperdollItemId(PAPERDOLL_ID);
            _inv[PAPERDOLL_ID][1] = player.getInventory().getPaperdollAugmentationId(PAPERDOLL_ID);
        }

        _mAtkSpd = player.getMAtkSpd();
        _pAtkSpd = player.getPAtkSpd();
        speed_move = player.getMovementSpeedMultiplier();
        _runSpd = (int) (player.getRunSpeed() / speed_move);
        _walkSpd = (int) (player.getWalkSpeed() / speed_move);

        _flRunSpd = 0; // TODO
        _flWalkSpd = 0; // TODO

        if(player.isFlying())
        {
            _flyRunSpd = _runSpd;
            _flyWalkSpd = _walkSpd;
        }
        else
        {
            _flyRunSpd = 0;
            _flyWalkSpd = 0;
        }

        _swimSpd = player.getSwimSpeed();
        _race = player.getBaseTemplate().race.ordinal();
        _sex = player.getSex();
        base_class = player.getBaseClassId();
        pvp_flag = player.getPvpFlag();
        karma = player.getKarma();

        speed_atack = player.getAttackSpeedMultiplier();
        col_radius = player.getColRadius();
        col_height = player.getColHeight();
        hair_style = player.getHairStyle();
        hair_color = player.getHairColor();
        face = player.getFace();
        if(clan_id > 0 && player.getClan() != null)
            clan_rep_score = player.getClan().getReputationScore();
        else
            clan_rep_score = 0;
        _sit = player.isSitting() ? 0 : 1; // standing = 1 sitting = 0
        _run = player.isRunning() ? 1 : 0; // running = 1 walking = 0
        _combat = player.isInCombat() ? 1 : 0;
        _dead = player.isAlikeDead() ? 1 : 0;
        private_store = player.isInObserverMode() ? Player.STORE_OBSERVING_GAMES : player.getPrivateStoreType();
        cubics = player.getCubics().toArray(new EffectCubic[player.getCubics().size()]);
        _abnormalEffect = player.getAbnormalEffect();
        _abnormalEffect2 = player.getAbnormalEffect2();
        rec_have = player.isGM() ? 0 : player.getRecomHave();
        class_id = player.getClassId().getId();
        _team = player.getTeam();

        _noble = player.isNoble() ? 1 : 0; // 0x01: symbol on char menu ctrl+I
        _hero = player.isHero() || player.isGM() && Config.GM_HERO_AURA ? 1 : 0; // 0x01: Hero Aura
        _fishing = player.isFishing() ? 1 : 0;
        _fishLoc = player.getFishLoc();
        _nameColor = player.getNameColor(); // New C5
        plg_class = player.getPledgeClass();
        pledge_type = player.getPledgeType();
        _transform = player.getTransformation();
        _agathion = player.getAgathionId();
        _isPartyRoomLeader = player.getMatchingRoom() != null && player.getMatchingRoom().getType() == MatchingRoom.PARTY_MATCHING && player.getMatchingRoom().getLeader() == player;
        _isFlying = player.isInFlyingTransform();
    }

    @Override
    protected final void writeImpl()
    {
        Player activeChar = getClient().getActiveChar();
        if(activeChar == null)
            return;

        if(_objId == 0){ return; }

        if(activeChar.getObjectId() == _objId)
        {
            _log.error("You cant send CharInfo about his character to active user!!!");
            return;
        }

        writeC(0x31);
        writeD(_loc.x);
        writeD(_loc.y);
        writeD(_loc.z + Config.CLIENT_Z_SHIFT);
        writeD(_clanBoatObjectId);
        writeD(_objId);
        writeS(_name);
        writeD(_race);
        writeD(_sex);
        writeD(base_class);

        for(int PAPERDOLL_ID : PAPERDOLL_ORDER)
            writeD(_inv[PAPERDOLL_ID][0]);

        for(int PAPERDOLL_ID : PAPERDOLL_ORDER)
            writeD(_inv[PAPERDOLL_ID][1]);

        writeD(0x01); //TODO talisman count(VISTALL)
        writeD(0x00); //TODO cloak status(VISTALL)

        writeD(pvp_flag);
        writeD(karma);

        writeD(_mAtkSpd);
        writeD(_pAtkSpd);

        writeD(0x00);

        writeD(_runSpd);
        writeD(_walkSpd);
        writeD(_swimSpd);
        writeD(_swimSpd);
        writeD(_flRunSpd);
        writeD(_flWalkSpd);
        writeD(_flyRunSpd);
        writeD(_flyWalkSpd);

        writeF(speed_move); // _cha.getProperMultiplier()
        writeF(speed_atack); // _cha.getAttackSpeedMultiplier()
        writeF(col_radius);
        writeF(col_height);
        writeD(hair_style);
        writeD(hair_color);
        writeD(face);
        writeS(_title);
        writeD(clan_id);
        writeD(clan_crest_id);
        writeD(ally_id);
        writeD(ally_crest_id);

        writeC(_sit);
        writeC(_run);
        writeC(_combat);
        writeC(_dead);
        writeC(0x00); // is invisible
        writeC(mount_type); // 1-on Strider, 2-on Wyvern, 3-on Great Wolf, 0-no mount
        writeC(private_store);
        writeH(cubics.length);
        for(EffectCubic cubic : cubics)
            writeH(cubic == null ? 0 : cubic.getId());
        writeC(_isPartyRoomLeader ? 0x01 : 0x00); // find party members
        writeD(_abnormalEffect);
        writeC(_isFlying ? 0x02 : 0x00);
        writeH(rec_have);
        writeD(mount_id);
        writeD(class_id);
        writeD(0x00);
        writeC(_enchant);

        writeC(_team.ordinal()); // team circle around feet 1 = Blue, 2 = red

        writeD(large_clan_crest_id);
        writeC(_noble);
        writeC(_hero);

        writeC(_fishing);
        writeD(_fishLoc.x);
        writeD(_fishLoc.y);
        writeD(_fishLoc.z);

        writeD(_nameColor);
        writeD(_loc.h);
        writeD(plg_class);
        writeD(pledge_type);
        writeD(_title_color);
        writeD(cw_level);
        writeD(clan_rep_score);
        writeD(_transform);
        writeD(_agathion);

        writeD(0x01); // T2

        writeD(_abnormalEffect2);
    }

    public static final int[] PAPERDOLL_ORDER =
    {
            Inventory.PAPERDOLL_UNDER,
            Inventory.PAPERDOLL_HEAD,
            Inventory.PAPERDOLL_RHAND,
            Inventory.PAPERDOLL_LHAND,
            Inventory.PAPERDOLL_GLOVES,
            Inventory.PAPERDOLL_CHEST,
            Inventory.PAPERDOLL_LEGS,
            Inventory.PAPERDOLL_FEET,
            Inventory.PAPERDOLL_BACK,
            Inventory.PAPERDOLL_LRHAND,
            Inventory.PAPERDOLL_HAIR,
            Inventory.PAPERDOLL_DHAIR,
            Inventory.PAPERDOLL_RBRACELET,
            Inventory.PAPERDOLL_LBRACELET,
            Inventory.PAPERDOLL_DECO1,
            Inventory.PAPERDOLL_DECO2,
            Inventory.PAPERDOLL_DECO3,
            Inventory.PAPERDOLL_DECO4,
            Inventory.PAPERDOLL_DECO5,
            Inventory.PAPERDOLL_DECO6,
            Inventory.PAPERDOLL_BELT
    };
}

сделано копирование всей инфы в поля класа CharInfo? (зачем эти "буферные поля"?)

[Smiler]

потому что сформированный пакет попадает в очередь на отправку.

и между моментом формирования пакет и реальной отправкой проходит какое то время.

за это время может многое поменяться в "игре" других классах откуда эти данные берутся.

[Hint]

Мне тоже смысл непонятен, хотя смотрю со стороны (никогда не занимался правкой кода L2J, только изучал).
Пакет сформировали (создали буфер, записали в него поля в нужном порядке), отправить можно когда угодно. Зачем что-то куда-то копировать?

[Smiler]

А почему нельзя сразу пакет сформировать и отложить?

он как раз сразу формируется и "откладывается" в очередь на отправку

[Hint]

он как раз сразу формируется и "откладывается" в очередь на отправку

А зачем поля копировать? Для удобства, чтобы код чище получался? Сначала сбор конкретных полей из данных (различные условия, преобразования и т. д.), а потом их запись.

Код:

_mAtkSpd = player.getMAtkSpd();
_pAtkSpd = player.getPAtkSpd();

[Smiler]

А зачем поля копировать? Для удобства, чтобы код чище получался? Сначала сбор конкретных полей из данных (различные условия, преобразования и т. д.), а потом их запись.

Код:

_mAtkSpd = player.getMAtkSpd();
_pAtkSpd = player.getPAtkSpd();

нет. я уже ответил во 2-м посте на этот вопрос.

данные в момент формирования и отправки пакета в других классах могут поменяться.

почитайте про многопоточные приложения и о том как работает Nio в java думаю многое проясниться.

[Hint]

нет. я уже ответил во 2-м посте на этот вопрос.

данные в момент формирования и отправки пакета в других классах могут поменяться.

почитайте про многопоточные приложения и о том как работает Nio в java думаю многое проясниться.

В процессе выполнения метода "public CharInfo(Creature cha)" данные не меняются. Кто мешает здесь же полностью составить пакет и отложить его до момента отправки? А не копировать поля, а потом уже формировать из них пакет во writeImpl.

[Smiler]

так в методе public CharInfo(Creature cha) (этот метод называется конструктором класса)
как раз и происходит формирование данных пакета.
нам же надо гдето хранить эти данные?
вот мы и сохраняем их в переменные экземпляра класса.

а потом дергается метод writeImpl который ложит данные из этого пакета в сетевой буфер.

а уже потом в отдельном потоке, "сетевой менеджер" назовем его так, при появлении данных в "сетевом буфере" отправляет данные клиенту.

[Gattsu]

Аналогия:
Синхронизация аудио и видео дорожки.

Смотреть видео с отстающим звуком как-то не ахти

[VISTALL]

В процессе выполнения метода "public CharInfo(Creature cha)" данные не меняются. Кто мешает здесь же полностью составить пакет и отложить его до момента отправки? А не копировать поля, а потом уже формировать из них пакет во writeImpl.

по той причине, что когда пишется конструктор - он не сует даные в Буфер.


А вот когда вызывается writeImpl, статы у чара могут изменится.

Это очень сильно важно для XYZ при движении, которые задержка в 100 мс, уже вылазит