Как правильно использовать транзакцию в цикле?

Question

[aopil]

$DB->begin(); // Start Transaction
try {
	$rs = $DB->queryUniqueObject("SELECT Balance FROM tbl_users WHERE UserId = '" . $userId . "' FOR UPDATE");
	$totalOrders = sizeof($arrOrders);
	for ($j = 0; $j < $totalOrders; $j++)
	{
		$finalBalance = $rs->Balance - $servicePrice;

		// много различного кода...

		$DB->execute("UPDATE tbl_users SET Balance = '" . $finalBalance . "' WHERE UserId = '" . $userId . "'");
	}
	$DB->commit(); // END Transaction
} catch (Exception $e) {
	$DB->rollback();
}

Теоретически, я предполагаю, что в моем коде будет выполнена 1 транзакция, но правильно ли это? Или я должен добавить

SELECT Balance FROM tbl_users WHERE UserId = '" . $userId . "' FOR UPDATE

внутрь for и тогда это будет N-ое количество транзакций в зависимости от кол-ва totalOrders?

P.S. То, что баланс нужно считать напрямую в SQL запросах я знаю, в данной ситуации это невозможно.

Comments

[FanatPHP]

в каких целях используется транзакция?

[FanatPHP]

также вопросы, что такое "$objDBCD14"?
и что за дебильная функция execute()?

[aopil]

FanatPHP,

в каких целях используется транзакция?

- Размещение заказа

также вопросы, что такое "$objDBCD14"?

это $DB класс базы.

и что за дебильная функция execute()?

execute - функция для запроса к бд.

[FanatPHP]

Для "Размещение заказа" транзакция не требуется.
если транзакция не нужна, то не надо её и использовать

[FanatPHP]

еще один вопрос
почему обновление баланса происходит $totalOrders раз, а не один?

[aopil]

FanatPHP, Почему не нужна? Тебе прилетают куча заказов по АПИ и если будет куча запросов параллельно - баланс пользователя может считаться не правильно.

[aopil]

FanatPHP, есть массив в котором пришло 5 заказов, они оформляются как 5 разных заказов, и используется цикл в котором разные проверки на ошибки, например из 5 заказов может быть мы примем только 3, остальные будут с ошибкой.

[FanatPHP]

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

[FanatPHP]

то что из 5 заказов может быть мы примем только 3 НИКАК не объясняет, почему обновление баланса происходит $totalOrders раз, а не один
В чем смысл обновлять 3 раза, а не один?
а если будет 100 заказов - сто раз обновлять?

[aopil]

FanatPHP,

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

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

[aopil]

FanatPHP,

то что из 5 заказов может быть мы примем только 3 НИКАК не объясняет, почему обновление баланса происходит $totalOrders раз, а не один
В чем смысл обновлять 3 раза, а не один?
а если будет 100 заказов - сто раз обновлять?

Вероятно вы правы, можно производить списание после выполнения цикла, но вопрос в данной теме у меня был другой.

Если вы советуете вытащить update баланса из цикла, то логично что и SELECT не нужно добавлять внутрь цикла чтобы запрашивался баланс каждый раз тоже не имеет смысла, верно?

[FanatPHP]

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

Ну то есть они становятся, но именно из-за блокировки. а транзакция здесь просто для красоты.

[aopil]

FanatPHP, какое-то противоречие в вашем ответе)) Так блокирует транзакция или нет, лично я на практике тестировал - с транзакцией скрипт отрабатывает правильно.

[FanatPHP]

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

[FanatPHP]

блокирует блокировка, а не транзакция
если это вдруг новость, то SELECT FOR UPDATE - это и есть блокировка.

[aopil]

FanatPHP,

если это вдруг новость, то SELECT FOR UPDATE - это и есть блокировка.

ну а блокировку снимается по завершению транзакции, верно?

[FanatPHP]

да при чем здесь транзакция-то?!

[aopil]

FanatPHP, всё при том же. У меня есть еще другие UPDATE запросы перед обновлением баланса для этого и нужна транзакция, поэтому и интересуюсь, получается блокировка от FOR UPDATE снимается после выполнения команды $DB_>commit();