Klärung von AdpaterEntwicklungs Fragen

Peoples

@UncleSam sagte in Suche Alternative zu adapter.getState mit Callback:

@Peoples Und noch ein grundsätzlicher Tipp: schau dir mal "gute" Adapter an, wie die geschrieben sind.

"Gut" kannst du selber definieren, aber ich glaube, du hast da ein gutes Bauchgefühl:

wenn ich mir den Code anschaue kriege ich schier das ko....

Wenn du bereit bist etwas mehr zu lernen, empfehle ich dir sehr, den Adapter gleich in TypeScript zu schreiben - das ist wirklich keine Magie!

Und verwende unbedingt den Adapter Creator: https://github.com/ioBroker/create-adapter
npx @iobroker/create-adapter [options]

Den Adapter-Creater habe ich benutzt und versuche eben nun erstmal alle Funktionen meines Scripts zum Laufen zu kriegen, das nächste Ziel ist dann den Code aufzuhübschen.
Wenn ich mich dann noch nicht gefragt habe warum ich das angefangen habe, werde ich mir das Thema TypeScript auch noch anschauen.

Aber für Laien ist das im Ganzen schon ein sehr forderndes Thema

UncleSam

@Peoples sagte in Suche Alternative zu adapter.getState mit Callback:

Aber für Laien ist das im Ganzen schon ein sehr forderndes Thema

Absolut. Ich wünsche dir auf jeden Fall viel Erfolg. Und du weisst ja: im Forum wirst du immer geholfen! ;-)

AlCalzone

@Peoples sagte in Suche Alternative zu adapter.getState mit Callback:

werde ich mir das Thema TypeScript auch noch anschauen.

==> https://forum.iobroker.net/topic/36493/einsteiger-cursus-demonstration-von-typescript :)

@Peoples sagte in Suche Alternative zu adapter.getState mit Callback:

aber wenn ich mir den Code anschaue kriege ich schier das ko....

Das kann ich dir bezüglich async/await noch ans Herz legen:
https://gist.github.com/AlCalzone/d14b854b69ce5e8a03718336cc650a95#

Peoples

@UncleSam @AlCalzone

Tausend Dank schonmal!

Ich habe das Ganze jetzt wie folgt geändert:

async function switchToHomeView() {
      try {
          const switchTimer = await adapter.getStateAsync('switchTimer');
          const lockViewActive = await adapter.getStateAsync('lockViewActive');
          const actualLockView = await adapter.getStateAsync('actualLockView');
          const actualHomeView = await adapter.getStateAsync('actualHomeView');
          const visInstance = await adapter.getForeignStateAsync('vis.0.control.instance');
          timerTout = setTimeout(function () {
                 let timer = parseInt(switchTimer.val, 10)
                 if(timer > 1){
                     if(lockViewActive.val === true){
                         if(timerTout) clearTimeout(timerTout);
                         adapter.setState('switchTimer', 0);
        //Woher kommt newState                       
                         if(actualLockView.val != newState.split('/').pop()){
                             switchToViewImmediate(project+'/'+state.val);
                         }
                     } else {
                        adapter.setState('switchTimer',timer - 1);
                        switchToHomeView(); 
                    }
                 } else {
                     adapter.setState('switchTimer', 0);
                     if(visInstance.val == undefined) adapter.setForeignState('vis.0.control.instance', 'FFFFFFFF');

                     adapter.setForeignState('vis.0.control.data', project + '/' + actualHomeView.val);
                     adapter.setForeignState('vis.0.control.command', 'changeView');

                 }
            }, 1000);

    } catch (error) {
      adapter.log.error(error);
    }
}

Rein optisch gefällt mir das nun schon sehr viel besser, ABER funktionieren tut das nun irgendwie nicht mehr richtig.

Der Timer läuft nicht mehr im Sekundentakt sondern im 2 Sekundentakt, man sieht zwar in den Objekten dass er sekündlich getriggert wird aber die Wertänderung ist nur alle 2.

