JavaRush/Помощь по Java задачам/Какая всё-таки последовательность методов в main?

5 уровень

15.03.2018
1453просмотров
10комментариев

Какая всё-таки последовательность методов в main?

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

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

Решен

Прошу помочь. Не принимает по двум пунктам, перепробовал ВСЁ, извёл кучу попыток, но так и не понял в чём ошибка. Рекомендация ментора: "Вывод метода main должен содержать удаленного пользователя (id пользователя - 124L) в секции "All deleted users"."

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); } } }

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

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

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

Ivan

Уровень 29

21 апреля 2021, 19:12

Тезка, проверь main, ты не все создал, что необходимо - поэтому у тя не должно запускаться (должна вылетать NullPointerException).

Алина Будаева

Уровень 25

1 августа 2018, 13:23

Добрый день. Попробуйте удалить строку *** она влияет на вывод, а здесь нужно просто удалить @Override public void deleteUserById(long id) { userService.deleteUser(id); *** modelData.setDisplayDeletedUserList(false); modelData.setUsers(getAllUsers()); }

Pavlo PlynkoJava-разработчик в CodeGym

Expert

4 апреля 2018, 16:25

В MainModel здесь:

public void loadDeletedUsers() {
    modelData.setDisplayDeletedUserList(true);
    modelData.setUsers(getAllUsers());
}

не должно быть getAllUsers().

ИванJava Developer в Фора-Банк

24 апреля 2018, 06:50

Приветствую. Надо так ?: public void loadDeletedUsers() { modelData.setDisplayDeletedUserList(true); modelData.setUsers(userService.getAllDeletedUsers()); } Не помогает, можно поподробнее. Долго не отвечал, потому что перешел к другим задачам, а сейчас вернулся к этой.

Pavlo PlynkoJava-разработчик в CodeGym

Expert

25 апреля 2018, 12:41

У меня твой код, с таким методом loadDeletedUsers() проходит валидацию. Видимо, ты внес еще какие-то изменения.

ИванJava Developer в Фора-Банк

27 апреля 2018, 12:26

решение

ОК. Благодарю, буду проверять.

Ярослав Java Developer

Master

16 марта 2018, 05:05

public void onUserDelete(long id) {
    model.deleteUserById(id);
    usersView.refresh(model.getModelData());
}

Ничего подозрительного?

public void fireEventUserDeleted(long id) {
        controller.onUserDelete(id);
    }

Это метод класса EditUserView, вызывается метод onUserDelete, в котором вы обновляете UsersView. Посмотрите на методы refresh этих классов. Они разные.

ИванJava Developer в Фора-Банк

16 марта 2018, 05:49

Доброе утро. Благодарю за ответ, но не могли бы Вы пояснить по поводу методов refresh и про "подозрительные" строки кода. Их я писал на предыдущих уровнях и проверку задача проходила. Честно скажу, что понимаю логику взаимодействия методов очень плохо, хочу допинать задачу до конца и там уже, имея всю картину в целом, основательно разобраться. А пока просто выполняю требования. Заранее благодарен.

Ярослав Java Developer

Master

16 марта 2018, 14:52

Похоже, что я немного ошибся. Вообще логика немного странная у всего приложения. Вот что еще вижу: у UsersView есть логическая переменная, благодаря которой показываются удаленные пользователи. Вы её не устанавливаете, потому удаленные пользователи не выводятся. Установить её в тру через специальный метод (сетер). Это должно выполнить требование ментора.

ИванJava Developer в Фора-Банк

24 апреля 2018, 06:54

Приветствую. Не смог найти у UsersView логических переменных. Не могли бы Вы поподробнее пояснить. Есть modelData.setDisplayDeletedUserList(true), но это не помогло. Долго не отвечал, потому что перешел к другим задачам, а сейчас вернулся к этой.