If Then mehrfach [gelöst]

[ceng]

@bahnuhr
Nachdem ich Dein Scriptschnipsel gesehen habe, passte ich mein Script folgendermassen an und siehe da, die Ausrufezeichen sind weg. Ich darf nur einmal "var beaufort" schreiben...

var aktuellerWind = getState("hm-rpc.2.0017D709AE3070.1.WIND_SPEED"/*HmIP-SWO-B 0017D709AE3070:1.WIND_SPEED*/).val;
var loggen = false;  // = false, wenn kein Logging gewünscht
var timestamp = Date();
var timestampUnix = Math.round(+new Date()/1000);

createState('status_Wind.Windstatus', 'INITIAL', {type: 'string'});           // Wetterstatus
createState('status_Wind.Beaufort', 'INITIAL', {type: 'string'});           // Wetterstatus

createState('status_Wind.timestampaktWind', 'INITIAL', {type: 'string'});       // Ausführungsdatum
createState('status_Wind.timestampaktWindUnix', timestampUnix, {type: 'number'});// Ausführungsdatum Unix Format
createState('status_Wind.windspeedakt', 0, {type: 'number'});                   // aktuelle Windgeschwindigkeit

createState('status_Wind.timestamp1Wind', 'INITIAL', {type: 'string'});         // Ausführungsdatum
createState('status_Wind.windspeed1', 0, {type: 'number'});                     // letzte Windgeschwindigkeit
createState('status_Wind.timestamp2Wind', 'INITIAL', {type: 'string'});         // Ausführungsdatum
createState('status_Wind.windspeed2', 0, {type: 'number'});                     // vorletzte Windgeschwindigkeit
createState('status_Wind.timestamp3Wind', 'INITIAL', {type: 'string'});         // Ausführungsdatum
createState('status_Wind.windspeed3', 0, {type: 'number'});                     // vorvorletzte Windgeschwindigkeit


function windstate() {
    var windspeedakt        = getState("hm-rpc.2.0017D709AE3070.1.WIND_SPEED"/*HmIP-SWO-B 0017D709AE3070:1.WIND_SPEED*/).val;             // Windgeschwindigkeit
    var windspeed1          = getState("javascript.0.status_Wind.windspeedakt"/*status_Wind.windspeedakt*/).val;             // letzte Windgeschwindigkeit
    var windspeed2          = getState("javascript.0.status_Wind.windspeed1"/*status_Wind.windspeedakt*/).val;             // vorletzte Windgeschwindigkeit
    var windspeed3          = getState("javascript.0.status_Wind.windspeed2"/*status_Wind.windspeedakt*/).val;             // vorletzte Windgeschwindigkeit
    var winddurchschnitt    = ( windspeedakt + windspeed1 + windspeed2 + windspeed3 ) / 4;
    
    var timestampaktWind    = timestamp;
    var timestampaktWindUnix= getState("javascript.0.status_Wind.timestampaktWindUnix").val;
    var timestamp1Wind      = getState("javascript.0.status_Wind.timestampaktWind"/*status_Wind.windspeedakt*/).val;        // Zeit letzte Windgeschwindigkeit
    var timestamp2Wind      = getState("javascript.0.status_Wind.timestamp1Wind"/*status_Wind.windspeedakt*/).val;          // Zeit vorletzte Windgeschwindigkeit
    var timestamp3Wind      = getState("javascript.0.status_Wind.timestamp2Wind"/*status_Wind.windspeedakt*/).val;          // Zeit vorvorletzte Windgeschwindigkeit
    
    var timediff            = timestampUnix-timestampaktWindUnix;
    
    var Beaufort            = 'undefined';
    if (0 < windspeedakt && windspeedakt < 1) {Beaufort='Windstill';}
    if (1 < windspeedakt && windspeedakt < 6) {Beaufort='leiser Zug';}
    if (6 < windspeedakt && windspeedakt < 12) {Beaufort='leichte Briese';}
    if (12 < windspeedakt && windspeedakt < 20) {Beaufort='schwacher Wind';}
    if (20 < windspeedakt && windspeedakt < 29) {Beaufort='mäßiger Wind';}
    if (29 < windspeedakt && windspeedakt < 39) {Beaufort='frischer Wind';}
    if (39 < windspeedakt && windspeedakt < 50) {Beaufort='starker Wind';}
    if (50 < windspeedakt && windspeedakt < 62) {Beaufort='steifer Wind';}
    if (62 < windspeedakt && windspeedakt < 75) {Beaufort='stürmischer Wind';}
    if (75 < windspeedakt && windspeedakt < 89) {Beaufort='Sturm';}
    if (89 < windspeedakt && windspeedakt < 103) { Beaufort='schwerer Sturm';}
    if (103 < windspeedakt && windspeedakt < 118) {Beaufort='orkanartiger Sturm';}
    if (windspeedakt > 118) {Beaufort='Orkan';}
    setState("status_Wind.Beaufort", Beaufort);                                 // Schreibt Beaufort Bezeichnung
    
    setState("status_Wind.timestampaktWind", timestampaktWind);                 // Schreibt Datum des Ausführung
    setState("status_Wind.timestampaktWindUnix", timestampUnix);                // Schreibt Datum des Ausführung zur Berechung
    setState("status_Wind.timestamp1Wind", timestamp1Wind);                     // überträgt Datum des letzten Ausführung
    setState("status_Wind.timestamp2Wind", timestamp2Wind);                     // überträgt Datum des vorletzten Ausführung
    setState("status_Wind.timestamp3Wind", timestamp3Wind);                     // überträgt Datum des vorvorletzten Ausführung
    
    setState("status_Wind.windspeedakt", windspeedakt);                         
    setState("status_Wind.windspeed1", windspeed1);                          
    setState("status_Wind.windspeed2", windspeed2);                         
    setState("status_Wind.windspeed3", windspeed3); 
    

    
    if(loggen) log("Zeitdifferenz seit letztem Status: " + timediff);
    if(loggen) log("Aktueller Wind: " + windspeedakt);
    if(loggen) log("1 Wind: " + windspeed1);
    if(loggen) log("2 Wind: " + windspeed2);
    if(loggen) log("3 Wind: " + windspeed3);
    if(loggen) log("Durchschnitt: " + winddurchschnitt);
    
    //if (windspeedakt>20){"status_Wind.Wetterstatus", 'Windig'}
            
}