Wie kann ich das beheben ausser dass ich das Timeout von 1000 auf 500 setze?

Asgothian

@Peoples ich denke du solltest den timeout vor die await Statements setzen. Der rennt dann schon einmal los während noch auf die Werte gewartet wird.

A.

Peoples

@Asgothian
Die Idee hatte ich auch schon dann klappt das mit dem await nicht mehr, da das Timeout keine async function ist :-)

Edit
Nach längerm suchen und probieren habe ich es dann doch hinbekommen einfach das Timeout auch als async:

async function switchToHomeView() {
      try {
          const switchTimer = await adapter.getStateAsync('switchTimer');
          const lockViewActive = await adapter.getStateAsync('lockViewActive');
          const actualLockView = await adapter.getStateAsync('actualLockView');
          const actualHomeView = await adapter.getStateAsync('actualHomeView');
          const visInstance = await adapter.getForeignStateAsync('vis.0.control.instance');
          timerTout = await setTimeout(async function () {
                 let timer = parseInt(switchTimer.val, 10)
                 if(timer > 1){
                     if(lockViewActive.val === true){
                         if(timerTout) clearTimeout(timerTout);
                         await adapter.setStateAsync('switchTimer', 0);
        //Woher kommt newState                       
                         if(actualLockView.val != newState.split('/').pop()){
                             switchToViewImmediate(project+'/'+state.val);
                         }
                     } else {
                        await adapter.setStateAsync('switchTimer',timer - 1);
                        switchToHomeView(); 
                    }
                 } else {
                     await adapter.setStateAsync('switchTimer', 0);
                     if(visInstance.val == undefined) await adapter.setForeignStateAsync('vis.0.control.instance', 'FFFFFFFF');

                     await adapter.setForeignStateAsync('vis.0.control.data', project + '/' + actualHomeView.val);
                     await adapter.setForeignStateAsync('vis.0.control.command', 'changeView');

                 }
            }, 1000);

    } catch (error) {
      adapter.log.error(error);
    }
}

Peoples

Ich habe jetzt mal den Titel geändert, um ein weiteres Problemchen hier mit einfügen zu können ohne einen neuen Thread auf zu machen

``

Jetzt bleibt aber noch eine Frage:
Ich erstelle nach folgendem Muster meine Datenpunkte:

adapter.setObjectNotExistsAsync('actualHomeView', {
        type: 'state',
        common: {
            name: 'View what is set as Home',
            type: 'string',
            role: 'indicator',
            read: true,
            write: true,
        },
        native: {},
    });

wenn ich diese jetzt mit deleteState lösche möchte passiert garnichts. Ich habe auch schon Beispiel mit einer Schleife gefunden und probiert aber auch ohne Erfolg.
Wie schaffe ich es bspw. den Ordner Demoview 2 inkl. aller darin enthaltenen Datenpunkte zu löschen?

viewswitch.0
- Datenpunkt
- Datenpunkt
   |_ Views (Unterordner)
     |_ Demoview 1(Unterordner)
         - Datenpunkt 1
         - Datenpunkt 2
         - Datenpunkt 3
         - Datenpunkt 4
     |_ Demoview 2(Unterordner)
          - Datenpunkt 1
          - Datenpunkt 2
          - Datenpunkt 3
          - Datenpunkt 4
     |_ Demoview 3(Unterordner)
         - Datenpunkt 1
         - Datenpunkt 2
         - Datenpunkt 3
         - Datenpunkt 4
- Datenpunkt 1
- Datenpunkt 2
- Datenpunkt 3
- Datenpunkt 4

AlCalzone

@Peoples sagte in Klärung von AdpaterEntwicklungs Fragen:

Wie schaffe ich es bspw. den Ordner Demoview 2 inkl. aller darin enthaltenen Datenpunkte zu löschen?

Wie hast du es denn versucht (Code)? Ist Demoview 2 ein Channel oder Folder-Objekt oder nur ein Teil der ID?

Peoples

