À l'aide de “if/else” avec OnClick

Tout d'abord, est-il même possible d'écrire des if/else directement en html avec l'attribut onclick? Et si oui, pourquoi mon code ne fonctionne pas?

C'est donc un bouton. Le "Calc.D'entrée.de la valeur" se réfère à une saisie de texte et je veux afficher un message d'erreur si le champ est 0 ou vide. Dans tous les autres cas, j'en veux d'autres choses comme vous pouvez le voir. Ce n'est pas pertinente si, comme tout ce qui est après le "else" a bien fonctionné avant.

<INPUT TYPE="button" NAME="drop" id="dropbutton" style="background-color:#D1E8D5;" VALUE="
Släpp ankare " OnClick="if ("Calc.Input.value == ''" || "Calc.Input.value == '0'")
{window.alert("Please enter a number");} else 
{document.getElementById('dropbutton').value=' Justera 
längd ';document.getElementById('length').innerHTML =
Calc.Input.value;document.getElementById('dropbutton').style.cssText = 'background-
color:#FFF6B3;';}">
Oy...Même si cela pourrait fonctionner (il n'est pas), qui n'a pas l'air trop jolie. :/ Pourquoi ne pas vous venez d'avoir un appel de fonction dans le onclick et effectuer les expressions conditionnelles au sein de la fonction?
Le Obtrusiveness brûle mes yeux!!! Ce n'est PAS de savoir comment bien le JavaScript doit être écrit
Vous avez cassé l'internet
Le if ("Calc.Input.value == ''" || "Calc.Input.value == '0'") ligne est étrange ...
Besoin de plus de document.write().

OriginalL'auteur NollProcent | 2013-03-12

  1. 18

    Il suffit de NE PAS le mettre dans l'attribut onclick.

    Utilisation

    <INPUT TYPE="button" NAME="drop" id="dropbutton" style="background-color:#D1E8D5;" VALUE="Släpp ankare ">
<script>
document.getElementById('dropbutton').onclick = function() {
    if (Calc.Input.value == '' || Calc.Input.value == '0') {
         window.alert("Please enter a number");
    } else {
        document.getElementById('dropbutton').value=' Justera längd ';
        document.getElementById('length').innerHTML = Calc.Input.value;
        document.getElementById('dropbutton').style.backgroundColor = '#FFF6B3';
    }
    return false;
}
</script>
    Le supplément de devis dans la if déclaration sont évidemment pas censé être là.
    J'irais même jusqu'à suggérer de quitter le reurn false à l'intérieur de la portée de la fonction...
    bon, je re-structuré ma réponse un peu.
    Je vais retirer mes chambres pour certains à se vautrer dans la honte...mais je vous remercie, cela fonctionne comme un charme 🙂

    OriginalL'auteur kay

  2. 4

    La raison que votre code n'est pas le travail, c'est parce que vous devez vous échapper de votre " guillemets, sinon il sera interprété comme l'attribut HTML symbole.

    Et je suis d'accord avec les autres, c'est une mauvaise pratique d'écrire votre JavaScript incorporé à l'intérieur de votre code HTML.

    OriginalL'auteur Amy

  3. 2

    sweetamylase de réponse est meilleur, mais vous pouvez également envisager de:

    Si votre onclick= affectation est entouré de guillemets doubles (onclick="...") puis essayez d'utiliser uniquement des guillemets simples à l'intérieur de ces guillemets doubles (ou vice versa).

    Cela peut rendre les choses plus facile quand vous avez besoin d'effectuer des affectations à l'événement onclick lors de l'insertion d'éléments de façon dynamique et à la séparation des fonctions de script n'est pas vraiment une bonne option.

    Exemple:

    <button onclick="alert('Do this'+' and that')">Press This</button>

    OriginalL'auteur washicd

  4. 1

    Hmm, mais que diriez-vous pour quelque chose comme ceci:

    <a href="#" onclick="if (a_function_to_get_language() == 'en') {
    alert('Some message');
}
else {
    alert('Some message in a different language');
}">Action</a>

    OriginalL'auteur klidifia

  5. 0

    • Alerte sur nav page
               <script>
              function myFunction() {
            alert("You Are Not Signed In!");
                   }
                 </script>
    Veuillez fournir une explication de votre code..

    OriginalL'auteur user7850612