2017-10-29 4 views
1

Wie könnte ich diesen Teil des Codes verkürzen und anstatt alle Optionen anzufügen, fügen Sie Wettersymbole für alle möglichen Situationen hinzu?Wie zu kürzerer Code für Wetter API?

if (desc == "clear sky") 
{ 
    $('div.clear').removeClass('hide'); 
} else if (desc == "broken clouds") 
    { 
    $('div.cloudy').removeClass('hide'); 
    } 
else if (desc == "few clouds") 
    { 
    $('div.cloudy').removeClass('hide'); 
    } 

und so weiter ...

else { 
    $('#desc').text("now it's "); 
} 
+0

Hinweis: Ich denke, dass diese Frage mehr Sinn machen würde auf dem [code review stackexchange] (https://codereview.stackexchange.com/) – Wndrr

Antwort

0

können Sie eine switch/case Anweisung verwenden, die wie dieser

switch(desc) 
    { 
    case "clear sky": 
    $('div.clear').removeClass('hide'); 
    break; 
    case "broken clouds": 
    $('div.cloudy').removeClass('hide'); 
    break; 
    case "few clouds": 
    $('div.cloudy').removeClass('hide'); 
    break; 
    case default: 
    $('#desc').text("now it's "); 
    break; 
    } 

Eine andere Möglichkeit, ein Wörterbuch zu machen wäre aussehen würde, wo der Schlüssel ist der case und der Wert ist eine auszuführende Funktion.