Promisify based solution for lab 8.2 .

I have tried a promisify based solution to lab 8.2 .
Share with you.
Any additional advice is welcome.
Solution 1:
'use strict' const { promisify } = require('util') const print = (err, contents) => { if (err) console.error(err) else console.log(contents) } const opA = (cb) => { setTimeout(() => { cb(null, 'A') }, 500) } const opB = (cb) => { setTimeout(() => { cb(null, 'B') }, 250) } const opC = (cb) => { setTimeout(() => { cb(null, 'C') }, 125) } // solution 1 const printHelper = (contents) => print(null, contents) const pOpA = promisify(opA) const pOpB = promisify(opB) const pOpC = promisify(opC) pOpA() .then(printHelper) .then(() => pOpB()) .then(printHelper) .then(() => pOpC()) .then(printHelper)
Solution 2:
'use strict' const { promisify } = require('util') const print = (err, contents) => { if (err) console.error(err) else console.log(contents) } const opA = (cb) => { setTimeout(() => { cb(null, 'A') }, 500) } const opB = (cb) => { setTimeout(() => { cb(null, 'B') }, 250) } const opC = (cb) => { setTimeout(() => { cb(null, 'C') }, 125) } // solution 2 const pOpA = promisify(opA) const pOpB = promisify(opB) const pOpC = promisify(opC) const operate = async () => { print(null, await pOpA()) print(null, await pOpB()) print(null, await pOpC()) } operate()
Comments
-
Hi,
I followed the .then approach according to the training Promises (Cont.) chapter where we want to execute serial each function.
const contentA = promisify(opA)
const contentB = promisify(opB)
const contentC = promisify(opC)contentA()
.then((contents) => {
print(contents)
return contentB()
})
.then((contents) => {
print (contents)
return contentC()
})
.then(print)0 -
@ghortat said:
Hi,I followed the .then approach according to the training Promises (Cont.) chapter where we want to execute serial each function.
const contentA = promisify(opA)
const contentB = promisify(opB)
const contentC = promisify(opC)contentA()
.then((contents) => {
print(contents)
return contentB()
})
.then((contents) => {
print (contents)
return contentC()
})
.then(print)That looks similar as mine. I abstracted the print function into a printHelper to make the code compact.
0 -
@krave said:
@ghortat said:
Hi,I followed the .then approach according to the training Promises (Cont.) chapter where we want to execute serial each function.
const contentA = promisify(opA)
const contentB = promisify(opB)
const contentC = promisify(opC)contentA()
.then((contents) => {
print(contents)
return contentB()
})
.then((contents) => {
print (contents)
return contentC()
})
.then(print)That looks similar as mine. I abstracted the print function into a printHelper to make the code compact.
You actually don't need to abstract the print function
const { promisify } = require("util"); const print = (err, contents) => { if (err) console.error(err); else console.log(contents); }; const opA = (cb) => { setTimeout(() => { cb(null, "A"); }, 500); }; const opB = (cb) => { setTimeout(() => { cb(null, "B"); }, 250); }; const opC = (cb) => { setTimeout(() => { cb(null, "C"); }, 125); }; const opAProm = promisify(opA); const opBProm = promisify(opB); const opCProm = promisify(opC); opAProm() .then(print) .then(() => opBProm()) .then(print) .then(() => opCProm()) .then(print);
0 -
@nosvalds said:
@krave said:
@ghortat said:
Hi,I followed the .then approach according to the training Promises (Cont.) chapter where we want to execute serial each function.
const contentA = promisify(opA)
const contentB = promisify(opB)
const contentC = promisify(opC)contentA()
.then((contents) => {
print(contents)
return contentB()
})
.then((contents) => {
print (contents)
return contentC()
})
.then(print)That looks similar as mine. I abstracted the print function into a printHelper to make the code compact.
You actually don't need to abstract the print function
const { promisify } = require("util"); const print = (err, contents) => { if (err) console.error(err); else console.log(contents); }; const opA = (cb) => { setTimeout(() => { cb(null, "A"); }, 500); }; const opB = (cb) => { setTimeout(() => { cb(null, "B"); }, 250); }; const opC = (cb) => { setTimeout(() => { cb(null, "C"); }, 125); }; const opAProm = promisify(opA); const opBProm = promisify(opB); const opCProm = promisify(opC); opAProm() .then(print) .then(() => opBProm()) .then(print) .then(() => opCProm()) .then(print);
Hi, nosvalds
If you look close at the code and set a debugger in the print function. Run the code in debug mode and you will notice that the actual method which log out A, B, C is
console.error
.I believe the intention of this function is for whatever error occurs. Not the normal workflow. So I adjusted the handler.
1 -
Prefer async/await to .then and .catch because
- it allows for async stack traces (stack traces over multiple event loop ticks that trace the awaiting of promises in async functions)
- if you forget the .catch handler you end up with a unhandled rejection handler
- less code noise with async await + less chance of deep nesting
1 -
@davidmarkclements Thank you for your advices.
0
Categories
- All Categories
- 49 LFX Mentorship
- 102 LFX Mentorship: Linux Kernel
- 548 Linux Foundation Boot Camps
- 294 Cloud Engineer Boot Camp
- 117 Advanced Cloud Engineer Boot Camp
- 52 DevOps Engineer Boot Camp
- 52 Cloud Native Developer Boot Camp
- 4 Express Training Courses
- 4 Express Courses - Discussion Forum
- 1.9K Training Courses
- 18 LFC110 Class Forum
- 6 LFC131 Class Forum
- 24 LFD102 Class Forum
- 150 LFD103 Class Forum
- 17 LFD121 Class Forum
- 61 LFD201 Class Forum
- LFD210 Class Forum
- LFD210-CN Class Forum
- 1 LFD213 Class Forum - Discontinued
- 128 LFD232 Class Forum
- LFD237 Class Forum
- 23 LFD254 Class Forum
- 596 LFD259 Class Forum
- 102 LFD272 Class Forum
- 1 LFD272-JP クラス フォーラム
- LFD273 Class Forum
- 2 LFS145 Class Forum
- 24 LFS200 Class Forum
- 739 LFS201 Class Forum
- 1 LFS201-JP クラス フォーラム
- 3 LFS203 Class Forum
- 69 LFS207 Class Forum
- 300 LFS211 Class Forum
- 54 LFS216 Class Forum
- 47 LFS241 Class Forum
- 41 LFS242 Class Forum
- 37 LFS243 Class Forum
- 11 LFS244 Class Forum
- 33 LFS250 Class Forum
- 1 LFS250-JP クラス フォーラム
- LFS251 Class Forum
- 138 LFS253 Class Forum
- 1K LFS258 Class Forum
- 10 LFS258-JP クラス フォーラム
- 92 LFS260 Class Forum
- 129 LFS261 Class Forum
- 32 LFS262 Class Forum
- 79 LFS263 Class Forum
- 15 LFS264 Class Forum
- 11 LFS266 Class Forum
- 17 LFS267 Class Forum
- 17 LFS268 Class Forum
- 23 LFS269 Class Forum
- 203 LFS272 Class Forum
- 1 LFS272-JP クラス フォーラム
- LFS281 Class Forum
- 220 LFW211 Class Forum
- 166 LFW212 Class Forum
- SKF100 Class Forum
- 901 Hardware
- 219 Drivers
- 74 I/O Devices
- 44 Monitors
- 115 Multimedia
- 208 Networking
- 101 Printers & Scanners
- 85 Storage
- 761 Linux Distributions
- 88 Debian
- 66 Fedora
- 15 Linux Mint
- 13 Mageia
- 24 openSUSE
- 141 Red Hat Enterprise
- 33 Slackware
- 13 SUSE Enterprise
- 356 Ubuntu
- 476 Linux System Administration
- 41 Cloud Computing
- 69 Command Line/Scripting
- Github systems admin projects
- 94 Linux Security
- 77 Network Management
- 108 System Management
- 49 Web Management
- 66 Mobile Computing
- 23 Android
- 29 Development
- 1.2K New to Linux
- 1.1K Getting Started with Linux
- 536 Off Topic
- 131 Introductions
- 216 Small Talk
- 21 Study Material
- 816 Programming and Development
- 275 Kernel Development
- 507 Software Development
- 927 Software
- 260 Applications
- 183 Command Line
- 3 Compiling/Installing
- 76 Games
- 316 Installation
- 59 All In Program
- 59 All In Forum
Upcoming Training
-
August 20, 2018
Kubernetes Administration (LFS458)
-
August 20, 2018
Linux System Administration (LFS301)
-
August 27, 2018
Open Source Virtualization (LFS462)
-
August 27, 2018
Linux Kernel Debugging and Security (LFD440)