O que você deve fazer só você pode decidir.
Eu sempre prefiro o simples. Sem, sem ser simplório. Nem sempre é fácil dizer o que é simples. E tem que observar outras coisas, em alguns casos é melhor deixar um pouco mais complicado se tiver alguma vantagem clara em fazer isto.
Eu odeio essa ideia de passar um booleano para dizer o que o fazer (embora há casos para uso). Fiz até uma pergunta sobre isto: Por que não se deve usar um parâmetro booleano?.
Neste caso não é só o booleano, mesmo usar um enum
mais semântico não me agrada. O .NET faz isso e eu não gosto. Mas não é o fim do mundo usar. Prefiro isto que violar o DRY, por exemplo, mas tem maneiras de ter mais de um método, um para resultado único e outro para resultado múltiplo sem violar o DRY, ainda que pode ficar um pouco mais complexo.
Este caso seu me parece que ter os dois métodos é o ideal, e o parâmetro todos
nem deveria existir.
Note que o LINQ faz exatamente isto, tem um um método para pegar tudo ou para pegar só o primeiro e outro para pegar o default se nem isso conseguir.
Outra possibilidade não muito ruim é fazer um método que segue a mesma sistemática do LINQ e retornar um enumerador e aí se você quer só um ou vários pode compor uma uma query que faça isso de forma eficiente. Pra falar a verdade na forma criada seria o melhor a fazer. Mas eu sei que provável menteprovavelmente não conseguirá fazer isto, estou colocando porque talvez seja a forma que eu eu faria. Tem
Tem muitas soluções, mas não são perfeitas.
O código tem vários outros problemas. O uso do .ToList()
aí parece errado. Tem certeza que precisa fazer isto? Se não sabe usar o LINQ da forma correta faça do jeito normal sem ele. Só chame o ToList()
se se quiser concretizar a lista, não parece ser o necessário nesse ponto. A solução também não é bem o que foi colocado na outra resposta, mas enfim, não vou conseguir resolver tudo aqui.
O segundo if
parece não ser necessário, ele deveria estar dentro do primeiro (se mantiver o todos
). Esse arquivos.Count == 0
parece estranho aí, principalmente porque depois tem um arquivos.Count == 1
, mas nem posso afirmar sem entender o que precisa. Essa flag flag apenasOPrimeiro
parece desnecessária e só existe porque o código é confuso. A questão é que esse código certamente pode ser escrito de forma mais simples.
Tenho medo desses tipos Pastas
e List
, isto parece algo feito errado.