C# Packets Bizarres!

[DevChris]

Bonsoir à tous,
Je remarque qu'il y a des packets "bizarre" que je n'arrive pas à implementer.

Je m'explique, normalement dans les sources du jeu il est dis quel type nous devons instancier, hors je m'apercois que ça peut varier.
En effet, dans le MapComplementaryInformationsDataMessage nous devons sois disant avoir une liste de InteractiveElement hors je vois en faisant ça :

for (interactiveElementsIndex = 0; (interactiveElementsIndex < interactiveElementsCount); interactiveElementsIndex = (interactiveElementsIndex + 1))
            {
                short protocol = (short)reader.ReadUShort();
                Console.WriteLine("*************************** " + protocol + " ***************************");
                if ((TypeEnum)protocol == TypeEnum.InteractiveElement)
                {
                    InteractiveElement objectToAdd = ProtocolTypeManager.GetInstance<InteractiveElement>(protocol);
                    objectToAdd.Deserialize(reader);
                    m_interactiveElements.Add(objectToAdd);
                }
                else if ((TypeEnum)protocol == TypeEnum.InteractiveElementWithAgeBonus)
                {
                    InteractiveElementWithAgeBonus tmp = ProtocolTypeManager.GetInstance<InteractiveElementWithAgeBonus>(protocol);
                    tmp.Deserialize(reader);
                    m_interactiveElements.Add(new InteractiveElement(tmp.EnabledSkills, tmp.DisabledSkills, tmp.ElementId, tmp.ElementTypeId, tmp.OnCurrentMap));
                }
            }

Que parfois il me recupere le TypeID du InteractiveElement, mais parfois du InteractiveElementWithAgeBonus.... j'ai essayer cette implementation, mais ça ne veut pas marcher quand je recois le MapComplementaryInformationsDataInHouseMessage par exemple... je ne sais pas pourquoi....

Merci

 

[w0dm4n]

Salut, c'est pareil pour les acteurs qui peuvent contenir des GameRolePlayCharacterInformations ou des GameRolePlayMerchantInformations par exemple, qui hérite tout deux de GameRolePlayActorInformations
les serialize/derialize doivent être donc fait en tant que GameRolePlayCharacterInformations ou GameRolePlayMerchantInformations et non GameRolePlayActorInformations

 

[DevChris]

Bizzare.... tu pourrais me montrer ton MapComplementaryInformationsDataMessage et ton FriendsList et FriendUpdate alors s'il te plait? merci

 

[Taykyue]

C'est de l'héritage.
Prenons l'exemple concernant les élements interactifs :

         var _loc6_:uint = param1.readUnsignedShort();
         var _loc7_:uint = 0;
         while(_loc7_ < _loc6_)
         {
            _loc18_ = param1.readUnsignedShort();
            _loc19_ = ProtocolTypeManager.getInstance(InteractiveElement,_loc18_);
            _loc19_.deserialize(param1);
            this.interactiveElements.push(_loc19_);
            _loc7_++;
         }

Dans la boucle while, on lit un ushort, qui correspond au type de l'élement. Il nous permet de récupérer le bon type en questionnant le ProtocolTypeManager. Le type récupéré doit donc être un enfant du type de base (ici InteractiveElement)

Ensuite, on appelle la méthode Deserialize (car si tu appeles DeserializeAs_InteractiveElement, l'héritage ne fonctionnera pas).

 

[DevChris]

Ahhhh je vois du coup le problème c'est mon ProtocolTypeManager.... je pourrais voir comment tu as fait le tient please? dicord si tu veux au pire : yovano_c#4592

 

[Taykyue]

Grâce à la réflection, je récupère tout les types qui contiennent l'interface INetworkType, puis je les met dans un Dictionary<int, Type> (int pour l'id du type).

Tu peux optimiser un peu plus en faisant un Dictionary<int, Func<object>>, où la Func retournera le type instancié. Il ne te restera plus qu'a le cast selon le type désiré

 

[DevChris]

Mais c'est ce que je fais lol avec le Func et tout pourtant ajoute moi discord please qu'on s'appelle...

 

[DevChris]

Du coup tu le cast comment ? deja est-ce que mes If sont bons ???

short protocol = (short)reader.ReadUShort();
                Console.WriteLine("************* FriendsList ************** " + protocol + " ***************************");
                if ((TypeEnum)protocol == TypeEnum.FriendInformations)
                {
                    FriendInformations objectToAdd = ProtocolTypeManager.GetInstance<FriendInformations>((short)reader.ReadUShort());
                    objectToAdd.Deserialize(reader);
                    m_friendsList.Add(objectToAdd);
                }
                else if ((TypeEnum)protocol == TypeEnum.FriendOnlineInformations)
                {
                    FriendInformations objectToAdd = ProtocolTypeManager.GetInstance<FriendOnlineInformations>((short)reader.ReadUShort());
                    objectToAdd.Deserialize(reader);
                    m_friendsList.Add(objectToAdd);
                }
                else
                {
                    FriendInformations objectToAdd = ProtocolTypeManager.GetInstance<FriendInformations>((short)reader.ReadUShort());
                    objectToAdd.Deserialize(reader);
                    m_friendsList.Add(objectToAdd);
                }

Ensuite comme tu vois, je cast comme ça, mais est-ce que c'est bon??? :

