8

Eu estou fazendo um pequeno programa que faz busca dentro de um vetor. O problema que eu to tendo é que se ele encontra o nome ele dá uma mensagem dizendo que foi encontrado mas logo em seguida dá outra mensagem dizendo que não foi encontrado, o código que eu to criando é esse:

string[] nomes = { "misael", "camila", "fernando" };

for (int i = 0; i < nomes.Length; i++) {
    if (txtbusca.Text == nomes[i])
    {
        MessageBox.Show("nome encontrado");
    }    
}

for (int i = 0; i < nomes.Length; i++) {
    if (txtbusca.Text != nomes[i])
    {
        MessageBox.Show("nome não encontrado");
    }    
}
3
  • Se alguma das repostas te ajudou marque como correta para que futuros visitantes tenham uma referência. 30/05/2017 às 12:50
  • E sou nova aqui, não sei como faz isso 1/06/2017 às 0:10
  • É só clicar no símbolo de checked imgur.com/a/sJi2a 1/06/2017 às 18:15

4 Respostas 4

12

O problema é que no segundo for a mensagem é apresentada mal encontre um nome diferente.

Não necessita de percorrer o array duas vezes.
Caso ele seja percorrido até ao fim sem encontrar, é porque o nome não está no array.

string[] nomes = { "misael", "camila", "fernando" };  

string message = "nome não encontrado";
for (int i = 0; i < nomes.Length; i++) {
    if (txtbusca.Text == nomes[i])
    {
        message = "nome encontrado";
        break; //Foi encontrado, não necessita de procurar mais
    }    
}
MessageBox.Show(message);

É claro que existem formas mais simples de fazer isso. No entanto, já que está aprendendo, é importante saber o que está errado no seu código e saber resolve-lo na forma "tradicional", antes de usar recursos mais "avançados" da linguagem.

1
12

Se puder usar LINQ (Contains()) pode resolver já no if que decide se encontrou ou não.

using static System.Console;
using System.Linq;

public class Program {
    public static void Main() {
        string[] nomes = { "misael", "camila", "fernando" };
        var txtBusca = "camila";
        if (nomes.Contains(txtBusca)) {
            WriteLine("Nome encontrado");
        } else {
            WriteLine("Nome não encontrado");
        }
    }
}

Veja funcionando no ideone. E no .NET Fiddle. Também coloquei no GitHub para referência futura.

Se quiser dá para simplificar ainda mais e até eliminar o if, então 5 linhas (ou 7 dependendo do estilo) podem virar apenas uma.

Obviamente troquei a variável e o método de escrita em tela para poder testar sem dependência da sua aplicação, o que alias me lembra que tela e regra de negócio deveriam estar separados. Se for só um exercício, tudo bem, mas tenha isto em mente.

No seu código ficaria assim:

string[] nomes = { "misael", "camila", "fernando" };
MessageBox.Show(nomes.Contains(txtBusca.Text) ? "Nome encontrado" : "Nome não encontrado");
6

Usa o Contains:

string[] nomes = { "misael", "camila", "fernando" };

if(nomes.contains(txtbusca.Text))
    MessageBox.Show("nome encontrado");
else
    MessageBox.Show("nome não encontrado");
4
  • 1
    muito obrigada pela ajuda 30/05/2017 às 9:50
  • 1
    Na verdade isto só funciona se estiver usando LINQ. E o método é Contains(). E falta fechar um parentese, mas esses dois erros são só digitação. E a falta de chaves não causará problemas aí, mas é um estilo que ajuda causar problemas de manutenção.
    – Maniero
    30/05/2017 às 12:52
  • @bigown, se não estou em erro, o método Contains() é para procurar dentro de uma string, enquanto o método contains() é para procurar dentro de um array. Pelo que percebi da pergunta ele queria procurar se um array continha uma string e não se uma string continha uma sequência de caracteres. 30/05/2017 às 15:49
  • 4
    Não, não existe contains(), nenhum método do .NET é minúsculo. Ele procura em qualquer objeto enumerável. O tipo string tem um Contains() especializado, mas se não tivesse, funcionaria com LINQ. Sem LINQ só se o tipo tiver e poucos têm, List é um dos que tem também. Na minha resposta eu até mostrei que funciona da forma esperada, a sua nem compila, pode testar.
    – Maniero
    30/05/2017 às 15:52
4

Você pode utilizar Linq para encontrar também

string[] nomes = { "misael", "camila", "fernando" };

if (nomes.Count(x => x == txtbusca.Text) > 0)
    MessageBox.Show("nome encontrado");
else
    MessageBox.Show("nome não encontrado");

No Count(x => x == txtbusca.Text) estou atribuindo uma variável x que terá o valor de cada item do array depois eu comparo o valor de x com o do seuTextBox, no final deste processo você terá a quantidade de nomes que foram encontrado.

Depois eu verifico se o sistema encontrou ao meno um nome com o > 0

2
  • 6
    Dica: não precisa fazer Where e depois Count. O Count também recebe um predicado.
    – Jéf Bueno
    30/05/2017 às 11:53
  • 3
    Complementando: Usar Any(x => x == txtbusca.Text) parece uma opção melhor ainda.
    – Jéf Bueno
    30/05/2017 às 12:08

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .