Read the statement by Michael Teeuw here.
How to best process an [object Promise]?
-
I have an API that is returning a JSON object as an
[object Promise]
. At first I was completely cluless to what this was. I now understand it’s because the API is using some asynchronous function to obtain and present the data. I have spent the last day or two to reading up about this. In particular aboutPromises
,async/await
,fetch/catch
, and evenGenerators
.But since I’m completely new to this kind of JS programming, I’m rather overwhelmed by the various kinds of methods used for doing this. I could need a small hint of guidance for what to look into. Basically a super simple crash-course to get people like me started in the framework of MM modules. In addition, I’ve read that even seasoned developers get this wrong regularly.
What I think I need to add in my node_helper is:
- read the config file to get an option to be used in the API call
- make the promised API call
- wait for correct JSON response from API call
- push JSON to MMM-module
All the above need to be repeated on a regular interval. (I’m not sure this “polling” need to be implemented separately here, or if the MM
config.js
reload mechanism is enough.What is the simplest method to implement that?
I’d love to hear from what you guys think.PS. I’ve read that
async/await
is syntactically simpler to use. -
@E3V3A said in How to best process an [object Promise]?:
I have an API that is returning a JSON object as an
[object Promise]
. At first I was completely cluless to what this was. I now understand it’s because the API is using some asynchronous function to obtain and present the data. I have spent the last day or two to reading up about this. In particular aboutPromises
,async/await
,fetch/catch
, and evenGenerators
.But since I’m completely new to this kind of JS programming, I’m rather overwhelmed by the various kinds of methods used for doing this. I could need a small hint of guidance for what to look into. Basically a super simple crash-course to get people like me started in the framework of MM modules. In addition, I’ve read that even seasoned developers get this wrong regularly.
What I think I need to add in my node_helper is:
- read the config file to get an option to be used in the API call
- make the promised API call
- wait for correct JSON response from API call
- push JSON to MMM-module
All the above need to be repeated on a regular interval. (I’m not sure this “polling” need to be implemented separately here, or if the MM
config.js
reload mechanism is enough.What is the simplest method to implement that?
I’d love to hear from what you guys think.PS. I’ve read that
async/await
is syntactically simpler to use.Hi maybe i can help. Have a look at the node_helper.js in the MMM-NOAA modules. Their i use the google translate api which also gives back a promise. U can try the solution i useed.
-
@tbbear Hi! Thank you, but that is quite an extensive code. Can you pinpoint where you do this?
-
@E3V3A of course no problem gimme a ffew minutes
-
@E3V3A I don’t think there’s a reload mechanism in MM’s
config.js
. For example, you can have anupdateInterval
variable in the module config, but it’s the module’s job to then implement the refresh – it isn’t done automagically by MM.See the code for the default
compliments
module for where it usesthis.config.updateInterval
to callsetInterval()
withupdateDom()
(to set a timer to reload the DOM after a delay) all within itsstart()
function. -
@ninjabreadman said in How to best process an [object Promise]?:
automagically
Absolutely the best term I have heard to date. :-)
-
@Mykle1 said in How to best process an [object Promise]?:
@ninjabreadman said in How to best process an [object Promise]?:
automagically
Absolutely the best term I have heard to date. :-)
:) Especially in this context! I’m only now starting to understand why it’s called a magic mirror… Whatever you guys manage to create, it seem to have been done by development magic…
But, anyway, my problem is surely just a lack of conceptual and technical knowledge.
In my node_helper.js I was tryign to do this:
const NodeHelper = require("node_helper"); const fs = require('fs'); const async = require('async'); const radar = require('radar-api'); // This API return improper JSON module.exports = NodeHelper.create({ ... async function radarPing() { console.log("ENTER (inside)"); try { const ping = await radar(-8.20917,114.62177,-9.28715,115.71243); console.log("GOT DATA: ", await ping); return await ping; } catch(error) { console.log("API ERROR: ", error); } //return "None"; console.log("EXIT (inside)"); }, readData: function() { var radarData = ""; radarData = radarPing(); console.log("The DATA:\n" + radarData); ...
but that ended up with some weird errors, like:
const ping = await radar(-8.20917,114.62177,-9.28715,115.71243); ^^^^^ SyntaxError: Unexpected identifier at Object.exports.runInThisContext (vm.js:76:16) at Module._compile (module.js:528:28)
Since, I’ve only started to learn about Promises, sync/await, just 2 days ago, I must be missing a lot. Because all the examples I’ve seen, are using more or less this way of writing.
So how the eeek do I make sure that what is returned by the api/poll function, is actual data and not an
[object Promise]
, and how can I handle thePromise { }
that is returned in advance?I’m overwhelmed…
-
I also tried another variant, but that failed too:
radarPing: function() { console.log("ENTER (inside)"); try { var ping = radar(-8.20917,114.62177,-9.28715,115.71243); ping.then({ return ping; //... }); console.log("GOT DATA: ", ping); //return await ping; } catch(error) { console.log("API ERROR: ", error); } //return "None"; console.log("EXIT (inside)"); }, readData: function() { var radarData = ""; radarData = this.radarPing; console.log("The DATA:\n" + radarData); ...
-
I’ve got a partially working function, but it’s a bit funny. For some reason the call to cleanup the JSON is not liked, because it seem that the actual result is not returned, but the
Promise
.This is the code snippet:
radarPing: function() { console.log("ENTER (inside)"); Promise.all([ radar(-8.20917,114.62177,-9.28715,115.71243) ]).then(function(results) { var ping = results; console.log("PING1:"); console.log(ping); var cleanData = jZen(ping); console.log("PING2: ", cleanData); self.sendSocketNotification("NEW_DATA", ping); //"PING" return cleanData; //ping; }); console.log("EXIT (inside)"); }, readData: function() { //const myfile = 'modules/MMM-FlightsAbove/demo.json'; // The demo API use improper JSON var radarData = ""; radarData = this.radarPing(); console.log("The DATA:\n", radarData); /* if ( radarData === "" ) { } */ //let cleanData = jZen(data); //let cleanData = jZen(radarData); let cleanData = radarData; if (isJSON(cleanData) ) { this.sendSocketNotification("NEW_DATA", cleanData); } else { // So WTF is it? console.log("- JSON: false"); console.log("- isAO(dirty): " + isAO(radarData)); console.log("- isAO(clean): " + isAO(cleanData)); console.log("- Data:\n", radarData); } }, ... }); // To check if something is JSON function isJSON(str) { try { return (JSON.parse(str) && !!str); } catch (e) { return false; } } // To check if something is an Array or Object (parsed JSON) function isAO(val) { return val instanceof Array || val instanceof Object ? true : false; } // -------------------------------------------------------------------------- // What: A dirt simple JSON cleanup function that also compactifies the data // NOTE: - Only use on flat and trustworthy ASCII JSON data! // - Cannot handle any characters outside [A-Za-z0-9_\-]. (e.g. UTF-8) // - Using remote data without further sanitation is a security risk! // -------------------------------------------------------------------------- const re1 = /([A-Za-z0-9_\-]+):/gm; // use const to make sure it is compiled function jZen(juice) { //let re1 = /([A-Za-z0-9_\-]+):/gm; // Find all ASCII words $1 before an ":" //let data = juice; let str = ""; str = juice.replace(/\s/gm, ''); // Remove all white-space str = str.replace(/\'/gm, '\"'); // Replace all ' with " str = str.replace(re1, '\"$1\":'); // Replace $1: with "$1": //console.log("Dirty JSON is:\n" + data.toString() ); //console.log("Clean JSON is:\n" + str); return str; }
The output is:
... ENTER (inside) EXIT (inside) The DATA: undefined - JSON: false - isAO(dirty): false - isAO(clean): false - Data: undefined PING1: [ [ { id: '108be389', timestamp: 1519647028, registration: 'PK-GQL', flight: 'QG8816', callsign: 'CTV8816', ...
As you can see, PING2 never happens…