Sua função é criar um objeto (ReadOnlyCollection
) que garanta o contrato que o objeto original não será alterado, desta forma você pode passar para métodos, possivelmente em outro contexto de execução (outra thread) e será garantido que ele será usado só como leitura. Portanto é uma forma mais semântica e mais segura de usar o objeto que deseja que nesse contexto seja imutável.
Note que ele não faz cópia alguma, não tem um processamento real, apenas mantém um objeto que é do mesmo tipo do original sem permitir as alterações no seu conteúdo. Em geral só faz sentido se for passar para outro método.
Todo o .NET foi modificado para aceitar este tipo de objeto e dar mais garantias, e seus códigos deveriam fazer o mesmo, afinal, fica mais seguro em todos os sentidos e por até ser mais performático, especialmente ajuda em ambientes concorrentes.
Se quer que o objeto original seja imutável sempre deve usar um ImmutableList
que faz parte de uma coleção de objetos que as pessoas desconhecem e devem ser escolhidos se não precisa fazer alterações nele. Infelizmente a maioria dos programadores não conhecem as novidades do C# e do .NET e continuam fazendo como era 15 anos atrás (por isso tenho uma palestra e planejo um dia ter um livro sobre C# Moderno).
Documentação.
Não faz mais sentido fazer da forma como fez, faça assim:
var list = new List<string> { "Maria", "Lucas", "Renata", "Yara", "Josias", "Morten Granau""Morten Granau", "Phaxe" };
Metodo(list.AsReadOnly());
Coloquei no GitHub para referência futura.
Considerando que a assinatura deste método seja mais ou menos Metodo(ReadOnlyCollection lista)
.
Em muitos casos o uso de um IReadOnlyCollection
pode ser suficiente.