Blockly: CreateMyState: Datenpunkte mit Unit/States

[ManfredHi]

@bananajoe
Wowh, vielen Dank für das Script. Gerade ausprobiert, funktioniert perfekt.
Eine Frage noch dazu: Das was man bei ValueName angibt wird sowohl für die ObjektID als auch für den Wert im Feld Name verwendet. Kann man das eventuell auch unterscheiden?

Ergänzung:
Hab das Script schon so angepasst, dass ich ein eigene Variable für den Namen habe.

DANKE!

[paul53]

@bananajoe sagte: Als States eine Liste übergeben mit der Anzahl der States, jeweils ein Textfeld im Format
Zahl:Text

Weshalb nicht gleich als Objekt?

Anstelle von TargetType könnte man auch

typeof TargetDefaultValue

verwenden. Außerdem vermisse ich die Rolle, die automatisch aus der Kombination von TargetType und TargetIsWritable gebildet werden könnte.

Sollten nicht alle eigenen Datenpunkte unter "0_userdata.0" erstellt werden?

[BananaJoe]

@paul53 Ok, die Idee mit dem Objekt hatte ich einfach nicht (ich arbeite einfach zu viel Strings).
Aber ... das ist es schon wieder fest verdrahtet. Meine Funktion ist ja entstanden weil ich dynamisch Datenpunkte anlegen wollte - in der Regel weil ich ein JSON verarbeiten will und gar nicht weis was ich bekomme.
Mit den Textfeldern kann ich die Liste auch per erstelle Text aus zusammenbauen, in dynamischer Länge indem ich dieser vorher erzeuge.

Das mit dem Automatischen Target-Type, ich weis nicht. Ich hätte das schon gern klar definiert. Aber die Möglichkeit da eine automatische Auswahl zusätzlich zu den festen einzubauen wäre eine Idee.

Ob man 0_userdata.0 da fest einbaut, könnte man. Besser wäre vielleicht zu prüfen ob der Datenpunkt unterhalb erstellt wird und dann eine Fehlermeldung zu geben.

@ManfredHi Ok, den Namen benutze ich nie - ich nutze auch nie die Objekt-Auswahl in Blockly sondern immer Textfelder für die Datenpunkte. Für mich ist das lesbarer.
Aber du hast ja einen Weg gefunden. Eventuell baue ich das in V5 ein, wenn man es leer lässt nimmt er dann einfach den ValueName

Allgemein: Hauptziel war eine Funktion für das Erstellen von Datenpunkten zu bauen wo ich jeden Aspekt aus anderen Dingen zusammen bauen kann.

Aber noch mal @paul53 Wie verarbeite ich wenn ich einen Parameter gar nicht angebe:

Im Moment prüfe ich auf

if (TargetStates != "") { }

oder

