Программно не было бы никакой причины. Причина может быть в удобочитаемости, хотя; Обертывание фрагмента произвольного кода в транзакции - это довольно хороший способ привлечь внимание будущих разработчиков к этому - или от чего-то другого.
Возможно также, что на момент написания этой статьи передовой практикой было заключать в транзакции любые операции с базой данных, так же, как упаковывать вещи в блоки try / catch.
Мое предложение: если код является более понятным и более читаемым из-за этого, или если все другие вызовы базы данных выполняются в транзакциях, оставьте его. В противном случае удалите его.