From 5b67f0e7ae35ad07e50aa3037724b00f6d8b88c4 Mon Sep 17 00:00:00 2001 From: CyrilAntoni Date: Tue, 11 Sep 2018 12:19:26 +0200 Subject: [PATCH] Update article.md Just added an 'if' on line 265. --- 6-async/01-callbacks/article.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/6-async/01-callbacks/article.md b/6-async/01-callbacks/article.md index 0cc13215..765f257b 100644 --- a/6-async/01-callbacks/article.md +++ b/6-async/01-callbacks/article.md @@ -262,7 +262,7 @@ function step3(error, script) { See? It does the same, and there's no deep nesting now, because we made every action a separate top-level function. -It works, but the code looks like a torn apart spreadsheet. It's difficult to read, you probably noticed that. One needs to eye-jump between pieces while reading it. That's inconvenient, especially the reader is not familiar with the code and doesn't know where to eye-jump. +It works, but the code looks like a torn apart spreadsheet. It's difficult to read, you probably noticed that. One needs to eye-jump between pieces while reading it. That's inconvenient, especially if the reader is not familiar with the code and doesn't know where to eye-jump. Also the functions named `step*` are all of a single use, they are created only to avoid the "pyramid of doom". No one is going to reuse them outside of the action chain. So there's a bit of a namespace cluttering here.