Помогите Проверить правильность написания профессии у жителей.

[Lisenochek]

Народ, мне необходимо сделать проверку на профессии жителей. Например, если в команде будет второй аргумент равен BUTCHER, то будет спавнится мясник, но если будет неправильное название профессии, например BYTHCER, то проверка сработает и игроку выведется сообщение о неправильном аргументе. Пытался сделать foreach, но не помогло, проверка постоянно срабатывала, даже если профессия была написана правильно. Помогите мне!

 

[Lisenochek]

Вот так делал форич:

                for (Villager.Profession prof : Villager.Profession.values()) {

                    if (!args[2].equalsIgnoreCase(prof.name())) {
                        p.sendMessage(C.getEntityPrefix() + C.c("&cПрофессии &6\"&7" + args[2] + "&6\" &cне существует!"));
                        p.sendMessage("");
                        p.sendMessage(C.c("&aСписок всех профессий:&7 BLACKSMITH, BUTCHER, FARMER, HUSK, LIBRARIAN, NITWIT, NORMAL, PRIEST&a!"));
                        return true;
                    }
                }

 

[Fenix78563]

Вот так делал форич:

                for (Villager.Profession prof : Villager.Profession.values()) {

                    if (!args[2].equalsIgnoreCase(prof.name())) {
                        p.sendMessage(C.getEntityPrefix() + C.c("&cПрофессии &6\"&7" + args[2] + "&6\" &cне существует!"));
                        p.sendMessage("");
                        p.sendMessage(C.c("&aСписок всех профессий:&7 BLACKSMITH, BUTCHER, FARMER, HUSK, LIBRARIAN, NITWIT, NORMAL, PRIEST&a!"));
                        return true;
                    }
                }

конечно, у тебя есть n профессий, и ты сравниваешь аргумент с ними в n-1 случаях он не совпадет и тебе скажут что ты питух.

 

[Lisenochek]

конечно, у тебя есть n профессий, и ты сравниваешь аргумент с ними в n-1 случаях он не совпадет и тебе скажут что ты питух.

Ты хоть сам понял, что сказал?

 

[Fenix78563]

Ты хоть сам понял, что сказал?

АА боже.

У тебя есть сколько то проффесий, пусть семь. теперь у тебя есть аргумент профессия (допустим в списке профессий такая есть и она не первая).

Запускаем форку.

При проходе форки первый раз у тебя проффесия не совпадет с аргументом, ибо он отвечает не за ту профессию.

Все, пролет.

Не умеешь понимать алгоритмы, не берись.

List<String> profs = new ArrayList<>();
for (Villager.Profession prof : Villager.Profession.values()) {
    profs.add(prof.name().toLowerCase());
}
if (!profs.contains(args[2].toLowerCase())){
    //профессии не существует
}

 

[xDark]

АА боже.

У тебя есть сколько то проффесий, пусть семь. теперь у тебя есть аргумент профессия (допустим в списке профессий такая есть и она не первая).

Запускаем форку.

При проходе форки первый раз у тебя проффесия не совпадет с аргументом, ибо он отвечает не за ту профессию.

Все, пролет.

Не умеешь понимать алгоритмы, не берись.

List<String> profs = new ArrayList<>();
for (Villager.Profession prof : Villager.Profession.values()) {
    profs.add(prof.name().toLowerCase());
}
if (!profs.contains(args[2].toLowerCase())){
    //профессии не существует
}

valueOf(..) ? Неее, зачем нам это

 

[Lisenochek]

valueOf(..) ? Неее, зачем нам это

Пробовал, ничего.

 

[Fenix78563]

valueOf(..) ? Неее, зачем нам это

ну человеку нужно name. в чем проблем то
Тему закрывать то будем, или как?

 

[alexandrage]

valueOf(..) ? Неее, зачем нам это

И ты отхватишь нпе.

 

[Fenix78563]

И ты отхватишь нпе.

ну так то try catch вполне удобен

 

[alexandrage]

ну так то try catch вполне удобен

Да удобен. Если не забыть toUpperCase().

 

[Fenix78563]

Да удобен. Если не забыть toUpperCase().

ну главные скиллы программиста - понимать что он хочет, понимать какой алгоритм сделает это, понимать почему твой алгоритм не сработал, и не забывать про такие мелочи