windstate();

on(
    {id: 'hm-rpc.2.0017D709AE3070.1.WIND_SPEED', change: "ne"}, function (obj) {
        windstate();
        console.log('Windspeed changed');
});

[ceng]

@paul53
Ok, was bedeutet "nur einmal deklarieren". Mit dem Beispiel von Bahnuhr wurde mir Deine Aussage dann klar.

[bahnuhr]

@bahnuhr sagte in If Then mehrfach [gelöst]:

Was machst du wenn der Wind = 20 ist.
Dann läuft das Script ins Leere.

Das Problem hast du aber immer noch.

[paul53]

@ceng Die Abfrage kann vereinfacht werden, da nur eine Variable gesetzt wird, die beliebig oft überschrieben werden kann.

    var Beaufort = 'Windstill';
    if (1 < windspeedakt)  Beaufort = 'leiser Zug';
    if (6 < windspeedakt)  Beaufort = 'leichte Briese';
    if (12 < windspeedakt) Beaufort = 'schwacher Wind';
    if (20 < windspeedakt) Beaufort = 'mäßiger Wind';
    if (29 < windspeedakt) Beaufort = 'frischer Wind';
    if (39 < windspeedakt) Beaufort = 'starker Wind';
    if (50 < windspeedakt) Beaufort = 'steifer Wind';
    if (62 < windspeedakt) Beaufort = 'stürmischer Wind';
    if (75 < windspeedakt) Beaufort = 'Sturm';
    if (89 < windspeedakt) Beaufort = 'schwerer Sturm';
    if (103 < windspeedakt) Beaufort = 'orkanartiger Sturm';
    if (windspeedakt > 118) Beaufort = 'Orkan';

[bahnuhr]

Paul,
die Werte passen aber nicht zu den Werten vom TE.

Bei 5 wollte er z.B. "leiser Zug"; bei dir steht jetzt "leichte Briese"

Aber egal, muss er halt anpassen so wie er es will.

mfg

[paul53]

@bahnuhr sagte:

Bei 5 wollte er z.B. "leiser Zug"; bei dir steht jetzt "leichte Briese"

??

[bahnuhr]

TE:

if (0 < windspeedakt && windspeedakt < 1) {var Beaufort='Windstill';}
    if (1 < windspeedakt && windspeedakt < 6) {var Beaufort='leiser Zug';}
    if (6 < windspeedakt && windspeedakt < 12) {var Beaufort='leichte Briese';}

Paul:

    if (1 < windspeedakt)  Beaufort = 'leiser Zug';
    if (6 < windspeedakt)  Beaufort = 'leichte Briese';
    if (12 < windspeedakt) Beaufort = 'schwacher Wind';

Bei Wert 5 ergibt oben was anderes als unten.
Oder lese ich falsch.
Ist aber auch nicht schlimm, kann er ja ändern.

[paul53]

@bahnuhr sagte:

Bei Wert 5 ergibt oben was anderes als unten.

Wirklich ? Wert 5 ergibt in beiden Fällen "leiser Zug".

[bahnuhr]

@paul53 sagte in If Then mehrfach [gelöst]:

Wirklich ? Wert 5 ergibt in beiden Fällen "leiser Zug".

Ok, hast recht. Hab falsch gelesen.
Entschuldigung.

[George_Best]

Wäre ein "switch - case" nicht die bessere Wahl?

So in der Art:

switch (windspeedakt) {
    case < 6:
        Beaufort = 'leichte Briese;
        break;
    case <12:
        Beaufort = 'schwacher Wind';  
        break;
    ...

    default:
       Beaufort = "irgendwas".;
     break;

[bahnuhr]

case funktioniert auch.

Aber vielleicht könnte mal ein Experte sagen, ob es da eine bessere Methode gibt.

Für mich hätte "if", "else if" oder case den gleichen Effekt.
Gibts da eine Auswahl "was ist besser ?"

[George_Best]

@bahnuhr sagte in If Then mehrfach [gelöst]:

case funktioniert auch.

Aber vielleicht könnte mal ein Experte sagen, ob es da eine bessere Methode gibt.

Für mich hätte "if", "else if" oder case den gleichen Effekt.
Gibts da eine Auswahl "was ist besser ?"

"Switch - Case" ist in der Ausführung schneller, meinte ich mal gehört zu haben. Das liegt wohl daran, dass der Compiler beim Kompilieren den Code effizienter gestalten kann.

Alternative wäre noch eine LookUp-Table, aber das funktioniert meines Wissens nicht mit Vergleichsoperatoren <, > sondern nur mit ===.