Error calling Function on my page

Question

I’m having trouble calling two functions to validate the fields txtnome and txtsobrenome, calling the roles validanome() and validasobrenome().

I tried to call them the following way:

if (!validanome(cad.txtnome.value)) {
    return;  
}
if (!validasobrenome(cad.txtsobrenome.value)) {
    return;  
}

The entire code of my page and script is here.

There are two functions, in Javascript.

I would like someone to briefly analyze and find out why it’s going wrong, what I’m failing. The functions validanome and validasobrenome that I quoted meet there at the end of the script, and I’m trying to call it in my Function validaFormulario().

Note: I took the else at the end of Function validanome() and validasobrenome(), but it hasn’t worked yet.

Answer 1

Include the ids id="txtnome" and id="txtsobrenome" in the fields name and surname respectively:

<input type="text" name="txtnome" id="txtnome" maxlength="40" required />

and

<input type="text" id="txtsobrenome" name="txtsobrenome" maxlength="30" required />

In the functions to validate these fields, it is necessary to put return false when it is invalid and return true when valid:

function validanome() {
   var nome = document.getElementById("txtnome").value;
   var padrao = /[^a-zà-ú]/gi;
   var valida_nome = nome.match(padrao);
   if( valida_nome || !nome ){
      alert("Nome possui caracteres inválidos ou é vazio, por favor, preencha-o corretamente.");
      return false;
   }
   return true;
}

function validasobrenome() {
   var sobrenome = document.getElementById("txtsobrenome").value;
   var padrao = /[^a-zà-ú]/gi;
   var valida_sobrenome = sobrenome.match(padrao);
   if( valida_sobrenome || !sobrenome ){
      alert("Sobrenome possui caracteres inválidos ou é vazio, por favor, preencha-o corretamente.");
      return false;
   }
   return true;
}

Other functions are also in trouble, such as muitoCurto(), for example. It should be like this:

function muitoCurto(campo, nome, tamanho) {
   if (campo.value.length < tamanho){
      alert("O conteúdo do campo '" + nome
      + "' deve ter pelo menos " + tamanho + " caracteres."
      + " Por favor, preencha-o corretamente.");
      return false;
   }
}

Then you adjust the other functions in the same way.

Answer 2

In the functions you want to call, there is already the call to the element that will be filtered and does not have the parameter in which would be informed which element should be filtered:

var nome = document.getElementById("txtnome").value;
var sobrenome = document.getElementById("txtsobrenome").value;

So the call should just be validanome() and validasobrenome() no parameters. However, the returns tbm do not agree with what you specified in if()s, the functions should remain so, without the parameters:

function validanome() {
    var nome = document.getElementById("txtnome").value;
    var padrao = /[^a-zA-Zà-úÀ-Ú]/gi;
    var valida_nome = nome.match(padrao);
    if( valida_nome || !nome ){
        return false;
    }else{
        return true;
    }
}

function validasobrenome(){
    var sobrenome = document.getElementById("txtsobrenome").value;
    var padrao = /[^a-zA-Zà-úÀ-Ú ]/gi;
    var valida_sobrenome = sobrenome.match(padrao);
    if( valida_sobrenome || !sobrenome ){
      return false;
    }else{
      return true;
    }
}

And the if()s for functions without parameters would be like this:

if (!validanome()){
     //codigo de retorno de erro
}
if (!validasobrenome()){
     //codigo de retorno de erro
}

Or with parameters:

function validanome(nome) {
    var padrao = /[^a-zA-Zà-úÀ-Ú]/gi;
    var valida_nome = nome.match(padrao);
    if( !valida_nome || !nome ){
        return false;
    }else{
        return true;
    }
}

function validasobrenome(sobrenome){
    var padrao = /[^a-zA-Zà-úÀ-Ú ]/gi;
    var valida_sobrenome = sobrenome.match(padrao);
    if( !valida_sobrenome || !sobrenome ){
      return false;
    }else{
      return true;
    }
}

And the if()s of the functions with parameters:

if (!validanome(document.getElementById("txtnome").value)){
     //codigo de retorno de erro
}
if (!validasobrenome(document.getElementById("txtsobrenome").value)){
     //codigo de retorno de erro
}