Stopping players from using all caps in a message.

Note: This may or may not belong in resources? Oh well.

This code is for people who want to check if a player used x amount of all capital words in a message.
And then do something else.

Code:java

 
@EventHandler
public void onChat(PlayerChatEvent event){
     String m = event.getMessage();
     String[] message = m.split(" ");
         if(message.length < 2)return;
         int x = (YOUR INT HERE);
              if(getCapsSize(message) > x){
              //DO STUFF
              //ex:
               event.getPlayer().sendMessage(ChatColor.RED + "Please don't use so many caps");
            event.setMessage(m.toLowerCase());
       }
}
 
private int getCapsSize(String[] args){
     int i = 0;
     for(String string: args){
         if(string.equals(new String(string.toUpperCase()))){
             i++;
         }
     }
return i;
}
 

 

Someone requested something similar a while back
http://forums.bukkit.org/threads/chat-propercaps.93842/#post-1290765

 

Your code could actually be optimized as well!

Code:

private int getCapsSize(String... args) { // This allows both String[] and arg1, arg2, arg3, arg4
    int i = 0; // Just use a counter instead of a list, as this takes less resources
    for(String s : args) {
        if(string.equals(string.toUpperCase())) // Shouldn't need "new String()"
            i++;
    }
   
    return i;
}

 

You actually do need new String() because i tried

Code:java

 
if(string.eqauls(string.toUpperCase()){
getLogger().info("true");
}else{
getLogger().info("false");
}

and it said false, so i tried new String(string.toUpperCase()); and it said true

 

Interesting... Doesn't mean my code is not slightly more optimized though!

 

it is, i didn't think about that, im gonna edit the post so people dont need to scroll down to see it Thanks Giant

 

AsyncPlayerChatEvent event

made something similar 2 days ago:

Code:java

public boolean capsCheck(String message, AsyncPlayerChatEvent evt,
String player) {
boolean capCheck = false;
int counter = 0;
 
if(message.length() > 3) {
 
for(int i = 3; i<message.length(); i++) {
if(Character.isUpperCase(message.charAt(i)) && Character.isLetter(message.charAt(i))) {
counter++;
}
}
 
if(counter > (message.length()*0.30)) {
capCheck = true;
evt.getPlayer().sendMessage(
ChatColor.DARK_RED + "You are now muted");
message = message.toLowerCase();
char[] msg = message.toCharArray();
msg[0] = Character.toUpperCase(msg[0]);
msg.toString();
message = new String(msg);
evt.setMessage(message);
}
}
 
return capCheck;
}

checks if there is more then 30% caps in the String and returns if the player used more caps then the limit and mutes him for 10 seconds on another class(that manages all my 'guards'[curses,links, etc]). Also prints the message, just lowercased and first letter capitalized.

btw, are there any better ways to do it?

 

This part seems quite useless to me... What .toString() does is return a String, not change something internally into a String.
Therefore doing "message = msg.toString();" should work just as fine

 

Yeah someone told me this yesterday, i forgot to change it

 

That would've been me