Помогите победить Валю!

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

CRUD 2

Java Core, 7 уровень, 10 лекция

Решен

Проверил каждый пункт, всё выдаётся как нужно. Чего я не вижу или не понимаю?

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 }

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

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

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

Ksenia VolkovaJava Developer в DXC

Master

7 октября 2020, 14:13

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

hidden #2322530

Уровень 41

7 октября 2020, 13:32

я вроде бы уже писал комментарий с указанием ошибок.

Nicha

Уровень 26

7 октября 2020, 14:05

Из указанного помогло только с методом, -1 был лишним. Сам метод мне нужен чтобы первый аргумент отсекать. Еще сам нашёл косяки, поправил их. Остальные ошибки непонятны.

public static List copyList(List list, String[] args){
    for (int i = 1; i < args.length; i++){
        list.add(args[i]);
    }

Nicha

Уровень 26

7 октября 2020, 14:07

Мне бы разжевать :) По отдельности всё верно выходит вроде как.

hidden #2322530

Уровень 41

7 октября 2020, 14:12

полезный

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

case "-i" :  synchronized (allPeople) {
// условие внимательно почитай в цикле.
                for (int i = 0; i < allPeople.size(); i++) {
                    System.out.println(allPeople.get(i).getName() + " " + allPeople.get(i).nameSex() + " " + simpleTwo.format(allPeople.get(i).getBirthDate()));
                }
            }
                break;

у тебя i не id, а индекс в списке аргументов. да и потом, ты когда получил ссылку на объект, его не надо перезаписывать, ты уже по этой ссылке изменяешь объект (обнуляешь) эту строку удалить в блоке "-d"

allPeople.set(i, current);

for (int i = 0; i < list.size(); i++) {
                    Person current = allPeople.get(i);
// а должно быть так:
Person current = allPeople.get(list.get(i));

опять та же ошибка, i у тебя индекс элемента в списке, а не id. id должен быть тогда list.get(i)

case "-u":  synchronized (allPeople) {
                count = (list.size() / 4);
                System.out.println(list.size());
                System.out.println(count);
                for (int i = 0; i < count; i++) {
                    id = i;

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

Nicha

Уровень 26

8 октября 2020, 07:41

Решил я задачу. Ты подтолкнул в правильную сторону. Но по факту, задача борьба с валидатором. Особенно пункт с -i.

hidden #2322530

Уровень 41

8 октября 2020, 07:53

пункт -i самый простой здесь. перебираешь список аргументов с 1го индекса, и для каждого выводишь человека allPeople.get(args[i]); если данный args[i] не выходит за пределы списка