@AlCalzone
Ich habe es versucht wie in diesem gefundenen Forumsthread: https://forum.iobroker.net/topic/948/adapter-alle-objekte-states-eines-adapters-l%C3%B6schen/4

dann kannst du es mit````
adapter.getChannels(function (err, channels) {
     for(var d = 0; d < channels.length, d++) {
          adapter.deleteChannel(channels[d]._id);
     }

Aber bei mir passiert garnichts, nichtmal ein Fehler wird ausgegeben. Meine Objektstrucktur sieht so aus:

Hintergrund ist folgender, wenn der Adapter gestartet wird, werden über die entsprechende vis-views.json alle vorhandenen Views eingelesen und als "Ordner" mit 4 States angelegt.

Nun beobachte ich mit watchfile die Datei und bei einer Änderung möchte ich eine Funktion aufrufen die prüft ob neue Views dazu gekommen sind und diese dann ergänzt oder nicht mehr vorhandene löscht.

AlCalzone

@Peoples
deleteChannel, deleteDevice etc. gehen davon aus, dass du dich an die vorgesehene Struktur device.channel.state hältst. Das ist hier nicht der Fall - du hast nur States ohne übergeordnete Device- und Channel-Objekte.
Das erkennst du daran, dass neben DemoView in der Typ-Spalte nicht channel steht.

Du musst also die zu löschenden States einzeln löschen, sprich

mit getStates die States auflisten
diese durchgehen und diejenigen mit deleteState löschen, die weg sollen.

Peoples

@AlCalzone
Wäre es besser Channel zu benutzen?
Mir ist ehrlich gesagt nicht ganz klar was der Unterschied ist.
Und wenn ich die states lösche wie kriege ich die Ordner dann weg

AlCalzone

@Peoples
Die "Ordner" sind nur virtuell, sobald alle States mit diesem ID-Abschnitt weg sind, sind die auch weg.

Streng genommen sollte keiner dieser "Ordner" ohne ein zugehöriges Objekt existieren. Objekte definieren unter anderem den Sinn dieses "Ordners" und Anzeigenamen, etc.. Die Objekttypen sind definiert unter https://www.iobroker.net/#en/documentation/dev/objectsschema.md?objecttypes

Die "klassische" Struktur, die wohl noch aus der Homematic-Welt stammt, ist device.channel.state, wobei ein device ein physisches Gerät beschreibt, channel eine Teilfunktion (z.B. Licht) und states einzelne Zustände dieser Teilfunktion.
Abweichen davon gibt es auch noch den Objekttyp folder, der zum Gruppieren von States oder weiteren Ordnern dient, die nicht 1:1 in diese Struktur passen. Wäre für dich vermutlich der richtige Anwendungsfall, kann aber dennoch nicht mit deleteChannel gelöscht werden. Hättest du die zugehörigen Objekte angelegt, müsstest du sie neben den States per deleteObject löschen.

@apollon77 noch was hinzuzufügen?

Peoples

@AlCalzone
Also ich hab jetzt zwar ewig gebraucht aber wenn ich es so mache:

adapter.getStates('Views.DemoView' + '.*', async function (err, states) {
        for (const idS in states) {            
            adapter.delObject(idS, function(err){
               // adapter.deleteState(idS, function(err){
               //     if (err) {
               //         adapter.log.error('cannot delete state : ' + idS + ' Error: ' + err);
               //     }
               // });
            });
            
        }
    });

ist der Ordner samt Inhalt weg, die Frage die ich mir stelle ist nur ob es wirklich notwendig ist deleteState zu nutzen da mit dem nur delObject auch alles weg ist.

OliverIO

@Peoples

die logik in iobroker ist

ein objekt hat 0 bis viele states
ein state gehört immer zu einem objekt

wenn du delState benutzt ist die objektdefinition immer noch da, auch wenn du das im objektbaum nicht mehr siehst. dort wird ein objekt nur angezeigt wenn es mindestens einen state hat.
wenn du dann das objekt bspw neu mit einem anderen typ erzeugen willst, dann funktioniert das nicht. es gibt aberr auch keine fehlermeldung.

bei der Erzeugung von objekten/states muss man 2 Sachen unterscheiden, die im objektbaum
nicht gleich ersichtlich sind.

1.) es gibt devices und channels, die eigene states haben
bspw "meinadapter.0.devicename" oder "meinadapter.0.channelname"
darunter kann es dann ein oder mehrere datenpunkte geben
bspw "meinadapter.0.devicename.datenpunkt1" oder "meinadapter.0.devicename.datenpunkt2"
Der Datenpunkt heißt in der internen Verwaltung genau so.
channel und devices müssen separat gelöscht werden

es gibt nur datenpunkte
bspw "meinadapter.0.datenpunk1" oder "meinadapter.0.datenpunkt2"
bspw "meinadapter.0.datenpunk1.datenpunkt11" oder "meinadapter.0.datenpunkt2.datenpunkt21"
Der Datenpunkt heißt in der internen Verwaltung genau so.
Wenn alle Unterdatenpunkte gelöscht sind, sind auch die "Ordner" weg

Das was du als Ordner im Objektbaum siehst, wird nur in der Visualisierung so dargestellt, dass jeder Punkt ab Instanzbezeichnung zu einem eigenen Ordner interpretiert wird. Der Ordner existiert zumindest bei 2) so nirgends.

