Skip to main content
adicionou 97 caracteres ao conteúdo
Fonte Link
Jéf Bueno
  • 69,8mil
  • 22
  • 134
  • 244
adicionou 310 caracteres ao conteúdo
Fonte Link
Jéf Bueno
  • 69,8mil
  • 22
  • 134
  • 244
  • Não capture exceções se não for fazer algo útil com elas, se você precisa ao menos transcrever isso pra algum lugar (mesmo que seja uma caixa de mensagem) faça isso no ponto de entrada do código (no método Main() ou algum similar);. Aqui no site tem algumas publicações que falam sobre isso, por exemplo:

  • O retorno do método Equals é um boolean, então isso:

    if(record[positionOfSearchTerm].Equals(searchTerm)) 
        return true; 
    else 
        return false;
    

    é exatamente a mesma coisa que isto:

    return record[positionOfSearchTerm].Equals(searchTerm);
    

    e a segunda forma é muito mais enxuta e simples de ler;

  • O método readRecord não deveria ser responsável pela mensagem "Record not found", isso deveria estar a cargo do método que o chama. Seria uma boa ideia se o readRecord apenas retornasse um conjunto vazio para o seu chamador quando não encontrar nenhum registro que atenda ao critério de busca;;

  • Não é nenhuma uma regra, mas há uma convenção de nomenclatura para C#, talvez seja uma referência bacana para te guiar na escolha de nomes.

  • Não capture exceções se não for fazer algo útil com elas, se você precisa ao menos transcrever isso pra algum lugar (mesmo que seja uma caixa de mensagem) faça isso no ponto de entrada do código (no método Main() ou algum similar);

  • O retorno do método Equals é um boolean, então isso:

    if(record[positionOfSearchTerm].Equals(searchTerm)) 
        return true; 
    else 
        return false;
    

    é exatamente a mesma coisa que isto:

    return record[positionOfSearchTerm].Equals(searchTerm);
    

    e a segunda forma é muito mais enxuta e simples de ler;

  • O método readRecord não deveria ser responsável pela mensagem "Record not found", isso deveria estar a cargo do método que o chama. Seria uma boa ideia se o readRecord apenas retornasse um conjunto vazio para o seu chamador quando não encontrar nenhum registro que atenda ao critério de busca;;

  • Não é nenhuma uma regra, mas há uma convenção de nomenclatura para C#, talvez seja uma referência bacana para te guiar na escolha de nomes.

  • Não capture exceções se não for fazer algo útil com elas, se você precisa ao menos transcrever isso pra algum lugar (mesmo que seja uma caixa de mensagem) faça isso no ponto de entrada do código (no método Main() ou algum similar). Aqui no site tem algumas publicações que falam sobre isso, por exemplo:

  • O retorno do método Equals é um boolean, então isso:

    if(record[positionOfSearchTerm].Equals(searchTerm)) 
        return true; 
    else 
        return false;
    

    é exatamente a mesma coisa que isto:

    return record[positionOfSearchTerm].Equals(searchTerm);
    

    e a segunda forma é muito mais enxuta e simples de ler;

  • O método readRecord não deveria ser responsável pela mensagem "Record not found", isso deveria estar a cargo do método que o chama. Seria uma boa ideia se o readRecord apenas retornasse um conjunto vazio para o seu chamador quando não encontrar nenhum registro que atenda ao critério de busca;;

  • Não é nenhuma uma regra, mas há uma convenção de nomenclatura para C#, talvez seja uma referência bacana para te guiar na escolha de nomes.

Fonte Link
Jéf Bueno
  • 69,8mil
  • 22
  • 134
  • 244

Tem algumas coisas pra arrumar no seu código e, de forma geral, a solução como um todo poderia ser um pouco melhor. Quando eu comecei a escrever a resposta pensei em propôr uma solução adaptada, mas isso mudaria muito seu código inicial e não sei se isso seria benéfico agora. Portanto, decidi apenas adaptar seu código original para atender o problema que você relata na sua publicação...

Alguns pontos que eu acho interessante citar antes de mostrar a solução:

  • Não capture exceções se não for fazer algo útil com elas, se você precisa ao menos transcrever isso pra algum lugar (mesmo que seja uma caixa de mensagem) faça isso no ponto de entrada do código (no método Main() ou algum similar);

  • O retorno do método Equals é um boolean, então isso:

    if(record[positionOfSearchTerm].Equals(searchTerm)) 
        return true; 
    else 
        return false;
    

    é exatamente a mesma coisa que isto:

    return record[positionOfSearchTerm].Equals(searchTerm);
    

    e a segunda forma é muito mais enxuta e simples de ler;

  • O método readRecord não deveria ser responsável pela mensagem "Record not found", isso deveria estar a cargo do método que o chama. Seria uma boa ideia se o readRecord apenas retornasse um conjunto vazio para o seu chamador quando não encontrar nenhum registro que atenda ao critério de busca;;

  • Não é nenhuma uma regra, mas há uma convenção de nomenclatura para C#, talvez seja uma referência bacana para te guiar na escolha de nomes.

Pra resolver seu problema, você precisa:

  1. Alterar a assintura do método readRecord para que seja possível retornar uma coleção de string[];

    IEnumerable<string[]> ReadRecord(string searchTerm, string filepath, int positionOfSearchTerm)
    
  2. Criar uma lista para adicionar todos os itens encontrados de acordo com o critério de busca;

    var retorno = new List<string[]>();
    
  3. Ao invés de retornar o primeiro registro encontrado, adicioná-lo na lista criada;

    retorno.Add(fields);
    
  4. Usar a lista como retorno do método.

Veja o método completo:

IEnumerable<string[]> ReadRecord(string searchTerm, string filepath, int positionOfSearchTerm)
{
    positionOfSearchTerm--;

    string[] lines = System.IO.File.ReadAllLines(filepath);

    var retorno = new List<string[]>(); // Lista para retornar todos itens encontrados
    for (int i = 0; i < lines.Length; i++)
    {
        string[] fields = lines[i].Split(',');
        if (recordMatches(searchTerm, fields, positionOfSearchTerm))
        {
            Console.WriteLine("Record found");
            retorno.Add(fields); // Adicionar resultados na lista
        }
    }
    return retorno;
}

Veja funcionando no Repl.it.