JavaRush/Помощь по Java задачам/Что не так?

61 уровень

10.12.2020
567просмотров
13комментариев

Что не так?

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

Java Multithreading, 4 уровень, 14 лекция

Решен

Все идентично заданию, но валидатор не принимает

1. Следуя принципу MVC аналогично реализации предыдущих методов сделай следующее:
напиши логику удаления пользователя. После удаления должен отображаться список пользователей.

Распредели следующие методы по классам MVC:
public void fireEventUserDeleted(long id)
public void onUserDelete(long id)
public void deleteUserById(long id)

Примечание: метод, который ты собираешься добавить в Вью нужно добавить в EditUserView.

2. Добавь в main вызов fireEventUserDeleted(124L) перед вызовом метода fireEventShowDeletedUsers().

3. Выполни рефакторинг класса MainModel. Теперь, когда есть удаленные пользователи, часть методов стала работать неправильно.
Почти во всех методах, где требуется список пользователей, нужно работать только с активными(живыми) пользователями.
Вынеси в отдельный приватный метод List<User> getAllUsers() получение списка всех активных пользователей.
Фильтрация активных пользователей у тебя уже есть - метод List<User> filterOnlyActiveUsers(List<User> allUsers).
Отрефактори все методы, которые используют список пользователей. Они должны использовать список живых пользователей.

4. Добавь в интерфейс Model метод, который ты поместил в Модель, реализуй его в FakeModel: выброси UnsupportedOperationException.

Требования:

Необходимо определить правильное расположение метода fireEventUserDeleted(long) и реализовать этот метод.
Необходимо определить правильное расположение метода onUserDelete(long) и реализовать этот метод.
Необходимо определить правильное расположение метода deleteUserById(long) и реализовать этот метод.
В методе main класса Solution необходимо вызвать метод fireEventUserDeleted(124L) у объекта класса EditUserView перед вызовом метода fireEventShowDeletedUsers().
В методе main вызов методов должен происходить в такой последовательности: fireEventShowAllUsers(), fireEventOpenUserEditForm(126L), fireEventUserDeleted(124L), fireEventShowDeletedUsers().
Необходимо реализовать приватный метод List<User> getAllUsers() в классе MainModel.
Необходимо выполнить рефакторинг: все методы класса MainModel, которые используют список пользователей, должны использовать список живых пользователей.
Интерфейс Model должен содержать объявление метода, который ты ранее реализовал в классе MainModel.
В классе FakeModel в теле метода, помещенного в интерфейс Model, необходимо бросить UnsupportedOperationException.

package com.javarush.task.task36.task3608; import com.javarush.task.task36.task3608.bean.User; public final class Util { public static final String DELETED_MARKER = " (deleted)"; public static boolean isUserDeleted(User user) { return user.getName().endsWith(DELETED_MARKER); } public static void markDeleted(User user) { if (User.NULL_USER != user && !Util.isUserDeleted(user)) { user.setName(user.getName() + DELETED_MARKER); } } }

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

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

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

hidden #2322530

Уровень 41

10 декабря 2020, 12:02

разве этот метод не должен у EditUserView вызываться?

fireEventOpenUserEditForm(126L),

кода нет под рукой посмотреть, на память не очень помню.

begem

Уровень 61

10 декабря 2020, 12:05

нет, все правильно, проверил

hidden #2322530

Уровень 41

10 декабря 2020, 12:06

я к другому. разве не так должно быть?

usersView.fireEventShowAllUsers();
editUserView.fireEventOpenUserEditForm(126L);
editUserView.fireEventUserDeleted(124L);
usersView.fireEventShowDeletedUsers();

и метод соответственно перенести туда. ведь у тебя метод вот что делает:

public void onOpenUserEditForm(long userId) {
    model.loadUserById(userId);
    editUserView.refresh(model.getModelData());
}

соответственно вызываем у editUserView, контроллер потом его же и обновляет.

begem

Уровень 61

10 декабря 2020, 12:08

в предыдущей задаче так делал, но валидатор не принимал, пришлось переделать и вызвать метод fireEventOpenUserEditForm() у объекта userView

hidden #2322530

Уровень 41

10 декабря 2020, 12:09

да, у других посмотрел сейчас. всё правильно в этом месте.

hidden #2322530

Уровень 41

10 декабря 2020, 12:13

метод попробуй вот так переписать в MainModel

public void loadUsers() {
        modelData.setDisplayDeletedUserList(false);
        modelData.setUsers(getAllUsers());
    }

begem

Уровень 61

10 декабря 2020, 12:18

сделал, но не проходит по тому же пункту

begem

Уровень 61

10 декабря 2020, 12:47

Уже как вариант взял готовое решение JR и скопировал в класс MainModel и Solution, все равно не пропускает. Наверное что-то с валидатором

hidden #2322530

Уровень 41

10 декабря 2020, 13:05

решение

@Override
public void refresh(ModelData modelData) {
    System.out.println("User to be edited:");
    System.out.println("\t" + modelData.getActiveUser());
    System.out.print("===================================================");
}

здесь должен быть println и здесь

@Override
public void refresh(ModelData modelData) {
    if (modelData.isDisplayDeletedUserList()) {
        System.out.println("All deleted users:");
    } else {
        System.out.println("All users:");
    }
    for (User user : modelData.getUsers()) {
        System.out.printf("\t%s%n", user);
    }
    System.out.print("===================================================");

}

begem

Уровень 61

10 декабря 2020, 13:16

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

begem

Уровень 61

10 декабря 2020, 13:25

Возможно из-за этого, но валидацию прошло. Очень странно, т.к. предыдущие задачи проверку проходили с таким кодом

hidden #2322530

Уровень 41

10 декабря 2020, 13:27

в прошлый раз не было вызовов новых методов

begem

Уровень 61

10 декабря 2020, 13:29

Понятно, благодарю! Эта задача слишком запутанная