JavaRush/Помощь по Java задачам/Сдаюсь! Выбрасывает исключение на switch (args[0])

33 уровень

09.11.2022
645просмотров
21комментариев

Сдаюсь! Выбрасывает исключение на switch (args[0])

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

Обсуждается

Сил моих больше нет! 😁 Помогите найти ошибки и объясните, пожалуйста, как для убогих 😁 Выбрасывает исключение: Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: Index 0 out of bounds for length 0 at com.javarush.task.task17.task1711.Solution.main(Solution.java:25)

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 }

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

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

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

hidden #598481

Уровень 27

9 ноября 2022, 12:06

Массив args - это набор параметров запуска программы. То есть он задаётся ДО запуска программы. Если использовать командную строку для запуска программы, то параметры перечисляются через пробел после имени запускаемого класса. Если использовать среду разработки, то нужно заполнить их в конфигурации запуска. В идее нажми Edit configuration и заполни поле Program arguments.

Юлия

Уровень 33

9 ноября 2022, 12:15

Я заполнила. Наверно, я в чем-то ошиблась при заполнении? Посмотри, пожалуйста)

Юлия

Уровень 33

9 ноября 2022, 12:20

Пробовала и брать в кавычки и без кавычек. Или дело не в них?))

hidden #598481

Уровень 27

9 ноября 2022, 13:32

Дело не в них. Кавычки нужны, если ты хочешь, чтобы аргумент внутри себя содержал пробелы. Если пробелов нет, то кавычки не нужны. Так а что теперь не так? Запусти дебаг и посмотри содержимое массива args. Или выполни первой командой

System.out.println(Arrays.toString(args));

Юлия

Уровень 33

9 ноября 2022, 13:52

теперь вот эту выдает Exception in thread "main" java.text.ParseException: Unparseable date: "м" at java.base/java.text.DateFormat.parse(DateFormat.java:399) at com.javarush.task.task17.task1711.Solution.main(Solution.java:40)

hidden #598481

Уровень 27

9 ноября 2022, 14:05

Так пора начинать читать, что написано 😉 Unparseable date: "м". Ты передаёшь "м" в парсер даты, что, очевидно, неправильно. Что ты передаёшь туда? Аргумент с индексом i+3, то есть 4й по порядку. Это правильно, но там должна быть дата, а у тебя там "м". Почему? Потому что у тебя перед этим какая-то единица в аргументах. Что она там делает? )

Юлия

Уровень 33

9 ноября 2022, 15:27

не читаю не потому что не хочу, увы( просто пока не понимаю, мы с дебагом никак не можем подружиться. Но я хочу это победить)))) В аргументах у меня цифрами написано id, ты об этом? Как правильно вписать параметр id?

hidden #598481

Уровень 27

9 ноября 2022, 15:35

А, я думал ты с "-c" разбираешь, там не должно быть айди в параметрах. Смотри, у тебя индексы поехали. У тебя первый параметр - это -u, у него индекс 0. Но этот параметр не повторяется, поэтому у тебя блок из четырёх параметров (айди, имя, пол, др) должен начинаться с единицы. А следующий блок будет начинаться с 1+4=5. А потом с 5+4=9 и т.д. Поэтому у тебя съезжают индексы сейчас, т.к. ты всегда начинаешь с нуля.

Юлия

Уровень 33

10 ноября 2022, 11:52

Точно! Блин как могла упустить, это же очевидно!)) Спасибо тебе большое! Но! Это еще не все 😁

Посмотри, пожалуйста, где косяк?

case "-c":
                synchronized (allPeople) {
                    for (int i = 1; i < args.length; i = i + 3) {
                        Date date = dateFormat.parse(args[i+2]);
                        allPeople.add((args[i+1].equals("м")) ? Person.createMale(args[i], date) : Person.createFemale(args[i], date));
                        System.out.println(allPeople.size() - 1);
                    }
                }
                break;

            case "-u":
                synchronized (allPeople) {
                    for (int i = 1; i < args.length; i = i +4) {
                        Date date1 = dateFormat.parse(args[i+3]);
                        person = allPeople.get(Integer.parseInt(args[i]));
                        person.setName(args[i + 1]);
                        if (args[i+2].equals("м")) {
                            person.setSex(Sex.MALE);
                        } else {
                            person.setSex(Sex.FEMALE);
                        }
                        person.setBirthDate(date1);
                    }
                }
                break;

Владимир

Уровень 30

10 ноября 2022, 16:53

Юлия, добрый вечер. Могу вам помочь, если нужно?

Юлия

Уровень 33

15 ноября 2022, 07:25

Добрый вечер! Владимир, буду вам очень благодарна!

pichitap

Уровень 51

15 ноября 2022, 14:30

Предполагаю, что не хватает скобок { } на каждый из case.

case "-u" : {
                //Мой код
            }

По крайней мере, сравнивая со своим решением, только в этом отличие.

Владимир

Уровень 30

15 ноября 2022, 14:54

Условие:
"-с" добавляет всех людей с заданными параметрами в конец списка allPeople
и выводит id каждого (index) на экран.

Первую часть требования вы выполняете:

allPeople.add();

