Не понимаю, почему через создание потоков не принимает

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

Обсуждается

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

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 }

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

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

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

Anonymous #3171134

Уровень 35

24 апреля 2023, 08:30

если использовать эти данные :args = new String[]{"-u","0", "Миронов", "м", "15/04/1990","1", "Миронова", "ж", "25/04/1997"}; то программа получает: Миронов ж 04-Mar-1991 Миронова ж 04-Jan-1999

Aleksandr

Уровень 33

23 марта 2023, 20:21

Увидел это: в строке 108 стоит точка с запятой и метод для "ж" не выполняется. в строке 127 цикл начинается с нуля, а в нуле у тебя в списке входных параметров - "-d", да и вообще - ты не вызываешь id из массива входных параметров, а просто людей по порядку.

Rustam Garant

Уровень 35

24 марта 2023, 04:40

Спасибо за замечания👍 в 127 строке заменил i на (Integer.parseInt(args[i])). Но так и не проходит валидацию

Стас ПасинковSoftware Developer в Zipy

Master

21 марта 2023, 19:01

нуу, прикольно, шо ви так вирішили зробити) але тут це не виправдано) спробуйте додати джойнів (щоб не переписувати все заново) і перевірити, чи пройде перевірку)

Rustam Garant

Уровень 35

22 марта 2023, 04:46

Если правильно понял, то сделал так, но не помогло

switch (args[0]) {
                case "-c":
                    synchronized (allPeople) {
                        Creation creation = new Creation(args);
                        creation.start();
                        creation.join();
                    }
                    break;
                case "-u":
                    synchronized (allPeople) {
                        Update update = new Update(args);
                        update.start();
                        update.join();
                    }
                    break;
                case "-d":
                    synchronized (allPeople) {
                        Delete delete = new Delete(args);
                        delete.start();
                        delete.join();
                    }
                    break;
                case "-i":
                    synchronized (allPeople) {
                        Info info = new Info(args);
                        info.start();
                        info.join();
                    }
                    break;

Стас ПасинковSoftware Developer в Zipy

Master

22 марта 2023, 08:07

32гу строку забрали, перш ніж здавати на перевірку?

Rustam Garant

Уровень 35

22 марта 2023, 09:05

закомментировал её

Стас ПасинковSoftware Developer в Zipy

Master

22 марта 2023, 17:34

тоді не знаю( спробуйте взагалі без тредів зробити. переписати просто на методи окремі. може не дарма змінна allPeople волатильна, може там тести в кілька потоків вашу програму ганяють, не знаю

Rustam Garant

Уровень 35

23 марта 2023, 04:30

без множества потоков сделал, всё ок. Просто хотел понять для себя, почему с задачей не справляются потоки. Поэтому и поднял этот вопрос. Может есть какая-то особенность или что-то, что я упускаю