Überprüfen, ob mehrere Funktionen wahr sind, dann tun Sie etwas

Dieser Teil ist in Ordnung (ich nahm mir die Freiheit der Festsetzung der Einrückung):

function checknewaccount(){
    if(emailvalid()&& checkname() && passwordcheck())
    {
        return true;
    }
    else{
        return false;
    }
}

Obwohl man könnte es verbessern:

function checknewaccount(){
    return emailvalid() && checkname() && passwordcheck();
}

Dieser Teil ist ein syntax-Fehler (um es milde auszudrücken):

function emailvalid(), checkname(), passwordcheck(){
if(condition){
    return true;}
else{return false;}

Wenn das nicht ein echtes Zitat aus Ihrem code, müssen Sie aktualisieren Sie Ihre Frage (auch wenn ich möglicherweise nicht hier, dann aktualisieren diese Antwort). Nicht viel Sinn, zu Fragen, code und dann zitieren pseudo-code in Frage. (Zumindest den pseudo-code fehlt die Letzte }.)

Die gleiche Art von Sache, die wahr ist für Ihre Funktionen in der form:

function emailvalid()
{
      if(email condition)
      {
          return true;
      }
      else {
          return false;
      }
}

Das ist in Ordnung (vorausgesetzt, dass email condition ist noch psuedocode), aber es gibt keine Notwendigkeit für die if:

function emailvalid()
{
    return email condition;
}

Im Sinne von "nichts passiert," stellen Sie sicher, dass debugging-tools, die Sie verwenden können. Chrome Dev-Tools gebaut, einfach die Tastenkombination Strg+Umschalt+I Für Firefox, installieren Sie die ausgezeichnete Firebug. Neuere Versionen des IE haben die dev-tools, die in diese integriert sind (für ältere Versionen können Sie eine Kostenlose version von Visual Studio, können plug-in den browser). Alle diese Ihnen sagen, über syntax-und andere Fehler, lass Sie gehen Sie durch Ihr code-Anweisung-by-Anweisung, etc., das ist entscheidend, um herauszufinden, was passiert ist.

Hier ist ein schoss-off-version von dem, was ich denke, Sie versuchen zu tun. Ich würde das so nicht, aber ich habe nur minimale änderungen machen, damit es funktioniert:

HTML:

<form action="http://www.google.com/search"
      method="GET" target="_blank"
      onsubmit="return checknewaccount()">
<input type="text" id="email" name='q' onblur="emailvalid()">
<input type="text" id="username" onblur="checkname()" >
<input type="password" id="password"  onkeyup="passwordcheck()">
<input type="submit" value="New">
</form>

Hinweise:

• Als Basiclife hingewiesen, Ihre form code Probleme hat. Diese sind fest über.
• Ich oben verwendet habe action="http://www.google.com/search" aber natürlich auch für Sie wäre es action="accountcode.php" (oder zumindest ich denke, es würde).
• Verwenden onsubmit für die Formular-handler, nicht onclick auf den Absenden-button. Sie können nicht Abbrechen eines Formulars zuverlässig cross-Browser über das submit-Schaltfläche onclick.
• In onsubmit, stellen Sie sicher, dass Sie return — z.B. onsubmit="return checknewaccount()", nicht onsubmit="checknewaccount()" — denn wir wollen sicherstellen, dass die event-Sachen sieht die Rückkehr Wert. Wir kümmern uns nicht, wenn die event-Sachen nicht sehen, die Rückkehr Wert unserer anderen Prüfungen (onblur="emailvalid()"), aber wenn wir es täten, müssten returns dort auch.
• Nur eines der Felder hat oben ein name Attribut; keiner von Euch tun. Nur Felder mit name Attribute vorgelegt bekommen mit Formularen. Ich habe nur eine einzige name für mein Beispiel, da ich nur senden will ein Feld für Google, sondern für Ihre Zwecke, Sie gehen zu wollen name Attribute auf allen drei Feldern. Dieser kurze Artikel hat eine Diskussion id vs. name und was Sie sind für. Sie wollen manchmal beide.
• Ich habe die Attribute in all lower-case, was ist best practice (und erforderlich, wenn Sie verwenden möchten, XHTML).
• Jedoch habe ich entfernt, die / von den enden der inputs. Das ist ein bisschen off-topic, aber bei der scheinbaren Ebene, die Sie arbeiten auf, die Sie nicht wollen, um zu versuchen, zu verwenden, XHTML, HTML zu verwenden. Mit XHTML korrekt ist technisch schwierig, sowohl im authoring-und server-Konfiguration, und selbst dann haben Sie es dienen als tag-soup zu IE oder es wird nicht behandeln, es richtig. XHTML hat seinen Platz, aber in der überwiegenden Mehrheit der Fälle es gibt keinen Grund, es zu benutzen.
• Mit den oben genannten kombiniert mit dem JavaScript-Code unten, es gibt keinen Zweck, der Handler auf die einzelnen Felder. Ich habe Sie gelassen, denn ich nehme an, du machst mehr als nur die Prüfungen unter — es ist ein Beispiel weiter unten zeigt diese Handler etwas nützliches tut.

