[help] all my erros

[reborn666]

Accoun logging baba IP >--> 5.236.154.198
System.NullReferenceException: Object reference not set to an instance of an object.
at UltimateCo.Character.UseSkill(UInt16 SkillId, UInt16 X, UInt16 Y, UInt32 TargID) in C:\backup £ost\UltimateCo V1.4\UltimateCo V1.4\Character.cs:line 2219
at UltimateCo.Character.Attack() in C:\backup £ost\UltimateCo V1.4\UltimateCo V1.4\Character.cs:line 3196




else if (AtkType == 21)
{
if (SkillLooping != 0)
{
if (SkillLoopingTarget != 0)
{
if (SkillLoopingTarget > 400000 && SkillLoopingTarget < 500000)
{
SingleMob Targ = (SingleMob)Mobs.AllMobs[SkillLoopingTarget];
if (Targ == null || !Targ.Alive)
return;
else
UseSkill(SkillLooping, (ushort)Targ.PosX, (ushort)Targ.PosY, SkillLoopingTarget);

}
else if (SkillLoopingTarget < 7000 && SkillLoopingTarget >= 5000)
{
SingleNPC Targ = (SingleNPC)NPCs.AllNPCs[SkillLoopingTarget];
if (Targ == null)
return;
else
UseSkill(SkillLooping, (ushort)Targ.X, (ushort)Targ.Y, SkillLoopingTarget);

}
else
{
Character Targ = (Character)World.AllChars[SkillLoopingTarget];
if (Targ == null || !Targ.Alive)
return;
else
UseSkill(SkillLooping, Targ.LocX, Targ.LocY, SkillLoopingTarget);
}
}
else
{
UseSkill(SkillLooping, SkillLoopingX, SkillLoopingY, 0); //here <<---
}
}
else
{
SkillLooping = 0;
}
}
Ready = true;

}
catch (Exception Exc) { General.WriteLine(Convert.ToString(Exc)); }
}







and


if (!Other.NoPK(LocMap))
if (PKMode == 2 || PKMode == 0)
{

Character Target = (Character)World.AllChars[TargID];
if (Target.Alive) //here <--
if (!Target.Flying || SkillAttributes[0] != 12)
{
X = Target.LocX;
Y = Target.LocY;

if someone can help me

[tao4229]

Target is null.

[Color="Blue"]if [/color][Color="Black"]([b][i]Target != null[/i][/b] && Target.Alive)[/color]

Should fix it.

[InfamousNoone]

Originally Posted by tao4229

Target is null.

[Color="Blue"]if [/color][Color="Black"]([b][i]Target != null[/i][/b] && Target.Alive)[/color]

Should fix it.

I'm not sure if C#'s pre-processor is smart enough to inline the null check to this, but it should be

if (Target != null)
{
if (Target.Active)
{

}
}

Or else when it evaluates the right hand side of the &&, if the left side is false (meaning it's null) you'll still get the error.

[tao4229]

Didn't know that, thanks =o.

[reborn666]

Originally Posted by InfamousNoone

I'm not sure if C#'s pre-processor is smart enough to inline the null check to this, but it should be

if (Target != null)
{
if (Target.Active)
{

}
}

Or else when it evaluates the right hand side of the &&, if the left side is false (meaning it's null) you'll still get the error.

ths i'm working with this code