Вопрос по synchronized

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

Решен

В данном случае разве не логично было бы использовать один блок

synchronized ?

на всеь блок

switch(args[0]) {
case ...
.
.
.
}

Ведь все равно все блоки блокируются по

synchronized (allPeople)

и одновременно может работать только один case.

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 }

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

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

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

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

Master

18 августа 2022, 08:32

решение

так не обязательно же все тело кейза оборачивать в синхронайзд :) синхронизировать надо только тот момент, когда мы обращаемся к этому списку. пример:

case "-c": {
    for (int i = 0; i < (args.length - 1) / 3; i++) {
        person = createPerson(Arrays.copyOfRange(args, 1 + i * 3, 1 + (i + 1) * 3));
        synchronized(allPeople) {
            allPeople.add(person);
            System.out.println(allPeople.size() - 1);
        }
    }
    break;
}

то-есть, мы будем блокировать доступ к allPeople только в тот момент, когда нам это надо, когда нам надо что-то в него записать/прочитать в остальное время, пока мы там что-то свое считаем - нам не нужно держать блокировку доступа к этому объекту, а значит другие нити в это время тоже смогут что-то в него писать/читать в случае, когда мы весь свитч синхронизируем - у нас получится однопоточная программа. потому что все нити будут стоять и ждать пока закончит работу первая, потом вторая, потом третья. а хотелось бы чтобы они все работали одновременно с этим кодом и блокировались только тогда, когда это действительно надо

sdr

Уровень 32

19 августа 2022, 13:43

Спасибо)