Я правильно понимаю, что в коде ниже мне достаточно обложить

Question

Я правильно понимаю, что в коде ниже мне достаточно обложить

геттер и сеттер TokenValue каким-нибудь ReaderWriterLockSlim, чтобы никаких проблем с многопоточностью не было?
https://gist.github.com/ChXIII/f1b68f10273f55edd2a3d9a9108c2273

#backend #dot #programming #russian

0

04.03.2021

24 ответов

4 просмотра

Dr. Friedrich von Never

А ты уверен, что нужен именно ReaderWriterLock? Операция записи значительное время занимает? По дефолту я бы врапнул в локи, или даже сделал volatile филду, да и дело с концом. Но зависит от того, какая семантика тебе нужна во время обновления токена. Если я обновляю токен вот прям щас, то другим людям ещё можно им пользоваться, или уже нет?

0

06.03.2021

Ilya L Автор вопроса

Dr. Friedrich von Never
А ты уверен, что нужен именно ReaderWriterLock? Оп...

Самое важное забыл написать. Читателей параллельных много будет. Каждому лок выдавать жирно как-то. А писатель, рефрешер токена, один. Старый токен работает ещё минут 5, как минимум, работает. Поэтому лок на запись планировал брать именно перед самой записью, после получения нового токена. Надо защитить момент, когда в поле класса пишется новая ссылка на строку, как я понимаю. Если описывать цель, то я пишу апи клиент с авторизацией oauth2 (oidc) для использования в тестах. Параллельные тесты, параллельная генерация сущностей для теста, ещё рефит на каждый интерфейс апи создаёт новый апи клиент. Рефреш токена по таймеру планирую запускать. Потенциально мне ещё словарь юзер-токен придётся делать и что-то вокруг костылить, потому что первое получение токена ресурсоёмкое, с запуском селениума. Но до этого кода я ещё не добрался :)

0

06.03.2021

Dr. Friedrich von Never

Ilya L
Самое важное забыл написать. Читателей параллельны...

Ок. Считаю, что reader-writer lock норм сработает.

0

06.03.2021

Mr. N

https://prnt.sc/10edkm9

0

06.03.2021

Dr. Friedrich von Never

Mr. N
https://prnt.sc/10edkm9

Мистер опять жестит.

0

06.03.2021

Mr. N

Dr. Friedrich von Never
Мистер опять жестит.

норм код.😁

0

06.03.2021

Ilya L Автор вопроса

Mr. N
https://prnt.sc/10edkm9

Прикольный код. Но теряется поддержка нескольких юзеров с разными токенами из-за статика. И не получится безопасно писать/читать в поле токена. Забей, спасибо за помощь.

0

06.03.2021

Mr. N

Ilya L
Прикольный код. Но теряется поддержка нескольких ю...

static тебе гарантирует безопасность

0

06.03.2021

Ilya L Автор вопроса

Mr. N
static тебе гарантирует безопасность

Тогда бы его назвали safe, а не static!

0

06.03.2021

Dr. Friedrich von Never

Mr. N
static тебе гарантирует безопасность

Ох ёпт, начинается.

0

06.03.2021

Ilya L Автор вопроса

Mr. N
static тебе гарантирует безопасность

А если серьёзно, то расскажи, откуда ты это взял. Это ж неверно.

0

06.03.2021

Mr. N

Ilya L
А если серьёзно, то расскажи, откуда ты это взял. ...

Ну хз, когда-то слышал что-то такое

0

06.03.2021

Mr. N

Ilya L
А если серьёзно, то расскажи, откуда ты это взял. ...

Когда-то у меня в многопоточном коде возникали коллапсы, потому что много потоков рвались к одному полю, ошибок не было, все потоки брали данные из static поля, и поэтому все становились в очеридь

0

06.03.2021

βλαδιμιρ

Mr. N
Когда-то у меня в многопоточном коде возникали кол...

в реббит очередь становились?

0

06.03.2021

Ilya L Автор вопроса

Mr. N
Когда-то у меня в многопоточном коде возникали кол...

Ну так у тебя только чтение тут. Я думаю, проблема изначально была из-за того, что инициализация не статик поля происходила одновременно с попытками чтения. А вот статик гарантирует инициализацию до первого обращения.

0

06.03.2021

Dr. Friedrich von Never

Ilya L
Ну так у тебя только чтение тут. Я думаю, проблема...

(ну, чаще всего гарантирует, а иногда нет)

0

06.03.2021

Mr. N

Dr. Friedrich von Never
(ну, чаще всего гарантирует, а иногда нет)

ну и. https://github.com/maksim2143/TestStaticFields/blob/master/TestStaticFields/Program.cs

0

06.03.2021