FriendInformations objectToAdd = ProtocolTypeManager.GetInstance<FriendOnlineInformations>((short)reader.ReadUShort());

 

[Taykyue]

Montre le code de ton ProtocolTypeManager

 

[DevChris]

using DofusBot.Utilities.Reflection;
using System;
using System.Collections.Generic;
using System.Reflection;
using System.Runtime.Serialization;

namespace DofusBot.Protocol
{
    public static class ProtocolTypeManager
    {

        private static readonly Dictionary<short, Type> types = new Dictionary<short, Type>(200);
        private static readonly Dictionary<short, Func<object>> typesConstructors = new Dictionary<short, Func<object>>(200);
        public static void Initialize()
        {
            Assembly asm = Assembly.GetAssembly(typeof(ProtocolTypeManager));

            foreach (Type type in asm.GetTypes())
            {
                if (type.Namespace == null || !type.Namespace.Contains("Protocol.Network.Types"))
                    continue;

                FieldInfo field = type.GetField("ProtocolId");

                if (field != null)
                {
                    short id = (short)(int)(field.GetValue(type));

                    types.Add(id, type);

                    ConstructorInfo ctor = type.GetConstructor(Type.EmptyTypes);

                    if (ctor == null)
                        throw new System.Exception(string.Format("'{0}' doesn't implemented a parameterless constructor", type));

                    typesConstructors.Add(id, ctor.CreateDelegate<Func<object>>());
                }
            }

            Console.WriteLine(string.Format("<{0}> type(s) loaded.", types.Count));
        }

        public static T GetInstance<T>(short id) where T : class
        {
            if (!types.ContainsKey(id))
            {
                Console.WriteLine(string.Format("Type <id:{0}> doesn't exist", id));
            }

            return typesConstructors[id]() as T;
        }

        [Serializable]
        public class ProtocolTypeNotFoundException : System.Exception
        {
            public ProtocolTypeNotFoundException()
            {
            }

            public ProtocolTypeNotFoundException(string message)
                : base(message)
            {
            }

            public ProtocolTypeNotFoundException(string message, System.Exception inner)
                : base(message, inner)
            {
            }

            protected ProtocolTypeNotFoundException(
                SerializationInfo info,
                StreamingContext context)
                : base(info, context)
            {
            }
        }

    }
}

 

[Taykyue]

Du coup tu le cast comment ? deja est-ce que mes If sont bons ???

short protocol = (short)reader.ReadUShort();
                Console.WriteLine("************* FriendsList ************** " + protocol + " ***************************");
                if ((TypeEnum)protocol == TypeEnum.FriendInformations)
                {
                    FriendInformations objectToAdd = ProtocolTypeManager.GetInstance<FriendInformations>((short)reader.ReadUShort());
                    objectToAdd.Deserialize(reader);
                    m_friendsList.Add(objectToAdd);
                }
                else if ((TypeEnum)protocol == TypeEnum.FriendOnlineInformations)
                {
                    FriendInformations objectToAdd = ProtocolTypeManager.GetInstance<FriendOnlineInformations>((short)reader.ReadUShort());
                    objectToAdd.Deserialize(reader);
                    m_friendsList.Add(objectToAdd);
                }
                else
                {
                    FriendInformations objectToAdd = ProtocolTypeManager.GetInstance<FriendInformations>((short)reader.ReadUShort());
                    objectToAdd.Deserialize(reader);
                    m_friendsList.Add(objectToAdd);
                }

Ensuite comme tu vois, je cast comme ça, mais est-ce que c'est bon??? :

FriendInformations objectToAdd = ProtocolTypeManager.GetInstance<FriendOnlineInformations>((short)reader.ReadUShort());

Tu ne dois pas gérer le cas par type. Ton code doit plutôt ressembler à ça :

FriendInformations objectToAdd = ProtocolTypeManager.GetInstance<FriendInformations>((short)reader.ReadUShort());
objectToAdd.Deserialize(reader);
this.m_friendsList.Add(objectToAdd);

 

[DevChris]

Ok bon bah mon probleme viens pas de là alors... tu peux dl mon projet et look please ?....
J'ai ces 4 packets qui ne fonctionne pas juste (pour l'instant en tout cas.. j'en ai ps vu d'autre) et je ne sais pas d'ou ça peut venir alors..pour moi les packets sont bon.. ça dois etre les types qui sont implementer dans ces messages....:

- FriendsListMessage
- FriendUpdateMessage
- MapComplementaryInformationsDataMessage
- GameRolePlayShowActorMessage

https://mega.nz/#!JTAVSarQ!AkuYI2rti4zuhMihMICPlAOJokIjYU17doeluppVgf8

 

[DevChris]

AH oui change le DofusPath dans le Main_Load pour ton chemin dofus aussi :3

 

[Gustavo Militão]

Yovano,

Certaines classes de votre projet sont la lecture des types de données dans les mauvaises positions. Probablement votre protocolbuilder l'a fait sans le vouloir. La mise en œuvre de la deserialize de classe doit être simple et passer ProtocolId pour GetInstance le ProtocolTypeManager et il résoudra le constructeur à appeler, ainsi que le type. Le problème est à la lecture des paquets de InteractiveElement enfants.

Câlins.

 

[DevChris]

Speak english please Gustavo your french is bad. So, I know that some things are misplaced. But I don't see them.... you can tell me where they are?