Jahresrückblick 2025 – unser neuer Blogbeitrag ist online! ✨

[haus-automatisierung]

@ben1983 Weiß nicht, ob es da den besten Weg gibt. Jeder hat andere Vorzüge. Ich sehe z.B. gern auf einen Blick, was im Script passiert. Und wenn da dann Dinge aus Global kommen, sehe ich das ja nicht mehr. Daher verwende ich den Ordner z.B. gar nicht.

Aber: Geschmackssache.

[Ben1983]

@haus-automatisierung OK.
ist ein gutes Argument.

Ich nutze allerdings auch ein Skript für globale Funktionen, die ich oft verwende.
Hab z.B. eine Funktion, die einfach ein senden an die Alexainstanzen wrapt

function sendAlexaMessage(message, deviceId)
{
    setState(deviceId + ".Commands.speak", message);
}

Die wird halt sehr oft in unterschiedlichen Skipten verwendet

[haus-automatisierung]

@ben1983 Das mache ich mit sendTo / onMessage

[Ben1983]

@haus-automatisierung OK

[Ben1983]

@haus-automatisierung nutzt Du immer sendTo?
Oder auch messageTo (innerhalb von Skripten)?

[haus-automatisierung]

@ben1983 sagte in auf globale Objekt kann nicht zugegriffen werden:

Oder auch messageTo (innerhalb von Skripten)?

Ist technisch genau das gleiche. messageTo nutzt auch sendTo.

https://github.com/ioBroker/ioBroker.javascript/blob/ae0a27b3edca7e346ddfbff81df40ff77fbfbcd8/lib/sandbox.js#L3526-L3573

https://github.com/ioBroker/ioBroker.javascript/blob/ae0a27b3edca7e346ddfbff81df40ff77fbfbcd8/main.js#L724-L725

[Ben1983]

@haus-automatisierung OK, danke

Ich bekomme nur bei der messageTo den log geschrieben, was ist denn beim SendTo falsch?

messageTo("Testmessage",{name:"Testname"},(callbackdata)=>{
    log(JSON.stringify(callbackdata));   
});

sendTo("javascript.0",'toScript', {
        script: 'script.js.x_testskripte._a_Mesagetest', 
        message: 'Testmessage', 
        data: {
            flag: true
        },function(res){
            log(JSON.stringify(res));
        } 
});

onMessage("Testmessage",(messagedata,callback) =>{
    const myData = {input: messagedata,
                    output:{name:"Test"}};
    callback(myData)
});

Ps. die funktionieren NUR mit callback,oder?
Also mit return kann man nichts zurückgeben, dass man das Ergebnis einer Funktion in einer Zeile direkt zuweisen kann.
Oder?

[haus-automatisierung]

@ben1983 sagte in auf globale Objekt kann nicht zugegriffen werden:

Also mit return kann man nichts zurückgeben, dass man das Ergebnis einer Funktion in einer Zeile direkt zuweisen kann.

Na Du kannst doch mit sendToAsync und await arbeiten. Ganz normal mit Promises halt.

https://github.com/ioBroker/ioBroker.javascript/blob/master/docs/en/javascript.md#sendtoasync

Oder halt messageToAsync:

https://github.com/ioBroker/ioBroker.javascript/blob/master/docs/en/javascript.md#messagetoasync

[Ben1983]

