5

Estou criando uma rotina para buscar palavras reservadas em um texto (nome de pessoas). Já fiz testes com Contains() e IndexOf(). Funciona bem na maioria das vezes. No entanto em algumas palavras o retorno não satisfaz. Exemplo: no nome CORALINA retorna a palavra reservada ORAL.

Abaixo o código realizado:

    public JsonResult GetPalavrasReservada1(string frase)
    {
        var palavras = db.TBL_PALAVRA_RESERVADA.Where(pal => pal.PLR_ATIVO == 0).Select(i => new {i.PLR_DESC}).ToList();

        var palavrareservada = "";

        for (int i = 0; i < palavras.Count; i++)
        {
            if (frase.ToUpper().Contains(palavras[i].PLR_DESC.ToString()))
            {
                return Json(palavras[i].PLR_DESC.ToString());
            }
        }

        return Json(palavrareservada);
    }

Estou utilizando lógica ou métodos errados?

3
  • 2
    E não tá certo? De fato o nome "Coralina" contém a palavra "oral".
    – Jéf Bueno
    Commented 1/11/2017 às 16:56
  • @LINQ não está de acordo com a definição de palavra aqui adotada. Você está pensando em substring, aqui uma palavra é um token devidamente delimitado. Como "coralina" forma um token só, ele não contém "oral" Commented 1/11/2017 às 21:30
  • Não seria o caso de verificar tem espaço em branco antes e depois da palavra? Commented 6/11/2017 às 11:39

2 Respostas 2

1

Se achar uma palavra reservada o algoritmo para e não acha outras, é isto que deseja?

Se deseja só pegar palavras inteiras tem que quebrar todas as palavras e fazer a verificação. Só que isto não é tão simples a não ser que controle o conteúdo do texto anteriormente e possa garantir que não tenha certos padrões. Se puder fazer isto quebrar a frase é só dar um Split() pelo espaço e/ou outros caracteres que quebram uma palavra. Mas é improvável que não tenha situações de exceção a esta regra, aí teria que tratar tudo na mão. Obviamente que isto exigiria um laço extra para testar todas as palavras reservadas em todas palavras da frase.

Seria mais performática fazer um parser especializado, mas complica mais ainda.

Pode ser que um RegEx ajude, mas não gosto da ideia, é fácil fazer errado. Tem exemplos no SO (outro e com LINQ).

Um erro claro é usar o ToUpper(), isto não é correto. Veja a forma mais correta.

A variável palavrareservada não tem função alguma neste código, pode ser eliminada.

Ficar pegando as palavras direto do banco de dados pode não ser a melhor estratégia.

2
  • Com relação a este problema fiz testes com a utilização do split e também regex que separam o texto em palavras e, neste caso, acaba resolvendo. 1. Com relação ao ToUpper() que você disse que está incorreto. Exatamente no que? 2. Se possuir uma palavra reservada com duas palavras (NAO CONSTA, por exemplo) o tratamento teria que ser diferente do que a utilizada com split ou regex. Commented 1/11/2017 às 17:22
  • É ineficiente e há casos (não em português, é verdade) que não dá o resultado esperado.
    – Maniero
    Commented 1/11/2017 às 17:27
0

Eu usaria Regex com o pattern \d para verificar a existência de palavras inteiras na frase.

Regex.IsMatch("CAROLINE", @"\bORAL\b",RegexOptions.IgnoreCase); // false
Regex.IsMatch("Discurso oral", @"\bORAL\b",RegexOptions.IgnoreCase); // true
Regex.IsMatch("Nada consta contra", @"\bNada Consta\b",RegexOptions.IgnoreCase); // true

No seu código, ficaria assim:

if ( Regex.IsMatch(frase, "\\b" + palavras[i].PLR_DESC.ToString() + "\\b",RegexOptions.IgnoreCase ))
1
  • Olá @Fernando, esta forma, utilizando o Regex.IsMatch acabou resolvendo o problema. Muito agradecido. Commented 6/11/2017 às 16:58

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 .