CommandExecutor

[Mojuuu]

    private static CommandMap cmap;
    private String command;
    private GroupType type;
    private List<String> alias;

    public CustomCommand(final String command) {
        this(command, null, null);
    }

    public CustomCommand(final String command, final GroupType type) {
        this(command, null, type);
    }

    public CustomCommand(final String command, final String[] aliases) {
        this(command, new ArrayList<>(Arrays.asList(aliases)));
    }

    public CustomCommand(final String command, final String[] aliases, final GroupType type) {
        this(command, new ArrayList<>(Arrays.asList(aliases)), type);
    }

    private CustomCommand(final String command, final List<String> aliases, final GroupType type) {
        this.command = command.toLowerCase();
        this.alias = aliases;
        this.type = type;
        register();
    }

    private CustomCommand(final String command, final List<String> aliases) {
        this.command = command.toLowerCase();
        this.alias = aliases;
        this.type = null;
        register();
    }

Ktoś mi powie co robię źle? https://scr.hu/1pqBr4

[MarcinWieczorek]

Jakbyś dał cast nulla na List to pewnie by poszło. Jak dasz dwa nulle to skąd wiadomo o który konstruktor chodzi? Chociaż z tego co pamiętam to javac ma inny error na taką sytuację. Ja bym dał ten field alias jako final i od razu tworzył tę listę a w konstruktorach (w this konkretnie) przekazywał jakąkolwiek listę.

[Mojuuu]

Jakbyś dał cast nulla na List to pewnie by poszło. Jak dasz dwa nulle to skąd wiadomo o który konstruktor chodzi? Chociaż z tego co pamiętam to javac ma inny error na taką sytuację. Ja bym dał ten field alias jako final i od razu tworzył tę listę a w konstruktorach (w this konkretnie) przekazywał jakąkolwiek listę.

Rzeczywiście dając cast na nulla poszło, dzięki. Mam jeszcze pytanie. Co masz na myśli aby przekazywać jakąkolwiek listę?

[MarcinWieczorek]

this(str, new ArrayList<>(), cośtam)
this(str, Collections.EMPTY_LIST, cośtam)

W drugim przypadku unikamy tworzenia listy która i tak nie jest używana i dajemy mocno do zrozumienia, że jest to lista przekazana tylko dla ładu i składu.

[Mojuuu]

Dzięki, do zamknięcia.