@haus-automatisierung AH ok... hatte mir mal etwas drum rum gebaut, weil damals ein fehler im sendToAsynch war... weiß leider nicht mehr was :-(

async function mySendToAsync(_adapter, cmd, msg) {
    return new Promise((resolve, reject) => {
        sendTo(_adapter, cmd, msg, res => {
            if (!res || res.error) {
                reject(res ? res.error : new Error('Unknown error'));
            } else {
                resolve(res);
            }
        });
    });
}

also kann man auch normal jetzt das messageToAsynch benutzen, oder wie?

Trotzdem macht er irgendwie meinen Callback nicht, ob wohl ich den aufrufe. (im sendTo)

[haus-automatisierung]

@ben1983 sagte in auf globale Objekt kann nicht zugegriffen werden:

also kann man auch normal jetzt das messageToAsynch benutzen, oder wie?

Hatte ich doch schon geschrieben, dass die beiden Funktionen praktisch identisch sind. Siehe Code. messageTo hat halt eine etwas andere Signatur und braucht weniger Infos. Siehe Doku.

Ich hatte das hier gefixt: https://github.com/ioBroker/ioBroker.javascript/pull/1343

Einfach die aktuellste Stable version nehmen dann passts.

[OliverIO]

@ben1983 sagte in auf globale Objekt kann nicht zugegriffen werden:

Schade,dass es nicht geht. Dann bekomme ich halt lauter Fehlermeldungen und weiß nicht, ob es wirklich Fehler sind.

du könntest, zumindest temporär, die zeilen in dem das objekt aufgebaut wird in dein skript kopieren. dann funktioniert auch die vervollständigung.
wenn dann alles getestet ist und es funktioniert, dann entfernst du die zeilen wieder und die zeilen aus global wirken.

das einmalige definieren in global ist vor allem dann gut, wenn du regelmäßig das objekt in seiner definition erweiterst und du dir arbeit sparen willst. dadurch werden fehler reduziert.

[Ben1983]

@haus-automatisierung OK. Cool.

Könntest DU mir freundlicherweise noch sagen, warum ich hier keinen Log der Daten im Callback bekomme?
Ich finde es nicht. Aufruf wird geloggt

sendTo("javascript.0",'toScript', {
        script: 'script.js.x_testskripte._a_Mesagetest', 
        message: 'Testmessage', 
        data: {
            flag: true
        },function(res){
            log(JSON.stringify(res));
        } 
});

onMessage("Testmessage",(messagedata,callback) =>{
    const myData = {input: messagedata,
                    output:{name:"Test"}};
    callback(myData)
    log("Aufruf ");
});

[haus-automatisierung]

@ben1983 sagte in auf globale Objekt kann nicht zugegriffen werden:

Könntest DU mir freundlicherweise noch sagen, warum ich hier keinen Log der Daten im Callback bekomme?

Deine Klammern sind falsch. Die Function ist teil des Msg-Objektes und nicht als separater Parameter definiert. Wenn Du den Code schöner formatieren würdest, siehst Du das auch auf einen Blick :)

So gehts:

sendTo(
    'javascript.0',
    'toScript',
    {
        script: 'script.js.test.playground', 
        message: 'Testmessage', 
        data: {
            flag: true
        }
    },
    (res) => {
        console.log(JSON.stringify(res));
    }
);

onMessage('Testmessage', (messagedata, callback) => {
    console.log('Aufruf');
    const myData = {
        input: messagedata,
        output: {
            name: 'Test'
        }
    };

    callback(myData);
});

[Ben1983]

@haus-automatisierung OK. Danke

Beim Verwendet von messageToAsynch tritt dieser Fehler auf:

messageToAsync("Testmessage",{name:"Testname"});

	javascript.0 (20591) script.js.x_testskripte._a_Mesagetest: TypeError: Cannot read properties of undefined (reading 'timeout')

[paul53]

@ben1983 sagte: "Besser" das Objekt nur dazu definieren, wo es gebraucht wird?

Ja. Die Gruppe "global" verwendet man für häufig verwendete eigene Funktionen.
Mit globalen Objektdefinitionen müllt man sich den RAM der Javascript-Instanz zu, da sie in jedem nicht globalen Skript kompiliert werden.

[haus-automatisierung]

@ben1983 sagte in auf globale Objekt kann nicht zugegriffen werden:

Beim Verwendet von messageToAsynch tritt dieser Fehler auf:

Ist wohl nen Bug wenn die Options fehlen. So gehts:

const res = await messageToAsync('Testmessage', { flag: true }, { timeout: 5000 });
console.log(JSON.stringify(res));

Machst einen GitHub Issue?

[ticaki]

@ben1983 sagte in auf globale Objekt kann nicht zugegriffen werden:

messageToAsync

ups falsch gelesen

EDIT aber trotzdem relevant und die Beschreibung/Definition sollte geändert werden.

function messageToAsync(target: iobJS.MessageTarget | string, data: any, options?: any): Promise<iobJS.MessageCallback | iobJS.MessageCallbackInfo>;

Der {timeout:30000} (zeit is nur ein Beispiel) sollte im Javascript-Adapter zumindest ein default sein oder die Angabe verpflichtend wie jetzt. Ohne timeout kann das an der Stelle hängen bleiben, wie ich schmerzlich feststellen mußt mit einem gleichzeitigen Defekt des Übertemperaturwächters am Boiler.

[haus-automatisierung]

@ticaki sagte in auf globale Objekt kann nicht zugegriffen werden:

Ohne timeout kann das an der Stelle hängen bleiben, wie ich schmerzlich feststellen mußt

Habe ich gerade das erste mal von gehört. Gibts dazu schon einen GitHub Issue? Sollte zumindest in der nächsten Version gefixt sein:

https://github.com/ioBroker/ioBroker.javascript/commit/41e59dfd72f99027a9a303beee8d4d5597826445

[ticaki]

@haus-automatisierung
Ach mist, nicht das es nicht das gleiche ist. Bei mir ist sendToAsync hängen geblieben. Sorry hab die beiden durcheinandergewürfelt.

[haus-automatisierung]

@ticaki sendToAsync funktioniert bei mir wunderbar