Annoying Unknown Error

[killersub]

Alright sorry for my massive posting but I have been testing out different sources amd understand how they are coded, a little better. I just need this one to figure out InGame how everything works but ofc I get errors.

I get an error on the 'public static void Load(GameClient GC)'

Here's the line of the error:

if (DateTime.Now > GC.MyChar.canload.AddSeconds(3) && GC.MyChar.cannload == false)

This is the error:

Object reference not set to an instance of an object

This is the first ProjectX source originally released here, that I'm currently using btw.

EDIT:
Now at times it wont give me any error now the client closes halfway loading without any error.

[pro4never]

The error is very general. It's saying that an object has not been set to any value.


For example if I told you "is player X over level 130" you can't check that if I haven't given you the information about who player x is.


Breakpoint it, check the line where the error is happening and find what is null then make sure you assign it before checking it.

[killersub]

Originally Posted by pro4never

The error is very general. It's saying that an object has not been set to any value.


For example if I told you "is player X over level 130" you can't check that if I haven't given you the information about who player x is.


Breakpoint it, check the line where the error is happening and find what is null then make sure you assign it before checking it.

I wouldn't have posted if I haven't tried all of that,sir.

The result is inevitable, client comes to a stop then closes rapidly.

The breakpoint did not help me because I got nothing out of it, btw the values is assigned to a 'new DateTime()' so that can't be it.

There must be some sort of hidden error, idk.

[pro4never]

GC is not null, mychar is not null and canload is a valid, initiated datetime value?

[BaussHacker]

canload is null if I'm correct. Initialize it in the constructor of the character class.

[killersub]

Originally Posted by pro4never

GC is not null, mychar is not null and canload is a valid, initiated datetime value?

Correct everything is assigned to a value. But for some reason it tells me GC in the parameters of the 'Load' void is null, when it cant be because it's the ClientState and it can't ne null ?

Originally Posted by BaussHacker

canload is null if I'm correct. Initialize it in the constructor of the character class.

sorry I'm still kinda new to some of this.

'public bool canload == false'

Has a value assigned to it already, has it not?

[Spirited]

canload must be null.
also, == is a Boolean expression.
= is an assignment.

[BaussHacker]

Argh, I don't have the source. Haven't got it since I released it, I will download it and take a look and get back to you

[killersub]

Originally Posted by Fаng

canload must be null.
also, == is a Boolean expression.
= is an assignment.

Sorry to break it to you sir, But I already have the basic knowledge to know the difference between "==" and "=".

this is "CANLOAD":

public DateTime canload = new DateTime();

this is "CANNLOAD":

public bool cannload = false;

Originally Posted by BaussHacker

Argh, I don't have the source. Haven't got it since I released it, I will download it and take a look and get back to you

I would so much appreciate it.

Any questions you have, please ask ahead.

[BaussHacker]

Before I look at it try change:
public DateTime canload = new DateTime();

to:
public DateTime canload = DateTime.Now;


You can also try catching ArgumentNullException or what it's named.

[killersub]

Originally Posted by BaussHacker

Before I look at it try change:
public DateTime canload = new DateTime();

to:
public DateTime canload = DateTime.Now;


You can also try catching ArgumentNullException or what it's named.

Will do.

will get back to you ASAP.

EDIT:

The error seems to occur when the client finishes creating the Account and the server sends the "Load" void to the client, that it when the

Object reference not set to an instance of an object

error pops up in the same line. And when I try to login again the client crashes.

[pwerty]

breakpoint. Check for null values in Client/entity

[BaussHacker]

Replace it with:

        public static void Load(Character C)
        {
            if (C.cannload == true)
            {
                if (DateTime.Now > C.canload.AddSeconds(3))
                {
                    C.CanBackup = DateTime.Now;

                    C.Loc.PreviousMap = 1002;
                    C.Loc.PreviousX = 400;
                    C.Loc.PreviousY = 400;

                    C.cannload = false;
                    C.CTT.Stop();
                    C.IsLoaded = true;
                    C.Teleport(C.Loc.Map, C.Loc.X, C.Loc.Y);
                }
            }
        }

Find the constructor for Character and replace it with:

            CTT = new System.Timers.Timer(5000.0);
            CTT.Start();
            CTT.Elapsed += delegate { Timer32.Timer.Load(this); };

Now find C.Loaded = true; in (Database.cs)

public static Game.Character LoadCharacter(string Name, ref string Account)

Put this under:

C.cannload = true;

Really bad way, but it should work. I can see a lot other **** mistakes I made back then and something like this should not even be in use.

[killersub]

Originally Posted by BaussHacker

Replace it with:

        public static void Load(Character C)
        {
            if (C.cannload == true)
            {
                if (DateTime.Now > C.canload.AddSeconds(3))
                {
                    C.CanBackup = DateTime.Now;

                    C.Loc.PreviousMap = 1002;
                    C.Loc.PreviousX = 400;
                    C.Loc.PreviousY = 400;

                    C.cannload = false;
                    C.CTT.Stop();
                    C.IsLoaded = true;
                    C.Teleport(C.Loc.Map, C.Loc.X, C.Loc.Y);
                }
            }
        }

Find the constructor for Character and replace it with:

            CTT = new System.Timers.Timer(5000.0);
            CTT.Start();
            CTT.Elapsed += delegate { Timer32.Timer.Load(this); };

Now find C.Loaded = true; in (Database.cs)

public static Game.Character LoadCharacter(string Name, ref string Account)

Put this under:

C.cannload = true;

Really bad way, but it should work. I can see a lot other shit mistakes I made back then and something like this should not even be in use.

smart fix, but still there is an error.

Object reference not set to an instance of an object

Error line (Character.cs) "TELEPORT VOID"

MyClient.AddSend(Packets.GeneralData(EntityID, Map, X, Y, 86));

[BaussHacker]

Might be Loc or Map that's empty.

Could be MyClient as well.