Plusieurs if et ELSE if en javascript

Donc ce qui se poussé à ce tableau est personne à charge sur une radio quelques boîtes. J'ai eu ce pour standard et en fauteuil roulant sièges:

if(document.getElementById('standardseat').checked) {
  //Standard seat is checked
seatsArray.push(e.posX, e.posY);
}
else if(document.getElementById('wheelchairseat').checked) {
  //Wheelchair seat is checked
seatsArray.push("Wheelchair " + e.posX, e.posY);
}

Et c'est l'équivalent du code de formulaire:

<input id="standardseat" type="radio" name="seat" value="standard" /> Standard seat
<input id="wheelchairseat" type="radio" name="seat" value="wheelchair" /> Wheelchair seat

Mais je veux ajouter un peu plus de la radio des boîtes, qui sont distinctes de la norme/un fauteuil siège:

<input id="backnave" type="radio" name="area" value="backnave" /> Back Nave
<input id="frontnave" type="radio" name="area" value="frontnave" /> Front nave
<input id="middlenave" type="radio" name="area" value="middlenave" /> Middle nave

Et je veux le pousser à également inclure ce. Pour expliquer, si l'utilisateur a coché l'option "Fauteuil roulant siège" et "Milieu de la nef", la commande doit de sortie ("MN, en Fauteuil roulant" + e.posX, e.posY). Est-il possible de faire cela sans manuellement, y compris beaucoup de monde si pour chaque résultat possible (je peut même ajouter un troisième ensemble de la radio, des boîtes)?

Merci!

OriginalL'auteur IceDragon | 2010-07-13

  1. 12

    Je voudrais construire la chaîne qui décrit le président avec un relativement petit nombre de if états, et ensuite appeler la push à la fin.

    Donc quelque chose comme:

    var desc = "";
if(document.getElementById('wheelchairseat').checked) {
  //Wheelchair seat is checked
  desc = "Wheelchair "+desc;
}

if(document.getElementById("backnave").checked) {
  desc = "BN, "+desc;
} else if(document.getElementById("middlenave").checked) {
  desc = "MN, "+desc;
} else if(document.getElementById("frontnave").checked) {
  desc = "FN, "+desc;
}

seatsArray.push(desc + e.posX, e.posY);

    Cela peut facilement être étendue pour tenir compte de d'autres groupes de blocs.

    Serait "desc = "BN, "+desc;" ne pas ajouter à la fois "desc"'s, par exemple, lui-même et l'autre? Ou serait-il juste d'ajouter l'autre? Merci
    Les deux desc? Pour l'autre? hein? Suivez la chaîne. Il commence à vide, si c'est un fauteuil roulant siège, il ajoute que la chaîne vide (c'est inutile, comme un siège normal de ne pas noter que c'est un siège normal) et, il ajoute celui de "BN" ou "MN", ou "le FN," Remarque, qu'il serait inutile de faire des virgules normal sièges.
    Je pense que c'est ce que vous vous dîtes: Le côté droit (évaluation) est calculé avant la cession, de sorte qu'il n'ajoute que quelle que soit la valeur était avant la ligne est exécutée. Laissez-moi savoir si ce n'est pas la réponse à votre question ou faire sens.
    Ah, oui. Cela me semble la meilleure interprétation.
    Il a obtenu, merci!

    OriginalL'auteur Jeffrey Blake

  2. 3

    Vous avez raison, IceDragon, il n'est pas judicieux d'utiliser plusieurs si/d'autre, parce que chaque fois que vous ajoutez des options, vous devrez réécrire votre code. Il y a un certain nombre de façons d'éviter cela. Ici n'est qu'une méthode:

    <html><body>
  <form>
    <p>
      <input type="radio" name="seat" onclick="chose(this, 'Standard')" /> Standard seat
      <input type="radio" name="seat" onclick="chose(this, 'Wheelchair')" /> Wheelchair seat
    </p><p>
      <input type="radio" name="area" onclick="chose(this, 'BN')" /> Back Nave
      <input type="radio" name="area" onclick="chose(this, 'FN')" /> Front nave
      <input type="radio" name="area" onclick="chose(this, 'MN')" /> Middle nave
    </p>
  </form>
  <script>
    //make sure to put these in the order you wish them to appear in the output
    var selections = { area: '', seat: '' };
    var seatsArray = [];
    function chose(input, value) {
      selections[input.name] = value;
    }
    //call this function when user clicks the floor plan:
    function image_clicked(e) {
      var desc = [];
      for (var i in selections) if (selections[i] != '') desc.push(selections[i]);
      seatsArray.push(desc.join(', ') + ' ' + e.posX, e.posY);
    }
  </script>
</body></html>

    Avis que, dans le selections objet, nous gardons la trace de sélections que l'utilisateur a fait jusqu'à présent. Ensuite, lorsque l'utilisateur clique sur l'image (ou tout ce qui déclenche le code de travail), la fonction simplement formats les valeurs déjà recueilli.

    Le seul inconvénient de la façon dont j'ai écrit ce code est que les navigateurs ont tendance à cache l'état des boutons radio, de sorte qu'un bouton radio peut être déjà sélectionné, mais l'a choisi() la fonction n'est pas appelée. Un rapide et sale solution de contournement consiste à ajouter un ID à la balise form et l'exécuter sur chargement de la page:

    document.getElementById('form').reset();

    Où la "forme" est l'attribut ID de la balise form.

    OriginalL'auteur Mark Eirich

  3. 1

    En règle générale, lorsque vous avez beaucoup de si/d'autre-si vous songez à la remplacer par un switch/case déclaration.

    http://www.javascriptkit.com/javatutors/switch.shtml

    Maintenant, ce ne serait pas nécessairement approprié pour votre situation, selon le type de siège est une condition distincte de sa position (front/middle/back).

    Il y a peut être un plus élégant que votre code, mais je ne suis pas complètement comprendre le contexte. Que faites-vous avec ce tableau?

    Le tableau va être écrit dans un fichier texte lorsque l'utilisateur choisit de l'épargner. L'ensemble du document est destiné à être utilisé pour activer le plan d'étage positions de siège sur une image numérisée en numérique, mobile positions (les coordonnées des pixels est la sortie à ce stade). Merci
    Manaster Merci. Votre pickiness semble être aussi forte que la mienne. 🙂

    OriginalL'auteur George Marian

  4. 0

    Faire la value ce que vous souhaitez voir apparaître dans la chaîne de résultat. Si vous voulez 'MN' faire la value 'MN'.

    Alors au lieu d'écrire un if déclaration pour chaque état possible vous pourrait faire une boucle par la input objets à trouver celui qui est coché.

    Obtenir toutes les entrées de chaque pour comme si...

    var chairFormElems = document.getElementById('chairFormId').elements;
var naveFormElems = document.getElementById('naveFormId').elements;

    Ensuite une boucle sur chaque et de trouver celle qui est cochée. Puis juste concat la value attributs pour faire de votre chaîne de résultat.

    OriginalL'auteur jasongetsdown