Java method arguments not applicable

Question

First, appologies for the lack of a better title. Hopefully someone with more experience in Java is able to change it to something more appropiate.

So I'm faced with the following exception:

The method copyPartialMatches(String, Iterable, T) in the type StringUtil is not applicable for the arguments (String, String[], List)

The documentation for this method states:

Parameters:
token - String to search for
originals - An iterable collection of strings to filter.
collection - The collection to add matches to

My code:

public class TabHandler implements TabCompleter {
    private static final String[] params = {"help", "menu", "once", "repeat", "infinite", "cancel"};

    @Override
    public List<String> onTabComplete(CommandSender sender, Command command, String alias, String[] args) {
        final List<String> completions = new ArrayList<>();
        StringUtil.copyPartialMatches(args[0], params, completions);

        Collections.sort(completions);
        return completions;
    }
}

I'm fairly certain the problem lies with the completions List. Perhaps this is not a valid collection? It was my understanding that it was, but right now I'm just at a loss here. So hopefully you guys can help me out.

score 2 · Accepted Answer · answered Aug 12 '19 at 01:04

Try passing in an actual List as the second parameter to StringUtil#copyPartialMatches:

@Override
public List<String> onTabComplete(CommandSender sender, Command command, String alias, String[] args) {
    final List<String> completions = new ArrayList<>();
    StringUtil.copyPartialMatches(args[0], Arrays.asList(params), completions);
    //                                     ^^^ change is here

    Collections.sort(completions);
    return completions;
}

Thank you! That solved the issue straight away ^^ Will accept when able to :) — icecub, Aug 12 '19 at 01:07

score 1 · Answer 2 · answered Aug 12 '19 at 01:04

1

String[] isn't Iterable. Change

StringUtil.copyPartialMatches(args[0], params, completions);

to pass a List<String> instead. Something like,

StringUtil.copyPartialMatches(args[0], Arrays.asList(params), completions);

answered Aug 12 '19 at 01:04

Elliott Frisch

198,278
20
158
249

Thanks, this indeed solved the issue. I will accept Tim Biegeleisen's answer because he was slightly faster. But both your answers are equaly as good :) – icecub Aug 12 '19 at 01:08
@icecub No problem. Glad the issue was resolved. – Elliott Frisch Aug 12 '19 at 01:11

score 1 · Answer 3 · answered Aug 12 '19 at 04:13

1

Why is an array not assignable to Iterable?

Arrays don't implement the Iterable interface. That's why the method signature isn't matching. Converting the array to a list works since the latter implements the interface.

The forEach loop for arrays is a special case (arrays don't implement the Iterable<T> interface yet the work with the forEach loop).

answered Aug 12 '19 at 04:13

Prashant Pandey

4,332
3
26
44

Thanks, I'm sure this information will be usefull to future readers – icecub Aug 12 '19 at 04:31

Java method arguments not applicable

3 Answers3