Не проходит по 4 пункту (-u) :(

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

Обсуждается

Товарищи, подскажите ! Валидатор говорит следующее : Убедись, что действительно информация о всех людях меняется в списке allPeople. Я дебажил, информация на самом деле изменяется для всех объектов листа allPeople. Думаю о том, что может быть создатели задачи имели ввиду изменения значений конкретно У САМИХ объектов? А то я создаю новый, и уже его set-аю в лист

CrUD Batch - multiple Creation, Updates, Deletion.

Программа запускается с одним из следующих наборов параметров:
-c name1 sex1 bd1 name2 sex2 bd2 ... -u id1 name1 sex1 bd1 id2 name2 sex2 bd2 ... -d id1 id2 id3 id4 ... -i id1 id2 id3 id4 ...

Значения параметров:
name - имя, String
sex - пол, "м" или "ж", одна буква
bd - дата рождения в следующем формате 15/04/1990
-с - добавляет всех людей с заданными параметрами в конец allPeople, выводит id (index) на экран в соответствующем порядке
-u - обновляет соответствующие данные людей с заданными id
-d - производит логическое удаление человека с id, заменяет все его данные на null
-i - выводит на экран информацию о всех людях с заданными id: name sex bd

id соответствует индексу в списке.
Формат вывода даты рождения 15-Apr-1990
Все люди должны храниться в allPeople.
Порядок вывода данных соответствует вводу данных.
Обеспечить корректную работу с данными для множества нитей (чтоб не было затирания данных).
Используй Locale.ENGLISH в качестве второго параметра для SimpleDateFormat.

Пример вывода для параметра -і с двумя id:
Миронов м 15-Apr-1990 Миронова ж 25-Apr-1997

Требования:

Класс Solution должен содержать public static volatile поле allPeople типа List<Person>.
Класс Solution должен содержать статический блок, в котором добавляются два человека в список allPeople.
При параметре -с программа должна добавлять всех людей с заданными параметрами в конец списка allPeople, и выводить id каждого (index) на экран.
При параметре -u программа должна обновлять данные людей с заданными id в списке allPeople.
При параметре -d программа должна логически удалять людей с заданными id в списке allPeople.
При параметре -i программа должна выводить на экран данные о всех людях с заданными id по формату указанному в задании.
Метод main класса Solution должен содержать оператор switch по значению args[0].
Каждый case оператора switch должен иметь блок синхронизации по allPeople.

package com.javarush.task.task17.task1711; public enum Sex { MALE, FEMALE }

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

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

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

Денис Enterprise Java Developer

14 сентября 2022, 07:54

Слушай, а ты можешь описать словами, что конкретно происходит в блоке кода ответственном за -u? Тебе передают строку в которой данные идут в конкретной последовательности - id, имя, пол, дата. Твоя задача найти существующих людей в твоем списке с указанным айди и поменять значение соответствующих полей. Я например не заметил у тебя работу с айди в принципе. Вместо обновления полей ты создаешь новые объекты. К методу сбора информации об именах и прочем тоже есть вопросы. Во первых это не читаемо, во вторых я даже не уверен что это работает верно во всех возможных случаях :)

Нейросеть

Уровень 41

14 сентября 2022, 08:01

Согласен. Тоже в глаза бросилось это). Создавать List для одного обекта как то не разумно. И не очень понял что там фильтруется, если и так поданы на вход готовые строки в нужном порядке.

Денис Enterprise Java Developer

14 сентября 2022, 08:05

Да логика понятна на самом деле. Человек хотел разбить строки на списки отдельных элементов, по идее индекс элемента в списке у данных относящихся к одному объекту должен совпадать, но во первых понятие id игнорируется как классово неприемлимое, во вторых, я не уверен как это будет работать на именах с пробелами или спец символами типа Д'Артаньян, во вторых выглядит как лютейший оверхед - зачем если мы знаем строгий порядок элементов и можем составлять объекты тупо читая строку последовательно. Ну и как я уже отметил выше - это совершенно нечитаемо. Я понимаю, что писать логику лень и хочется, что бы ее за тебя реализовал добрый дядя из JDK, но так работает не всегда :)

Нейросеть

Уровень 41

14 сентября 2022, 07:44

Всё правильно ты думаешь. Создавать новых в условии не требовалось. Требовалось изменить старых. Тебе ж не надо новый акк создавать чтобы скажем имя сменить где бы то не было