Help Problem ;|

[Procustration90]

@Override

public boolean doDie(L2Character killer)

{

if (!super.doDie(killer))

return false;

{

   if (this.getNpcId() == 13016)

killer.teleportto(20808,145755,-3144);

   {

   player.sendMessage("You are the lucky one!");

   return;

   }

 

this doesnt work, it gives me errors what do i need to change?

 

i want to make when a mob(13016) gets killed the player who killed it to be teleported to the location above 

Edited June 17, 2014 by Procustration90

[Nightw0lf]

this is so wrong...

[Boorinio]

this must be in L2Npc or a separate instance of the custom mob...not in l2pcinsatance....

[Procustration90]

it's in L2Npc

[Stewie]

 

@Override

public boolean doDie(L2Character killer)

{

if (!super.doDie(killer))

return false;

{

   if (this.getNpcId() == 13016)

killer.teleportto(20808,145755,-3144);

   {

   player.sendMessage("You are the lucky one!");

   return;

   }

 

this doesnt work, it gives me errors what do i need to change?

 

i want to make when a mob(13016) gets killed the player who killed it to be teleported to the location above 

 

 

Did you code this yourself?

If yes ... noone can help you.

For me this is wrong:

{
if (!super.doDie(killer))
return false;
{

Edited June 18, 2014 by Stewie

[Procustration90]

Ok i wrote the code by myself so i needed help from someone to what to change to make it work and what i did wrong..

[Boorinio]

 

Did you code this yourself?

If yes ... noone can help you.

For me this is wrong:

{
if (!super.doDie(killer))
return false;
{

 

you have 0 clue what those lines do please stay out of this topic...

 

tell me where it gives you error cause the only "mistake" here is the message which should be send to killer and not to player..other than that you might lost an uppercase or something...

Edited June 19, 2014 by Boorinio

[Tryskell]

Lol Stewie, those are the only correct  lines in the method ^^.

 

First,

   if (this.getNpcId() == 13016)
killer.teleportto(20808,145755,-3144);
   {
   player.sendMessage("You are the lucky one!");
   return;
   }

is wrong anyway, you probably want to do

 

   if (this.getNpcId() == 13016 && killer instanceof L2PcInstance)
   {
      killer.teleportto(20808,145755,-3144);
      killer.sendMessage("You are the lucky one!");
      return;
   }

Second, this refer to the victim, meaning in that case it should be at least setted in L2Npc or L2Attackable doDie. The point to set in the good doDie section is only for performance then, thanks to the npcid check. It has to be setted at least after the "super." call.

 

You also have to do a check regarding killer, because you would feel idiot if another NPC kills (Confusion skill and such) the NPC, being teleported and being the "lucky one".

[xdem]

doDie is boolean so you must use return true; instead of return

[xdem]

you don't even need return on that block it must be at the end of the method anyways

[Nightw0lf]

 

Lol Stewie, those are the only correct  lines in the method ^^.

 

First,

   if (this.getNpcId() == 13016)
killer.teleportto(20808,145755,-3144);
   {
   player.sendMessage("You are the lucky one!");
   return;
   }

is wrong anyway, you probably want to do

 

   if (this.getNpcId() == 13016 && killer instanceof L2PcInstance)
   {
      killer.teleportto(20808,145755,-3144);
      killer.sendMessage("You are the lucky one!");
      return;
   }

Second, this refer to the victim, meaning in that case it should be at least setted in L2Npc or L2Attackable doDie. The point to set in the good doDie section is only for performance then, thanks to the npcid check. It has to be setted at least after the "super." call.

 

You also have to do a check regarding killer, because you would feel idiot if another NPC kills (Confusion skill and such) the NPC, being teleported and being the "lucky one".

 

already told him in private but he seems not understand a thing...

[Tryskell]

We tried at least :(.

[Stewie]

 

Lol Stewie, those are the only correct  lines in the method ^^.

 

First,

   if (this.getNpcId() == 13016)
killer.teleportto(20808,145755,-3144);
   {
   player.sendMessage("You are the lucky one!");
   return;
   }

is wrong anyway, you probably want to do

 

   if (this.getNpcId() == 13016 && killer instanceof L2PcInstance)
   {
      killer.teleportto(20808,145755,-3144);
      killer.sendMessage("You are the lucky one!");
      return;
   }

Second, this refer to the victim, meaning in that case it should be at least setted in L2Npc or L2Attackable doDie. The point to set in the good doDie section is only for performance then, thanks to the npcid check. It has to be setted at least after the "super." call.

 

You also have to do a check regarding killer, because you would feel idiot if another NPC kills (Confusion skill and such) the NPC, being teleported and being the "lucky one".

 

 

Well atleast i'm glad i still find correct one in the code.

However u are the boss, respect, cya.