Graal Forums - gs2 debug error?

Graal Forums (https://forums.graalonline.com/forums/index.php)

- Tech Support (https://forums.graalonline.com/forums/forumdisplay.php?f=7)

- - gs2 debug error? (https://forums.graalonline.com/forums/showthread.php?t=59227)

gs2 debug error?

just testing what would happen i made this:

PHP Code:


		
			
function onActionserverside(acc,weapon){

  e = findplayer(acc);

  if(e != NULL) echo("Found Player: " @ e);

    else echo("Player NOT found:" @ acc);

  e.addweapon(weapon);



}



//#CLIENTSIDE

function onPlayerchats(){

 tokens = player.chat.tokenize();

   if(tokens[0] == "/findplayer"){

     triggeraction(0,0,"serverside","Sern",tokens[1],tokens[2]);

  }

}

and when weapon = NULL it returns a function error to the RC.

yes i could just add if(weapon != NULL) but i wanted to see if gs2 would look for the function addweapon if no weapon was specified. this also happens to removeweapon

this also happens if the weapon doesn't exist
GraalScript: Function addweapon not found at line 5 in script of Sern (in level test.nw at pos (0, 0))

Your script is logically wrong. It is not erroring because the weapon is not found, it is erroring because it has not found a valid player object.

HTML Code:

function onActionServerSide(temp.acc, temp.weap)

{

  temp.p = findplayer(temp.acc);

  

  if (temp.p != null)

  {

    sendToNC("Found Player: " @ temp.p.account);

    temp.p.addWeapon(temp.weap);

  }

  else

  {

    sendToNC("Player NOT found:" @ temp.acc);

  }

}



//#CLIENTSIDE

function onPlayerChats()

{

  tokens = player.chat.tokenize();

  

  if (tokens[0] == "/findplayer")

  {

    triggeraction(0,0, "serverside", "Sern", tokens[1], tokens[2]);

  }

}

Quote:

Originally Posted by ZeroTrack

just testing what would happen i made this:

PHP Code:


		
			
function onActionserverside(acc,weapon){

  e = findplayer(acc);

  if(e != NULL) echo("Found Player: " @ e);

    else echo("Player NOT found:" @ acc);

  e.addweapon(weapon);



}



//#CLIENTSIDE

function onPlayerchats(){

 tokens = player.chat.tokenize();

   if(tokens[0] == "/findplayer"){

     triggeraction(0,0,"serverside","Sern",tokens[1],tokens[2]);

  }

}

Rick, it is kind of weird to use a temp. prefix on function parameters. Aren't they treated as temp variables automatically?

Quote:

Originally Posted by Kaimetsu

Rick, it is kind of weird to use a temp. prefix on function parameters. Aren't they treated as temp variables automatically?

Yes, personal preference... ^^

Quote:

Originally Posted by Rick

Yes, personal preference... ^^

Quote:

Originally Posted by Rick

Your script is logically wrong. It is not erroring because the weapon is not found, it is erroring because it has not found a valid player object.

OH YEA ..... whoops stupid mistake, but why would it error that the function addweapon wasn't found? because e = null so it looks for the function addweapon???

Quote:

Originally Posted by ZeroTrack

whoops stupid mistake, but why would it error that the function addweapon wasn't found?

Because you are referring to a function of a non-existent object. Non-existent objects do not have functions.

Quote:

Originally Posted by Kaimetsu

Because you are referring to a function of a non-existent object. Non-existent objects do not have functions.

ahh i see , thanks =D

I would say onActionServerSide(temp.acc, temp.weap) is not correct because this would create temp.temp.acc or so, have not tested that though. temp. is just referring to the current callstack entry, so you could do onActionServerSide(acc, weap) and then access the parameters as "acc" or "temp.acc"