2

Alguém consegue me explicar por que sempre retorna false????

function ValidationProcessDoc( ext ){
    if ( ext === 'TXT' || 'DOC' || 'XLS' || 'PPT' ) {
        return false;
    }
    else if ( ext === 'PDF' || 'HTM' || 'HTML' ) {
        console.log(ext);
        return true;
    }
    else {
        return false;
    }
}
1

1 Resposta 1

3

Ele sempre retorna falso, pois a primeira condição sempre será verdadeira, independente do parâmetro que você passe.

Isso acontece por conta da seguinte verificação:

if ( ext === 'TXT' || 'DOC' || 'XLS' || 'PPT' ) {

Caso o valor de ext seja diferente de 'TXT', as demais condições todas são verdadeiras, pois javascript compreende que uma string não vazia é verdadeiro, veja esse exemplo:

if ("Teste") {
  console.log("(\"Teste\") = Verdadeiro");
}

if ("") {
  console.log("(\"\") = Verdadeiro");
}

Perceba que se eu faço uma if utilizando uma string com valor, a condição é atendida.


Caso queira corrigir seu código de forma simples, faça as demais verificações utilizando a variável ext novamente:

function ValidationProcessDoc( ext ){
  if ( ext === 'TXT' || ext === 'DOC' || ext === 'XLS' || ext === 'PPT' ) {
      return false;
  }
  else if ( ext === 'PDF' || ext === 'HTM' || ext === 'HTML' ) {
      return true;
  }
  else {
      return false;
  }
}

console.log( ValidationProcessDoc('PDF') );
console.log( ValidationProcessDoc('TXT') );
console.log( ValidationProcessDoc('XISTO') );


Você pode alterar sua função, deixando ela bem mais simples, retorno diretamente a verificação dos tipos que retornam true:

function ValidationProcessDoc( ext ){
  return ext === 'PDF' || ext === 'HTM' || ext === 'HTML';
}

console.log( ValidationProcessDoc('PDF') );
console.log( ValidationProcessDoc('TXT') );
console.log( ValidationProcessDoc('XISTO') );

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 .