Objekte aufräumen

[A Former User]

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

@nis nein - die braucht man eigentlich nur für die "monsterprüfung" - dort sind mir diese sache erst aufgefallen - ich werd das erst morgen nochmal ansehen

Warum nur für die Monsterprüfung?

[A Former User]

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

@nis ah - jetzt kommt der adapter profi zum vorschein

Naja, nicht wirklich. Ich sehe nur täglich welche Probleme entstehen wenn das alles wild durcheinander verwendet wird...

[liv-in-sky]

@nis

ich wollte eigentlich nur eigene und die javascript datenpunkte checken - da hatte ich bei keine probleme - wurde alles gefunden - dann fiel mir ein , ich könnte ja mal versuchen alle instanzen abzuchecken - und dann gab es halt plötzlich enorm viele warnungen und dateityp vertauschungen - und script abstürze, weil ein fehlender "type" das script crashte - ich arbeite micht halt stück für stück vor und bin froh um input

[sigi234]

@nis sagte in Objekte aufräumen:

Rollen mit sensor.* müssen boolean und read-only sein.
Rollen mit button.* müssen boolean und write-only sein.
usw...

Gute Idee, das wird ein mächtiges Skript wenn es fertig ist.

@liv-in-sky
Überlegung:
Wäre da vielleicht nicht ein Adapter interessant?

[A Former User]

@sigi234 Oder es wird direkt in ioBroker bereitgestellt. Finde es nur sinnvoll, wenn das eigentliche ioBroker System auch die erstellten Regeln auf Einhaltung prüft.

[liv-in-sky]

@sigi234 mal sehen, wie sich das weiterentwickelt - wer weiß, vielleicht komme ich ja an grenzen bei der programmierung

[liv-in-sky]

@nis sagte in Objekte aufräumen:

@sigi234 Oder es wird direkt in ioBroker bereitgestellt. Finde es nur sinnvoll, wenn das eigentliche ioBroker System auch die erstellten Regeln auf Einhaltung prüft.

hat schon jemand vorgeschlagen - aber lass mal erst das script etwas wachsen und sehen, ob es wirklich gebraucht wird

[sigi234]

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

ob es wirklich gebraucht wird

Na sicher! Ich schon.

[liv-in-sky]

eventuell für den info adapter interessant ?

[paul53]

@nis sagte:

oftmals entdecke ich solche Angaben die falsch sind:

Es ist insofern falsch, als die states (Zustandstexte) nicht direkt unter common, sondern unter common.states deklariert werden müssen. Sie werden als JSON im Datenpunkt gepeichert, aber als Objekt zur Verfügung gestellt.

[A Former User]

@paul53 Da war ich zu schnell im Tippen. Hab's oben korrigiert. Der angesprochene Fehler ist der falsche Datentyp.

[paul53]

@nis sagte:

Fehler ist der falsche Datentyp.

Es ist JSON. Die ersten beiden Beispiele wirst Du in Datenpunkten nicht finden.

[liv-in-sky]

@paul53

könntest du das bitte noch kurz erklären

https://forum.iobroker.net/post/312032

[liv-in-sky]

@nis sagte in Objekte aufräumen:

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"
}}}

@nis - verstehe ich dich richtig:

du würdest gerne eine prüfung haben, indem das zweite beispiel richtig ist - unter states sollen immer an erster stelle zahlen stehen, die nicht in anführungszeichen sind - also sicherstellen, das dort number-werte anstatt strings stehen

[AlCalzone]

@sigi234 sagte in Objekte aufräumen:

Bei über 21000 Objekte es händisch zu machen ist sehr aufwendig.

Hast du den BLE-Adapter ohne Filter laufen lassen?

[sigi234]

@AlCalzone sagte in Objekte aufräumen:

Hast du den BLE-Adapter ohne Filter laufen lassen?

Hab den nicht.

[A Former User]

@liv-in-sky Ich muss nochmal nachsehen welche Fehler genau auftreten. Ich glaube dass @paul53 Recht hat und es ein JSON Objekt immer Attribute vom Typ String hat.

[liv-in-sky]

@nis ok - geb mir bescheid, wenn du näheres weißt

ihr kennt euch da besser aus - wenn man einen datenpunkt als auswahlliste anlegt, sieht das wieder anders aus:

[paul53]

@liv-in-sky sagte:

wenn man einen datenpunkt als auswahlliste anlegt, sieht das wieder anders aus:

Das ist die ebenfalls zulässige String-Schreibweise für states. Wenn man states manuell eingibt, ist diese Schreibweise einfacher zu handhaben.
Admin berücksichtigt automatisch die Schreibweise. In Javascript sollte man eine Wandlung in ein Objekt vornehmen.

function statesStr2Obj(states) {
    if (typeof states == 'string') {
        var arr = states.split(';');
        states = {};
        for(var i = 0; i < arr.length; i++) {
            var ele = arr[i].split(':');
            states[ele[0]] = ele[1];
        }
    }
    return states;
}

[paul53]

@liv-in-sky sagte:

darin sind object-werte geschrieben obwohl string,number oder boolean im dp deklariert ist

console.log(typeof null); // ist object 

Wenn leere Datenpunkte beim Typvergleich nicht auftauschen sollen, dann fasse die Prüfungen zusammen.

function checkTypes(select) {
    $(select).each(function (id, i) {
        var ida = id.split('.');
        if(ida[2] != 'scriptEnabled' && ida[2] != 'scriptProblem') {
            if(getState(id).val === null) console.log(id + ': Leer !');
            else {
                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);
                }
            }
        }
    });
}

checkTypes('javascript.*');

oder filtere die leeren Datenpunkte aus

function checkTypes(select) {
    $(select).each(function (id, i) {
        var ida = id.split('.');
        if(ida[2] != 'scriptEnabled' && ida[2] != 'scriptProblem' && getState(id).val !== null) {
            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);
            }
        }
    });
}
 
checkTypes('javascript.*');