http://ayende.comAyende @ RahienCopyright (C) Ayende Rahien 2004 - 2021 (c) 202660Yeah get rid of the ugly out parameter and rename it to something else. It were quit close to the solution I provided :) http://ayende.com/4453/refactor-this-results#comment4http://ayende.com/4453/refactor-this-results#comment4Tue, 06 Apr 2010 15:45:45 GMTnice. I'd replace the SuccessfulCompletion method to return a struct of two booleans, IsSuccessful and WasDisposed @ICR: Better than having an unneeded member, and nicer than the out param http://ayende.com/4453/refactor-this-results#comment3http://ayende.com/4453/refactor-this-results#comment3Tue, 06 Apr 2010 14:19:58 GMTIsn't there some way to make the creator of the TransactionScope responsible for disposing it? It's hard to see why that should be a concern of this method or class. http://ayende.com/4453/refactor-this-results#comment2http://ayende.com/4453/refactor-this-results#comment2Tue, 06 Apr 2010 13:27:38 GMTWhy not put txDisposed as a member of the class? That would avoid the ugly out parameter. http://ayende.com/4453/refactor-this-results#comment1http://ayende.com/4453/refactor-this-results#comment1Tue, 06 Apr 2010 11:43:38 GMT