Hilfe für Javascript Legastheniker und js v8.3.x

oFbEQnpoLKKl6mbY5e13

Was mich einfach nachdenklich macht und zweifeln lässt, ist, dass ich durchaus die Umstellung gemacht habe. Dazu habe ich auch https://www.iobroker.net/#en/adapters/adapterref/iobroker.javascript/README.md
genutzt. Ich habe ja nicht einfach danach gefragt, ob mir das jemand bitte umstellen kann, damit ich mich nicht damit beschäftigen muss. Trotzdem tappe ich total im Dunkeln...

Homoran

@ofbeqnpolkkl6mby5e13 ja, ich will es auch versuchen zu können, wenigstens nachvollziehen.

mir scheint, bei denen wo es nicht klappt ist jedesmal die Funktion readJson (url, function(err, response)) mit beteiligt, wobei der url identisch zu dem httpGet url ist.

das finde ich bei dir jedoch nicht

oFbEQnpoLKKl6mbY5e13

@homoran

In dem Skript mit request gab es den Parameter url. Den habe ich entfernt, weil er für httpGet nicht benötigt wird. Die Abfrage der Daten funktioniert ja auch. Keine Ahnung...

Homoran

@ofbeqnpolkkl6mby5e13 sagte in Hilfe für Javascript Legastheniker und js v8.3.x:

httpGet(options), (error, response)

oooh, da hatte ich mein Beispiel zu stark vereinfacht.
Ich glaube nicht, dass da wörtlich options stehen darf. Da muss je nach Anwendung eine Option, z.B. der url mit timeout usw. rein

EDIT: DU hast options als variable deklariert!
alles gut

oFbEQnpoLKKl6mbY5e13

@homoran

Richtig, options hat der Entwickler als Variable deklariert.

Homoran

@ofbeqnpolkkl6mby5e13 sagte in Hilfe für Javascript Legastheniker und js v8.3.x:

@homoran

Richtig, options hat der Entwickler als Variable deklariert.

damit hast du 2x httpGet
einmal mit direkter Eingabe des url in Zeile 27, mit Ausgabe des response in Zeile 30, und einmal in Zeile 46 mit dem url in der Variable options
Da wäre es doch interessant

@tt-tom sagte in Hilfe für Javascript Legastheniker und js v8.3.x:

Setze das mal in Zeile 48 mit rein, damit du sehen kannst was da kommt.

ob es da auch ankommt

oFbEQnpoLKKl6mbY5e13

@homoran

Die Funktion updateMuell (Zeile 43) wird nur ausgeführt wenn "schedule('{"time":{"exactTime":true,"start":"12:17"},"period":{"days":1}}'".

Beim ersten Start muss trotzdem aktualisiert bzw. die Objektstruktur erstellt werden.

Edit:
Nee, stimmt nicht. Die wird auch beim Starten ausgeführt.

Homoran

@ofbeqnpolkkl6mby5e13 sagte in Hilfe für Javascript Legastheniker und js v8.3.x:

Edit

danke! ich wollte gerade für heute Schluss machen weil ich keine andere Stelle mehr fand

also mal per log prüfen ob options klappt

oFbEQnpoLKKl6mbY5e13

@homoran

httpGet(options), (error, response)
    console.log(response.statusCode);
    console.log(response.data);

Wird nichts ausgegeben.

Homoran

@ofbeqnpolkkl6mby5e13 sagte in Hilfe für Javascript Legastheniker und js v8.3.x:

@homoran

httpGet(options), (error, response)
    console.log(response.statusCode);
    console.log(response.data);

Wird nichts ausgegeben.

dann wird auch nichts angelegt.

Nur warum?
ist options nicht korrekt zusammengebaut?

console.log(options) sagt das gleiche was in Zeile 27 steht?

oFbEQnpoLKKl6mbY5e13

@homoran

Wenn ich das mache:

httpGet(options), (error, response)
    console.log(response.statusCode);
    console.log(response.data);
    console.log("test");

Wird "test" nicht ausgegeben. Das Skript ruft also die Funktion gar nicht auf.

Homoran

@ofbeqnpolkkl6mby5e13
Sorry, dann mach ich doch Schluss für heute.
mir raucht der Kopf. Das hemmt die Kreativität

Mal sehen was abdere bis morgen rausfinden

paul53

@ofbeqnpolkkl6mby5e13 sagte: Wird nichts ausgegeben.

Es fehlt auch was.

