Ребятки, что не нравится валику, подскажите, плиз! Всё в класе MainModel

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

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

Решен

По 3 пункту ментор: "В классе MainModel метод deleteUserById(long) должен установить список всех активных пользователей объекту modelData."

public void deleteUserById(long id) {
        userService.deleteUser(id);
        loadUsers(); // -> тут устанавливаю активных пользователей объекту modelData.
    }

По 6 пункту ментор: "Ты не добавил в класс MainModel метод getAllUsers()." Вот, добавил:

private List<User> getAllUsers(List<User> all) {
    return userService.filterOnlyActiveUsers(all);
}

По 7 пункту ментор: "В методе loadUsers() используй метод getAllUsers(), и не забудь обновить модель."

public void loadUsers() {
        List<User> userList100 = userService.getUsersBetweenLevels(1, 100);
        userList100 = getAllUsers(userList100); //фильтруем "живых" пользователей // -> использую метод getAllUsers()
        modelData.setDisplayDeletedUserList(false);
        modelData.setUsers(userList100);
    }

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

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

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

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

Максим

Уровень 27

22 октября 2018, 15:56

решение

Всё решилось рефакторингом рефакторинга. Было:

public void loadUsers() {
      List<User> userList100 = userService.getUsersBetweenLevels(1, 100);
      userList100 = getAllUsers(userList100); //фильтруем "живых" пользователей
      modelData.setDisplayDeletedUserList(false);
      modelData.setUsers(userList100);
  }
  private List<User> getAllUsers(List<User> all) {
      return userService.filterOnlyActiveUsers(all);
  }

Стало:

public void loadUsers() {
//        List<User> userList100 = userService.getUsersBetweenLevels(1, 100);
//        userList100 = getAllUsers(userList100); //фильтруем "живых" пользователей
        modelData.setDisplayDeletedUserList(false);
        modelData.setUsers(getAllUsers());
    }
    private List<User> getAllUsers() {
        List<User> userList100 = userService.getUsersBetweenLevels(1, 100);
        return userService.filterOnlyActiveUsers(userList100);
    }

По-русски я бы назвал это - "кал месить"

+25