JavaRush/Помощь по Java задачам/3е условие

16 уровень

08.06.2022
447просмотров
22комментариев

3е условие

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

Решен

не работает

В классе Solution есть список сотрудников waitingEmployees, которые ждут зарплату, и список сотрудников alreadyGotSalaryEmployees, которые её уже получили. Необходимо реализовать метод paySalary(String), который в качестве аргумента принимает имя сотрудника, желающего получить зарплату.
Метод должен проверить наличие сотрудника в списке waitingEmployees, и если он не получал зарплату, выдать её:

Если параметр метода paySalary(String) равняется null, ничего не нужно делать.
Добавить имя сотрудника в список alreadyGotSalaryEmployees.
В списке waitingEmployees заменить имя сотрудника на null.

Требования:

Метод paySalary(String) должен добавлять в список alreadyGotSalaryEmployees имя сотрудника, если он есть в списке waitingEmployees.
Метод paySalary(String) должен заменять имя сотрудника на значение null, если он есть в списке waitingEmployees.
Метод paySalary(String) не должен изменять списки waitingEmployees и alreadyGotSalaryEmployees, если переданного имени нет в списке waitingEmployees.
Если в метод paySalary(String) передается null, метод не должен добавлять это значение в waitingEmployees или alreadyGotSalaryEmployees.

package com.javarush.task.pro.task12.task1209; import java.util.ArrayList; /* Бухгалтерия */ public class Solution { public static ArrayList<String> waitingEmployees = new ArrayList<>(); public static ArrayList<String> alreadyGotSalaryEmployees = new ArrayList<>(); public static void initEmployees() { waitingEmployees.add("Гвинно"); waitingEmployees.add("Гунигерд"); waitingEmployees.add("Боргелейф"); waitingEmployees.add("Нифрод"); waitingEmployees.add("Альбиуф"); waitingEmployees.add("Иногрим"); waitingEmployees.add("Фриле"); } public static void main(String[] args) { initEmployees(); System.out.println(waitingEmployees); System.out.println(alreadyGotSalaryEmployees); } public static void paySalary(String name) { for (int i = 0 ;i < waitingEmployees.size();i++) { if(name != null ) { name = waitingEmployees.get(i); alreadyGotSalaryEmployees.add(name); waitingEmployees.set(i, null); } } } }

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

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

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

Виктор

Уровень 45

8 июня 2022, 16:04

Вы в этом методе получаете на вход String name, потом в цикле (цикл то зачем?) переопределяете ее name = waitingEmployees.get(i); зачем то, и дальше с этим значением работаете, так неправильно и незабывайте что это Arraylist, а не String[], методы немножко другие

public static void paySalary(String name) {
    for (int i = 0 ;i < waitingEmployees.size();i++) {
        if(name != null  ) {
        name = waitingEmployees.get(i);
        alreadyGotSalaryEmployees.add(name);
        waitingEmployees.set(i, null);
        }
    }
}

Dexter Dangerous

Уровень 16

8 июня 2022, 16:06

Понял ,спасибо ,как дальше подправить этот код ? if(name != null && waitingEmployees.contains(name)) { alreadyGotSalaryEmployees.add(name); waitingEmployees.set(i, null); }

Виктор

Уровень 45

8 июня 2022, 16:17

если именно этот код, то там цикл не нужен - contains определяет наличие и i должно быть определено indexOf

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

Виктор

Уровень 45

8 июня 2022, 15:57

решение

if(name==null)return;
if(waitingEmployees.contains(name)){
    alreadyGotSalaryEmployees.add(name);
    int index = waitingEmployees.indexOf(name);
    waitingEmployees.set(index,null);
}

ну как бы думаю тут все понятно

Dexter Dangerous

Уровень 16

8 июня 2022, 16:02

Виктор ,а на примере моего кода можете сделать что бы подошло под условие ,без index.Of ? if(name != null && waitingEmployees.contains(name)) { alreadyGotSalaryEmployees.add(name); waitingEmployees.set(i, null); }

Виктор

Уровень 45

8 июня 2022, 16:11

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

for (int i = 0 ;i < waitingEmployees.size();i++) {
    if(name != null  ) {
    String nameForTest = waitingEmployees.get(i);
    if(nameForTest.equals(name)){
           alreadyGotSalaryEmployees.add(name);
           waitingEmployees.set(i, null);
     }
    }
}

Dexter Dangerous

Уровень 16

9 июня 2022, 09:25

Спасибо ,как думаете теперь будет ? жаль они про indexOf не написали в лекции ....... public static void paySalary(String name) { for (int i = 0 ;i < waitingEmployees.size();i++) { if(name != null && waitingEmployees.contains(name)) { int position = alreadyGotSalaryEmployees.indexOf(name); alreadyGotSalaryEmployees.add(name); waitingEmployees.set(position, null); } } }

Виктор

Уровень 45

9 июня 2022, 12:42

решение

Вот смотрите:

int position = alreadyGotSalaryEmployees.indexOf(name);

индекс чего Вы хотите получить? и главное где? по факту это выражение в принципе не имеет смысла т.к элемент name Вы добавляете следующей строчкой, но что самое главное Вы не в том списке ищете Вам нужен индекс name в списке waitingEmployees чтобы после выдачи зп обнулить имя в нем исправьте на

