Fix some mistakes in Chapters 8 and 9

marijnh · marijnh · commit 747fa81cc909 · 2018-01-22T17:04:47.000+01:00
diff --git a/08_error.md b/08_error.md
@@ -201,8 +201,8 @@ it annoying, it will also necessarily be ineffective, since it takes
 too much time to exhaustively test everything every time you make a
 change.
 
-Computers are good at repetetive tasks, and testing is the ideal
-repetetive task. Automated testing is the process of writing a program
+Computers are good at repetitive tasks, and testing is the ideal
+repetitive task. Automated testing is the process of writing a program
 that tests another program. Writing tests is a bit more work than
 testing manually, but once you've done it you gain a kind of
 superpower: it only takes you a few seconds to verify that your
@@ -832,7 +832,7 @@ Make sure you handle only the exceptions you are trying to handle.
 {{if interactive
 
 ```{test: no}
-class MultiplicatorUnitFailure extends Error () {}
+class MultiplicatorUnitFailure extends Error {}
 
 function primitiveMultiply(a, b) {
   if (Math.random() < 0.2)
diff --git a/09_regexp.md b/09_regexp.md
@@ -1385,14 +1385,14 @@ let number = /^...$/;
 // Tests:
 for (let str of ["1", "-1", "+15", "1.55", ".5", "5.",
                  "1.3e2", "1E-4", "1e+12"]) {
-  if (!number.test(s)) {
-    console.log("Failed to match '" + s + "'");
+  if (!number.test(str)) {
+    console.log(`Failed to match '${s}'`);
   }
 }
 for (let str of ["1a", "+-1", "1.2.3", "1+1", "1e4.5",
                  ".5.", "1f5", "."]) {
-  if (number.test(s)) {
-    console.log("Incorrectly accepted '" + s + "'");
+  if (number.test(str)) {
+    console.log(`Incorrectly accepted '${s}'`);
   }
 }
 ```
diff --git a/code/solutions/08_2_the_locked_box.js b/code/solutions/08_2_the_locked_box.js
@@ -1,7 +1,8 @@
 function withBoxUnlocked(body) {
   var locked = box.locked;
-  if (!locked)
+  if (!locked) {
     return body();
+  }
 
   box.unlock();
   try {

Original file line number	Diff line number	Diff line change
`@@ -1385,14 +1385,14 @@ let number = /^...$/;`
`1385`	`1385`	`// Tests:`
`1386`	`1386`	`for (let str of ["1", "-1", "+15", "1.55", ".5", "5.",`
`1387`	`1387`	`"1.3e2", "1E-4", "1e+12"]) {`
`1388`		`- if (!number.test(s)) {`
`1389`		`- console.log("Failed to match '" + s + "'");`
	`1388`	`+ if (!number.test(str)) {`
	`1389`	+ console.log(`Failed to match '${s}'`);
`1390`	`1390`	`}`
`1391`	`1391`	`}`
`1392`	`1392`	`for (let str of ["1a", "+-1", "1.2.3", "1+1", "1e4.5",`
`1393`	`1393`	`".5.", "1f5", "."]) {`
`1394`		`- if (number.test(s)) {`
`1395`		`- console.log("Incorrectly accepted '" + s + "'");`
	`1394`	`+ if (number.test(str)) {`
	`1395`	+ console.log(`Incorrectly accepted '${s}'`);
`1396`	`1396`	`}`
`1397`	`1397`	`}`
`1398`	`1398`	```