AlCalzone

@OliverIO sagte in Klärung von AdpaterEntwicklungs Fragen:

ein objekt hat 0 bis viele states

Das stimmt so nicht ganz.
Ein Objekt hat 0 oder 1 direkt zugeordneten State, kann aber im Fall von Device/Channel/Folder untergeordnete Objekte haben, die ihre eigenen States besitzen.

@OliverIO sagte in Klärung von AdpaterEntwicklungs Fragen:

dort wird ein objekt nur angezeigt wenn es mindestens einen state hat.

Stimmt auch nicht:

Alle 3 dieser Objekte haben keinen State.

Ich denke du wirfst hier Objekte und die Baumstruktur im Admin durcheinander.

OliverIO

@AlCalzone
hm, hab ich so unter 1) geschrieben.
in iobroker gibt es doch keine folder, nur devices und channels.
die ordner/folder werden nur in der darstellung so angezeigt.
ein eigenständigen eintrag für ordern/folder gibt es nicht (ausser channel und devices)

AlCalzone

@OliverIO sagte in Klärung von AdpaterEntwicklungs Fragen:

ein eigenständigen eintrag für ordern/folder gibt es nicht

Doch! https://www.iobroker.net/#en/documentation/dev/objectsschema.md?objecttypes --> letzter Eintrag "folder"

Hatte ich oben übrigens schon geschrieben.

OliverIO

@AlCalzone
ah ok, noch nie was von typ folder in einem adapter gesehen.
allerdings hab ich bisher noch keine vorteile gesehen, diese Unterscheidungen von devices, channels und folders zu treffen.
in meinen adapter verwende ich mittlerweile immer nur states unterhalb einer adapter instanz.

AlCalzone

@OliverIO sagte in Klärung von AdpaterEntwicklungs Fragen:

allerdings hab ich bisher noch keine vorteile gesehen, diese Unterscheidungen von devices, channels und folders

In Z-Wave 2 hab ich ein gutes Bespiel:
Der Adapter spricht über den Z-Wave Stick mit mehreren Geräten, die alle als device abgebildet sind. Jedes Gerät hat mehrere Funktionen (z.B. Batterie, schalten, ...), die in Channels gruppiert sind. Jeder Status bzw. Steuermöglichkeit dieser Funktionen sind States innerhalb der Channels.

folder ist eher dafür gedacht, States zu gruppieren, die nicht in diese strikte Struktur passen bzw. den "Ordnern" im Admin eine Definition mitzugeben. Streng genommen muss jeder dargestellte Ordner ein Objekt haben (ist aber oft nicht korrekt umgesetzt).

Peoples

@OliverIO @AlCalzone
was mich halt verwirrt:

