Objekte aufräumen

[liv-in-sky]

@paul53 das ist wohl der grund, warum der browser aussteigt, wenn ich scriptProblem nicht ausfiltere, während ich die datenpunkte checke

• da explodieren die warnungen bei leeren datenpunkten - noch dazu kommen knapp 1000 datenpunkte unter scriptEnabeld

[sigi234]

@liv-in-sky sagte in Objekte aufräumen:

@sigi234 sagte in Objekte aufräumen:

@paul53 sagte in Objekte aufräumen:

@liv-in-sky sagte:

alle sind leer - egal welche instanz

Das ist ein Zeichen dafür, dass diese Datenpunkte (noch) nicht benutzt werden (ich wüsste auch nicht - wofür).

Mann muss aber aufpassen, den manche Werte kommen erst nach einen Cron oder Adapterstart.
Bespiel, mein Yamaha Adapter das stehen keine Werte drinnen, Erst wenn ich den Yamaha einschalte werden diese befüllt.

aber sie bleiben drin stehen, nachdem du den adapter wieder stoppst - oder ?

ist doch nur beim ersten mal so - dachte ich

Teilweise:

Rot: Werte leer
Grün: Werte teilweise befüllt

Das Skript würde jetzt leer anzeigen, wäre schlecht wenn ich diese lösche, da es ein command ist und ich damit den Yamaha steuere.

[paul53]

@sigi234 sagte:

Das Skript würde jetzt leer anzeigen, wäre schlecht wenn ich diese lösche

Ja, man sollte sich nicht an Adapter-Objekten vergreifen, sondern nur an eigenen Datenpunkten (Leichen).

[liv-in-sky]

@sigi234 aus diesem grund habe ich erst garnicht mit den adapter angefangen sondern nur eigenedatenpunkte und die javascriptdatenpunkte (und diese ohne scriptEnabled und scriptProblem)

die leeren datenpunkte können natürlich auch durch ein update des adapters kommen - also am besten adapter mal stoppen - datenpunkte löschen - adapter starten und nachsehen, ob immer noch leere datenpunkte da sind - und dann den erzeuger des adapters informieren, dass leere datenpunkte "unerwünscht" sind

[sigi234]

Hm, wenn ich die objects.json lösche, wird diese nicht neu angelegt?

[paul53]

@liv-in-sky sagte:

1580 datenpunkte nur unter scriptProblem

Dann ist es sinnvoll, diese auszufiltern.

function checkEmpty(select) {
    $(select).each(function (id, i) {
        var ida = id.split('.');
        if(ida[2] != 'scriptEnabled' && ida[2] != 'scriptProblem' && getState(id).val === null) {
            console.log(id + ': Leer !');
        }
    });
}

checkEmpty('javascript.*');
checkEmpty('meineDP.*');

[paul53]

@sigi234 sagte:

wenn ich die objects.json lösche, wird diese nicht neu angelegt?

Sie wird bei Start von ioBroker eingelesen.

[sigi234]

@paul53
@liv-in-sky

Kann man die Skripte zusammenfassen und ev. später wenn das Skript soweit läuft auch als Blockly reinstellen?

[liv-in-sky]

@paul53 eine kleine hilfe bräuchte ich noch - wie bekomme ich alle installierten instanzen heraus - nur die "überordner"

also alle ordner , die .0 oder .2 oder .3 .... haben und danch nichts mehr

[liv-in-sky]

@sigi234 sagte in Objekte aufräumen:

@paul53
@liv-in-sky

Kannst man die Skripte zusammenfassen und ev. später wenn das Skript soweit läuft auch als Blockly reinstellen?

ich werde mich mal daran versuchen und ein gerüst darum bauen - wenn dabei etwas rauskommt werd ich dich informieren - ich bin nicht so schnell wie @paul53 l und werde etwas zeit brauchen AUSSER @paul53 will da ein script machen ??

[paul53]

@liv-in-sky sagte:

wie bekomme ich alle installierten instanzen heraus

Das Skript zeigt auch an, ob sie laufen:

$('system.adapter.*.alive').each(function(id, i) {
    var ida = id.split('.');
    console.log(ida[2] + '.' + ida[3] + ': ' + getState(id).val);
});

[paul53]

@sigi234 sagte:

Kann man die Skripte zusammenfassen

function checkEmpty(select) {
    $(select).each(function (id, i) {
        var ida = id.split('.');
        if(ida[2] != 'scriptEnabled' && ida[2] != 'scriptProblem' && getState(id).val === null) {
            console.log(id + ': Leer !');
        }
    });
}

