Keywords in config

Very very new to java and plugins so forgive me if this is very simple.

I am learning from tutorials and have copied a "helloworld" plugin. But i want to change it slightly. This is the part i want to change

if(message_lower.contains("hi") && message_lower.contains("server")){
p.sendMessage(RED + "[server] " + WHITE + "Hello " + p.getName());
}

So what i want to do is, instead of the plugin looking for "hi" and "server", i want to create a config in which u can add keywords which it will look for, and an answer which it will send.

so config would like something like this:

Keywords: hi, server
Answer: Hello %p

Keywords: Where, sell
Answer: You can sell items at the Market. Use /warp market

and so on.

Can anyone help me please?

Try something like this:

Code:

      Map<String, Integer> m = new HashMap<String, Integer>();
@EventHandler
public void (PlayerChatEvent event) {
String keywords = plugin.getConfig().getString(Keywords);
for(String keyValue : keywords.split(",")) {
  String[] words = keyValue.split(",", 2);
  m.put(words[0], words.length == 1 ? "" : pairs[1]);
}
 
String[] message = event.getMessage().split(" ");
for (i = 0; i<message.length; i ++ ) {
if (m.contains(message[i])) {
event.getPlayer().sendMessage(plugin.getConfig().getString("Answer");
break;
}

Not sure if this works as I haven't tested it yet but that's basically what you would do.

ok, some bits in there i dont understand but will give it a try. Thanks

@the_merciless You will have to make new keyword entries for each question. For example, for your hello server option you would need "hello-server-keywords:" and "hello-server-answer".

In your code you would need these lines of code to get each keyword and set the answer:

Code:java

 
String[] keywords = plugin.getConfig().getString("hello-server-keywords").replace(" ", "").split(",");
String answer = plugin.getConfig().getString("hello-server-answer");
answer = answer.replace("%p", " + p.getName() + ");
answer.trim();
while(answer.endsWith("+")){
    answer.replace("" + answer.charAt((answer.length() -1)), "");
    answer.trim();
}
 

that way you can split each keyword into it's own argument, which then you can look for and find the keyword itself. The whole bit with the while and trimming and stuff makes sure the answer is in the right format later on.

You will need to create 2 new integers, one to reference how many arguments there are, and one to use to check which argument you're currently looking for. You'll also need a boolean to print the message once everything checks out.

Code:java

 
Integer argAmount = keywords.length;
Integer currentArg = 0;
Boolean containsAll = true;
 

you then will need a while statement to check each argument.

Code:java

 
/**this while method sets "contains all" to false if any keywords arent said*/
while(currentArg <= argAmount){
    currentArg += 1;
    if(!message_lower.contains(keywords[currentArg]))
        containsAll = false;
}
 

And finally, you need to say the message if containsAll is true

Code:java

 
if(containsAll)
    p.sendMessage(ChatColor.RED + "[server] " + ChatColorWHITE + answer);
 

Edit: i've edited this a couple times, but it should be good now. Lol didn't mean to overshadow you @r0306. This dang 502 error spree made me not see your post before I posted

@Musaddict
No problem! Glad that someone else contributed as I am not entirely sure that my code works. I'm having trouble posting this right now lol.

So the amount of Q's and A's i could have in the config would be limited to the amount entries i put in?

Correct (if I understand your question). Here's a test plugin I made. Turns out I needed to tweak a few things in my above post. Feel free to use this to your liking. Please note, this plugin is contained only in the main class, but I recommend making a new class with all your responses.

Code:java

 
 
package musaddict.serverreply;
 
 
import java.io.File;
import java.util.logging.Level;
import java.util.logging.Logger;
 
import org.bukkit.ChatColor;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.EventPriority;
import org.bukkit.event.Listener;
import org.bukkit.event.player.PlayerChatEvent;
import org.bukkit.plugin.PluginDescriptionFile;
import org.bukkit.plugin.java.JavaPlugin;
 
 
 
public class ServerReply extends JavaPlugin implements Listener {
 
    ServerReply plugin;
 
    public static PluginDescriptionFile info;
    static Logger logger = Logger.getLogger("Minecraft");
    public static final String mainDirectory = "plugins/ServerReply";
 
 
    @Override
    public void onEnable() {
        info = getDescription();
        logger = Logger.getLogger("Minecraft");
 
        Log(Level.INFO, "is enabled, version: " + info.getVersion());
        Log(Level.INFO, "written by [Musaddict]");
 
        new File(mainDirectory).mkdir(); // makes the ColorKeys directory/folder in the plugins
                                            // directory if it dosen't exist.
 
        getConfig().options().copyDefaults(true);
        saveConfig();
 
        getServer().getPluginManager().registerEvents(this, this);
    }
 
 
    @Override
    public void onDisable() {
 
        Log(Level.INFO, "was successfully disabled.");
    }
 
 
    public static void Log(String message) {
        Log(Level.INFO, message);
    }
 
 
    public static void Log(Level logLevel, String message) {
        logger.log(logLevel, "[" + info.getName() + "] " + message);
    }
 
 
    @EventHandler(priority = EventPriority.HIGH)
    public void onPlayerChat(PlayerChatEvent event) {
        serverHello(event);
        //list more methods for more responses (name them anything you want)
    }
 
 
    private void serverHello(PlayerChatEvent event) {
        Player p = event.getPlayer();
        String message_lower = event.getMessage().toLowerCase();
        String [] keywords = this.getConfig().getString("hello-server-keywords").replace(" ", "").split(",");
        String answer = this.getConfig().getString("hello-server-answer");
        Integer argAmount = keywords.length;
        Integer currentArg = 0;
        Boolean containsAll = true;
        answer = answer.replace("%p", p.getName());
        answer.trim();
        do {
            if (!message_lower.contains(keywords[currentArg]))
                containsAll = false;
            currentArg = currentArg + 1;
        }
        while (currentArg < argAmount);
        if (containsAll)
            p.sendMessage(ChatColor.RED + "[server] " + ChatColor.WHITE + answer);
    }
}
 

Code:

# ServerReply Configuration
#
# USE 4 SPACES INSTEAD OF TABS
 
hello-server-keywords: 'hello,server'
hello-server-answer: 'Hello %p'

Code:

name: ServerReply
main: musaddict.serverreply.ServerReply
version: 1.0.0 [1.2.5-R1.0]
description: reply to messages!
authors: [Musaddict]

Download test plugin

EDIT: I was attacked by a wild 502 error!! :O

Well you can make a whole list in the config, and you wont have to edit code if you want to add another word. But the answer that the server gives back in the examples given will always be the same.

Also, for convenience, this is how I would do it

Code:java

// Remember the imports! They're... IMPORTant!!! lolololol
import java.util.List;
import org.bukkit.event.player.PlayerChatEvent;
 
// ...
public void onPlayerChat(PlayerChatEvent event){
List<String> string_list = config.getStringList("PATH.TO.LIST");
String msg = event.getMessage().toLowerCase();
 
if(containsAll(msg, string_list)){
event.getPlayer().sendMessage(config.getString("PATH.TO.MESSAGE"));
}
}
 
public boolean containsAll(String string, List<String> contents){
for(String content : contents){
if(!string.contains(content)) return false;
}
return true;
}

This would reference the following config:

Code:

PATH:
    TO:
        LIST:
            - hi
            - server
        MESSAGE: "Sup, playa"

Actually, with the method I posted, you wont have to edit the code at all if you add a word, and it also allows different responses for different keywords. For ever new response you want, you will have to copy and paste the "serverHello" method, and name it to whatever you want, then register it in the onPlayerChat method.

@Musaddict, thats sill editing code. You can simply loop through the top level keys of config and then each key has a keywords item and a message item, therefore you simply add a new key in config an it shiws a new message. No need to edit plugin at all.

Wow thanks for your help guys. @Musaddict you helped alot but @ryan7745 has the idea. the whole idea of this plugin is that you can add as many questions and answers to the config as you like and the code will find it without having to edit the code each time, i thought you may be able to loop but no idea how to do it, also wanted to add a default answer if no keywords match, eg "sorry i dont have an answer to your question right now" Can you show me how i would make a loop and give a default answer?" I think i gotta watch some new tuts cos alot of this going straight over my head!

i just realised that option for a default message prob doesnt make sense. let me clear it up. i will add "katie" to all the keywords questions so when the player asks a question they will only get a response if they include the word katie. essentialy they will be asking the server a question but will be calling it katie. for example "hello katie" or Katie how can i join a town" so if they dont add katie it will not respond, and if they type katie with no other keywords it will give the default answer.

Code:java

if(args.length ==1){
if(args[0].equalsignorecase("katie"))
//do nothing we won't react on calling katie's name
}

Or maybe:

Code:java

if(args.length ==1 || args.length ==2){
if(args.contains("katie"))
//do nothing we won't react on calling katie's name
}

If this is in fact what your asking...

@Musaddict i hope this doesnt sound stupid but couldnt u just change "hello-server-keywords" to a number, make it a variable and have it ++1 if false to serch through all questions without having to code them all?

In theory it should work, but I've never done anything like that.

@the_merciless Got it working the new code you need is this:

Code:java

 
    @EventHandler(priority = EventPriority.HIGH)
    public void onPlayerChat(PlayerChatEvent event) {
        serverRespond(event);
    }
 
 
    private void serverRespond(PlayerChatEvent event) {
        Player p = event.getPlayer();
        String message_lower = event.getMessage().toLowerCase();
        Integer i = -1;
        for (i = 0; i < this.getConfig().getKeys(false).size(); i++) {
            Integer argAmount = null;
            String [] keywords = null;
            String answer = null;
            try {
                keywords = this.getConfig().getString("" + i + "k").replace(" ", "").split(",");
                answer = this.getConfig().getString("" + i + "a");
                argAmount = keywords.length;
                answer = answer.replace("%p", p.getName()).replace("%w", p.getWorld().getName());
                answer.trim();
            }
            catch (NullPointerException e) {
                continue;
            }
            Integer currentArg = 0;
            Boolean containsAll = true;
            do {
                if (!message_lower.contains(keywords[currentArg]))
                    containsAll = false;
                currentArg = currentArg + 1;
            }
            while (currentArg < argAmount);
            if (containsAll)
                p.sendMessage(ChatColor.RED + "[Server] " + ChatColor.WHITE + answer);
        }
    }
 

it searches for Xk for keywords, and Xa for answers (where X is any int less than or equal to the number of entries in the config).

Ok so i made a new class for the responses and all looked good. exported well and no errors show up. But when running the server i get an error saying it found { and expected ; Went ovet the code again and cant spot the problem. what have i done wrong?

oh cool, just saw your last comment after posting that ^ Still out of interest can u guess what i done wrong.

Probly did something like:
player.sendMessage("blah"){

idk, never seen that error before.

Edit: Also, I realized something that would improve efficiency of the code I posted. Change this line:

Code:java

 
for (i = 0; i < this.getConfig().getKeys(false).size(); i++) {
 

to this:

Code:java

 
for (i = 0; i < Math.ceil(this.getConfig().getKeys(false).size()/2)+1; i++) {
 

That way it only searches for half the amount of entries (which when you think about it, there's double the number of entries to start, 1 for keywords, 1 for answers), so it still searches all of them, but it doesn't search for ones that don't exist.

I got it working, turned out to be a problem with the names of the package/class not sure why it gave that error tho.
Anyway this is working PERFECTLY and i cant thank you enough for your help. Although i ended up copying your code i learnt alot from this thread, trying to work out what u guys had done and how it works etc etc. Oh 1 minor problem i got is the answer appears on screen above what they asked. Is it possible to add a 1 second delay or something?

change the "if(containsAll){" to this:

Code:java

 
            if (containsAll){
                final String answer2 = answer;
                final Player p2 = p;
                this.getServer().getScheduler().scheduleSyncDelayedTask(this, new Runnable() {
                    public void run() {
                        p2.sendMessage(ChatColor.RED + "[Server] " + ChatColor.WHITE + answer2);
                    }
                }, 10L);
            }
 

the "10L" at the end is "10 ticks". 20 ticks is 1 second, so this is a 0.5 second delay. Glad to help

Edited, take a second look over what I changed.

Can I use || in the config. For example 1k: can||could,I,be,op or Is there another way to do this.

I bet there's some way to do an OR method, but I think it would be a huge work around. At this point I think it'd be simpler to:
1) copy and paste (with the other word)
or
2) include both words in the same line

Idk tho. I've never thought to lookup strings with OR methods.

1 last thing. tried to add a reload command for when i have made changes to the config and failed miserably.

Code:

public boolean onCommand(CommandSender sender, Command cmd, String commandLabel, String[] args){
        if(cmd.getName().equalsIgnoreCase("ask reload")){
            this.saveConfig();
            this.reloadConfig();
            sender.sendMessage(ChatColor.DARK_RED + "ASKKATIE has been reloaded");
            return true;
        }
        return false;
        }

im not sure if saveconfig then reload config is right. The command doesnt even work.

Try just reloading the config. What I think saving does is put the temporary values into the config.

Ergo, if an option was originally FALSE (and it loaded FASLE), and you change the value manually to TRUE, then you save the config, it overwrites the TRUE back to FALSE, then loads the FALSE again, lol.