Jump to content
  • 0
Sign in to follow this  
asdfycacaa

Help AdminCommandHandler

Question

Posted (edited)

Hello i have a error when i use //create_fake_players command, anyone can help me a little bit please?

the problem is the array is outofbounds.

 

Error:

 errorconsole.png.6213cfc5520e9e733158cdd2cf7ffcad.png

 

AdminFakePlayers.java:

adminfakeplayers.png.b9514961484b0fd72a45d924daeaa6a7.png

AdminCommandHandler.java

 

admincommandhandler.png

 

 

Edited by asdfycacaa

Share this post


Link to post
Share on other sites

5 answers to this question

Recommended Posts

  • 0

debug wordlist array in order to see what strings are saved. print the size or loop it

Share this post


Link to post
Share on other sites
  • 0
Posted (edited)

<= check that, also arrays start on 0 and why you start from index 1 instead of 0?

also a debug on the command as it comes print it before split after split look at your values until the value comes to the error line so you can control the output, thats called debug

Edited by Nightw0lf

Share this post


Link to post
Share on other sites
  • 0
Posted (edited)

The whole structure must be change. As I can see from he's code

wordlist[0] is the gm command and the wordlist[1] is the value. Its like 

{"Admin_gmspeed"}

{"4"}

Thats why it takes direclty pos[1]

So. If im not mistaken you forgot to write your value? 

//admin_fake_players 50

 

 

try this one:

 

switch(command)
{
	case admin_create_fake_players:
		if (wordList[1] == null || wordList[1].isEmpty())
		{
			activeChar.sendMessage("create_fake_players [val]");
			return false;
		}
		int count = 0;
		try
		{
			count = Integer.parseInt(wordList[1]);
		}
		catch(NumberFormatException e)
		{
			activeChar.sendMessage("value must be a number.");
		}
		for (int i =0; i< count; i++)
			new FakeGameClient(null);
		break;
}
return true;
                             
                             
		
		

in case you dont want to add more commands here, remove the whole switch case.... If statement is enough

 

Edited by melron

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
Sign in to follow this  

  • Recently Browsing   0 members

    No registered users viewing this page.



×