if (TargetUnit.length === 0 && s_states.length === 0) {

Wenn ich den Parameter einfach leer lasse, gibt es einen Fehler

[paul53]

@bananajoe sagte: Im Moment prüfe ich auf

Prüfe einfach

if (TargetStates) { }

und

if (TargetUnit) { }

Das genügt um undefined und Leerstrings auszuklammern.

Meine Lösung sehe etwa so aus:

if(!id.startsWith('0_userdata.0')) id = '0_userdata.0.' + id;
const type = typeof def;
if(!role) {
    if(write && type == 'boolean') role = 'switch';
    else if(write && type == 'number') role = 'level';
    else if(!write && type == 'boolean') role = 'indicator';
    else if(!write && type == 'number') role = 'value';
    else role = 'state';
}
const common = {
    type: type,
    name: Name,
    def: def,
    read: true,
    write: !!write,
    role: role
};
if(desc) common.desc = desc;
if(unitOrStates) {
    if(typeof unitOrStates == 'object') common.states = unitOrStates;
    else if(type == 'number' && typeof unitOrStates == 'string') common.unit = unitOrStates;
}
createState(id, def, common);

Die letzten 3 Parameter sind optional (leer lassen oder Leerstring).

[BananaJoe]

@paul53 sagte in Blockly: CreateMyState: Datenpunkte mit Unit/States:

@bananajoe sagte: Im Moment prüfe ich auf

Prüfe einfach

if (TargetStates) { }

und

if (TargetUnit) { }

Das genügt um undefined und Leerstrings auszuklammern.

Danke! Habe gerade festgestellt das

if (testVar === null) { }

auch geht, aber deines ist natürlich noch einfacher. Ich baue - vermutlich - morgen mal eine V5

[BananaJoe]

@paul53 sagte in Blockly: CreateMyState: Datenpunkte mit Unit/States:

Meine Lösung sehe etwa so aus:

und stimmt, deine Variante common zunächst zu definieren und dann weitere Attribute wie if(desc) common.desc = desc; ist natürlich auch schlauer. Besser und eleganter als meine ganzen if Entscheidungen.

[ManfredHi]

@bananajoe
Hab mir jetzt noch eine zusätzliche Variable mit dem Objektpfad angelegt und wenn ich keinen eigenen Namen angebe, wird der ValueName für den Namen gesetzt.

if(!objektpfad.endsWith('.')) objektpfad = objektpfad + '.';
if(!objektpfad.startsWith('0_userdata.0.')) objektpfad = '0_userdata.0.' + objektpfad;
objektpfad = objektpfad + id;
const type = typeof def;
if(!role) {
    if(write ?? type == 'boolean') role = 'switch';
    else if(write ?? type == 'number') role = 'level';
    else if(!write ?? type == 'boolean') role = 'indicator';
    else if(!write ?? type == 'number') role = 'value';
    else role = 'state';
}
const common = {
    type: type,
    def: def,
    read: true,
    write: write,
    role: role
};
if(desc) common.desc = desc;
if(Name) common.name = Name
    else common.name = id;
if(unitOrState) {
    if(typeof unitOrState == 'object') common.states = unitOrState;
    else if(type == 'number' && typeof unitOrState == 'string') common.unit = unitOrState;
}
createState(objektpfad, DefaultValue, def, common);

[paul53]

@manfredhi
Korrigiere Zeilen 6 bis 9 und ersetze ?? durch &&.
Zeile 26: Defaultvalue? Du hast im Skript def ausgewertet.

[ManfredHi]

@paul53 sagte in Blockly: CreateMyState: Datenpunkte mit Unit/States:

Zeile 26: Defaultvalue? Du hast im Skript def ausgewertet.

Das mit dem def war mir nicht ganz klar. Kann ich darin z.B. als Defaultwert einen Wert eines anderen Datenpunktes eintragen? Dann steht dieser Wert aber auch im common-Block des Datenpunktes drinnen oder? Außer ich nehm den eintrag defaus dem commonheraus oder?

if(!objektpfad.endsWith('.')) objektpfad = objektpfad + '.';
if(!objektpfad.startsWith('0_userdata.0.')) objektpfad = '0_userdata.0.' + objektpfad;
objektpfad = objektpfad + id;
const type = typeof DefaultValue;
if(!role) {
    if(write && type == 'boolean') role = 'switch';
    else if(write && type == 'number') role = 'level';
    else if(!write && type == 'boolean') role = 'indicator';
    else if(!write && type == 'number') role = 'value';
    else role = 'state';
}
const common = {
    type: type,
    read: true,
    write: write,
    role: role
};
if(desc) common.desc = desc;
if(Name) common.name = Name
    else common.name = id;
if(unitOrState) {
    if(typeof unitOrState == 'object') common.states = unitOrState;
    else if(type == 'number' && typeof unitOrState == 'string') common.unit = unitOrState;
}
createState(objektpfad, DefaultValue, common);

[paul53]

@manfredhi sagte: Kann ich darin z.B. als Defaultwert einen Wert eines anderen Datenpunktes eintragen?

Ja, wieso nicht?