この例だけで言うと、実際に試してみてトランザクションが正常にコミットできていれば、副次的な問題が起きる可能性は低いと思います。
以下は余談ですが、例外以外の要因でトランザクションを阻害するケースを考えてみました。
問題が起きる可能性がああるのはdo
ブロック内でreturn
してしまった場合が考えられます。
ブロック内でのreturn
はブロックを呼び出しているyield
の終了だけでなく、ブロックの呼び出し元の終了も行うためです。
-- hoge.rb --
def aaa
yield
p 123
end
def hoge
aaa do
p 'aiueo'
return 200
end
100
end
ret = hoge
p ret
上記のスクリプトを実行すると以下のようになります。
$ruby hoge.rb
"aiueo"
200
# "123"の出力もないし、戻り値が100でなくてブロック内でreturnした200になってる
この例から分るようにブロック内のreturn
は結構混乱要因です。ActiveRecord::Base.transaction
の実装次第では危険があります。
begin
yield
ensure
コミット
end
という実装であればブロック内でreturn
されようが必ずコミットはされるでしょうが、例外が出たらロールバックすることを考えると、
begin
yield
コミット
rescue Exception=> e
ロールバック
end
という実装である可能性もあるかもしれません。
後者の実装の場合、ActiveRecord::Base.transaction
のdo
ブロックでreturn
を使うと、まずコミットされないままトランザクションだけは継続し、同じくdo
ブロック呼び出し元のコントローラメソッドもおそらく即終了となるため、何らかのDB例外につながると推測できそうです。
以下追記
ActiveRecord::Base.transaction
のソースコードをざっと見てみたところ、
ConnectionAdapters::DatabaseStatements#transaction
に定義がされています。
def transaction(options = {})
options.assert_valid_keys :requires_new, :joinable, :isolation
if !options[:requires_new] && current_transaction.joinable?
if options[:isolation]
raise ActiveRecord::TransactionIsolationError, "cannot set isolation when joining a transaction"
end
yield
else
transaction_manager.within_new_transaction(options) { yield }
end
rescue ActiveRecord::Rollback
# rollbacks are silently swallowed
end
これだけを見ると、yieldで呼び出されたdoブロックでreturnが使われると果たして動作がどうなるのか分からない感じがします。そもそもこのyieldに到達するための条件すらよく分かりませんが、とにかくこのパスを通過する場合に限っては、do
ブロック内のreturn
は危ないかもしれません。
それ以外の場合はActiveRecord::ConnectionAdapters::TransactionState
クラスの#within_new_transaction
が処理を行うようです。
def within_new_transaction(options = {})
transaction = begin_transaction options
yield
rescue Exception => error
rollback_transaction if transaction
raise
ensure
unless error
if Thread.current.status == 'aborting'
rollback_transaction if transaction
else
begin
commit_transaction
rescue Exception
transaction.rollback unless transaction.state.completed?
raise
end
end
end
これを見る限り、ActiveRecord
はyield
がreturn
で抜けられても、その後ensure
ブロックにてエラーでチェックを行い、正常ならコミットをしてるっぽいです。
このような実装なのでActiveRecord
に関しては、一部不明な点がありますが、多分、大体、return
を使っても安全そうです。