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

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.

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. Mas eu sei que provável mente 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.

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.

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.

Fonte Link
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. Mas eu sei que provável mente 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.

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.