коллеги, это потокобезопасно? / Java

ReSQL.ru

Мобильная версия Контакт Правила FAQ Помощь

Гость

Войти | Регистрация | Профиль | Очистить

Новые сообщения | Избранное

Форумы | Пользователи | Статистика | Мод. лог | Поиск

Цитировать

Написать

Автор*:

Ввести пароль для входа

Тема*:

Сообщение

Данное сообщение тематическое

Сообщение содержит картинки или видео 18+

Автор:

ВНИМАНИЕ! На данном подфоруме действуют строгие правила. Удостоверьтесь, что ваше сообщение соответствует им!

Форум или тема закрыты для гостей. Необходима авторизация!

Загрузить последнюю сохраненную версию

Вложение:

Вставить как галерею

Максимальный размер вложений: 4,0 МБ, аудио/видео: 8,0 МБ. Картинки большего размера ужимаются, если возможно.

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

Отправляя сообщение, я выражаю свое согласие с правилами форума и принимаю пользовательское соглашение.

Доб. в избранное | Игнор. тему | Прикреп. тему | Пометить прочит. / непрочит. | Фильтр

Форумы / Java [игнор отключен] [закрыт для гостей] / коллеги, это потокобезопасно?

11 сообщений из 11, страница 1 из 1

коллеги, это потокобезопасно?

#39709452

rabiter

Участник

Откуда: SPb

Сообщения: 677

Рейтинг: 0 / 0

коллеги, подскажите, такой класс потокобезопасен?

Код: java

1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
19.
20.
21.
22.

public class ExportProgress {

    private final long total;
    private final AtomicLong done = new AtomicLong(0);

    public ExportProgress(long total) {
        this.total = total;
    }

    public void setDone(long done) {
        this.done.set(done);
    }

    public void incrementDone() {
        this.done.incrementAndGet();
    }

    public int getPercentageDone() {
        return total == 0 ? 0 : (int) Math.min(done.get() * 100 / total, 100);
    }

}

...

Рейтинг:

0 / 0

27.09.2018, 18:00

| Ответить | Цитировать | Написать

коллеги, это потокобезопасно?

#39709466

забыл ник

Участник

Сообщения: 3 596

Рейтинг: 0 / 0

rabiter,
Что в вашем понимании потокобезопасно? Следует отличать потокобезопасность, гарантию видимости изменений и корректность.
В вашем случае конструктор потокобезопасный, это +. Далее два метода мутирующих сосстояние, setDone и incrementDone, оба изменяют значение одного и того же атомик счетчика - значит проблем с видимостью изменений тоже не будет, опять +.
Далее сложнее, доказать корректность - тут нужен контекст, если побыть телепатом, то скорее всего у вас ошибки. Зачем два метода изменения? Причем один инкрементит а второй все затирает и ставит новое значение? Допустим два потока инкрементят себе, инкрементят, а потом приходит лесник и выставляет setDone(0)... wtf?

Назначение переменной total тоже туманное.

Если резюмировать, то код выглядит "потокобезопасным" в разрезе именно конкурентного доступа к данным и видимостью, но я почти уверен что это работает не так как вы ожидаете

...

Рейтинг:

0 / 0

27.09.2018, 18:23

| Ответить | Цитировать | Написать

коллеги, это потокобезопасно?

#39709471

rabiter

Участник

Откуда: SPb

Сообщения: 677

Рейтинг: 0 / 0

забыл ник,

да, все верно,туманно! но вы ответили на мой вопрос, спасибо!

а вообще контекст такой, что один поток будет только читать, а второй поток - обновлять значения. По поводу лесника - да, но в реальности такого не будет.

...

Рейтинг:

0 / 0

27.09.2018, 18:30

| Ответить | Цитировать | Написать

коллеги, это потокобезопасно?

#39709472

rabiter

Участник

Откуда: SPb

Сообщения: 677

Рейтинг: 0 / 0

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

...

Рейтинг:

0 / 0

27.09.2018, 18:32

| Ответить | Цитировать | Написать

коллеги, это потокобезопасно?

#39709474

rabiter

Участник

Откуда: SPb

Сообщения: 677

Рейтинг: 0 / 0

изначально переменная done вообще была просто long volatile, но потребовалась функция инкремента и я подумал вместо блокировки попробовать атомик

...

Рейтинг:

0 / 0

27.09.2018, 18:34

| Ответить | Цитировать | Написать

коллеги, это потокобезопасно?

#39709494

cossack5

Участник

Сообщения: 182

Рейтинг: 0 / 0

Метод getPercentageDone непотокобезопасен, т.к. у вас там есть и чтение и запись, и может получиться ArithmeticException. Можно сделать так:

Код: java

1.
2.
3.
4.

    public int getPercentageDone() {
        int tempTotal=total;
        return tempTotal == 0 ? 0 : (int) Math.min(done.get() * 100 / tempTotal, 100);
    }

...

Рейтинг:

0 / 0

27.09.2018, 19:16

| Ответить | Цитировать | Написать

коллеги, это потокобезопасно?

#39709511

rabiter

Участник

Откуда: SPb

Сообщения: 677

Рейтинг: 0 / 0

cossack5,

да не, вроде не должно, поле total же не меняется, оно финальное

...

Рейтинг:

0 / 0

27.09.2018, 20:01

| Ответить | Цитировать | Написать

коллеги, это потокобезопасно?

#39709525

mayton

Участник

Откуда: loopback

Сообщения: 53 422

Рейтинг: 2 / 0

rabiter

Код: sql

1.
2.
3.

    public int getPercentageDone() {
        return total == 0 ? 0 : (int) Math.min(done.get() * 100 / total, 100);
    }

Потокобезопасно и некорректно. Что за фигню он будет выдавать после 100%.
Ты воздействуешь событиями на ExportProgress а он не оказывает влияния на результат.

...

Рейтинг:

0 / 0

27.09.2018, 20:41

| Ответить | Цитировать | Написать

коллеги, это потокобезопасно?

#39709531

rabiter

Участник

Откуда: SPb

Сообщения: 677

Рейтинг: 0 / 0

mayton,

после 100 он будет выдавать 100! это лучше чем если мы увидим 150%. может конечно бросать исключение при попытке перевыполнить план, но я уверяю вас, в моец ситуации это абсолютно излишне.
но спасибо за заботу :-)

...

Рейтинг:

0 / 0

27.09.2018, 20:55

| Ответить | Цитировать | Написать

коллеги, это потокобезопасно?

#39709532

mayton

Участник

Откуда: loopback

Сообщения: 53 422

Рейтинг: 2 / 0

А минус сто пятьдесят? Вобщем... странный дизайн.

...

Рейтинг:

0 / 0

27.09.2018, 20:56

| Ответить | Цитировать | Написать

коллеги, это потокобезопасно?

#39709534

rabiter

Участник

Откуда: SPb

Сообщения: 677

Рейтинг: 0 / 0

mayton,

а да, при переполнении можно и в минус уйти запросто

...

Рейтинг:

0 / 0

27.09.2018, 21:05

| Ответить | Цитировать | Написать

11 сообщений из 11, страница 1 из 1

Форумы / Java [игнор отключен] [закрыт для гостей] / коллеги, это потокобезопасно?

Читали тему (0):

Читали форум (0):

Пользователи онлайн (0):

start [/forum/topic.php?fid=59&msg=39709511&tid=2121756]:	0ms
get settings:	8ms
get forum list:	12ms
check forum access:	3ms
check topic access:	3ms
track hit:	32ms
get topic data:	10ms
get forum data:	2ms
get page messages:	51ms
get tp. blocked users:	1ms
others:	371ms

total:	493ms