Merge pull request microsoft#748 from the1812/patch-1

mhegazy · web-flow · commit a90c7f221029 · 2018-04-09T08:44:56.000-07:00
Improved code in 'Modules'
diff --git a/pages/Modules.md b/pages/Modules.md
@@ -769,8 +769,9 @@ class ProgrammerCalculator extends Calculator {
 
     constructor(public base: number) {
         super();
-        if (base <= 0 || base > ProgrammerCalculator.digits.length) {
-            throw new Error("base has to be within 0 to 16 inclusive.");
+        const maxBase = ProgrammerCalculator.digits.length;
+        if (base <= 0 || base > maxBase) {
+            throw new Error(`base has to be within 0 to ${maxBase} inclusive.`);
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -769,8 +769,9 @@ class ProgrammerCalculator extends Calculator {`
`769`	`769`
`770`	`770`	`constructor(public base: number) {`
`771`	`771`	`super();`
`772`		`- if (base <= 0 \|\| base > ProgrammerCalculator.digits.length) {`
`773`		`- throw new Error("base has to be within 0 to 16 inclusive.");`
	`772`	`+ const maxBase = ProgrammerCalculator.digits.length;`
	`773`	`+ if (base <= 0 \|\| base > maxBase) {`
	`774`	+ throw new Error(`base has to be within 0 to ${maxBase} inclusive.`);
`774`	`775`	`}`
`775`	`776`	`}`
`776`	`777`