2017-05-20 4 views
0

Je viens d'entrer dans Javascript et je travaille sur l'édition d'un pdf pour une feuille de personnage dans un RPG moi et quelques amis jouent, pour une facilité d'utilisation. Je veux qu'il remplisse automatiquement les formulaires - et jusqu'ici je l'ai fait fonctionner, mais je me demande s'il n'y a pas moyen de condenser un peu mon code et de le rendre moins répétitif?PDF Javascript - Réglage des valeurs du champ

var one = this.getField("Stat.0.0"); 
var statfield = this.getField("Text100.0.1"); 

if(one.value=="1"){ 
statfield.value="-30"; 
} 
if(one.value=="2"){ 
statfield.value="-20"; 
} 
if(one.value=="3"){ 
statfield.value="-10"; 
} 
if(one.value=="4"){ 
statfield.value="-5"; 
} 
if(one.value=="5"){ 
statfield.value="0"; 
} 
if(one.value=="6"||one.value=="7"){ 
statfield.value="5"; 
} 
if(one.value=="8"||one.value=="9"){ 
statfield.value="10"; 
} 
if(one.value=="10"){ 
statfield.value="15"; 
} 
if(one.value=="11"||one.value=="12"){ 
statfield.value="20"; 
} 
if(one.value=="13"||one.value=="14"){ 
statfield.value="25"; 
} 
if(one.value=="15"){ 
statfield.value="30"; 
} 
if(one.value=="16"||one.value=="17"){ 
statfield.value="35"; 
} 
if(one.value=="18"||one.value=="19"){ 
statfield.value="40"; 
} 
if(one.value=="20"){ 
statfield.value="45"; 
} 

Avez-vous des conseils pour le nettoyer?

+0

https://developer.mozilla.org/hu/docs/Web/JavaScript/Reference/ Déclarations/commutateur – cske

Répondre

1

Utilisez des instructions switch() au lieu de if. Ne pas oublier break « s:

var one = this.getField("Stat.0.0").value(); 
var statfield = this.getField("Text100.0.1"); 

var oneval = one.value; 
switch (oneval) { 
    case "1": 
     statfield.value = "-30"; 
     break; 
    //case... 
    case ("1" || "7"): 
     statfield.value = "5"; 
     break; 
    //case... 
    default: 
     break; 
} 

ou (avis break uniquement sur case "7"):

switch (oneval) { 
    //case... 
    case "1": 
    case "7": 
     statfield.value = "5"; 
     break; 
    //case... 
}