В чем подвох и чем моё решение может быть не правильным?

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

Решен

Я написал своё решение и из любопытства посмотрел на готовое решение. Они отличаются, но валидатор пропустил. Я попробовал разные кейсы прогнать и вроде всё ок. Возможно я не вижу чем моё решение хуже и когда оно неправильно работает

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

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

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

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

Anonymous #3131865

Уровень 15

4 сентября 2022, 13:44

Тоже вполне рабочий вариант public static void paySalary(String name) { //напишите тут ваш код for (int i = 0; i < waitingEmployees.size(); i++) { if (waitingEmployees.contains(name)) { alreadyGotSalaryEmployees.add(name); waitingEmployees.set(i, null); } } }

Стас Пасинков Software Developer в Zipy

Master

10 августа 2022, 21:50

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

Haivvatha Пытаюсь переобуться в QA в Секрет

11 августа 2022, 12:04

Хорошо, спасибо. Просто подумал, что моё решение не будет работать в каком-то случае и хотел это узнать

An DongMeng Java Developer

9 августа 2022, 09:55

Не вижу формулировку задачи и остальной код, но сразу могу сказать: если в name null (waitingEmployees.contains(name) && name != null) выдаст NullPointerException (name != null && waitingEmployees.contains(name)) вернет false

Haivvatha Пытаюсь переобуться в QA в Секрет

10 августа 2022, 12:28

Формулировку и тд не дало прикрепить тк я её уже решил. Но вот условие.

Кейс с name = null я тоже проверял. Эксепшенов не было. Вот весь код

public class Solution {
    public static ArrayList<String> waitingEmployees = new ArrayList<>();
    public static ArrayList<String> alreadyGotSalaryEmployees = new ArrayList<>();

    public static void initEmployees() {
        waitingEmployees.add(null);
        waitingEmployees.add("Гунигерд");
        waitingEmployees.add("Боргелейф");
        waitingEmployees.add("Нифрод");
        waitingEmployees.add("Альбиуф");
        waitingEmployees.add("Иногрим");
        waitingEmployees.add("Фриле");
    }

    public static void main(String[] args) {
        initEmployees();
        paySalary(null);
    }

    public static void paySalary(String name) {
        if (waitingEmployees.contains(name) && name != null) {
            if (!(alreadyGotSalaryEmployees.contains(name)) && name != null) {
                alreadyGotSalaryEmployees.add(name);
                waitingEmployees.set(waitingEmployees.indexOf(name), null);
            }
        }
        System.out.println(waitingEmployees);
        System.out.println(alreadyGotSalaryEmployees);
    }
}

Alejandro Antoniuk

Уровень 34

19 августа 2022, 13:53

public static void paySalary(String name) {
		if (name != null && waitingEmployees.contains(name) && !alreadyGotSalaryEmployees.contains(name)) {
			alreadyGotSalaryEmployees.add(name);
			waitingEmployees.set(waitingEmployees.indexOf(name), null);
		}
	}