asynchron, callback, promise, await -> Hilfe

bahnuhr

Habe log eingebaut.
Man sieht dass das array 4 mal beschrieben wird. Bis die 4 Tel.Nr. durchlaufen sind; passt auch soweit.

Nur die Zeile 14 und 15 wird doch gar nicht ausgeführt. Ansonsten müsste doch ein log Eintrag wg. Zeile 14 kommen.
Kann das die Lösung sein?

J.A.R.V.I.S.

@bahnuhr probier mal folgendes:

// Variablen

//var obj = JSON.parse(getState("fritzbox.0.history.allTableJSON").val);

const obj = JSON.parse(getState("javascript.0.testjson").val);    // Test-json

const logging = true;

let array;

async function main() {

    array = [];

    obj = [{externalNumber: '+49 1573 1234567'}];

    for (let x = 0; x < obj.length; x++) {

        await test(obj[x].date, obj[x].externalNumber).then(async (prom) => {
            
            array.push(prom);

        }).catch((reason) => {
            log(reason, 'error');
        });
    }
    
    setState('javascript.0.Status.Telefon.json_Anrufe', JSON.stringify(array));
}

async function test(datum, tel) {

    return new Promise(function (resolve, reject) {

        sendTo('contact.0', 'query', { phonenumber: tel }, (obj) => {

            if(obj.error) {

                reject('Error: ' + obj.error);

            } else {

                datum = "datum".substring(0, 6) + " : " + "datum".substring(9, 14); // hier wird das Datum gekürzt auf z.B.:  "02.02. : 10:35"
                const name = (obj.contact.fullName) ? obj.contact.fullName : tel; // Telefon-Nr. soll erscheinen wenn Nr. nicht existiert in Kontakte

                resolve({ Datum: datum, VollName: name });
            }
        });
    });
}

main();

AlCalzone

@bahnuhr @J-A-R-V-I-S sendTo gibt keinen Promise zurück, also kann man die Funktion nicht awaiten. Der Ansatz mit new Promise ist richtig und dürfte funktionieren.

Allerdings stellen sich mir bei diesem await/.then-Mischmasch die Fußnägel hoch:

await test(obj[x].date, obj[x].externalNumber)
  .then(async prom => {
    array.push(prom);
  })
  .catch(reason => {
    log(reason, "error");
  });

so ist doch viel schöner:

try {
  const prom = await test(obj[x].date, obj[x].externalNumber);
  array.push(prom);
} catch (reason) {
  log(reason, "error");
}

bahnuhr

Ich kann nur sagen: Klasse, beides funktioniert.

Ihr beide seit super; da diskutiert ihr was schöner ist.

Und ich bin froh wenn es überhaupt läuft

Nochmals Danke.

bahnuhr

So, mein Script ist Dank @AlCalzone und @J-A-R-V-I-S fertig.

Stelle ich hiermit zur Verfügung.
Das Script nimmt die json Tabelle der fritzbox und ändert die Nr. in bekannte Namen ab.
Weiterhin kleinere Formatierungen. Kann natürlich jeder für sich ändern wie er es möchte.


// Script analysiert den DP:  fritzbox.0.history.allTableJSON
// Hierbei sollen die Tel.Nr. im json gegen die Namen (Adapter "contact") ausgetauscht werden.
// Nach Ermittlung aller Namen soll wieder ein json erzeugt werden
// um dann die Darstellung in VIS zu ermöglichen.
//
// Insbesondere meinen Dank an: @AlCalzone und @J-A-R-V-I-S
// 01/2020 bahnuhr


// Variablen
const obj = JSON.parse(getState("fritzbox.0.history.allTableJSON").val);
const logging = false; let array;

