feat(2018 day-14): simplify initialization steps for recipes and elves

developher-net · developher-net · commit b008964e5437 · 2018-12-23T07:55:55.000-08:00
Initialize recipes with an array instead of a single value

Breaks foo = new Recipes(x), use foo = new Recipes([x,y]) instead
diff --git a/2018/day-14/recipes.js b/2018/day-14/recipes.js
@@ -2,11 +2,14 @@
  * Circular linked list of recipes
  */
 class Recipes {
-  constructor (recipe) {
+  constructor (recipes) {
     this.head = null
     this.tail = null
     this.length = 0
-    this.addFirst(recipe)
+    this.addFirst(recipes[0])
+    for (let x = 1; x < recipes.length; x++) {
+      this.addRecipe(recipes[x])
+    }
   }
 
   addFirst (recipe) {
diff --git a/2018/day-14/recipes.test.js b/2018/day-14/recipes.test.js
@@ -9,22 +9,29 @@ const {
 } = require('./recipes')
 
 describe('--- Day 14: Chocolate Charts ---', () => {
+  let recipes
+  let elves
   describe('Part 1:', () => {
+    beforeEach(() => {
+      elves = [3, 7]
+      recipes = new Recipes(elves)
+      elves[0] = recipes.tail
+      elves[1] = recipes.head
+    })
     describe('new Recipes()', () => {
       it('builds a linked list', () => {
-        const recipes = new Recipes(0)
+        expect(recipes.head.value).to.equal(7)
         for (let x = 1; x <= 5; x++) {
           recipes.addRecipe(x)
         }
-        expect(recipes.length).to.equal(6)
+        expect(recipes.length).to.equal(7)
         expect(recipes.head.value).to.equal(5)
-        expect(recipes.tail.value).to.equal(0)
+        expect(recipes.tail.value).to.equal(3)
         expect(recipes.tail.prev).to.equal(recipes.head) // circular linked list for prev
         expect(recipes.head.next).to.equal(recipes.tail) // circular linked list for next
       })
       describe('scoreRecipes()', () => {
         it('adds new recipes based on the provided score', () => {
-          const recipes = new Recipes(0)
           for (let x = 1; x <= 5; x++) {
             recipes.addRecipe(x)
           }
@@ -47,18 +54,8 @@ describe('--- Day 14: Chocolate Charts ---', () => {
     describe('loopRecipeForEleves()', () => {
       it('loops through the recipe object for the specified elves the specified number of times', () => {
         const expected = '37101012451589167792' // list of recipe values in the last iteration of the example
-        const elves = [3, 7]
-        const recipes = new Recipes(elves[0])
         let actual = ''
 
-        elves.forEach((elf, idx) => {
-          if (idx === 0) {
-            elves[0] = recipes.head
-          } else {
-            elves[idx] = recipes.addRecipe(elf)
-          }
-        })
-
         loopRecipesForElves(elves, recipes, 15)
 
         let iterator = recipes.tail.next
@@ -73,139 +70,43 @@ describe('--- Day 14: Chocolate Charts ---', () => {
     })
     describe('calculateXAfterY(x, y, recipe, elves)', () => {
       it('predicts the next X results after the elves have executed Y', () => {
-        const elves = [3, 7]
-        const recipes = new Recipes(elves[0])
-        let actual = ''
-
-        elves.forEach((elf, idx) => {
-          if (idx === 0) {
-            elves[0] = recipes.head
-          } else {
-            elves[idx] = recipes.addRecipe(elf)
-          }
-        })
-
-        actual = calculateXAfterY(10, 9, recipes, elves)
+        let actual = calculateXAfterY(10, 9, recipes, elves)
         expect(actual).to.equal('5158916779')
       })
       it('predicts the next X results after the elves have executed Y', () => {
-        const elves = [3, 7]
-        const recipes = new Recipes(elves[0])
-        let actual = ''
-
-        elves.forEach((elf, idx) => {
-          if (idx === 0) {
-            elves[0] = recipes.head
-          } else {
-            elves[idx] = recipes.addRecipe(elf)
-          }
-        })
-
-        actual = calculateXAfterY(10, 5, recipes, elves)
+        const actual = calculateXAfterY(10, 5, recipes, elves)
         expect(actual).to.equal('0124515891')
       })
       it('predicts the next X results after the elves have executed Y', () => {
-        const elves = [3, 7]
-        const recipes = new Recipes(elves[0])
-        let actual = ''
-
-        elves.forEach((elf, idx) => {
-          if (idx === 0) {
-            elves[0] = recipes.head
-          } else {
-            elves[idx] = recipes.addRecipe(elf)
-          }
-        })
-
-        actual = calculateXAfterY(10, 18, recipes, elves)
+        const actual = calculateXAfterY(10, 18, recipes, elves)
         expect(actual).to.equal('9251071085')
       })
       it('predicts the next X results after the elves have executed Y', () => {
-        const elves = [3, 7]
-        const recipes = new Recipes(elves[0])
-        let actual = ''
-
-        elves.forEach((elf, idx) => {
-          if (idx === 0) {
-            elves[0] = recipes.head
-          } else {
-            elves[idx] = recipes.addRecipe(elf)
-          }
-        })
-
-        actual = calculateXAfterY(10, 2018, recipes, elves)
+        const actual = calculateXAfterY(10, 2018, recipes, elves)
         expect(actual).to.equal('5941429882')
       })
     })
     describe('findPattern()', () => {
       it('counts the number of recipes to the left of the specified pattern', () => {
-        const elves = [3, 7]
-        const recipes = new Recipes(elves[0])
-        let actual = ''
-
-        elves.forEach((elf, idx) => {
-          if (idx === 0) {
-            elves[0] = recipes.head
-          } else {
-            elves[idx] = recipes.addRecipe(elf)
-          }
-        })
-
-        actual = findPattern('51589', recipes, elves)
+        const actual = findPattern('51589', recipes, elves)
         expect(actual).to.equal(9)
       })
     })
     describe('findPattern()', () => {
       it('counts the number of recipes to the left of the specified pattern', () => {
-        const elves = [3, 7]
-        const recipes = new Recipes(elves[0])
-        let actual = ''
-
-        elves.forEach((elf, idx) => {
-          if (idx === 0) {
-            elves[0] = recipes.head
-          } else {
-            elves[idx] = recipes.addRecipe(elf)
-          }
-        })
-
-        actual = findPattern('01245', recipes, elves)
+        const actual = findPattern('01245', recipes, elves)
         expect(actual).to.equal(5)
       })
     })
     describe('findPattern()', () => {
       it('counts the number of recipes to the left of the specified pattern', () => {
-        const elves = [3, 7]
-        const recipes = new Recipes(elves[0])
-        let actual = ''
-
-        elves.forEach((elf, idx) => {
-          if (idx === 0) {
-            elves[0] = recipes.head
-          } else {
-            elves[idx] = recipes.addRecipe(elf)
-          }
-        })
-
-        actual = findPattern('92510', recipes, elves)
+        const actual = findPattern('92510', recipes, elves)
         expect(actual).to.equal(18)
       })
     })
     describe('findPattern()', () => {
       it('counts the number of recipes to the left of the specified pattern', () => {
-        const elves = [3, 7]
-        const recipes = new Recipes(elves[0])
-        let actual = ''
-
-        elves.forEach((elf, idx) => {
-          if (idx === 0) {
-            elves[0] = recipes.head
-          } else {
-            elves[idx] = recipes.addRecipe(elf)
-          }
-        })
-
-        actual = findPattern('59414', recipes, elves)
+        const actual = findPattern('59414', recipes, elves)
         expect(actual).to.equal(2018)
       })
     })
diff --git a/2018/day-14/solution.js b/2018/day-14/solution.js
@@ -7,15 +7,9 @@ const {
 const input = 540561
 
 let elves = [3, 7]
-let recipes = new Recipes(elves[0])
-
-elves.forEach((elf, idx) => {
-  if (idx === 0) {
-    elves[0] = recipes.head
-  } else {
-    elves[idx] = recipes.addRecipe(elf)
-  }
-})
+let recipes = new Recipes(elves)
+elves[0] = recipes.tail
+elves[1] = recipes.head
 
 const answer = calculateXAfterY(10, input, recipes, elves)
 