JavaScript:

function checknewaccount() {
  return emailvalid() && checkname() && passwordcheck();
}

function emailvalid() {
  var element;

  //Get the email element
  element = document.getElementById('email');

  //Obviously not a real check, just do whatever your condition is
  return element.value.indexOf('@') > 0;
}

function checkname() {
  var element;

  //Get the username element
  element = document.getElementById('username');

  //Obviously not a real check, just do whatever your condition is
  return element.value.length > 0;
}

function passwordcheck() {
  var element;

  //Get the username element
  element = document.getElementById('password');

  //Obviously not a real check, just do whatever your condition is
  return element.value.length > 0;
}

Live-Kopie

Dinge etwas ändern, wenn die emailvalid, et. al., Funktionen gehen, etwas zu tun, damit der Benutzer wissen, die Felder ungültig sind, wie die Hervorhebung des labels:

HTML:

<form action="http://www.google.com/search"
      method="GET" target="_blank"
      onsubmit="return checknewaccount()">
<label>Email:  
<input type="text" id="email" name='q' onblur="emailvalid()"></label>
<br><label>Username:
<input type="text" id="username" onblur="checkname()" ></label>
<br><label>Password:
<input type="password" id="password"  onkeyup="passwordcheck()"/></label>
<br><input type="submit" value="New">
</form>

JavaScript:

function checknewaccount() {
  var result;
  //Because we're actually doing something in each of the
  //three functions below, on form validation we want to
  //call *all* of them, even if the first one fails, so
  //they each color their field accordingly. So instead
  //of a one-liner with && as in the previous example,
  //we ensure we do call each of them:
  result = emailvalid();
  result = checkname() && result;
  result = passwordcheck() && result;
  return result;
}

function emailvalid() {
      var element, result;

  //Get the email element
  element = document.getElementById('email');

  //Obviously not a real check, just do whatever your condition is
  result = element.value.indexOf('@') > 0;

  //Update our label and return the result
  updateLabel(element, result);
  return result;
}

function checkname() {
  var element, result;

  //Get the username element
  element = document.getElementById('username');

  //Obviously not a real check, just do whatever your condition is
  result = element.value.length > 0;

  //Update our label and return the result
  updateLabel(element, result);
  return result;
}

function passwordcheck() {
  var element, result;

  //Get the username element
  element = document.getElementById('password');

  //Obviously not a real check, just do whatever your condition is
  result = element.value.length > 0;

  //Update our label and return the result
  updateLabel(element, result);
  return result;
}

function updateLabel(node, valid) {
  while (node && node.tagName !== "LABEL") {
    node = node.parentNode;
  }
  if (node) {
    node.style.color = valid ? "" : "red";
  }
}

Live-Kopie

InformationsquelleAutor T.J. Crowder