А вторую - нет!

System.out.println(allPeople.size() - 1);

Метод size() выводит количество элементов в листе. (Size() – 1) не уверен, что вы этим хотели сказать) В листе есть метод indexOf (Object o) – возвращает индекс 1ого совпавшего элемента. Как раз то, что нужно выводить.

Условие:
"-u" обновляет данные людей с заданными id.
bd - дата рождения в следующем формате dd/MM/yyyy

Вы же используете один и тот же dateFormat с шаблоном "dd-MMM-yyyy". Присвойте той же переменной в начале синхронизированного блока:

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

Владимир

Уровень 30

15 ноября 2022, 15:14

Фигурные скобки в синтаксисе case: блока не нужны! Каждый case - последовательность statements разделенных break;

Юлия

Уровень 33

15 ноября 2022, 15:29

Поправила, спасибо! Все равно не работает 😁 По дате, кстати, вроде верный шаблон, как в условии. В аргументах у меня "dd/MM/yyyy", а выводим на консоль "dd-MMM-yyyy".

Владимир

Уровень 30

15 ноября 2022, 15:30

Покажите весь main код

Владимир

Уровень 30

15 ноября 2022, 20:29

Юлия, было бы гораздо удобнее видеть ваш код перед глазами, тем не менее, по задаче: 4 типа параметров: -с; -u; -d; -i; В трех из них нужно использовать SimpleDateFormat 'er В -с и -u используется шаблон "на ввод" - "dd/MM/yyyy" В -i шаблон "на вывод" - "dd-MMM-yyyy" Соответственно, если вы используете одну переменную dateFormat, то вам нужно в каждом кейсе переопределять ее. Как вариант, можно создать отдельную переменную "на вывод" и применять каждую по необходимости.

Юлия

Уровень 33

16 ноября 2022, 08:50

public class Solution {
    public static volatile List<Person> allPeople = new ArrayList<Person>();


    static {
        allPeople.add(Person.createMale("Иванов Иван", new Date()));  //сегодня родился    id=0
        allPeople.add(Person.createMale("Петров Петр", new Date()));  //сегодня родился    id=1
    }

    public static void main(String[] args) throws ParseException {
        //System.out.println(Arrays.toString(args));
        Person person;
        SimpleDateFormat dateFormat = new SimpleDateFormat("dd-MMM-yyyy", Locale.ENGLISH);

        switch (args[0]) {
            case "-c":
                synchronized (allPeople) {
                    for (int i = 1; i < args.length; i = i + 3) {
                        Date date = dateFormat.parse(args[i + 2]);
                        Person newPerson = (args[i + 1].equals("м")) ? Person.createMale(args[i], date) : Person.createFemale(args[i], date);
                        allPeople.add(newPerson);
                        System.out.println(allPeople.indexOf(newPerson));
                    }
                }

                break;

            case "-u":
                synchronized (allPeople) {
                    for (int i = 1; i < args.length; i = i + 4) {
                        Date date1 = dateFormat.parse(args[i + 3]);
                        person = allPeople.get(Integer.parseInt(args[i]));
                        person.setName(args[i + 1]);
                        if (args[i + 2].equals("м")) {
                            person.setSex(Sex.MALE);
                        } else {
                            person.setSex(Sex.FEMALE);
                        }
                        person.setBirthDate(date1);
                    }
                }

                break;

Юлия

Уровень 33

16 ноября 2022, 08:51

Не поместился в одном сообщении))

case "-d":
                synchronized (allPeople) {
                    for (int i = 1; i < args.length; i++) {
                        allPeople.get(Integer.parseInt(args[i])).setName(null);
                        allPeople.get(Integer.parseInt(args[i])).setSex(null);
                        allPeople.get(Integer.parseInt(args[i])).setBirthDate(null);
                    }
                }

                break;

            case "-i":
                synchronized (allPeople) {
                    for (int i = 1; i < args.length; i++) {

                        Date date3 = allPeople.get(Integer.parseInt(args[i])).getBirthDate();
                        System.out.println(allPeople.get(Integer.parseInt(args[i])).getName() + " ");
                        System.out.println(allPeople.get(Integer.parseInt(args[i])).getSex() == Sex.MALE ? "м" : "ж" + " ");
                        System.out.println(dateFormat.format(date3));
                    }
                }

                break;
            default:
            throw new IllegalStateException("Unexpected value: " + args[0]);
        }
    }
}

Юлия

Уровень 33

16 ноября 2022, 09:07

Валидатор не ругается на дату. Попробовала скорректировать как вы сказали, но все-таки он пишет ошибки в -с Убедись, что действительно все люди добавляются в список allPeople. и в -u Убедись, что при обновлении данных людей размер списка allPeople не меняется.

Vic

Уровень 25

29 декабря 2022, 15:54

Я на том же затупил)))) Так же не принималось по -c и -u. Невнимательно прочитал задание и не создал 2-й SimpleDateFormat("dd/MM/yyyy". Как только перечитал задание, всё встало на свои места... Кстати, в решении JR недоработка , в IDEA, без проверки - if (id < allPeople.size()) { в "-d" и"-i", выскакивает IndexOutOfBoundsException