async function main() {
    array = [];
    for (let x = 0; x < obj.length; x++) {
        try { 
            const prom = await test(obj[x].date, obj[x].externalNumber, obj[x].callSymbolColor, obj[x].durationForm); 
            array.push(prom);
        } 
            catch (reason) { log(reason, "error"); 
        }
    }
    if (logging) {log (JSON.stringify(array)); }
    setState('javascript.0.Status.Telefon.json_Anrufe', JSON.stringify(array));
}

 
async function test(sdatuhr, stel, ssymbol, sdauer) {
    return new Promise(function (resolve, reject) {
        sendTo('contact.0', 'query', { phonenumber: stel }, (obj) => {
            if(obj.error) {
                reject('Error: ' + obj.error);
            } else {
                sdatuhr = sdatuhr.substring(0, 6) + " : " + sdatuhr.substring(9, 14);     // hier wird das Datum gekürzt auf z.B.:  "02.02. : 10:35"
                const sname = obj.contact.fullName ? obj.contact.fullName : stel;   // Telefon-Nr. soll erscheinen wenn Nr. nicht existiert in Kontakte
               // gesprochene Zeit prüfen; wenn Sek. dann 00:0 oder 00 voran setzen
                    sdauer = sdauer.trim();
                    v1 = sdauer.match(':') ? true : false;
                    v2 = sdauer.match('-') ? true : false;
                    if (v1 === false && v2 === false) { sdauer<10 ? sdauer="00:0"+sdauer : sdauer="00:"+sdauer; }
                    if (sdauer.length == 4) { sdauer = "0" + sdauer };
                    if (logging) { log (sdauer.length + " " + sdauer) };
                resolve({ Datum: sdatuhr, Telefon: stel, VollName: sname, Symbol: ssymbol, Dauer: sdauer});
            }
        });
    });
}

main();


on({id: "fritzbox.0.history.allTableJSON", change: "ne"}, function(obj) {   
    main();
});

In vis sieht das ganze dann so aus:

Viel Spaß damit und nochmals Danke für die Unterstützung.

mfg
Dieter

bahnuhr

Noch ne Ergänzungsfrage:

Ich hab ja unten die folgenden Zeilen:

main();
on({id: "fritzbox.0.history.allTableJSON", change: "ne"}, function(obj) {   
    log ("hier1");
    main();
});

