What should I be looking for?

05/27/2012 03:05 denominator#1
PlayerPacket.cs

Code:
public byte Job
        {
            get { return ReadByte(67); }
            set { WriteByte(value, 67); }
        }
Client.cs

Code:
private byte _Job;
Code:
public byte Job
            {
                get { return _Job; }               
            }
Code:
public void Jobs(StatusType[] Data, Client client)
            {
                if (Ninja(client))
                {
                    XpSkill = 6011;
                    XpAttackSpeed = 800;
                    ActionSpeed = 100;
                    XpJumpSpeed = 1000;
                    XpLootSpeed = 300;
                }
                if (Monk(client))
                {
                    XpSkill = 10390;
                    XpAttackSpeed = 300;
                    ActionSpeed = 100;
                    XpJumpSpeed = 800;
                    XpLootSpeed = 300;
                }
                /*  if (Taoist(Client))
                  {
                      XpSkill = 1280;
                      XpAttackSpeed = 150;
                      ActionSpeed = 100;
                      XpJumpSpeed = 800;
                      XpLootSpeed = 300;
                  }
                 */
                if (Warrior(client))
                {
                    XpSkill = 1025;
                    XpJumpSpeed = 300;
                    XpAttackSpeed = 150;
                    ActionSpeed = 100;
                    XpLootSpeed = 300;
                }
                if (Trojan(client))
                {
                    //XpSkill = 1015; //Accuracy
                    XpSkill = 1110;
                    XpAttackSpeed = 800;
                    ActionSpeed = 100;
                    XpJumpSpeed = 1000;
                    XpLootSpeed = 300;
                }
            }
Code:
#region JobBools

            public static bool Warrior(Client client)
            {
                return (client.Job > 16 && client.Job < 26);
            }
            public static bool Trojan(Client client)
            {
                return (client.Job > 9 && client.Job < 16);
            }
            public static bool Archer(Client client)
            {
                return (client.Job > 39 && client.Job < 46);
            }
            public static bool Taoist(Client client)
            {
                return (client.Job > 131 && client.Job < 136);
            }
            public static bool Monk(Client client)
            {
                return (client.Job > 59 && client.Job < 65);
            }
            public static bool Ninja(Client client)
            {
                return (client.Job > 49 && client.Job < 56);
            }
            #endregion
I'm guessing the reason it doesn't do anything is this?
Code:
public byte Job
        {
            get { return ReadByte(67); }
            set { WriteByte(value, 67); }
        }
05/27/2012 14:16 pro4never#2
... you have two different versions of Job...


One is an access or for the private variable _Job (love of god... use lower case if you're using private variables... it bothers me lol) and then you have a second version which looks to be referring to a byte array (The actual spawn packet for your character).

This looks to be a proxy settings system so there's no reason to be storing your spawn information. Just populate the value initially from the hero information packet when the client logs in and then store it as a normal variable as you have. I see no reason to use private/public Variable in this case... it's being set once and then looked up to change your settings. Just public byte Job {get;set;} would work just fine.
05/27/2012 15:44 denominator#3
Client.cs

Code:
            #region JobBools

            public static bool Warrior(Client client)
            {
                return (client.Job > 16 && client.Job < 26);
            }
            public static bool Trojan(Client client)
            {
                return (client.Job > 9 && client.Job < 16);
            }
            public static bool Archer(Client client)
            {
                return (client.Job > 39 && client.Job < 46);
            }
            public static bool Taoist(Client client)
            {
                return (client.Job > 131 && client.Job < 136);
            }
            public static bool Monk(Client client)
            {
                return (client.Job > 59 && client.Job < 65);
            }
            public static bool Ninja(Client client)
            {
                return (client.Job > 49 && client.Job < 56);
            }
            #endregion
            public byte job
            {
                get { return Job; }
            }
Code:
public byte Job { get; set; }
PlayerPacket.cs

Code:
public byte Job
        {
            get { return ReadByte(67); }
            set { WriteByte(value, 67); }
        }
So like that correct?
05/27/2012 16:02 pro4never#4
... why do you have player packet at all?

Either read the character info from memory or simply read from the packet in a single method.

Unlike with pservers, there's no reason to hold onto the packet and edit it... The server tells you the information and it only changes if the server updates it.
05/27/2012 16:08 denominator#5
Okay well I'll take it out of playerpacket.cs :D

Playerpacket.cs was already in the source

And the result is.....nothing happening o.0? Obviously not doing or adding something that needs to be added >.<