Bedingungen werden doppelt aktiviert

[Ecopunt]

glaube, dass wenn ich der SWDO auf "nur prüfen" einstell, das dann das Programm nicht mehr direkt reagiert sobalt der SWDO öffnet oder schliest. Stimmt?

Du solltest nur einen von beiden auf nur prüfen stellen. Der andere triggert ja auf jeden Fall

Okay, verstanden. So meinst du?

[MichaelN]

Hast du aus Änderung ein Aktualisierung gemacht?

[Matsch]

Auf die Lösung hast du übrigens gar nicht reagiert. Absicht?

In deinem Skript abfragen, ob der Aktor bereits seinen soll Zustand hat.

sag ich ja:

Entscheidend ist aber, dass im Script dann unnötige Mehrfachausgaben verhindert werden.
[...]
Und ja, die werden auch mehrfach getriggert, macht aber nichts, wenn der Aufruf im Script ins Leere läuft.

[Ecopunt]

Hast du aus Änderung ein Aktualisierung gemacht?

Entschuldigung, Fehler.

Ich bin Holländer, also Deutsch ist nicht meine Muttersprache

Ich versuch jetzt mal so:

[Ecopunt]

Hast du aus Änderung ein Aktualisierung gemacht?

Entschuldigung, Fehler.

Ich bin Holländer, also Deutsch ist nicht meine Muttersprache

Ich versuch jetzt mal so:


Naamloos3.jpg

Resultat ist jetzt, beim schliessen der SWDO:


11:54:54 09.06.2023 R001 status Fancoil: 0 | Temp: 24,8 | Gewenst: 17,0 | Deur: open
11:54:56 09.06.2023 R001 status Fancoil: 3 | Temp: 24,8 | Gewenst: 17,0 | Deur: dicht

Hier passiert also: Der absenktemperatur ist schon von 12 auf 17 hochgegangen (logisch, weil Gruppe), und direkt demnächst wird der neue Status von SWDO als neuer trigger verarbeitet.

[Ecopunt]

Im Skript mache ich eigentlich nur dies, richtung der DRSI. Der wird weiter im skript angesteuert an hand vom variabele "fancoil":

Code: Alles auswählen

if ((status_SWDO == 0) && (actual >= threshold_cooling))  {
  if		(delta > (minimum_delta + 2*fancoil_delta))	{ fancoil = 3; }
  elseif	(delta > (minimum_delta + fancoil_delta))	{ fancoil = 2; }
  elseif	(delta > (minimum_delta))			{ fancoil = 1; }
}

[Matsch]

Du setzt die Variable auf einen Wert, ohne vorher zu testen, ob sie nicht schon den Wert hat. Das ist mit Abfangen gemeint.
Allerdings stellt das mehrfache Beschreiben einer (lokalen) Variablen ja auch keinen Beinbruch dar - es sei denn, man benutzt sie für eine SV und wertet diese auch wieder mit Bei Aktualisierung aus.

Allerdings zeigst du uns ja nicht, wie die Variablen definiert sind und was du mit dem zugewiesenen Wert NACH der if-Bedingung überhaupt machst.

Es steht zu befürchten, dass du die Ausgabe letztlich auch wieder ohne Abfangen von Werten ohne Änderung ausführst.

[MichaelN]

Das Skript ist doch nicht vollständig?

[Matsch]

Das Skript ist doch nicht vollständig?

Nicht mehr als 10 g Salami am Tag, weißt du doch!

[Ecopunt]

Anbei das ganze skript, die Gruppe und die settings der Gruppe.

Danke für's mitdenken.

Edit 9-6-2023 18:15: Script code teilweise gelöscht, weil Auftrag von Kunde. Bitte schick ein Privat für info.

Code: Alles auswählen

...

var sysvar =			dom.GetObject(variable).Value();
var fancoil =			0;

var status_SWDO =		dom.GetObject(SWDO#":1.STATE").Value();

if ((status_SWDO == 0) && (actual >= threshold_cooling))  {
  if		(delta > (minimum_delta + 2*fancoil_delta))	{ fancoil = 3; }
  elseif	(delta > (minimum_delta + fancoil_delta))	{ fancoil = 2; }
  elseif	(delta > (minimum_delta))			{ fancoil = 1; }
}

if(fancoil != sysvar) {    
  if (fancoil == 0) {
     dom.GetObject(DRSI#":6.STATE").State(0);
     dom.GetObject(DRSI#":10.STATE").State(0);
     dom.GetObject(DRSI#":14.STATE").State(0);
     dom.GetObject(DRSI#":18.STATE").State(0);
  }
  elseif (fancoil == 1) {
     dom.GetObject(DRSI#":6.STATE").State(1);
     dom.GetObject(DRSI#":10.STATE").State(1);
     dom.GetObject(DRSI#":14.STATE").State(0);
     dom.GetObject(DRSI#":18.STATE").State(0);
  }
  elseif (fancoil == 2) {
     dom.GetObject(DRSI#":6.STATE").State(1);
     dom.GetObject(DRSI#":10.STATE").State(0);
     dom.GetObject(DRSI#":14.STATE").State(1);
     dom.GetObject(DRSI#":18.STATE").State(0);
  }
  elseif (fancoil == 3) {
     dom.GetObject(DRSI#":6.STATE").State(1);
     dom.GetObject(DRSI#":10.STATE").State(0);
     dom.GetObject(DRSI#":14.STATE").State(0);
     dom.GetObject(DRSI#":18.STATE").State(1);
  }
}