httpGet(options, (error, response) => {

oFbEQnpoLKKl6mbY5e13

@paul53

Danke!

2024-05-26 22:19:52.596  - [32minfo[39m: javascript.0 (3416) Start JavaScript script.js.common._Tests.Test_17 (JavaScript/js)
2024-05-26 22:19:52.597  - [31merror[39m: javascript.0 (3416) script.js.common._Tests.Test_17 compile failed:
at script.js.common._Tests.Test_17:118
2024-05-26 22:19:52.598  - [31merror[39m: javascript.0 (3416) })();
2024-05-26 22:19:52.598  - [31merror[39m: javascript.0 (3416)  ^
2024-05-26 22:19:52.598  - [31merror[39m: javascript.0 (3416) SyntaxError: Unexpected token ')'
2024-05-26 22:19:52.598  - [31merror[39m: javascript.0 (3416)     at new Script (node:vm:99:7)
2024-05-26 22:19:52.598  - [31merror[39m: javascript.0 (3416)     at createVM (/opt/iobroker/node_modules/iobroker.javascript/main.js:1917:21)
2024-05-26 22:19:52.599  - [31merror[39m: javascript.0 (3416)     at prepareScript (/opt/iobroker/node_modules/iobroker.javascript/main.js:2183:37)
2024-05-26 22:19:52.599  - [31merror[39m: javascript.0 (3416)     at /opt/iobroker/node_modules/iobroker.javascript/main.js:2281:17
2024-05-26 22:19:52.599  - [31merror[39m: javascript.0 (3416)     at Immediate._onImmediate (/opt/iobroker/node_modules/iobroker.javascript/main.js:1703:17)
2024-05-26 22:19:52.599  - [31merror[39m: javascript.0 (3416)     at process.processImmediate (node:internal/timers:478:21)
2024-05-26 22:19:55.316  - [32minfo[39m: javascript.0 (3416) Stopping script script.js.common._Tests.Test_17

An welcher Stelle ist die Klammer zu viel?

paul53

@homoran sagte: nur noch mit timeout of x msec exceeded und nicht mehr die ganzen null Meldungen

paul53

@ofbeqnpolkkl6mby5e13 sagte: An welcher Stelle ist die Klammer zu viel?

?? Die komplette Testfunktion:

httpGet(options, (error, response) => {
    console.log(response.statusCode);
    console.log(response.data);
});

oFbEQnpoLKKl6mbY5e13

@paul53

Aber das war nur ein winziger Ausschnitt aus einem größeren Skript:

const city_id = XXXXX;
const area_id = XXX;
const logging = false;
const name = 'Müllabfuhr';
const numberOfEntries = 69;

const instanz = '0_userdata.' + instance + '.muell-test';

if (logging) log('starting muell.' + instanz);

extendObject(instanz, {
       type: "device",
       common: {
           name: name
       }
   }, function (err) {
       if (err) {
           log('could not create device: ', 'warn');
           return;
       }
   
   }
);
let baseData = {};

// query names first
httpGet('https://mymuell.jumomind.com/mmapp/api.php?r=trash&city_id=' + city_id + '&area_id=' + area_id, { timeout: 2000 }, (error, response) => {
   if (!error && response.statusCode == 200) {
       console.log(response.statusCode);
       console.log(response.data);
   } else {
       console.error(error);
   var data = JSON.parse(response.data);  // info ist ein Objekt
           data.forEach((v, i) => {
               baseData[v._name] = v;
           })
   updateMuell();	
   }
});

schedule('{"time":{"exactTime":true,"start":"12:17"},"period":{"days":1}}', updateMuell);

function updateMuell() {
   var options = 'https://mymuell.jumomind.com/webservice.php?idx=termins&city_id=' + city_id + '&area_id=' + area_id + '&ws=3'

   httpGet(options), (error, response) => {
   console.log(response.statusCode);
   console.log(response.data);
   if (!error && response.statusCode == 200) 
       var info = JSON.parse(response.data);  // info ist ein Objekt
           if (info[0].Ack === 'Success') {
               var data = info[0]._data;  // xy ist eine Eigenschaft des Objektes info
               let counter = 0;
               const date = (new Date())
               const todayStr = date.getFullYear() + "-" + ('00' + (date.getMonth() + 1 )).slice(-2) + "-" + ('00' + date.getDate()).slice(-2)
               log(todayStr);
               data.forEach((v, i) => {
                   if (todayStr < v.cal_date && counter <= numberOfEntries) {
                       
                       const basePath = instanz + '.' + ('000' + counter).slice(-3);
                       log(v.cal_date + ' -> ' + basePath, 'debug');
                       // States erstellen
                       extendObject(basePath, {
                               type: "channel",
                               common: {
                                   name: v.cal_date_normal
                               }
                           }, function (err) {
                               if (err) {
                                   log('could not create device: ', 'warn');
                                   return;
                               }
                           
                           }
                       );
                       createState(basePath + '.date', v.cal_date_normal, {
                           name: 'Datum',
                           desc: 'Datum der Abholung',
                           type: 'string',
                           read: true,
                           write: false
                       }, () => {
                           setState(basePath + '.date', v.cal_date_normal, true);
                       });
                       
                       createState(basePath + '.desc', baseData[v.cal_garbage_type].title, {
                           name: 'Beschreibung',
                           desc: 'Beschreibung der Abholung',
                           type: 'string',
                           read: true,
                           write: false
                       }, () => {
                           setState(basePath + '.desc', baseData[v.cal_garbage_type].title, true);
                       });
                       createState(basePath + '.color', {
                           name: 'Farbe',
                           desc: 'Farbe',
                           type: 'string',
                           role: 'level.color',
                           read: true,
                           write: false
                       }, () => {
                           setState(basePath + '.color', baseData[v.cal_garbage_type].color, true);
                       });

                       counter++;
                   }
               });
           }
       };

TT-Tom

@ofbeqnpolkkl6mby5e13 sagte in Hilfe für Javascript Legastheniker und js v8.3.x:

httpGet(options), (error, response) => {

Zeile 46 hinter options die ) muss weg.

oFbEQnpoLKKl6mbY5e13

@tt-tom

2024-05-26 22:46:45.108  - [31merror[39m: javascript.0 (3416) script.js.common._Tests.Test_17 compile failed:
at script.js.common._Tests.Test_17:111
2024-05-26 22:46:45.109  - [31merror[39m: javascript.0 (3416)         };
2024-05-26 22:46:45.109  - [31merror[39m: javascript.0 (3416)         ^
2024-05-26 22:46:45.109  - [31merror[39m: javascript.0 (3416) SyntaxError: missing ) after argument list
2024-05-26 22:46:45.109  - [31merror[39m: javascript.0 (3416)     at new Script (node:vm:99:7)
2024-05-26 22:46:45.110  - [31merror[39m: javascript.0 (3416)     at createVM (/opt/iobroker/node_modules/iobroker.javascript/main.js:1917:21)
2024-05-26 22:46:45.110  - [31merror[39m: javascript.0 (3416)     at prepareScript (/opt/iobroker/node_modules/iobroker.javascript/main.js:2183:37)
2024-05-26 22:46:45.110  - [31merror[39m: javascript.0 (3416)     at /opt/iobroker/node_modules/iobroker.javascript/main.js:2281:17
2024-05-26 22:46:45.110  - [31merror[39m: javascript.0 (3416)     at Immediate._onImmediate (/opt/iobroker/node_modules/iobroker.javascript/main.js:1703:17)
2024-05-26 22:46:45.111  - [31merror[39m: javascript.0 (3416)     at process.processImmediate (node:internal/timers:478:21)
2024-05-26 22:46:46.955  - [32minfo[39m: javascript.0 (3416) Stopping script script.js.common._Tests.Test_17

TT-Tom

@ofbeqnpolkkl6mby5e13

da fehlen irgendwo noch Klammern, bin grad mal am suchen

Edit: probiere mal das jetzt.

function updateMuell() {
    var options = 'https://mymuell.jumomind.com/webservice.php?idx=termins&city_id=' + city_id + '&area_id=' + area_id + '&ws=3'

    httpGet(options, (error, response) => {
        console.log(response.statusCode);
        console.log(response.data);
        if (!error && response.statusCode == 200) {
            var info = JSON.parse(response.data);  // info ist ein Objekt
            if (info[0].Ack === 'Success') {
                var data = info[0]._data;  // xy ist eine Eigenschaft des Objektes info
                let counter = 0;
                const date = (new Date())
                const todayStr = date.getFullYear() + "-" + ('00' + (date.getMonth() + 1)).slice(-2) + "-" + ('00' + date.getDate()).slice(-2)
                log(todayStr);
                data.forEach((v, i) => {
                    if (todayStr < v.cal_date && counter <= numberOfEntries) {

                        const basePath = instanz + '.' + ('000' + counter).slice(-3);
                        log(v.cal_date + ' -> ' + basePath, 'debug');
                        // States erstellen
                        extendObject(basePath, {
                            type: "channel",
                            common: {
                                name: v.cal_date_normal
                            }
                        }, function (err) {
                            if (err) {
                                log('could not create device: ', 'warn');
                                return;
                            }

                        }
                        );
                        createState(basePath + '.date', v.cal_date_normal, {
                            name: 'Datum',
                            desc: 'Datum der Abholung',
                            type: 'string',
                            read: true,
                            write: false
                        }, () => {
                            setState(basePath + '.date', v.cal_date_normal, true);
                        });

                        createState(basePath + '.desc', baseData[v.cal_garbage_type].title, {
                            name: 'Beschreibung',
                            desc: 'Beschreibung der Abholung',
                            type: 'string',
                            read: true,
                            write: false
                        }, () => {
                            setState(basePath + '.desc', baseData[v.cal_garbage_type].title, true);
                        });
                        createState(basePath + '.color', {
                            name: 'Farbe',
                            desc: 'Farbe',
                            type: 'string',
                            role: 'level.color',
                            read: true,
                            write: false
                        }, () => {
                            setState(basePath + '.color', baseData[v.cal_garbage_type].color, true);
                        });

                        counter++;
                    }
                });
            }
        };
    })
}

NEWS

Hilfe für Javascript Legastheniker und js v8.3.x

Support us

861
Online

31.9k
Users

80.1k
Topics

1.3m
Posts

NEWS

Hilfe für Javascript Legastheniker und js v8.3.x

Support us

861Online

31.9kUsers

80.1kTopics

1.3mPosts

861
Online

31.9k
Users

80.1k
Topics

1.3m
Posts