warten auf exec befehl [resolved]

[MarkusK]

@fastfoot Danke

[Karel Puhli]

Nabend!
Das Thema ist schon etwas älter, aber passt so ziemlich genau auf meinen Anwendungsfall. Ich möchte mir per Telegram Bilder von der Cam zuschicken.
Folgendes Script liegt zu Grunde:

const idTestTrigger = "0_userdata.0.Fenster1";


async function fotoMachen() {
    return new Promise ((resolve) => {
        exec("wget --output-document /home/iobroker/IP_Cam/Snapshots/Garten.jpg http://10.146.26.135:8765/picture/2/current/");
        exec("wget --output-document /home/iobroker/IP_Cam/Snapshots/Wintergarten.jpg http://10.146.26.135:8765/picture/1/current/");
        resolve();
    });
}


async function fotoSenden() {
    await fotoMachen();
    sendTo(`telegram`, `/home/iobroker/IP_Cam/Snapshots/Garten.jpg`);
    sendTo(`telegram`, `/home/iobroker/IP_Cam/Snapshots/Wintergarten.jpg`);
}

on({id: idTestTrigger, change: "any"}, async function () {
    await fotoSenden();
})

Es funktioniert trotz promise, trotz await und sonst was leider nicht. Die Lösung mit einem Timeout vor dem sendTo kenne ich, sie funktioniert auch. Nur wollte ich es etwas eleganter machen. Leider habe ich wohl einen Nagel im Kopf. Hat jemand ne Idee

Log:

elegram.0	2023-06-15 20:22:02.837	error	Cannot send photo [chatId - 1271149145]: Error: ETELEGRAM: 400 Bad Request: file must be non-empty

LG Karel

[ticaki]

@karel-puhli

Hatte erst was mit callbacks geschrieben aber geht auch komplizierter Code hab ich bei fastfood geklaut

function some_execAsync() {
    return new Promise((resolve, reject) => {
        exec("wget --output-document /home/iobroker/IP_Cam/Snapshots/Garten.jpg http://10.146.26.135:8765/picture/2/current/", function (error, stdout, stderr) {
            log("1 exec ist fertig");
            resolve();
        });
    })
}

[Karel Puhli]

@ticaki
Oh man, so hatte ich es vorhin tatsächlich schon mal stehen. Anscheinend war irgendwo doch ein "Schreibfehler" oder so drin...
Trotzdem vielen Dank natürlich an Dich!!!

[fastfoot]

@ticaki sagte in warten auf exec befehl [resolved]:

bei fastfood geklaut

das geht garnicht, ich schenk' ihn dir
hier noch eine Alternative:

const util = require("util");
const execAsync = util.promisify(require("child_process").exec);

const test = async (cmd) => {
    let erg = await execAsync(cmd);

    log('1 ' + erg.stdout)
    log('2 ' + erg.stderr)
}

test('ls')

[ticaki]

@fastfoot
den will ich auch geschenkt haben

[fastfoot]

@ticaki sagte in warten auf exec befehl [resolved]:

@fastfoot
den will ich auch geschenkt haben

nur zu wichtiger als diese Schnipsel ist aber die Tatsache dass man mit beiden Methoden fast alles was einen callback nutzt in ein Promise wandeln kann, auf welches man dann ganz bequem mit await warten kann. Gut für ältere Tools welche callbacks verwenden, neuere bieten ja fast immer auch eine Version mit Promise an. Auch wichtig, eine async function zu erstellen macht nur dann Sinn wenn man darin auch await verwendet

[Karel Puhli]

@ticaki sagte in warten auf exec befehl [resolved]:

erst was mit callbacks geschrieben aber geht auch komplizierter

Naja, ich finde callbacks eher kompliziert zum Lesen.

@fastfoot sagte in warten auf exec befehl [resolved]:

eine async function zu erstellen macht nur dann Sinn wenn man darin auch await verwendet

Blockly bildet ja eigentlich immer async ab, ob await vorkommt, oder nicht. Ist ja auch nicht schädlich, oder?

[fastfoot]

@karel-puhli in JS schadet es der Lesbarkeit bzw Verständlichkeit, ansonsten schadet es nicht. In Blockly vermute ich ist es der Einfachheit wegen, async davor und passt immer

[Axel21]

@fastfoot
Beim Suchen nach Exec und wait bin ich hierher gekommen. Auch wenn das Thema etwas älter ist, es bleibt aktuell. Und danke für die vielen Anregungen!
Für alle, die in BLOCKLY eine execcmd- Funktion möchten, die auf das Ergebnis des Kommandos wartet und das Ergebnis dann auch direkt an eine Variable weitergeben kann: Hier die entsprechende JS-Funktion, die man in den Block "JS-Funktion mit Ergebnis" innerhalb Blockly -> Funktionen einfügt.

return new Promise((resolve, reject) => {
        exec(cmd, function (error, stdout, stderr) {
            resolve(stdout);
        });
    });

Den Unterschied zum klassischen exec-Block sieht man an folgendem Beispiel (Timestamps beachten).
Wer mit asynchroner JS-Programmierung superfit ist, und verschachtelte callbacks mag, braucht das natürlich nicht Vielleicht hilft es ja jemandem, der wie ich danach gesucht hat.