wenn ich delState mache wird der Inhalt des States gelöscht -> sprich das True oder die Zahl oder was auch immer.(Also der "Inhalt")
wenn ich nun mit delObject den Inhalt des "Überordners" lösche ist dessen Inhalt -> sprich alle States die darin waren "augenscheinlich" weg. Muss ich dann den State für sich noch löschen, denn er ist ja schon nicht mehr da (zu sehen).

Und wenn ich das delObject weg lasse und mit deleteState den State löschen möchte bekomme ich bei nachfolgender Funktion diesen Fehler:

viewswitch.0	2020-10-19 22:54:34.475	error	(13553) cannot delete state : viewswitch.0.Views.einfacheinetestview.isHomeView Error: Not exists
viewswitch.0	2020-10-19 22:54:34.475	error	(13553) cannot delete state : viewswitch.0.Views.einfacheinetestview.isLockView Error: Not exists
viewswitch.0	2020-10-19 22:54:34.475	error	(13553) cannot delete state : viewswitch.0.Views.einfacheinetestview.sWSec Error: Not exists
viewswitch.0	2020-10-19 22:54:34.475	error	(13553) cannot delete state : viewswitch.0.Views.einfacheinetestview.showIAV Error: Not exists

Funktion:

async function deleteVisObjects(arr){
        adapter.getStates('Views'+ '.*', async function (err, states) {
            for (const idS in states) {
                let nmb = idS.split('.');
                if(arr.includes(nmb[3])){
                    adapter.log.info('View vorhanden: '+idS)
                } else {
                    adapter.log.info('View NICHT vorhanden: '+idS)
                    adapter.deleteState(idS, function(err){
                        if (err) {
                            adapter.log.error('cannot delete state : ' + idS + ' Error: ' + err);
                        }
                 });
              }
        }
});

Wobei die Objekte aber noch vorhanden sind, da ich sie noch ausgeben kann:

viewswitch.0	2020-10-19 22:58:25.435	info	(13621) View NICHT vorhanden: viewswitch.0.Views.einfacheinetestview.isHomeView
viewswitch.0	2020-10-19 22:58:25.435	info	(13621) View NICHT vorhanden: viewswitch.0.Views.einfacheinetestview.isLockView
viewswitch.0	2020-10-19 22:58:25.435	info	(13621) View NICHT vorhanden: viewswitch.0.Views.einfacheinetestview.sWSec
viewswitch.0	2020-10-19 22:58:25.434	info	(13621) View NICHT vorhanden: viewswitch.0.Views.einfacheinetestview.showIAV
viewswitch.0	2020-10-19 22:58:25.434	info	(13621) View vorhanden: viewswitch.0.Views.test.isHomeView
viewswitch.0	2020-10-19 22:58:25.434	info	(13621) View vorhanden: viewswitch.0.Views.test.isLockView
viewswitch.0	2020-10-19 22:58:25.434	info	(13621) View vorhanden: viewswitch.0.Views.test.sWSec
viewswitch.0	2020-10-19 22:58:25.434	info	(13621) View vorhanden: viewswitch.0.Views.test.showIAV
viewswitch.0	2020-10-19 22:58:25.434	info	(13621) View vorhanden: viewswitch.0.Views.DemoView.isHomeView
viewswitch.0	2020-10-19 22:58:25.433	info	(13621) View vorhanden: viewswitch.0.Views.DemoView.isLockView
viewswitch.0	2020-10-19 22:58:25.433	info	(13621) View vorhanden: viewswitch.0.Views.DemoView.sWSec
viewswitch.0	2020-10-19 22:58:25.426	info	(13621) View vorhanden: viewswitch.0.Views.DemoView.showIAV

Nur zur erklärung das "Nicht mehr vorhanden" heißt dass sie in der Projekt Json nicht mehr vorhanden ist. Als Objekt aber schon

Community Forum

NEWS

Klärung von AdpaterEntwicklungs Fragen

Support us

606

32.7k

82.3k

1.3m