[Frage] Einfaches Script verursacht JS Absturz - Warum

[Blackeye]

Hallo zusammen,

das Problem gibt es mit Version 3.4.0 noch immer:

avascript.0 ReferenceError: e is not defined at Request. (script.js.common.Aktionen.Aktion_aus_wenn_nicht_gebraucht:4:116) at emitOne (events.js:96:13) at Request.emit (events.js:188:7) at Request.onRequestError (/opt/iobroker/node_modules/iobroker.javascript/node_modules/request/request.js:878:8) at emitOne (events.js:96:13) at ClientRequest.emit (events.js:188:7) at Socket.socketOnEnd (_http_client.js:345:9) at emitNone (events.js:91:20) at Socket.emit (events.js:185:7) at endReadableNT (_stream_readable.js:974:12)

Und bei mir kommt es auch nur zu dem Fehler falls die Adresse die angesteuert werden soll nicht mehr erreichbar ist.

Gruß

Lars

[apollon77]

Wie sieht denn Der js Code aus?

[Dutchman]

@Blackeye:

Und bei mir kommt es auch nur zu dem Fehler falls die Adresse die angesteuert werden soll nicht mehr erreichbar ist. `

Das war doch eigentlich teil eines fixes in version 3.3.4?

3.3.4 (2017-04-01)

(bluefox) Catch error by request if host unavailable
(bluefox) add "request" to script namespace

Hat sich der fehler ab ein bestimmten version eingeschlichen ?

[Blackeye]

Hi,

kann ich nicht sagen, da ich neu in dem Thema bin. Aber es ist genau der beschriebene Fehler:

Der deaktivierte Teil verursacht den Fehler…..

[apollon77]

Aaahhh Blockly … das fehlt im Thread Namen!!

[Dutchman]

@Blackeye:

Hi,

kann ich nicht sagen, da ich neu in dem Thema bin. Aber es ist genau der beschriebene Fehler:

Unbenannt.PNG

Der deaktivierte Teil verursacht den Fehler….. `

Koenntest du mal deinen javascript adapter downgraden nach 3.4.4 oder 3.3.6 und chaun ob das problem dan auch noch da ist ?

Experten modus aktivieren :

Version downgraden :

[Blackeye]

Hi,

bei beiden Versionen folgende Fehler:

<code>host.ioBroker-RasPi	2017-10-29 10:31:04.189	error	instance system.adapter.javascript.0 terminated with code 0 (OK)
javascript.0	2017-10-29 10:31:04.096	error	ReferenceError: e is not defined at Request. (script.js.common.Aktionen.Aktion_aus_wenn_nicht_gebraucht:4:116) at emitOne (events.js:96:13) at Request.emit (events.js:188:7)
javascript.0	2017-10-29 10:31:04.096	error	uncaught exception: e is not defined
javascript.0	2017-10-29 10:31:04.096	error	ReferenceError: e is not defined at Request. (script.js.common.Aktionen.Aktion_aus_wenn_nicht_gebraucht:4:116) at emitOne (events.js:96:13) at Request.emit (events.js:188:7)
javascript.0	2017-10-29 10:31:04.096	error	uncaught exception: e is not defined
[code][/code]</code>

@Dutchman:

@Blackeye:

Koenntest du mal deinen javascript adapter downgraden nach 3.4.4 oder 3.3.6 und chaun ob das problem dan auch noch da ist ?

Gruß

Lars

[apollon77]

Bitte

1.) Das Blockly auf machen, irgend eine kleine Änderung machen das der Javascript-code neu generiert wird

2.) ann schauen ob es wieder passiert

3.) Wenn ja, generiertes JavaScript (Zeige Script) posten

[Blackeye]

Hi,

habe alles gemacht nd bekomme die gleichen Fehler, daher hier der Code…

schedule("* * * * *", function () {
  if (getState("harmony.0.Stube.activities.currentActivity").val != 'Fernseher') {
    try {
      require("request")('http://192.168.188.30/web/powerstate?newstate=1').on("error", function () {console.error(e);});
    } catch (e) { console.error(e); }
    console.log("request: " + 'http://192.168.188.30/web/powerstate?newstate=1');
  }
  if (getState("harmony.0.Stube.activities.currentActivity").val == 'PowerOff') {
    setState("harmony.0.Stube.Sony-Fernseher.PowerOff"/*Sony-Fernseher:PowerOff*/, 1);
  }
});

@apollon77:

Bitte

1.) Das Blockly auf machen, irgend eine kleine Änderung machen das der Javascript-code neu generiert wird

2.) ann schauen ob es wieder passiert

3.) Wenn ja, generiertes JavaScript (Zeige Script) posten `

[pix]

Hallo,

kenn mich nicht Blockly gar nicht aus, aber @Blackeye:

schedule("* * * * *", function () {
  if (getState("harmony.0.Stube.activities.currentActivity").val != 'Fernseher') {
    try {
      require("request")('http://192.168.188.30/web/powerstate?newstate=1').on("error", function () {console.error(e);});
    } catch (e) { console.error(e); }
    console.log("request: " + 'http://192.168.188.30/web/powerstate?newstate=1');
  }
  if (getState("harmony.0.Stube.activities.currentActivity").val == 'PowerOff') {
    setState("harmony.0.Stube.Sony-Fernseher.PowerOff"/*Sony-Fernseher:PowerOff*/, 1);
  }
});
```` `  

