Кто-нибудь использовал конструкцию вида такого вида?
Ошибка = Ложь;
Попытка
НачатьТранзакцию();
что-то делаем;
Если НЕ Ошибка Тогда
ЗафиксироватьТранзакцию
Иначе
ОтменитьТранзакцию()
КонецЕсли;
Исключение
Если ТранзакцияАктивна() Тогда
ОтменитьТранзакцию();
КонецЕсли;
Что-то делаем;
КонецПопытки;

Может ли привести к каким-то проблемам?

Если ошибка не связана с работой транзакции, то ты об этом не узнаешь, например деление на ноль

(1) имелось ввиду, возможно ли падение программы при такой конструкции? Предполагается, что причины ошибки фиксируются в "что-то делается"

на вскидку - вроде все очень даже правильно сделано, я подобные конструкции постоянно использую

(0)Проблема только одна - бездумное втыкание транзакции, где она вообще не нужна, это 99% проблем.

Все по феншую: https://its.1c.ru/db/metod8dev#content:2313:hdoc

(4) она там точно нужна

(6) сомневаюсь

(7) почему же? Там происходит проведение ряда документов по данным из другой базы. В момент проведения любого может возникнуть ошибка или что-то такое. И нужно откатиться будет.

(0)нет вывода никакой инфы юзеру о проблемах - это уже жирный минус.

(9) как нету?
ведь есть Что-то делаем; после исключения...вот там оно и будет...вот прям стопудова...я хорошо думаю о людях, пока они не докажут обратное)

(8) вот-вот. Явное втыкание одной большой транзакции туда, куда не надо.

(11) Ну а как тогда правильно? Есть определенный ответ от внешнего источника. Мы его или получаем или откатываемся к моменту до получения.

(11) а почему не надо?
а как тогда отменить создание кучи связанных сущностей, если где-то в коде произошла ошибка?

(0) код, который после попытки, может скопытиться с формулировкой "В данной транзакции уже происходили ошибки"

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

(14) а что для этого должно произойти?

(15) ты это как определил? Обычно такие костыли лежат в модуле, который вызывается из модуля, который вызывается ПриЗаписи. Это же "haha classic"!

(12) ты проводишь 200 документов в одной большой транзакции, не провёлся один - ты откатываешь транзакцию.
В этот момент (в момент твоей транзакции, база нервно курит в сторонке)

Вопрос, и нафига оно надо ? Потом повторно опять проводить 200 документов, заместо одного ?

(13) они не связаны

(16) в (5) описано

(18) не 200. А около 4-5. Цепочка вида: Заказ-Реализация-Корректировка... что-то вроде того.

(18) судя по (20) таки связаны

(17) Не, у меня перед глазами картина такая, что код автора работает выделенно (внешняя обработочка какая-нибудь или кнопка)

(17) нет. Это вызывается регламентным заданием, отдельная процедура.

(23) иногда бывает, что транзакции начинаются и неявно.
В общем, тебе виднее. Мне добавить нечего.

(20) А не логичнее будет обернуть в Попытку только запись документа, а при исключении - отменять транзакцию и выходить из цикла/функции или что там.

(0) Вообще непонятна вложенность.
Классика жанра :

НачатьТранзакцию();
...
Попытка
   ЗафиксироватьТранзакцию();
Исключение
   ОтменитьТранзакцию();
КонецПопытки

То есть внутри попытка-исключение действия при успешному / неуспешном выполнению транзакции. А не все внутри попытки.

+ (26) То есть в попытку завернуть только потенциально опасные действия. А не все.

(27) "..." надо тоже в попытку заворачивать

А может сломаться именно на НачатьТранзакцию?

(29) нет, может сломаться на "..."

(30) Исправлюсь.
(0) Вообще непонятна вложенность.
Классика жанра :

НачатьТранзакцию();
...(1)
Попытка
   ...(2)
   ЗафиксироватьТранзакцию();
Исключение
   ОтменитьТранзакцию();
   ...(3)
КонецПопытки
, где
...(1) - подготовительные действия, ошибка в которых не приводит к изменению объектов БД.
...(2) - потенциально опасные действия, ошибка в которых может привести к нежелательным изменениям объектов БД. (Например нарушению ссылочной целостности)
...(3) - действия при возникновении исключительной ситуации.

(31) чуточку упростил без потери наглядности:

НачатьТранзакцию();
   // подготовительные действия, ошибка в которых не приводит к изменению объектов БД
Попытка
   //потенциально опасные действия, ошибка в которых может привести к нежелательным изменениям объектов БД. (Например нарушению ссылочной целостности)
   ЗафиксироватьТранзакцию();
Исключение
   ОтменитьТранзакцию();
   //действия при возникновении исключительной ситуации
КонецПопытки

(32) я бы перетащил НачатьТранзакцию внутрь Попытка
Так будет и наглядней и правильней.

Народ, а если в процедуре есть ряд действий, которые могут вызвать ошибку, но есть и действия, которые ошибки в БД не вызовут, то лучше использовать одну Попытка - Исключение или для каждого действия с БД отдельные?

(0) Я пользовался вот таким:

НачатьТранзакцию();

Попытка
    что-то делаем....
    ОтменитьТранзакцию();
Исключение

    ОтменитьТранзакцию();
    ВызватьИсключение;
конецПопытки

+(35) да-да - именно отменить независимо от того ошибка там или не ошибка ))

(36) может вместо 1го Отменить все таки Зафиксировать? ;-)

(37) Неа - надо сформировать документы с заданными параметрами, получить из них данные, а потом стереть документы. ))