Uma forma que eu aprendi onde eu trabalho é deixar bem claro o nome das variáveis pro que eu quero fazer.
Isso já evita muitos comentários desnecessários.
Vou dar um exemplo:
foreach (var perfilDoUsuario in listaDePerfisDoUsuario)
{
if (perfilDoUsuario.Value.ToUpper().Contains((ListaDePerfisDoRelatorio.Gerente).ObterDescricao().ToUpper()))
{
listaDeTiposDePerfis.Remove((int)ListaDePerfisDoRelatorio.Gerente);
listaDeTiposDePerfis.Remove((int)ListaDePerfisDoRelatorio.Administrador);
}
}
foreach (var perfilDoUsuario in listaDePerfisDoUsuario)
{
if (perfilDoUsuario.Value.ToUpper().Contains((ListaDePerfisDoRelatorio.Gerente).ObterDescricao().ToUpper()))
{
listaDeTiposDePerfis.Remove((int)ListaDePerfisDoRelatorio.Gerente);
listaDeTiposDePerfis.Remove((int)ListaDePerfisDoRelatorio.Administrador);
}
}
Nesse trecho aí eu deixo bem claro quais perfis eu que quero remover da lista que eu tenho.
Claro que isso não deixa absolutamente explicado o porquê de eu ter feito isso, mas já ajuda um pouco.
Antigamente eu colocava o número direto do perfil, sem identificá-lo.
É só uma dica adicional.