Tenho duas funções, uma retorna apenas o primeiro resultado e a outra também retorna o primeiro resultado ou retorna mais que um resultado.
Criei o parâmetro "string todos" pra definir se ira retornar um ou mais resultados na segunda função. Mas a segunda função se tornou mais complexa e eu poderia ter feito mais simples pra funcionar apenas pra retornar vários resultados
Devo continuar programando complexo igual a segunda função ou deixar-la mais simples e com um objetivo só?
Chamada pra teste:
Pastas pastas1 = new Pastas(@"C:\New folder\Nova pasta\");
Pastas.Teste1(pastas1, "3.txt");
Código:
// Mapeia todo as pastas e subpastas de um diretório
class Pastas
{
public DirectoryInfo Diretorio;
public List<Pastas> SubDiretorios = new List<Pastas>();
public FileInfo[] Arquivos;
public Pastas(string pasta)
{
Diretorio = new DirectoryInfo(pasta);
foreach (DirectoryInfo di in Diretorio.GetDirectories())
{
SubDiretorios.Add(new Pastas(di.FullName));
}
Arquivos = Diretorio.GetFiles();
}
// Primeira Função
// Percorre as pastas e retorna o primeiro arquivo pelo nome
public FileInfo GetArquivo(string nome)
{
FileInfo arquivo = Arquivos.FirstOrDefault(x => x.Name == nome);
if (arquivo is null)
{
foreach (Pastas item in SubDiretorios)
{
arquivo = item.GetArquivo(nome);
if (arquivo != null)
break;
}
}
return arquivo;
}
// Segunda função
public List<FileInfo> GetArquivo(string nome, bool todos)
{
List<FileInfo> arquivos = new List<FileInfo>();
if (todos)
arquivos = Arquivos.Where(x => x.Name == nome).ToList();
else
{
FileInfo arquivo = Arquivos.FirstOrDefault(x => x.Name == nome);
if (arquivo != null)
arquivos.Add(arquivo);
}
if (todos || arquivos.Count == 0)
{
foreach (Pastas item in SubDiretorios)
{
arquivos.AddRange(item.GetArquivo(nome, todos));
bool apenasOPrimeiro = !todos;
if (apenasOPrimeiro && arquivos.Count == 1)
break;
}
}
return arquivos;
}
public static void Teste1(Pastas pastas, string nomearquivo)
{
FileInfo arquivo = pastas.GetArquivo(nomearquivo);
if (arquivo != null)
Debug.Print(arquivo.FullName);
else
Debug.Print("arquivo nulo");
}
public static void Teste2(Pastas pastas, string nomearquivo)
{
List<FileInfo> arquivos = pastas.GetArquivo(nomearquivo, true);
if (arquivos.Count > 0)
arquivos.ForEach(x => Debug.Print(x.FullName));
else
Debug.Print("arquivo nulo");
}
// GetArquivo(..., false)
public static void Teste3(Pastas pastas, string nomearquivo)
{
List<FileInfo> arquivos = pastas.GetArquivo(nomearquivo, false);
if (arquivos.Count > 0)
arquivos.ForEach(x => Debug.Print(x.FullName));
else
Debug.Print("arquivo nulo");
}
}