fix: empty array throw error

ashokdey · ashokdey · commit 580e3ba89d97 · 2019-10-17T21:38:10.000+05:30
diff --git a/src/_DataStructures_/Trees/BinaryTree/index.js b/src/_DataStructures_/Trees/BinaryTree/index.js
@@ -2,10 +2,10 @@ const Node = require('./Node');
 
 class BinaryTree {
   constructor(arr) {
-    if (!Array.isArray(arr)) {
+    if (!Array.isArray(arr) || !arr.length) {
       throw new Error('Invalid argument to create a Binary Tre');
     }
-    this.root = this.createBinaryTree(this.root, arr, 0);
+    this.root = this.createBinaryTree((this.root = null), arr, 0);
   }
 
   // eslint-disable-next-line class-methods-use-this

Original file line number	Diff line number	Diff line change
`@@ -2,10 +2,10 @@ const Node = require('./Node');`
`2`	`2`
`3`	`3`	`class BinaryTree {`
`4`	`4`	`constructor(arr) {`
`5`		`- if (!Array.isArray(arr)) {`
	`5`	`+ if (!Array.isArray(arr) \|\| !arr.length) {`
`6`	`6`	`throw new Error('Invalid argument to create a Binary Tre');`
`7`	`7`	`}`
`8`		`- this.root = this.createBinaryTree(this.root, arr, 0);`
	`8`	`+ this.root = this.createBinaryTree((this.root = null), arr, 0);`
`9`	`9`	`}`
`10`	`10`
`11`	`11`	`// eslint-disable-next-line class-methods-use-this`