Zeile 4 sieht mir nicht valide aus.

[Blackeye]

@pix:

Hallo,

kenn mich nicht Blockly gar nicht aus, aber @Blackeye:

schedule("* * * * *", function () {
  if (getState("harmony.0.Stube.activities.currentActivity").val != 'Fernseher') {
    try {
      require("request")('http://192.168.188.30/web/powerstate?newstate=1').on("error", function () {console.error(e);});
    } catch (e) { console.error(e); }
    console.log("request: " + 'http://192.168.188.30/web/powerstate?newstate=1');
  }
  if (getState("harmony.0.Stube.activities.currentActivity").val == 'PowerOff') {
    setState("harmony.0.Stube.Sony-Fernseher.PowerOff"/*Sony-Fernseher:PowerOff*/, 1);
  }
});
```` `  

Zeile 4 sieht mir nicht valide aus. `

Aber da ich nur mit Blockly gearbeitet habe, spricht ja etwas für einen Fehler.

Danke

[apollon77]

Also in der aktuellen Version des JavaScript Adapters sollte der Code korrekt erzeugt werden:

            return 'try {\n  require("request")(' + URL + ').on("error", function (e) {console.error(e);});\n} catch (e) { console.error(e); }\n' +
                logText;

Exportiere mal das Blockly, lösche dasd Srkitp und erstelle neu und importiere. Und sicherstellen das Javascript Adapter aktuell ist. AM besten von Github installieren

[Blackeye]

Hi,

alles gemacht aber das Ergebnis bleibt. Ist es möglich das irgendwie anders zu lösen?

@apollon77:

Also in der aktuellen Version des JavaScript Adapters sollte der Code korrekt erzeugt werden:

            return 'try {\n  require("request")(' + URL + ').on("error", function (e) {console.error(e);});\n} catch (e) { console.error(e); }\n' +
                logText;

Exportiere mal das Blockly, lösche dasd Srkitp und erstelle neu und importiere. Und sicherstellen das Javascript Adapter aktuell ist. AM besten von Github installieren `

[blauholsten]

Hi,

ich verfolge die letzten Tage diesen Thread, nun will ich doch mal versuchen etwas beizutragen bzw. eine Idee einzubringen. Bitte haut nicht auf mich ein, falls der folgende Text kompletter Quatsch ist, ich habe nur einige Grundkenntnisse von JS. (Von Blocky gar keine)

Für mich ist der Programmcode falsch, denn TRY / CATCH kann keine asyncronen Aufrufe verarbeiten. Im vorliegenden Fall ist der Request aber Asyncron, und der auftretende Fehler tritt auf wenn try schon längst abgearbeitet ist.

Wenn man hier den TRY / CATCH Block weglässt, wird die Instanz nicht mehr abstürzen, denn die ERROR Behandlung wird über````
.on("error“,……

DAS IST NUR EINE VERMUTUNG; ALSO BITTE NICHT LACHEN…….

[blauholsten]

Keine Antwort von den Profis…. :oops:

[AlCalzone]

@blauholsten:

Wenn man hier den TRY / CATCH Block weglässt, wird die Instanz nicht mehr abstürzen, denn die ERROR Behandlung wird über.on("error“,…… abgefangen. `

Ne das ist Quatsch. on("error") ist keine Fehlerbehandlung an sich, sondern gibt Zugriff auf Fehler, die im Request-Modul bereits abgefangen wurden. Try/Catch verhindert Adapterabstürze durch Fehler, die davon nicht abgefangen wurden. Mit dem asynchronen hast du aber Recht. Weglassen von try/catch wird nicht dafür sorgen, dass Abstürze plötzlich nicht mehr zum Absturz führen (außer in catch sind neue Fehler eingebaut)

[blauholsten]

@AlCalzone:

@blauholsten:

Wenn man hier den TRY / CATCH Block weglässt, wird die Instanz nicht mehr abstürzen, denn die ERROR Behandlung wird über.on("error“,…… abgefangen. `

Ne das ist Quatsch. on("error") ist keine Fehlerbehandlung an sich, sondern gibt Zugriff auf Fehler, die im Request-Modul bereits abgefangen wurden. Try/Catch verhindert Adapterabstürze durch Fehler, die davon nicht abgefangen wurden. Mit dem asynchronen hast du aber Recht. Weglassen von try/catch wird nicht dafür sorgen, dass Abstürze plötzlich nicht mehr zum Absturz führen (außer in catch sind neue Fehler eingebaut) `
Hi,

sorry ich habe mich verkehrt ausgedrückt, ich meinte damit Fehler im request wie UNREACH usw….

habe mir auf der ersten Seite die Fehler nachmal angeschaut, es scheint ja so als ob im callback "e" nicht übergeben wurde....

Hier hilft try/catch nicht, da der callback asynchron aufgerufen wird.

 .on("error", function () {console.error(e);});

aber die Entwickler haben das anscheinend behoben, sodass das eigentlich behoben ist :oops:

[Blackeye]

Hi,

ich habe für mich eine Lösung gefunden.

Ich habe den Ping Adapter installiert und frage die Erreichbarkeit des Gerätes ab.

Gruß

Lars