Frage zu JS Code (Neuling kenn sich noch nicht so aus)
-
Das würde in diesem Falle bedeuten das dieser Code immer läuft. ist das richtig oder falsch. `
Falsch. Dieser Code ist permanent compiliert im RAM vorhanden und wartet auf Ereignisse (Funktion on()).Übrigens: Das Programm funktioniert so nicht. Korrektur:
var ALabwesend = getState('radar.0.Andrea_Handy.here').val; var CLabwesend = getState('radar.0.Christian_Tablet.here').val; /** Anwesenheit überprüfen **/ function anwesenheit_pruefen() { if (ALabwesend && CLabwesend) setState("hm-rega.0.950"/*Anwesenheit*/, false); else setState("hm-rega.0.950"/*Anwesenheit*/, true); } // Prüfen bei Skriptstart anwesenheit_pruefen(); /** Andrea Handy auf anwesenheit prüfen **/ on({id: 'radar.0.Andrea_Handy.here', change: 'ne'}, function (obj) { var value = obj.state.val; var oldValue = obj.oldState.val; ALabwesend = value; anwesenheit_pruefen(); }); /** Christian Tablet auf anwesenheit prüfen **/ on({id: 'radar.0.Christian_Tablet.here', change: 'ne'}, function (obj) { var value = obj.state.val; var oldValue = obj.oldState.val; CLabwesend = value;; anwesenheit_pruefen(); }); -
Das heist ich brauch sowas wie einen Startbefehl (mal nachdenken von wo ich den hernehme),
am besten irgendein Ereignis bei Systemstart
und dann doch eine permanente Schleife (do –- loop, wenns sowas gibt im JS),
so das der Code ständig ausgeführt wird.
Den geänderten Code muss ich erst in Ruhe studieren.
Da kommen die Fragen dann extra.
-
-
Wenn ich das jetzt richtig verstehe, ist das hier,
on({id: 'radar.0.Andrea_Handy.here', change: 'ne'}, function (obj)dann soviel wie ein Interrupt.
Wenn sich am Datenpunkt "radar.0.Andrea_Handy.here" was tut, egal was,
muss ich mich nicht mehr selber darum kümmern das er was macht.
Oder versteh ich da wieder was falsch.
-
-
nächste Frage
Das hier versteh ich nicht ganz.
function anwesenheit_pruefen() { if (ALabwesend && CLabwesend) setState("hm-rega.0.950"/*Anwesenheit*/, false); else setState("hm-rega.0.950"/*Anwesenheit*/, true); }Mir fehlt da der Vergleich zu "true"
Sollte das nicht so sein
function anwesenheit_pruefen() { if (ALabwesend && CLabwesend === true) setState("hm-rega.0.950"/*Anwesenheit*/, false); else setState("hm-rega.0.950"/*Anwesenheit*/, true); } -
if (ALabwesend && CLabwesend)ist eine verkürzte Schreibweise und funktioniert wie
if (ALabwesend == true && CLabwesend == true)Das Ergebnis eines Vergleiches ist false oder true, also kann man den Vergleich auch weglassen.
Mit zusätzlicher Typprüfung (boolean) ist es so richtig:
if (ALabwesend === true && CLabwesend === true) -
Wenn ich das jetzt richtig verstehe, ist das hier,
…
dann soviel wie ein Interrupt. `
Ja. Hast Du mal Assembler programmiert ?
@PapaChriLo:Wenn sich am Datenpunkt "radar.0.Andrea_Handy.here" was tut, egal was,
muss ich mich nicht mehr selber darum kümmern das er was macht. `
Ja, aber: Nicht egal was, sondern das, was angegeben wurde,change: 'ne'in diesem Fall bei jeder Änderung des Wertes. https://github.com/ioBroker/ioBroker.javascript/blob/master/doc/en/javascript.md#on–-subscribe-on-changes-or-updates-of-some-state.
-
Ja. Hast Du mal Assembler programmiert ? `
Nicht wirklich, komme eher aus der Ecke VB und C++.
OK ich schau mir das jetzt dann nochmal in Ruhe an und muss mal einiges überdenken und neu ordnen.
Muss mich noch mehr mit JS beschäftigen dann wirds schon…..
Hoff ich zumindestens.
Was mir noch ein wenig fehlt ist, wie die ganze Sache beim ioBroker im Hintergrund abläuft, da blicke ich noch nicht ganz durch aber es wird.
Naja ist ja auch kein Projekt das man in 3 Wochen erledigt.
-
komme eher aus der Ecke VB und C++. `
Mit Erfahrung in C++ sollte die Einarbeitung in Javascript nicht schwer fallen. Mir hat das https://wiki.selfhtml.org/wiki/JavaScript dabei sehr geholfen.
@PapaChriLo:wie die ganze Sache beim ioBroker im Hintergrund abläuft, `
https://nodejs.org/dist/latest-v8.x/docs/api/events.html liefert Funktionen für Events. -
Also ich bin etwas ratlos.
Der Code funktioniert so nicht.
var ALanwesend = getState('radar.0.Andrea_Handy.here'.val); var CLanwesend = getState('radar.0.Christian_Tablet.here'.val); /** Anwesenheit überprüfen **/ function anwesenheit_pruefen() { if (ALanwesend === false && CLanwesend === false) setState("hm-rega.0.950", false); else setState("hm-rega.0.950", true); } /** Prüfen bei Skriptstart **/ anwesenheit_pruefen(); /** Andrea Handy auf anwesenheit prüfen **/ on({id: 'radar.0.Andrea_Handy.here', change: 'ne'}, function (obj) { var value = obj.state.val; var oldValue = obj.oldState.val; ALanwesend = value; anwesenheit_pruefen(); }); /** Christian Tablet auf anwesenheit prüfen **/ on({id: 'radar.0.Christian_Tablet.here', change: 'ne'}, function (obj) { var value = obj.state.val; oldValue = obj.oldState.val; CLanwesend = value; anwesenheit_pruefen(); });Bin jetzt bei der Analyse und diese beiden Zeilen versteh ich erhlich gesagt nicht
var value = obj.state.val; var oldValue = obj.oldState.val;Gebe zu die beiden Zeilen hab ich aus dem Beispiel von paul53 übernommen, ohne zu wissen was das tut oder soll.
Ich mein OK, die erste Zeile versteh ich noch, da wird der Variable 'valu' der Wert des Objektes übergeben.
Aber bitteschön für was brauch ich die zweite Zeile.
Ja und warum muss oder soll ich den Wert des Objekts der Variable 'valu' übergeben, kann ich den Wert nicht gleich an meine eigen Variable übergeben?
-
Was genau heißt denn "funktioniert nicht"???
Und - ja du kannst natürlich den Wert gleich direkt an deine Variable übergeben. Und den "oldState" brauchst du (im Moment) auch nicht - das hast du eben wie du richtig schreibst einfach nur kopiert
Und dann bin ich mir nicht sicher ob deine If-Abfrage syntaktisch richtig ist - habe aber die JS-Syntax noch nicht 100%ig intus. Ich würde das jedenfalls so schreiben:
function anwesenheit_pruefen() { if (ALanwesend === false && CLanwesend === false) { setState("hm-rega.0.950", false);} else { setState("hm-rega.0.950", true);} } -
es müsste imho
getState(id).valsein und nicht````
getState(id.val)Die Wertzuordnung am Anfang wird vorgenommen, damit beim erstmaligen Start die Anwesenheit bestimmt wird und die hm-rega korrekt gesetzt wird.var ALanwesend = getState('radar.0.Andrea_Handy.here').val;
var CLanwesend = getState('radar.0.Christian_Tablet.here').val;
var debug = true; // zum debuggen)/** Anwesenheit überprüfen **/
function anwesenheit_pruefen() {
if (ALanwesend === false && CLanwesend === false) setState("hm-rega.0.950", false);
else setState("hm-rega.0.950", true);if(debug) log("Anwesenheit Andrea: "+ ALanwesend + " .Anwesenheit Christian: "+ CLanwesend);
}/** Prüfen bei Skriptstart **/
anwesenheit_pruefen();/** Andrea Handy auf anwesenheit prüfen **/ // hier werden die Auslöser definiert. Entweder Änderung beim Status Anwesenheit Andrea oder Christian
on({id: 'radar.0.Andrea_Handy.here', change: 'ne'}, function (obj) {
var value= obj.state.val;
// var oldValue = obj.oldState.val --> sehe nicht, wozu das notwendig wäre, kann imho raus
ALanwesend = value;
anwesenheit_pruefen();
if(debug) log("Andreas Anwesenheitsstatus hat von " +obj.oldState.val + " auf " + value "+ gewechselt);
});/** Christian Tablet auf anwesenheit prüfen **/
on({id: 'radar.0.Christian_Tablet.here', change: 'ne'}, function (obj) {
var value = obj.state.val;
CLanwesend = value;
anwesenheit_pruefen();
if(debug) log("Christians Anwesenheitsstatus hat von " +obj.oldState.val + " auf " + value "+ gewechselt);
}); -
Was genau heißt denn "funktioniert nicht"???
Und - ja du kannst natürlich den Wert gleich direkt an deine Variable übergeben. Und den "oldState" brauchst du (im Moment) auch nicht - das hast du eben wie du richtig schreibst einfach nur kopiert
Und dann bin ich mir nicht sicher ob deine If-Abfrage syntaktisch richtig ist - habe aber die JS-Syntax noch nicht 100%ig intus. Ich würde das jedenfalls so schreiben:
function anwesenheit_pruefen() { if (ALanwesend === false && CLanwesend === false) { setState("hm-rega.0.950", false);} else { setState("hm-rega.0.950", true);} } ```` `die geschweiften Klammern bei einer if Abfrage brauchst du nur, wenn du mehrere Folgen hast. Ist einfach nur kürzer ohne, wenn man eh nur eine Folge hat
-
Danke #Thisoft
Danke #tempestas
Damit komm ich, vermutlich weiter.
Eine Farge (von noch vielen anderen :? ) hätte ich da aber trotzdem noch.
Warum wird das in JS so geschrieben …
var value = obj.state.val; CLanwesend = value;und nocht so …
CLanwesend = obj.state.val;hat das einen bestimmten Grund?
Ich komm aus der VB / C++ Fraktion, da waren solche Sachen mehr als verpöhnt.
Jede Zeile Code zu viel wurde da schwer abkassiert in die Kaffekasse.
-
sollte auch kürzer gehen wie von dir vorgeschlagen
-
Aber bitteschön für was brauch ich die zweite Zeile.
Ja und warum muss oder soll ich den Wert des Objekts der Variable 'valu' übergeben, kann ich den Wert nicht gleich an meine eigen Variable übergeben? `
Das habe ich so vom ersten Skript übernommen, dessen Vorlage offenbar mit Blockly erstellt wurde. Blockly erzeugt die beiden Variablen value und oldValue per default. Selbst würde ich auch nicht den Umweg über eine Zwischenvariable value machen.Dein Skript mit Korrekturen und kürzer:
var ALanwesend = getState('radar.0.Andrea_Handy.here').val; var CLanwesend = getState('radar.0.Christian_Tablet.here').val; /** Anwesenheit überprüfen **/ function anwesenheit_pruefen() { if (ALanwesend || CLanwesend) setState("hm-rega.0.950", true); else setState("hm-rega.0.950", false); } /** Prüfen bei Skriptstart **/ anwesenheit_pruefen(); /** Andrea Handy auf anwesenheit prüfen **/ on('radar.0.Andrea_Handy.here', function (obj) { // Triggert bei Wertänderung ALanwesend = obj.state.val; anwesenheit_pruefen(); }); /** Christian Tablet auf anwesenheit prüfen **/ on('radar.0.Christian_Tablet.here', function (obj) { CLanwesend = obj.state.val; anwesenheit_pruefen(); }); -
die geschweiften Klammern bei einer if Abfrage brauchst du nur, wenn du mehrere Folgen hast. Ist einfach nur kürzer ohne, wenn man eh nur eine Folge hat `
Danke - gut zu wissen dass das auch so geht. Ich bin ja auch grundsätzlich faul
und dieser ganzen geschweiften Klammerei stehe ich sowieso nicht gerade freundschaftlich gegenüber da ich auch aus der VB und .NET - Ecke komme und dort geht es (sehr gut) ohne diese zusätzliche Tipperei -
OK Danke nochmal an alle.
Ich fang mit JS grade erst mal an und da klaube ich mir halt alles so irgendwie zusammen was momentan für mich Sinn ergibt.
Danach mach ich mich dann ans analysieren und versuch das dann halt mit meinen eigenen Codes.
Mittlerweile komm ich ja auch auf einen grünen Zweig
wenn der auch noch sehr sehr kurz ist :roll: -
Hallo,
hab das mal schnell im Online-Editor geschrieben. Das sollte auch gehen und nutzt RegExp. Als Beispiel und Denkanstoß
-
debug kann man weglassen (natürlich dann auch die Logzeile)
-
Einzeilige Kommentare mit // beginnen, mehrzeilige Kommentare in /* und */ einschließen
-
Hinweis: in der if-Abfrage ist das Ausrufezeichen genutzt. Das ist die Kurzschreibweise für die Abfrage auf false.
Grundsätzlich gilt ja: wenn mindestens ein Gerät auf true, dann anwesend. Wenn alle Geräte auf false, dann abwesend.
/* Andrea und Christian Handy auf Anwesenheit prüfen Regexp auf radar.0.Christian_Tablet.here und radar.0.Andrea_Handy.here */ var debug = true; // Bei Änderung on(/^radar\.[0-9]\.*here$/, function (obj) { // Triggert bei Wertänderung if (debug) log('Anwesenheitserkennung: ' + obj.common.name + ' auf ' + obj.state.val + ' gesetzt ' >; if (obj.state.val) setState("hm-rega.0.950", true); // wenn ein Trigger true, dann anwesend else { if (!getState('radar.0.Andrea_Handy.here').val && !getState('radar.0.Christian_Tablet.here').val) setState("hm-rega.0.950", false); // wenn jetzt beide false dann abwesend } }); // bei Skriptstart if (getState('radar.0.Andrea_Handy.here').val || getState('radar.0.Christian_Tablet.here').val) setState("hm-rega.0.950", true); else setState("hm-rega.0.950", false);Ach, mit der Vorschau Funktion im Forum-Editor kann man den Text nochmal prüfen und ggf. Schreibfehler finden
Gruß
Pix
-
