Форум: "Начинающим";
Поиск по всему сайту: delphimaster.net;
Текущий архив: 2011.12.18;
Скачать: [xml.tar.bz2];




Вниз

правильное использование транзакций 


LDV   (2011-09-08 15:15) [0]

есть функция, которая добавляет товар в указанный раздел либо разделы (определяется параметром CategoryIds). Вопрос касается использования транзакций. Стоит ли делать разделение (как у меня в коде), когда товар добавляется только в один раздел (запись только одна, использовать транзакции не имеет смысла) и когда товар добавляется в несколько разделов (тут несколько insert-запросов, использование очевидно);


function TProduct.Add(const CategoryIds: string): Integer;
var
 CategoryIdList: TStrings;
 I: Integer;
begin
 CategoryIdList := TStringList.Create;
 try
   CategoryIdList.Text := CategoryIds;
   for I := 0 to CategoryIdList.Count - 1 do
     if not IsIntegerValue(StrToInt(CategoryIdList[I])) or not
       CategoryIsValid(StrToInt(CategoryIdList[I])) then
       raise Exception.CreateRes(
         @ERROR_INVALID_VALUE_FOR_CATEGORY_IDS_PARAMETER);
   if CategoryIdList.Count > 1 then
     Database.StartTransaction;
   try
     // Insert query here
     if CategoryIdList.Count > 1 then
       Database.Commit;
   except
     if CategoryIdList.Count > 1 then
       Database.Rollback;
     raise;
   end;
 finally
   FreeAndNil(CategoryIdList);
 end;
end;



Ega23 ©   (2011-09-08 15:23) [1]


>  Стоит ли делать разделение

Зачем? Есть список - его и вставляем. Проверить только, чтобы Count <> 0, а остальное всё должно быть единообразно.



OW ©   (2011-09-08 15:50) [2]


>  всё должно быть единообразно.

+1

да даже и для одной записи можно начать транзакцию. (она все равно неявно сама начинается)
Зато заморочек меньше.




Форум: "Начинающим";
Поиск по всему сайту: delphimaster.net;
Текущий архив: 2011.12.18;
Скачать: [xml.tar.bz2];




Наверх






Память: 0.75 MB
Время: 0.024 c
1-1277298765      granulated            2010-06-23 17:12  2011.12.18  
EInvalidPointer после выхода из функции.


2-1315307915      alexx777              2011-09-06 15:18  2011.12.18  
сменить пользователя


2-1315587791      Rucosinus             2011-09-09 21:03  2011.12.18  
Просмотр шрифтов из папки


1-1277470105      harisma               2010-06-25 16:48  2011.12.18  
Окно StayOnTop


4-1252399230      Jeyson                2009-09-08 12:40  2011.12.18  
одно и тоже консольное приложение работает по разному