Help Search

[multiverse]

Help Search

[AkiTensai]

Option 1: Search text is only ever a single word. Page number set second.
Con: goes against what the current system uses of int after help.

//To handle the following:
///mv help
///mv help [text]
///mv help [text] [number]
//where [Text] is a string without spaces and [number] is an int or treated as 1 when invalid.

String sSearchTerm;
int iPage;

if (args.size() == 0)
{
    sSearchTerm = "";
    iPage = 1;            
} else if (args.size() == 1) {
    sSearchTerm = args.get(1);
    iPage = 1;
} else if (args.size() == 2) {
    sSearchTerm = args.get(1);
    try {
         iPage = args.get(2);
    }
    catch (NumberFormatException ex) {
        iPage = 1;
    }
}

//At this point, iPage and sSearchTerm are set.
//[Insert search for relevant return results.  

[AkiTensai]

Option 2: Search text may be more then one word.
I don't know the syntax for getting the array of args for the foreach. From what I've read, the continue should be correctly written.

//To handle the following:
///mv help
///mv help [number]
///mv help [number] [text]
///mv help [text]
//where [Text] is a string that may contain spaces and [number] is an int or treated as 1 when invalid.

String sSearchTerm;
int iPage;
boolean isPageProvided = false;

if (args.size() == 0)
{
    sSearchTerm = "";
    iPage = 1;            
} else {

    //Get the page number.  If it is a string, then it is a search
    //term and paging defaults to first page.
    try {
         iPage = args.get(0);
         isPageProvided = true;
    }
    catch (NumberFormatException ex) {
        iPage = 1;
        isPageProvided = false;
    }

    //Bring all of the search terms together, skipping the page number if
    //it was provided.
    boolean isFirstArg = true;
    for(string sArg = args.getarray())
    {
        if(isFirstArg) {
            isFirstArg = false;

            if(isPageProvided) {
                //When page is provided, skip the first arg, as it a number
                //instead of a search term.
                continue;
            }
        } else {
            //Each additional search term needs spaced from the last.
            sSearchTerm += " ";
        }
        sSearchTerm += sArg;
    }

}

//At this point, iPage and sSearchTerm is set and ready to go.

[fernferret]

Ah yea, I was quite tired last night. If you look at the Info Command one of the params is WORLD, which if not provided is determined IF the sender was a player. That's why it was hairy... Since you've basically done all the work... I'll go add this tonight!

[AkiTensai]

Updated the comment. I was using base 1 instead of base 0 for the args.

//Was
iPage = args.get(1);

//Now
iPage = args.get(0);

If you ever want help hammering out variable validations from commands, ping me.

[fernferret]

Finished in: dbdc6d9