feat/ exercise now accepts objects

Author: nik-rev

15_totalIntegers/README.md

@@ -1,7 +1,8 @@
 # Exercise 15 - totalIntegers
 
-Write a function that when given a multi-dimensional integer array, return the total number of integers stored inside this array
+Write a function that takes in an arbitrarily deep array or object and returns the total number of integers stored inside this array or object.
 
 ```javascript
 totalIntegers([[[5], 3], 0, 2, ['foo'], [], [4, [5, 6]]]); // returns 7
+totalIntegers({ a: 1, b: { a: [5, 10], b: 11 } }); // returns 4
 ```

15_totalIntegers/solution/totalIntegers-solution.js

@@ -1,19 +1,12 @@
-const isInteger = (number) => {
-  switch (true) {
-    case (Number.isNaN(number)):
-    case (typeof number !== 'number'):
-    case (number % 1 !== 0):
-      return false;
-    default:
-      return true;
-  }
-};
+const totalIntegers = function(obj, count = 0) {
+
+  if (typeof obj !== 'object' || obj === null) return;
+
+  const elements = Array.isArray(obj) ? obj : Object.values(obj)
 
-const totalIntegers = function(array, count = 0) {
-  if (!Array.isArray(array)) return;
-  for (const el of array) {
-    if (isInteger(el)) ++count;
-    if (Array.isArray(el)) count += totalIntegers(el);
+  for (const el of elements) {
+    if (Number.isInteger(el)) ++count;
+    if (typeof el === 'object' && el !== null) count += totalIntegers(el);
   }
   return count;
 };

15_totalIntegers/solution/totalIntegers-solution.spec.js

@@ -13,9 +13,8 @@ describe('totalIntegers', () => {
   test('Works with floats', () => {
     expect(totalIntegers([5, 7.7, 7, [45, 1, 0], [4.0, 7, [7.77777, 4567.4]], [5477.654]])).toBe(7);
   });
-  test('Only accepts arrays', () => {
+  test('Only accepts arrays or objects', () => {
     expect(totalIntegers('2')).toBe(undefined);
-    expect(totalIntegers({})).toBe(undefined);
     expect(totalIntegers(() => {})).toBe(undefined);
     expect(totalIntegers(42)).toBe(undefined);
     expect(totalIntegers(NaN)).toBe(undefined);
@@ -26,4 +25,7 @@ describe('totalIntegers', () => {
   test('Works with a nested array of all kinds of things', () => {
     expect(totalIntegers([NaN, [[{}], 555 ], '444', [], 74.0, undefined, [[() => {}], [4], Infinity, [[[], -44.0], [null, '-4'], NaN [[]], 6]], () => {}, [[], [-Infinity, ['4'], [4.7, -46.7], NaN]]])).toBe(5);
   });
+  test('Works with arrays containing objects containing integers', () => {
+    expect(totalIntegers([4, 6, { a: 1, b: { a: [5, 10], b: 11 } }, 9])).toBe(7);
+  });
 });