Why isn't this working?

Hey everyone,

Really simple problem here, I'm probably just being a bit dozy and missing out something, but as far as I can tell - this should work..

Code:java

@EventHandler
    public void onCommandEvent(PlayerCommandPreprocessEvent event) {
        System.out.println("Matches: " + event.getMessage().startsWith("/test")); //just a debug statement
        if(!(event.getMessage().startsWith("/test"))) {
            event.setCancelled(true);
            event.getPlayer().sendMessage("Your command wasn't /test!");
        }
    }

The problem is, even when I do /test, it still says the command isn't /test. I put that return statement there which returns whether it matches or not, and it returns true whenever I do /test - so I don't see why it doesn't work when it comes to the if statement.

Thanks in advance!

 

remove the / in if(!(event.getMessage().startsWith("/test"))){
so its startsWith("test"))) {

not sure but the think the e.getMessage() returns the string without the "/"

 

Nah, it definitely does pick-up on the /, I've been testing this out for a while :/

 

Maybe instead of startsWith try contains?

 

That seems to work great, thanks for suggesting that!

Another issue I've just encountered is, for some reason it doesn't seem to work on multiple commands, so for example:

Code:java

@EventHandler
public void onCommandEvent(PlayerCommandPreprocessEvent event) {
if(!(event.getMessage().contains("/test1")) || !(event.getMessage().contains("/test2"))) {
event.setCancelled(true);
event.getPlayer().sendMessage("Your command wasn't /test1 or test2!");
}
}

Any ideas anyone?

 

i dont know why that wouldnt work, but just try doing them in different if clauses?

 

Tried else if and just normal if, and it doesn't work.

 

You use OR where AND should be.

 

Try using equalsIgnoreCase instead of contains

Code:java

@EventHandler
public void onCommandEvent(PlayerCommandPreprocessEvent event) {
if(!(event.getMessage().equalsIgnoreCase("/test1")) || !(event.getMessage().equalsIgnoreCase("/test2"))) {
event.setCancelled(true);
event.getPlayer().sendMessage("Your command wasn't /test1 or test2!");
}
}