int position = waitingEmployees.indexOf(name);

и все будет отлично работать

Dexter Dangerous

Уровень 16

9 июня 2022, 13:15

Спасибо , совсем мысли в кучу собрать не могу , спасибо за уделенное время и помощь !!!!!!

Ada

Уровень 1

8 июня 2022, 14:31

У вас нет проверки на то, есть ли переданный аргумент в списке, у вас только проверка на null

Dexter Dangerous

Уровень 16

8 июня 2022, 14:35

if(name != null && waitingEmployees.contains(name) { } Если я добавляю такую проверку , то 3ий и 4ый пункт засчитываются , но 1ый и 2ой нет

Ada

Уровень 1

8 июня 2022, 14:45

А строка 32 вам для чего? Допустим, у вас такой код:

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

Вы проверили, что переданное имя в списке есть (переменная name), а затем присваиваете в name значение из waitingEmployees.get(i)

Денис Enterprise Java Developer

8 июня 2022, 14:56

Ваша ошибка в том, что вы просто берете первое попавшееся значение из массива waiting и кладете его в already. Строка 32.

public static void paySalary(String name) {
        if (name != null) {
            for (int i = 0; i < waitingEmployees.size(); i++) {
                if (name.equals(waitingEmployees.get(i))) {
                    alreadyGotSalaryEmployees.add(name);
                    waitingEmployees.set(i, null);
                }
            }
        }
    }

Dexter Dangerous

Уровень 16

8 июня 2022, 15:44

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

Dexter Dangerous

Уровень 16

8 июня 2022, 15:49

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

Ada

Уровень 1

8 июня 2022, 16:13

Вы можете посмотреть правильный код нажав на "показать правильное решение" в задаче. однако все же попробуйте посмотреть на 32-ую строку и разобраться, что она сейчас делает, и почему это противоречит условию задачи

Dexter Dangerous

Уровень 16

8 июня 2022, 16:16

Да ,спасибо ,что name я зачем то менял я понял. к сожалению ,случайно ткнул при просмотре верного решения не на запуск а на проверку и ответ засчитали ,теперь не могу проверить свой .... как думаете ,так верно ? if(name != null && waitingEmployees.contains(name)) { alreadyGotSalaryEmployees.add(name); waitingEmployees.set(i, null); }

Виктор

Уровень 45

8 июня 2022, 16:21

нет, не правильно, если там еще цикл вверху, то просто обнулите waitingEmployees, а если цикла нет, то что такое переменная i ? contains уже определит наличие, цикл не нужен

Денис Enterprise Java Developer

8 июня 2022, 16:56

Как будет правильно я написал в том же сообщении что и свой предыдущий комментарий.

Ada

Уровень 1

8 июня 2022, 17:27

Этот код неверный:

public static void paySalary(String name) {
    for (int i = 0 ;i < waitingEmployees.size();i++) {
        if(name != null && waitingEmployees.contains(name))  {
            alreadyGotSalaryEmployees.add(name);
            waitingEmployees.set(i, null);
        }
    }
}

Так как waitingEmployees.contains(name) проверяет, если ли в списке значение name, то вот, что у вас получается:

public static void paySalary(String name) {
    for (int i = 0 ;i < waitingEmployees.size();i++) {
        if(name != null && waitingEmployees.contains(name))  { // если в списке есть переменная name (на любой позиции)
            alreadyGotSalaryEmployees.add(name); // добавить имя в другой список
            waitingEmployees.set(i, null); // заменить элемент i на null
        }
    }
}

Обратите внимание на последнюю строчку. Что там за элемент i? Там может быть любой элемент.

Ada

Уровень 1

8 июня 2022, 17:27

Предположим, у вас есть список waitingEmployees с такими именами: Алексей, Иван, Ольга. Вы используете ваш метод и передаете туда значение "Ольга". Вот, что будет внутри метода:

for (int i = 0 ;i < waitingEmployees.size();i++) { // здесь i равно 0
        if(name != null && waitingEmployees.contains(name))  { // имя "Ольга" в списке есть, значит заходим внутрь условия
            alreadyGotSalaryEmployees.add(name); // добавляем имя "Ольга" в список alreadyGotSalaryEmployees
            waitingEmployees.set(i, null); // берем из списка waitingEmployees элемент под индексом 0 и меняем его на null
        }
    }
}

Опять внимание к последней строчке: waitingEmployees.set(i, null); // берем из списка waitingEmployees элемент под индексом 0 и меняем его на null элемент с индексом 0 — это имя "Алексей". Его вы меняете на null. А затем и все остальные элементы (просто меняйте значение переменной i, и смотрите, как дальше будет работать код)

Dexter Dangerous

Уровень 16

9 июня 2022, 09:24

Ada ,я попробовал вот так ,к сожалению не могу проверить,как думаете будет работать ? public static void paySalary(String name) { for (int i = 0 ; i < waitingEmployees.size(); i++) { if(name != null && waitingEmployees.contains(name)) { int position = alreadyGotSalaryEmployees.indexOf(name); alreadyGotSalaryEmployees.add(name); waitingEmployees.set(position, null); } } }