Добрый день.
Подскажите, есть регистр Штрихкоды, у него одно измерение Штрихкод (числовое значение).
Стоит задача написать функцию, которая будет возвращать следующее значение по порядку, и это значение должно записаться в этот регистр. Обращение к функции будет частое и с разных сеансов.
Насколько корректна след. функция.
    НачатьТранзакцию();
    
    Попытка
    
          БлокировкаДанных = Новый БлокировкаДанных;
          ЭлементБлокировкиДанных = БлокировкаДанных.Добавить("РегистрСведений.Штрихкоды");
          ЭлементБлокировкиДанных.Режим = РежимБлокировкиДанных.Исключительный;
          БлокировкаДанных.Заблокировать();
        
        НовыйШтрихкод = 100000;
        
        Запрос = Новый Запрос("ВЫБРАТЬ
                              |    МАКСИМУМ(Штрихкоды.Штрихкод) КАК Штрихкод
                              |ИЗ
                              |    РегистрСведений.Штрихкоды КАК Штрихкоды);
        Выборка = Запрос.Выполнить().Выбрать();
        Если Выборка.Следующий() Тогда
            Если Выборка.Штрихкод <> NULL Тогда
                НовыйШтрихкод = Выборка.Штрихкод + 1;
            КонецЕсли;
        КонецЕсли;
        
        НоваяЗапись = РегистрыСведений.Штрихкоды.СоздатьМенеджерЗаписи();
        НоваяЗапись.Штрихкод = НовыйШтрихкод;
        НоваяЗапись.Записать();    
        
        ЗафиксироватьТранзакцию();
    
    Исключение
    
        НовыйШтрихкод = 0;
        ОтменитьТранзакцию();
        ВызватьИсключение "Ошибка при создании штрихкода. Попробуйте снова. " + ОписаниеОшибки();
    
    КонецПопытки;
    
    Возврат НовыйШтрихкод;

1. Чем не устроил механизм генерации штрихкодов из типовых?

2. Если будет реально большая параллельная нагрузка, лучше использовать документ.

(0)

Я конечно извиняюсь:
1) У вас только "свои" штрихкоды ?
Ибо, если и "свои" и чужие -- может получиться нехорошо.
2) Число для хранения штрихкода. Не строка ?

(1) база самописка, особо не трогали БСП -шные функции
ну я бы не сказал чтоб прям много, просто вероятность небольшая есть, что одномоментно могут запустить процедуру.
(2) да свои только штрихкоды
ну у нас число, так проще

Хотелось просто понять, функция корректная, а то ранее блокировки не использовал, мало ли что упустил

(4) Мне РС для таких целей нравится больше
ТЗ на такие штрихкоды выдал производитель мед. оборудования, все считывается корректно.

(3) Код содержит несколько мин замедленного действия.

как самое простое - все действия по подготовке блокировки вынести из транзакции, оставить только Заблокировать()

(0) А зачем здесь вообще блокировка? Чтобы что?

(6) это каких? выглядит норм

(8) Чтобы поставить в очередь два параллельных сеанса.

может через очереь писать?

Что-то похожее было то ли в УТ 10.2, то ли 10.3.
Переделывал: завел отдельный РС (кажется, не помню) для хранения максимального значения ШК - блокировки регистра ШК замучили.

(6) например?
(0) почему Измерение, а не Ресурс?

(10) А смысл? Внутри попытки пишешь в регистр. Либо записал, либо нет. Вот и все. Зачем здесь блокировка?

(14) 2 сеанса могут прочитать одно и тоже значение.

(9) (13) Посмотрел еще раз и уже сам не вижу. :) По блокировкам, во всяком случае.

(14) Если нет, что дальше?

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

(18) Даже не говорите про типовые ШК :-). Каждый раз приходится переносить номенклатуру в Измерения, когда для разных номенклатур необходимо заводить одинаковые ШК.

(19) А в каком случае это нужно? ШК как бы для идентификации задуман. :)

(20) Всякие подарочные упаковки в алкоголе. Оформляются как отдельная позиция номенклатуры (так уж повелось), а ean13 тот же.

(21) а нет такой практики, в этом случае - приклеить свой ШК? Упаковка не красивая будет или что? Алкашовскую же марку все равно надо будет читать, а она вроде уникальная же?

Это через модификаторы решается,но везде они есть.
Тогда позиция + можификатор - это подарочная упаковка и товар.

(17) Все то же, что у ТС в блоке ИСКЛЮЧЕНИЕ, только без ОтменитьТранзакцию()

Блокировка на весь регистр? Это что за экстренные меры?

Если выборка. Следующий (). О боги. Если уж весь регистр блокируешь, экономь каждую копеечку. Запрос. Пустой()

Плюс ситуация : у пользователя тремор, нажал кнопку 45 раз. Что он получит в итоге?

А то у нас тут как раз случай :пользователь жмет "получить счёт" 20 раз, все 20 раз отрабатывают, пользователь что там получает, супер. Только у нас 19 дублей.

(24) OK, вызывающий получает бяку и.. что делает?

(25) А на что надо?

(30) так на диск же

(29) То же, что и раньше. В чем  вопрос? В приведенном куске кода блокировка и транзакция лишние

(15) Этот код никогда не вернет два одинаковых и не нулевых значения для двух сеансов.

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

(34) Вы сломали регистр сведений? Что-то не верится

(34) Двумя сеансами прочитать одно и то же значение из регистра можно. Записать нельзя.

(35) почему сломали, просто сеансы поочереди переписали последнее значение, то есть они одномоментно считали одно и то последнее значение, а потом в регистр записали поочереди это последнее значение +1.

(37) И вы получили в регистре сведений неуникальные значения измерения? Не верю

(38) вы с РС работали, этот код если выполнить дважды с одним и тем же штрихкодом, просто перезапишет существующую запись
НоваяЗапись = РегистрыСведений.Штрихкоды.СоздатьМенеджерЗаписи();
        НоваяЗапись.Штрихкод = НовыйШтрихкод;
        НоваяЗапись.Записать();

(39) Понятно

(31) Не понял

(39) Можно было бы написать НоваяЗапись.Записать(ложь) вместо блокировки

(42) Ты на машине ездишь?
Две машины на перекрестке. Если обе газанут, то обе побьются, но одна может проскочить. Ты предлагаешь алгоритм "если не проскочил, пробуй еще, пока не проскочишь или не развалишься". Но во всем мире используют светофоры.

Так понятно?

(42) Но не нужно.

(44) Блокировка так или иначе будет установлена. Вопрос в том, когда. Устанавливать ли блокировку пораньше - вопрос спорный

(45) вот именно, что должна быть установлена ранее

(45) Нет там ничего спорного.

(45) Вопрос пораньше или попозже начинает иметь значение, когда там десятки сеансов в секунду долбятся.

А в данному случае блокировка пораньше позволяет каждому сеансу получить свой номер без проблем. А не городить сложную логику обработки ошибок и долго вылавливать в ней баги.

(48) Десятки сеансов в секунду долбятся почитать. Пара сеансов в минуту долбятся записать. Иногда, раз в год, эти два сеанса в минуту совпадают...

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