Не говнокод ли?

Вопрос по задаче

Решен

В правильном решении много проверок и тд, но мое решение тоже прошло, мне интересно насколько это читаемое решение, может валидатор принял, но с решением все таки что то не так, слишком просто выглядит. if(name != null && waitingEmployees.contains(name)){ alreadyGotSalaryEmployees.add(name); waitingEmployees.set(waitingEmployees.indexOf(name),null); }

Комментарии (5)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

Dmitry

Уровень 19

5 декабря 2022, 13:13

у меня принял такое решение... if (waitingEmployees.contains(name) { alreadyGotSalaryEmployees.add(name); waitingEmployees.set(waitingEmployees.indexOf(name), null); }

very junior java developer

Уровень 51

23 ноября 2022, 19:08

полезный

Твой код хорош. Ты только одно условие пропустил

!alreadyGotSalaryEmployees.contains(name)

у меня вот так было.... в одну строку не писал, ибо слишком длинная строка получается, проверку на null вынес отдельно

if (name != null){
    if(!alreadyGotSalaryEmployees.contains(name) && waitingEmployees.contains(name)){
        alreadyGotSalaryEmployees.add(name);
        waitingEmployees.set(waitingEmployees.indexOf(name), null);
    }
}

Dmitriy Kvitka

Уровень 31

14 января 2023, 17:21

А зачем проверять наличие параметра в alreadyGotSalaryEmployees? В требованиях ведь нет такого. А в остальном решил идентично.

Tace HexJava Developer в РЦИТ

27 февраля 2023, 11:05

Человеческий фактор скорее. Может произойти такая ситуация, что человек одновременно получил зарплату и не получил. Мы заранее проверяем это, дабы уйти от лишних ошибок в будущем.

RIBERIOORIX

Уровень 15

26 сентября 2023, 03:12

Зачем вообще проверять на null? Если У переменной совпало имя, то там точно уже не null, и наоборот. Это взаимоисключаемые вещи. Решил без этой странной проверки и все получилось