Skip to main content
2 de 2
adicionou 209 caracteres ao conteúdo
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil

O que você deve fazer só você pode decidir.

Eu sempre prefiro o simples, 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 provavelmente não conseguirá fazer isto, estou colocando porque talvez seja a forma que eu eu faria.

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 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 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.

Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil