2016-04-11 18 views
1

Sadece hızlı olanı Bu kodu nasıl değiştiririm, eğer ifadelerden biri gerçekleştiğinde geri kalanını atlar? Yardımlarınız için tekrarBaşka bir deyim doğruysa nasıl durulur ve nasıl ifade edilir

if (Guitar > Percussion && Guitar > Brass && Guitar > Keyboard && Guitar > Woodwind) // Here I'm checking to see if Guitar is greater than the others 
      { 
      var guitarpage = document.getElementById("GuitarPage"); // If guitar is then get the Element ID "GuitarPage" 
      guitarpage.style.display ="block"; // Then Display that page 
      } 

     if (Percussion > Guitar && Percussion > Brass && Percussion > Keyboard && Percussion > Woodwind) 
      { 
      var percussionpage = document.getElementById("PercussionPage"); 
      percussionpage.style.display ="block"; 

      } 

     if (Brass > Guitar && Brass > Brass && Percussion > Keyboard && Brass > Woodwind) { 
      var brasspage = document.getElementById("BrassPage"); 
      brasspage.style.display ="block"; 
      } 

     if (Keyboard > Guitar && Keyboard > Keyboard && Percussion > Brass && Keyboard > Woodwind) { 
      var keyboardpage = document.getElementById("KeyboardPage"); 
      keyboardpage.style.display ="block"; 
      } 

     if (Woodwind > Guitar && Woodwind > Brass && Woodwind > Keyboard && Woodwind > Percussion) { 
      var woodpage = document.getElementById("WoodwindPage"); 
      woodpage.style.display ="block"; 
         } 

     else { 
      var nonepage = document.getElementById ("NoPage"); 
      nonepage.style.display ="block"; 

      } 

Teşekkür

+1

