Что не так?

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

Архивный

Требования: В классе UniversityGroup должно быть публичное не статическое поле students типа List<String>. Оно есть. Что не так?

У нас есть группа в университете и метод, который должен отчислять конкретного студента, но он почему-то не работает.
Твоя задача — найти ошибку и исправить, чтобы метод exclude смог отчислить студента.

Требования:

В классе UniversityGroup должно быть публичное не статическое поле students типа List<String>.
В классе UniversityGroup должен быть публичный не статический метод exclude(String) с типом возвращаемого значения void.
Метод exclude должен удалять из списка переданного студента.

package com.javarush.task.pro.task11.task1123; import java.util.ArrayList; import java.util.List; /* Почувствуй себя деканом */ public class UniversityGroup { public List<String> students = new ArrayList<>(); public UniversityGroup() { students.add("Сергей Фрединский"); students.add("Виталий Правдивый"); students.add("Максим Козыменко"); students.add("Наталия Андрющенко"); students.add("Ира Величенко"); students.add("Николай Соболев"); students.add("Снежана Слободенюк"); } public void exclude(String excludedStudent) { students.removeIf(student -> student.equals(excludedStudent)); } public static void main(String[] args) { UniversityGroup universityGroup = new UniversityGroup(); universityGroup.exclude("Виталий Правдивый"); universityGroup.students.forEach(System.out::println); } }

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

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

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

Володимир Вікторович Надточей

Уровень 23

19 января 2021, 08:18

Мне помогла эта статья https://javarush.com/groups/posts/1935-udalenie-ehlementa-iz-spiska-arraylist Правда решил ее не так как в варианте с правильным решением. Просто использовал цикл for each что проверить есть ли в списке нужный мне элемент и за пределами цикла удалил его с помощью метода remove(). Сложность єтой задачи в том что методы remove() или removeIf() нельзя использовать во время перебора элементов. Потому я удалил просто за пределами цикла. В статье также немного описывается класс Класс Iterator, который позволяет удалять элементы во время перебора. К слову списки и работа с ними будут изучаться в моем курсе только на следующем уровне, а цикл for each через два уровня.

Виталий Прончатов

Уровень 6

23 ноября 2020, 20:40

Пытался решить эту задачу дня два, искал информацию, как ее можно решить. В итоге так и не получилось. Посмотрел решение и понял, что для моего уровня это слишком сложно и информации в текстовом уроке явно недостаточно для решения. Иногда это обескураживает.

Илья

Уровень 33

24 сентября 2020, 13:37

решение

При этом подсказка "Убедись, что поле students не инициализировано при объявлении." Разбей объявление и инициализацию students.

Анонимус #2402320

Уровень 17

24 сентября 2020, 16:09

В упор не вижу эту подсказку... Но спасибо, помогло! :)

Павел студент на стероидах в Java Syntax Pro beta

10 октября 2020, 13:37

Эта подсказку выдаёт "ментор" при валидаци. К слову у меня метод exclude() вываливался с эксепшеном конкарент итерейшн там что-то. Заменил на просто students.remove(excludedStudent); даже не понял в чём была фишка.

hidden #2322530

Уровень 41

17 сентября 2020, 18:22

первоначальный код программы какой?

Анонимус #2402320

Уровень 17

18 сентября 2020, 07:10

такой:

package com.javarush.task.pro.task11.task1123;

import java.util.ArrayList;
import java.util.List;

/*
Почувствуй себя деканом
*/

public class UniversityGroup {
    public List<String> students = new ArrayList<>();

    public UniversityGroup() {
        students.add("Сергей Фрединский");
        students.add("Виталий Правдивый");
        students.add("Максим Козыменко");
        students.add("Наталия Андрющенко");
        students.add("Ира Величенко");
        students.add("Николай Соболев");
        students.add("Снежана Слободенюк");
    }

    public void exclude(String excludedStudent) {
        for (String s : students) {
            students.removeIf(student -> excludedStudent.equals(student));
        }
    }

    public static void main(String[] args) {
        UniversityGroup universityGroup = new UniversityGroup();
        universityGroup.exclude("Виталий Правдивый");
        universityGroup.students.forEach(System.out::println);
    }
}