skat_sakh Jul 2 2013 at 07:00

Ещё один пример использования замыканий в PHP

2 min

8.3K

Website development * PHP *

From sandbox

Comments 7

sectus Jul 2 2013 at 07:52

1. Вы используете не замыкания (даже если они так называются в PHP), а анонимные функции.
2. Вы очень странно начинаете транзакции. Почему нельзя делать begin?

skat_sakh Jul 2 2013 at 08:35

1. Вы используете не замыкания (даже если они так называются в PHP), а анонимные функции.

хоть вы и ответили чуть ниже на это, но я всё же напишу, что это всё-таки замыкания.

2. Вы очень странно начинаете транзакции. Почему нельзя делать begin?

Если вы имеете ввиду mysqli_begin_transaction, то версия должна быть PHP 5 >= 5.5.0

sectus Jul 2 2013 at 08:52

Не знал, что такая функция отсутствует в mysqli (обычно имею дело с PDO). Но можно использовать BEGIN; или START TRANSACTION; как запрос. И за одно isAutocommitOn переименовать в inTransaction.

Не, Ваш код работает, но понятность его можно повысить.

sectus Jul 2 2013 at 07:59

1. Отменяется… потому что $this это как раз ссылка на переменную, которая не объявлена в функции…
_{чуть-чуть не успел отредактировать комментарий}
_{а сейчас ещё и промахнулся...}

tvolf Jul 2 2013 at 08:29

У вас в методе transaction() отсутствует код, который, собственно, и выполняет обратное включение автокоммита для случая, когда это необходимо, то есть, когда $need_to_off равна TRUE. Вероятно, просто упустили этот момент, так как в самом тексте статьи этот функционал имеется )
ps. Если, конечно, я все правильно понял )

skat_sakh Jul 2 2013 at 09:57

Да, я пропустил его просто. Уже исправил)

sectus Jul 2 2013 at 14:23

А, вообще, я было хотел воспользоваться таким подходом. Даже название придумал 'расширение транзакции'. Только мне не понравилось, что при внутреннем откате откатывается вся транзакция.