Kullanım 'başka if' veya' switch'. Amandan [güzel cevap] (http://stackoverflow.com/a/36539726/2025923) var, neden bunu kullanmıyorsunuz? – Tushar

+0

Hala bunu çözemiyorum! Ama bunun için teşekkürler! – user3500310

+0

Eğer gerçekleşmişse, eğer doğruysa, gerçekleşmişse, başka bir şey farketmez mi? – user3500310

cevap

0
if (Guitar > Percussion && Guitar > Brass && Guitar > Keyboard && Guitar > Woodwind) // Here I'm checking to see if Guitar is greater than the others 
      { 
      var guitarpage = document.getElementById("GuitarPage"); // If guitar is then get the Element ID "GuitarPage" 
      guitarpage.style.display ="block"; // Then Display that page 
      } 

else if (Percussion > Guitar && Percussion > Brass && Percussion > Keyboard && Percussion > Woodwind) 
      { 
      var percussionpage = document.getElementById("PercussionPage"); 
      percussionpage.style.display ="block"; 

      } 

else if (Brass > Guitar && Brass > Brass && Percussion > Keyboard && Brass > Woodwind) { 
      var brasspage = document.getElementById("BrassPage"); 
      brasspage.style.display ="block"; 
      } 

else if (Keyboard > Guitar && Keyboard > Keyboard && Percussion > Brass && Keyboard > Woodwind) { 
      var keyboardpage = document.getElementById("KeyboardPage"); 
      keyboardpage.style.display ="block"; 
      } 

else if (Woodwind > Guitar && Woodwind > Brass && Woodwind > Keyboard && Woodwind > Percussion) { 
      var woodpage = document.getElementById("WoodwindPage"); 
      woodpage.style.display ="block"; 
         } 

else { 
      var nonepage = document.getElementById ("NoPage"); 
      nonepage.style.display ="block"; 

      } 
+0

Hey Bunu denedim ama diğer değişkenlerden biri diğerine eşitse hala görünüyor? – user3500310

+0

sadece yukarıdaki koşullardan hiçbiri "true" değerine eşit değilse gösterilir. Yukarıdaki tüm koşulların hatalarını ayıklamak ve bunlardan en az birinin doğru sonuçlandığından emin olmalısınız. Bu olmazsa, diğer blok yürütülür. –

+0

Yani bunlardan biri kesinlikle doğruydu. 5 oldu. İki tane daha vardı ve iki sayfa da çıktı? – user3500310

0

var indices = ["GuitarPage","PercussionPage","BrassPage","KeyboardPage", "WoodwindPage"]; 
var allNumbers = [Guitar,Percussion,Brass,Keyboard,Woodwind]; 
var maxValue = Math.max.apply(null,allNumbers); 
var index = allNumbers.indexOf(maxValue); 
if (maxValue > 0) 
{ 
    document.getElementById(indices[index]).style.display ="block"; 
} 
else 
{ 
    document.getElementById ("NoPage").style.display ="block"; 
} 
0

Sen değilse zaten bir fonksiyon içine ve bir koşul ise fonksiyonundan çıkmak olabilir bu basit mantık deneyin doğru. Gerisi atlandı.

function compare() { 
    if (Guitar > Percussion && Guitar > Brass && Guitar > Keyboard && Guitar > Woodwind) // Here I'm checking to see if Guitar is greater than the others 
    { 
     var guitarpage = document.getElementById("GuitarPage"); // If guitar is then get the Element ID "GuitarPage" 
     guitarpage.style.display = "block"; // Then Display that page 
     return; 
    } 

    if (Percussion > Guitar && Percussion > Brass && Percussion > Keyboard && Percussion > Woodwind) { 
     var percussionpage = document.getElementById("PercussionPage"); 
     percussionpage.style.display = "block"; 
     return; 
    } 

    if (Brass > Guitar && Brass > Brass && Percussion > Keyboard && Brass > Woodwind) { 
     var brasspage = document.getElementById("BrassPage"); 
     brasspage.style.display = "block"; 
     return; 
    } 

    if (Keyboard > Guitar && Keyboard > Keyboard && Percussion > Brass && Keyboard > Woodwind) { 
     var keyboardpage = document.getElementById("KeyboardPage"); 
     keyboardpage.style.display = "block"; 
     return; 
    } 

    if (Woodwind > Guitar && Woodwind > Brass && Woodwind > Keyboard && Woodwind > Percussion) { 
     var woodpage = document.getElementById("WoodwindPage"); 
     woodpage.style.display = "block"; 
     return; 
    }   else { 
     var nonepage = document.getElementById("NoPage"); 
     nonepage.style.display = "block"; 

    } 
} 

compare(); 
+1

Bu işe yaradı! Teşekkürler! – user3500310

0
function pageCond(Guitar, Percussion, Brass, Keyboard, Woodwind) 
{ 
    if (Guitar > Percussion && Guitar > Brass && Guitar > Keyboard && Guitar > Woodwind) // Here I'm checking to see if Guitar is greater than the others 
    { return "GuitarPage"; } 

    if (Percussion > Guitar && Percussion > Brass && Percussion > Keyboard && Percussion > Woodwind) 
    { return "PercussionPage"; } 

    if (Brass > Guitar && Brass > Brass && Percussion > Keyboard && Brass > Woodwind) 
    { return "BrassPage"; } 

    if (Keyboard > Guitar && Keyboard > Keyboard && Percussion > Brass && Keyboard > Woodwind) 
    { return "KeyboardPage"; } 

    if (Woodwind > Guitar && Woodwind > Brass && Woodwind > Keyboard && Woodwind > Percussion) 
    { return "WoodwindPage"; } 

    return "NoPage"; 
} 

var nodeID = pageCond(Guitar, Percussion, Brass, Keyboard, Woodwind); 
document.getElementById(nodeID).style.display ="block"; 
0

Shorter: (güncellenmiş)

var text = 'Guitar,Percussion,Brass,Keyboard,Woodwind'; 
var list = text.split(',').join('Page,').split(','); 
var vars = eval('['+text+']'); 
var page = list[vars.indexOf(Math.max.apply(null,vars))]; 

document.getElementById(page || 'NoPage').style.display ="block"; 
İlgili konular