Гость
Войти | Профиль | Очистить
Действия ...
Форумы / Caché, Ensemble, DeepSee, MiniM, IRIS, GT.M [игнор отключен] [закрыт для гостей] / TS TRY CATCH TC или TRY TS TC CATCH ? / 13 сообщений из 13, страница 1 из 1
14.12.2016, 17:03
    #39367041
u78
u78
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
TS TRY CATCH TC или TRY TS TC CATCH ?
Вариант 1
Код: plaintext
1.
2.
3.
4.
5.
6.
 try {
 TSTART  
 TCOMMIT
 }
 catch {
 TROLLBACK
 }
или
Вариант 2
Код: plaintext
1.
2.
3.
4.
5.
6.
7.
8.
9.
 TSTART
 try {
  
 
 }
 catch {
 
 }
 IF ok TCOMMIT
 Else  TROLLBACK

Какой вариант благославляется, а какой не благославляется? Почему?
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
14.12.2016, 18:49
    #39367133
Блок А.Н.
Блок А.Н.
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
TS TRY CATCH TC или TRY TS TC CATCH ?
Второй вариант странный, зачем он?
Я бы делал trollback 1, иначе можете откатить чужую транзакцию.
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
14.12.2016, 18:50
    #39367134
servit
servit
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
TS TRY CATCH TC или TRY TS TC CATCH ?
u78 ,

Например, в исходниках %SYS встречается только вариант №1 (с вариациями). Но возможно плохо искал.
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
14.12.2016, 18:57
    #39367139
u78
u78
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
TS TRY CATCH TC или TRY TS TC CATCH ?
Блок А.Н.,

да, надо TROLLBACK 1, но сейчас вопрос не в этом
спрашивая коллег, столкнулся с тем - что оба этих варианта коллеги отстаивают как логичные и богоугодные, вот решил коммьюнити спросить
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
14.12.2016, 19:22
    #39367162
Шваров Евгений
Шваров Евгений
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
TS TRY CATCH TC или TRY TS TC CATCH ?
u78,

Предлагаю спросить еще и это комьюнити - вдруг кто-то еще отстаивает вариант 2 и с аргументацией?
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
15.12.2016, 07:40
    #39367314
Ptn
Ptn
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
TS TRY CATCH TC или TRY TS TC CATCH ?
u78,

Наверное вкусовщина. Глазу и логике в отдельном блоке приятен первый вариант.

Второй вариант получается сам собой же при вызовах
Код: plaintext
1.
2.
3.
4.
5.
 TSTART

 SET ok = Вариант 1

 IF ok TCOMMIT
 Else  TROLLBACK
Что до аргументов, то Вариант 1 ловит исключение при выполнении TCOMMIT, а вот во втором варианте - нет.
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
15.12.2016, 09:11
    #39367342
servit
servit
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
TS TRY CATCH TC или TRY TS TC CATCH ?
u78 ,

Кроме того - при прочих равных - во втором варианте выполняется дополнительное условие. Причём всегда.
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
15.12.2016, 12:18
    #39367527
Petr0vi4444
Petr0vi4444
Гость
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
TS TRY CATCH TC или TRY TS TC CATCH ?
Я за гибридный вариант:
Код: plaintext
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
 TSTART
 Try {
    Set sc = ..doSomething()
 } Catch exception {
    Set sc = exception.AsStatus()
 }
 If ('sc) {
    TROLLBACK 1
 } Else {
    TCOMMIT
 }
 Quit sc
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
15.12.2016, 12:32
    #39367550
servit
servit
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
TS TRY CATCH TC или TRY TS TC CATCH ?
Petr0vi4444Я за гибридный вариант:Это второй вариант.

Здесь будет гораздо ценнее аргументация (вторая часть вопроса ТС).
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
15.12.2016, 14:00
    #39367702
Petr0vi4444
Petr0vi4444
Гость
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
TS TRY CATCH TC или TRY TS TC CATCH ?
servitЗдесь будет гораздо ценнее аргументация (вторая часть вопроса ТС).Во втором варианте единый блок обработки результата с типом %Status.
В первом варианте если в блоке Try не возникло исключения, но операция выполнилась неуспешно, после Catch нужно проверять результат и делать TROLLBACK в случае ошибки.
Это приводит к дублированию кода и если помимо TROLLBACK необходимо сделать ещё какие-нибудь операции, код становится слишком громоздским, например:
Код: plaintext
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
 TSTART
 Try {
    Set sc = ..save()
 } Catch exception {
    TROLLBACK 1
    Kill %objlasterror
    Do ##class(Log).addException(exception)
 }
 Quit:$D(exception) ..doSomethingIfError(exception.AsStatus())
 If ('sc) {
    TROLLBACK 1
    Kill %objlasterror
    Do ##class(Log).addError(sc)
    Quit ..doSomethingIfError(sc)
 }
 TCOMMIT
 Quit sc
 
P.S.Никак не пойму какими тэгами оформлять код
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
15.12.2016, 14:19
    #39367733
Блок А.Н.
Блок А.Н.
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
TS TRY CATCH TC или TRY TS TC CATCH ?
Petr0vi4444,

Непонятно, зачем вы это делаете. Почему бы в случае ошибки не выбросить исключение?
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
15.12.2016, 15:00
    #39367799
servit
servit
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
TS TRY CATCH TC или TRY TS TC CATCH ?
Petr0vi4444Во втором варианте единый блок обработки результата с типом %Status.
В первом варианте если в блоке Try не возникло исключения, но операция выполнилась неуспешно, после Catch нужно проверять результат и делать TROLLBACK в случае ошибки.Разве не проще сделать так, как сделано, например, в %SYSTEM.SQL?
Код: plaintext
1.
2.
3.
4.
5.
6.
7.
8.
9.
 Set sc = $$$OK
 Try {
    TSTART
    $$$ThrowOnError(..doSomething())
    TCOMMIT
 } Catch exception {
    TROLLBACK 1
    Set sc = exception.AsStatus()
 }
 Quit sc
 
P.S.Petr0vi4444Никак не пойму какими тэгами оформлять код
Вы и так оформляете правильными тегами.
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
15.12.2016, 15:48
    #39367880
DAiMor
DAiMor
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
TS TRY CATCH TC или TRY TS TC CATCH ?
Petr0vi4444,

Исходники оформлять нужно тегом [src], а у servit есть под рукой инструмент для корректной расскраски кода, что он и делает после публикации.
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
Форумы / Caché, Ensemble, DeepSee, MiniM, IRIS, GT.M [игнор отключен] [закрыт для гостей] / TS TRY CATCH TC или TRY TS TC CATCH ? / 13 сообщений из 13, страница 1 из 1
Целевая тема:
Создать новую тему:
Автор:
Найденые пользователи ...
Разблокировать пользователей ...
Читали тему (0):
Читали форум (0):
Пользователи онлайн (0):
Польз. соглашение
созд. / загр.: 416ms
Закрыть
start [/forum/topic.php?fid=39&tablet=1&tid=1556400]:
 0ms
get settings:
 8ms
get forum list:
 15ms
check forum access:
 4ms
check topic access:
 4ms
track hit:
 47ms
get topic data:
 13ms
get forum data:
 2ms
get page messages:
 52ms
get tp. blocked users:
 2ms
others: 269ms
total:  416ms
x
x
Закрыть


Просмотр
0 / 0
Close
Debug Console [Select Text]