JavaRush/Помощь по Java задачам/Наварил макарон

27 уровень

28.07.2020
460просмотров
9комментариев

Наварил макарон

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

Архивный

Не знаю почему, но данную задачу захотелось решить при помощи регулярных выражений. Проблема такова, что Валидатор не принимает ни один из пунктов -c -u -d -i Не могу дойти до истины, в intellij idea работает как надо ( насчет пункта -d не совсем уверен, хотя без него должны были хоть какие-то пункты пройти), вывод соответствует тому, что требуется ( или же я где-то слепо на код смотрю). Прошу помочь найти проблему, также дать совету по этому коду ( так как он мне не сильно нравится, хотелось бы что-то красивее)

CRUD - Create, Read, Update, Delete.

Программа запускается с одним из следующих наборов параметров:
-c name sex bd -r id -u id name sex bd -d id

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

id соответствует индексу в списке.
Все люди должны храниться в allPeople.
Используй Locale.ENGLISH в качестве второго параметра для SimpleDateFormat.

Пример параметров:
-c Миронов м 15/04/1990

Пример вывода для параметра -r:
Миронов м 15-Apr-1990

Если программа запущена с параметрами, то они попадают в массив args (аргумент метода main - String[] args).
Например, при запуске программы c параметрами:
-c name sex bd
получим в методе main
args[0] = "-c"
args[1] = "name"
args[2] = "sex"
args[3] = "bd"
Для запуска кода с параметрами в IDE IntellijIDEA нужно их прописать в поле Program arguments в меню Run -> Edit Configurations.

Требования:

Класс Solution должен содержать public static поле allPeople типа List<Person>.
Класс Solution должен содержать статический блок, в котором добавляются два человека в список allPeople.
При запуске программы с параметром -с программа должна добавлять человека с заданными параметрами в конец списка allPeople, и выводить id (index) на экран.
При запуске программы с параметром -r программа должна выводить на экран данные о человеке с заданным id по формату указанному в задании.
При запуске программы с параметром -u программа должна обновлять данные человека с заданным id в списке allPeople.
При запуске программы с параметром -d программа должна логически удалять человека с заданным id в списке allPeople.

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

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

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

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

Ksenia VolkovaJava Developer в DXC

Master

28 июля 2020, 23:38

А если имя латиницей? или там еще какие-то символы? Петров-Водкин, O'Reily...

Pavel Lysenkov

Уровень 27

28 июля 2020, 23:42

Это несложно добавить в регулярное выражение, беда в том, что даже пункт с -i хотя там всего 6 строчек кода ( Валидатор твердит, чтобы я убедился, что данные попадают в список (массив) Но данные действительно попадают туда, в самом конце я могу вывести массив и все данные там будут. Также вариант: когда имя может содержать сколько угодно слов у меня обработан

Ksenia VolkovaJava Developer в DXC

Master

29 июля 2020, 01:10

решение

Чтобы добавить, надо знать, что именно добавлять. А нам неизвестно, какие символы могут использоваться в имени, а какие нет. Регулярки тут вовсе не запрещены, но у них свои нюансы. Проблема твоего кода - чтение с консоли. "Программа запускается с параметрами" - это не ввод с консоли, это параметры args

Pavel Lysenkov

Уровень 27

29 июля 2020, 12:35

Вот это лажа) спасибо за помощь

Pavel Lysenkov

Уровень 27

29 июля 2020, 12:42

Ничего сложного тогда нет в этой задачи, да и не было )

Pavel Lysenkov

Уровень 27

29 июля 2020, 13:16

Короткое и простое решение тогда выходит)

SimpleDateFormat sdf = new SimpleDateFormat("dd/MM/yyyy", Locale.ENGLISH);

if (args[0].equals("-i")){
    System.out.println(allPeople.get(Integer.parseInt(args[1])));
} else if (args[0].equals("-d")){
    allPeople.get(Integer.parseInt(args[1])).deletePerson();
} else if (args[0].equals("-c")){
    if (args[2].equals("м")){
        allPeople.add(Person.createMale(args[1], sdf.parse(args[3])));
    } else {
       allPeople.add(Person.createFemale(args[1], sdf.parse(args[3])));
    }
    System.out.println(allPeople.size() - 1);
} else if (args[0].equals("-u")){
    int id = Integer.parseInt(args[1]);
    allPeople.get(id).setName(args[2]);
    allPeople.get(id).setBirthDate(sdf.parse(args[4]));
    if (args[3].equals("м")) {
        allPeople.get(id).setSex(Sex.MALE);
    } else {
        allPeople.get(id).setSex(Sex.FEMALE);
    }
}

Vergilius

Уровень 22

28 июля 2020, 23:28

Вместо регулярок используй split и trim. Рзаделяй код на методы - этапы обработки с промежуточными результатами. Используй LocalDate.

Pavel Lysenkov

Уровень 27

28 июля 2020, 23:31

Интересно, почему регулярные выражения то запрещены тут ?( Насчет разделения методов, это очевидно, но красить код, который валидатор не принимает, не очень хочется.🙃

Vergilius

Уровень 22

28 июля 2020, 23:38

Они не запрещены просто это overengineering. Если не хочется, то определенно не стоит.