Goku, já que a sua pergunta faz uso do Linq to SQL, então vou considerar apenas as transações envolvendo Banco de Dados e vou desconsiderar as transações com objetos instaciados.
Então, para se obter o bom uso de uma Transação, devemos respeitar o ACID.:
Atomicidade: Uma transação deve ser uma unidade atômica de trabalho; ou todas as suas modificações de dados são executadas ou
nenhuma delas é executada.¹
Já que as transações devem ser atomicas e por boa pratica os métodos precisam ter uma unica responsabilidade, então é interresante que se tenha apenas uma Transação por método.
Consistência: Quando concluída, uma transação deve deixar todos os dados em um estado consistente. Em um banco de dados relacional,
todas as regras devem ser aplicadas às modificações da transação para
manter toda a integridade dos dados. Todas as estruturas de dados
internas, tais como índices em árvore B ou listas duplamente
vinculadas, devem estar corretas ao término da transação.¹
Isolamento: Modificações feitas por transações simultâneas devem ser isoladas das modificações feitas por qualquer outra transação
simultânea. Uma transação reconhece os dados no estado em que estavam
antes de outra transação simultânea tê-los modificado ou reconhece os
dados depois que a segunda transação tiver sido concluída, mas não
reconhece um estado intermediário. Isso é chamado serializabilidade
porque resulta na capacidade de recarregar os dados iniciais e
reexecutar uma série de transações de modo que os dados obtidos
estejam no mesmo estado em que estavam depois que as transações
originais foram executadas.¹
Durabilidade: Depois que uma transação tiver sido concluída, seus efeitos ficam permanentemente no sistema. As modificações persistem
até mesmo no caso de uma queda do sistema.¹
Então tendo isto em mente, então talvez o seu método faria um melhor proveito da TransactionScope se fosse implementado da seguinte forma.:
ModeloColDataContext dm = DataContextFactory.GetContext(usuario);
{
if (documento > 0)
{
try
{
using (TransactionScope tr = new TransactionScope())
{
var updateDocumento = dm.ExecuteQuery<String>(@"
UPDATE TABELA_DOCUMENTOS
SET CLIENTE = {0}
WHERE CLIENTE = {1} AND DOCUMENTO = {2}
", clienteNovo, clienteAtual, documento);
if (updateDocumento == valorEsperado)
{
// realiza mais algunas operações no Banco
// note, caso o updateDocumento não tenha o valor esperado ou ocorra algum erro durante as consultas posteriores, o "UPDATE TABELA_DOCUMENTOS..." será desfeito.
tr.Complete();
}
}
}
catch (Exception err)
{
//realiza algum tratamento para o erro
}
}
}
Porém, se os seus comandos DMLs são completamente indepedentes e devem ser persitidos indepedente do resultado dos demais (que é o que está ocorrendo no seu exemplo), então não há a necessidade de utilizar o TransactionScope, afinal as suas operações não precisam ser Atomicas.
P.S.: outro ponto que achei estranho no seu exemplo, foi a não parametrização da consulta. Ao parametrizar você ganha em segurança (evita injeção de SQL) e desempenho (o banco consegue colocar o plano de execução da consulta em cache).
¹ - Transações (Mecanismo de Banco de Dados)