Graal Forums - Token problem

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

- NPC Scripting (https://forums.graalonline.com/forums/forumdisplay.php?f=8)

- - Token problem (https://forums.graalonline.com/forums/showthread.php?t=73840)

I'm making a gang system, but I'm having a small problem.
I have this in a class:

PHP Code:


		
			
function onPlayerchats()

{

  if (player.chat.starts(":"))

  {

    findNpc("Gang Control").leaderCommand(player.chat);

  }

}

and I have this in the Gang Control npc

PHP Code:


		
			
public function leaderCommand(toks)

{

  switch (temp.toks[0])

  {

    case ":addmember":

      player.chat = "testing";

    break;

  }

}

when I say :addmember alone, it switches my test to "testing" which is what I want, sort of (since I'm trying to debug it) but if I say :addmember blahwhatever, it doesn't do anything.

even if I changed it to player.chat = temp.toks[1]; , it doesn't return temp.toks[1] what am I doing wrong?

sorry if I don't explain it that well, I tried my best.

Change public function leaderCommand(toks) to (cmd) or something, and then immediately after, do:

PHP Code:


		
			
temp.toks = temp.cmd.tokenize();

Strings don't tokenize themselves; in your script, temp.toks is the full chat string, not a tokenized array from tokenize();.

Ha ha, I was going to post something similar before Mr. Gscript ( phrase originally coined by Twinny ) came in to save the day! :P

Quote:

Originally Posted by Sum41Freeeeek (Post 1305757)

I'm making a gang system, but I'm having a small problem.
I have this in a class:

PHP Code:


		
			
function onPlayerchats()

{

  if (player.chat.starts(":"))

  {

    findNpc("Gang Control").leaderCommand(player.chat);

  }

}

and I have this in the Gang Control npc

PHP Code:


		
			
public function leaderCommand(toks)

{

  switch (temp.toks[0])

  {

    case ":addmember":

      player.chat = "testing";

    break;

  }

}

When I use switch( var ), I find it helpful to do it like this:

PHP Code:


		
			
switch ( var )

{

  case "test":

  {

    // Stuff Here

    break;

  }

  case "blah":

  {

    // More stuff here.

    break;

  }

}

The braces organize it better, I think atleast.

I agree. I usually do it like that, but someone told me I didn't really need it for functionallity(sp) so this time I didn't bracket in the case's.

PHP Code:


		
			
temp.toks = player.chat.tokenize();
(@"Gang Control").leaderCommands( temp.toks[0], temp.toks[1], temp.toks[2], temp.toks[3]);

...
public function leaderCommand( command, param1, param2, param3 )
  {
  if ( ! isLeader( player.account ) )
    return;

  switch ( command )
    {
    default:
      echo( "Command ["@ command @"]: Is not recognized." );
    break;
    }
  }