function checkTypes(select) {
    $(select).each(function (id, i) {
        var ida = id.split('.');
        if(ida[2] != 'scriptEnabled' && ida[2] != 'scriptProblem') {
            var dpType = getObject(id).common.type;
            var valType = typeof getState(id).val;
            if(dpType != valType && !(dpType == 'array' && valType == 'object')) {
                console.log(id + ': Datenpunkttyp: ' + dpType + ', Wert: ' + valType);
            }
        }
    });
}

checkEmpty('javascript.*');
// checkTypes('javascript.*');

[A Former User]

an so ein Kontrolskript denke ich auch schon länger, ich sehe immer Folgefehler die durch Kraut&Rüben in den Onjekten verursacht werden...

Nett wäre noch die Überprüfung der obj.common.states, dort ist auch oft ein falscher Type angegeben z.B. boolean mit "OPEN" "CLOSED"
Ausserdem wäre es interessant die korrekte Verwendung der Role zu prüfen anhand der offiziellen Vorgaben auf Datentyp und Read/Write Angaben: https://github.com/ioBroker/ioBroker/blob/master/doc/STATE_ROLES.md

[liv-in-sky]

@nis
bis jetzt habe ich mal sowas gemacht

im script kann man dann zusätzlich zu überprüfende instanzen einsetzen - alle zu checken machte keinen sinn - habe schon 2-3 adapter entdeckt, die datenpunkte leer lassen - das werden zu viele fehler auf einmal - aber evtl mach ich ein auswahlmenu, um das script je nach bedarf checken zu lassen

[darkiop]

Das hört sich Interessant an!
Grundsätzlich könnte ich mir so eine House-Keeping-Funktion aber auch als Funktion im js-controller Vorstellen 'iobroker check ...' o.Ä.

[liv-in-sky]

weiter geht es hier: https://forum.iobroker.net/topic/25846/script-datenpunkte-info-sammlung

@sigi234 , @nis @darkiop

[liv-in-sky]

@sigi234

hi sigi

leider wird das nix mit blockly - aber das geht auch so

[liv-in-sky]

@paul53 sagte in Objekte aufräumen:

@sigi234 sagte:

kann man im Reiter Objekte eigentlich alte oder verweiste Einträge automatisch löschen?
Gibt es ein Skript dafür?

Mit automatischem Löschen sollte man sehr vorsichtig sein, weil danach etwas nicht mehr funktionieren kann. Da man auf die von Adaptern erzeugten Datenpunkte keinen Einfluss hat, sollte man sich ohnehin nur auf eigene Datenpunkte beschränken. Deshalb habe ich das Skript dahingehend abgewandelt.

function checkTypes(select) {
    $(select).each(function (id, i) {
        var dpType = getObject(id).common.type;
        var valType = typeof getState(id).val;
        if(dpType != valType && !(dpType == 'array' && valType == 'object')) {
            log(id + ': Datenpunkttyp: ' + dpType + ', Wert: ' + valType);
        }
    });
}

checkTypes('javascript.*');
checkTypes('meineDP.*');

Datenpunkte, die keinen Wert enthalten, erzeugen eine Warnung im Log.

hallo @paul53 ,

ich bin mit meinem script schon etwas weitergekommen und hätte noch eine frage an dich . du hattest mir diese zeile empfohlen bei der auswahl für die type abfrage - was auch sehr gut war

        if(dpType != valType && !(dpType == 'array' && valType == 'object')) {

im folgenden bild bekomme ich bei meiner suche folgende auswertung:

darin sind object-werte geschrieben obwohl string,number oder boolean im dp deklariert ist - da bei diesem beispiel apollon77 den adapter gemacht hat (und auch bei vielen anderen adaptern), gehe ich davon aus, dass dies korrekt ist und so benutzt werden kann - meine frage : kann ich alle diese kombinationen ausschliessen, da es gängig ist? im object-tab sehe ich sie als leer - irgendwie bin ich verwirrt, was das nun bedeutet? diese datenpunkte werden aber nicht als leer angezeigt.
also :

• id+valType(ist) -> dpType(soll)
• id+object ->string
• id+object ->number
• id+object -> boolean
• id+object ->array

[A Former User]

Weitere Prüfungen könnten noch sein:

Prüfen der einzelnen Angaben unter obj.common.states
boolean müsset sein:

{
 id: "irgendetwas mit fenster",
common: {
type: "boolean",
states: {
true: "offen",
false: "geschlossen"
}}}

number:

{
 id: "irgendetwas mit fenster",
common : {
type: "number",
states: {
0: "offen",
1: "geschlossen",
2: "gekippt"
}}}

oftmals entdecke ich solche Angaben die falsch sind da die Werte 0,1,2 String sind:

{
 id: "irgendetwas mit fenster",
common : {
type: "number",
states: {
"0": "offen",
"1": "geschlossen",
"2": "gekippt"
}}}

[liv-in-sky]

@nis probier doch mal das script aus, ob du es brauchen kannst