Wenn ich das Script manuell starte wird ja (main()) ausgeführt.
Der on Befehl klappt aber nicht.
Genauer:
log ("hier1"), wird im log ausgegeben. (also ist das Script doch in der on drin.
Der Befehl (main()) wird jedoch nicht ausgeführt.

Warum ist das denn so ?

AlCalzone

@bahnuhr Die Variable obj wird im obigen Skript nur beim Start 1x beschrieben. Die Aktualisierung bekommt sie gar nicht mit. Am besten gibst du ihr einen anderen Namen, z.B. table und änderst den Trigger wie folgt ab (ungetestet):

on({id: "fritzbox.0.history.allTableJSON", change: "ne"}, function(obj) {   

    table = obj.state.val;

    main();

});

bahnuhr

@AlCalzone sagte in asynchron, callback, promise, await -> Hilfe:

@bahnuhr wie sieht main() denn aus?

1 Beitrag weiter oben hatte ich das Script eingestellt.
Aber nun nochmal.

 
// Script analysiert den DP:  fritzbox.0.history.allTableJSON
// Hierbei sollen die Tel.Nr. im json gegen die Namen (Adapter "contact") ausgetauscht werden.
// Nach Ermittlung aller Namen soll wieder ein json erzeugt werden
// um dann die Darstellung in VIS zu ermöglichen.
//
// Insbesondere meinen Dank an: @AlCalzone und @J-A-R-V-I-S
// 01/2020 bahnuhr
 
 
// Variablen
const obj = JSON.parse(getState("fritzbox.0.history.allTableJSON").val);
const logging = false; let array;
 
async function main() {
    array = [];
    for (let x = 0; x < obj.length; x++) {
        try { 
            const prom = await test(obj[x].date, obj[x].externalNumber, obj[x].callSymbolColor, obj[x].durationForm); 
            array.push(prom);
        } 
            catch (reason) { log(reason, "error"); 
        }
    }
    if (logging) {log (JSON.stringify(array)); }
    setState('javascript.0.Status.Telefon.json_Anrufe', JSON.stringify(array));
}
 
 
async function test(sdatuhr, stel, ssymbol, sdauer) {
    return new Promise(function (resolve, reject) {
        sendTo('contact.0', 'query', { phonenumber: stel }, (obj) => {
            if(obj.error) {
                reject('Error: ' + obj.error);
            } else {
                sdatuhr = sdatuhr.substring(0, 6) + " : " + sdatuhr.substring(9, 14);     // hier wird das Datum gekürzt auf z.B.:  "02.02. : 10:35"
                const sname = obj.contact.fullName ? obj.contact.fullName : stel;   // Telefon-Nr. soll erscheinen wenn Nr. nicht existiert in Kontakte
               // gesprochene Zeit prüfen; wenn Sek. dann 00:0 oder 00 voran setzen
                    sdauer = sdauer.trim();
                    v1 = sdauer.match(':') ? true : false;
                    v2 = sdauer.match('-') ? true : false;
                    if (v1 === false && v2 === false) { sdauer<10 ? sdauer="00:0"+sdauer : sdauer="00:"+sdauer; }
                    if (sdauer.length == 4) { sdauer = "0" + sdauer };
                    if (logging) { log (sdauer.length + " " + sdauer) };
                resolve({ Datum: sdatuhr, Telefon: stel, VollName: sname, Symbol: ssymbol, Dauer: sdauer});
            }
        });
    });
}
 
main();
 
 
on({id: "fritzbox.0.history.allTableJSON", change: "ne"}, function(obj) {   
    main();
});

AlCalzone

@bahnuhr Ja hab auch nochmal nachgedacht, siehe mein vorletzter Post

AlCalzone

@AlCalzone @bahnuhr Die Variable muss dann aber mit let deklariert werden. Außerdem hab ich gerne die Trigger und das initiale Auslesen zusammen an einer Stelle.

bahnuhr

@AlCalzone sagte in asynchron, callback, promise, await -> Hilfe:

@AlCalzone @bahnuhr Die Variable muss dann aber mit let deklariert werden. Außerdem hab ich gerne die Trigger und das initiale Auslesen zusammen an einer Stelle.

Mann, Mann: manchmal sieht man den Wald vor lauter Bäumen nicht.

Hab die Variablen mal in main() gepackt.
Läuft jetzt.
Anbei das komplette Script.


// Script analysiert den DP:  fritzbox.0.history.allTableJSON
// Hierbei sollen die Tel.Nr. im json gegen die Namen (Adapter "contact") ausgetauscht werden.
// Nach Ermittlung aller Namen soll wieder ein json erzeugt werden
// um dann die Darstellung in VIS zu ermöglichen.
//
// Insbesondere meinen Dank an: @AlCalzone und @J-A-R-V-I-S
// 01/2020 bahnuhr



async function main() {
    // Variablen
    const obj = JSON.parse(getState("fritzbox.0.history.allTableJSON").val);
    const logging = true; let array;

    array = [];
    for (let x = 0; x < obj.length; x++) {
        try { 
            const prom = await test(obj[x].date, obj[x].externalNumber, obj[x].callSymbolColor, obj[x].durationForm); 
            array.push(prom);
        } 
            catch (reason) { log(reason, "error"); 
        }
    }
    if (logging) {log (JSON.stringify(array)); }
    setState('javascript.0.Status.Telefon.json_Anrufe', JSON.stringify(array));
}

 
async function test(sdatuhr, stel, ssymbol, sdauer) {
    return new Promise(function (resolve, reject) {
        sendTo('contact.0', 'query', { phonenumber: stel }, (obj) => {
            if(obj.error) {
                reject('Error: ' + obj.error);
            } else {
                sdatuhr = sdatuhr.substring(0, 6) + " : " + sdatuhr.substring(9, 14);     // hier wird das Datum gekürzt auf z.B.:  "02.02. : 10:35"
                const sname = obj.contact.fullName ? obj.contact.fullName : stel;   // Telefon-Nr. soll erscheinen wenn Nr. nicht existiert in Kontakte
               // gesprochene Zeit prüfen; wenn Sek. dann 00:0 oder 00 voran setzen
                    sdauer = sdauer.trim();
                    v1 = sdauer.match(':') ? true : false;
                    v2 = sdauer.match('-') ? true : false;
                    if (v1 === false && v2 === false) { sdauer<10 ? sdauer="00:0"+sdauer : sdauer="00:"+sdauer; }
                    if (sdauer.length == 4) { sdauer = "0" + sdauer };
                resolve({ Datum: sdatuhr, Telefon: stel, VollName: sname, Symbol: ssymbol, Dauer: sdauer});
            }
        });
    });
}

main();


on({id: "fritzbox.0.history.allTableJSON", change: "ne"}, function() {   
    main();
});

mfg
Dieter

sigi234

@bahnuhr

Hallo, muss da nicht der alte FB-Adapter installiert sein?

Diesen DP fritzbox.0.history.allTableJSON gibt es im Community Adapter nicht.

Dieser DP javascript.0.Status.Telefon.json_Anrufe muss manuel angelegt werden?

bahnuhr

@sigi234 sagte in asynchron, callback, promise, await -> Hilfe:

@bahnuhr

Hallo, muss da nicht der alte FB-Adapter installiert sein?

Diesen DP fritzbox.0.history.allTableJSON gibt es im Community Adapter nicht.

Ja, stimmt. hab ich ja auch.
Vom TR hab ich aber doch auch nichts geschrieben.

In meinem anderen thread hab ich auch gerade festgestellt, dass die Duration beim community TR nicht stimmt.
Oder es ist bewusst so.

mfg
Dieter

sigi234

@bahnuhr sagte in asynchron, callback, promise, await -> Hilfe:

So die Idee war nun, die json Tabelle der fritzbox als cacheselektor zu nehmen; die Nr. raus zu holen, eingeben in query um den Namen zu holen und dann wieder ein json in einem seperaten DP zu erzeugen.

Und für was brauche ich das jetzt? Stehe auf der Leitung.

bahnuhr

@sigi234
Ich lasse mir in vis die Anrufe anzeigen. (wie so viele andere auch).

Und bei dieser Tabelle möchte ich den Namen angezeigt bekommen; und nicht die Nr..
Weiterhin wollte ich das Datum sowie die Duration verändert angezeigt bekommen.

sieht dann so aus:

sigi234

@bahnuhr sagte in asynchron, callback, promise, await -> Hilfe:

Vom TR hab ich aber doch auch nichts geschrieben.

Also du meinst diesen:

https://github.com/ruhr70/ioBroker.fritzbox/blob/master/README.md

bahnuhr

@sigi234

Der, der bei den Adaptern aufgelistet ist:

sigi234

@bahnuhr sagte in asynchron, callback, promise, await -> Hilfe:

@sigi234

Der, der bei den Adaptern aufgelistet ist:

Ok, den habe ich nicht mehr. Nur mehr den TR-064 Community.
Muss mal @apollon77 fragen ob der noch betreut wird.

bahnuhr

@sigi234 sagte in asynchron, callback, promise, await -> Hilfe:

@bahnuhr sagte in asynchron, callback, promise, await -> Hilfe:

@sigi234

Der, der bei den Adaptern aufgelistet ist:

Ok, den habe ich nicht mehr. Nur mehr den TR-064 Community.
Muss mal @apollon77 fragen ob der noch betreut wird.

Der fritz.box funktioniert ja.
Frag lieber mal warum die duration im tr nicht stimmt.
Danke.

mfg

sigi234

@bahnuhr sagte in asynchron, callback, promise, await -> Hilfe:

Frag lieber mal warum die duration im tr nicht stimmt.

Ich GLAUBE mal was gelesen zu haben , das diese erst ab min. 30 Sekunden gezählt wird. Auch hängt es mit dem Einstellungen vom Anrufbeantworter zusammen.

NEWS

asynchron, callback, promise, await -> Hilfe

Support us

1.5k
Online

30.7k
Users

77.3k
Topics

1.2m
Posts

NEWS

asynchron, callback, promise, await -> Hilfe

Support us

1.5kOnline

30.7kUsers

77.3kTopics

1.2mPosts

1.5k
Online

30.7k
Users

77.3k
Topics

1.2m
Posts