fix(2018 day-12): 0th generation needs padding to match the rules requriements

developher-net · developher-net · commit bc721decae63 · 2018-12-22T12:53:50.000-08:00
diff --git a/2018/day-12/plants.js b/2018/day-12/plants.js
@@ -3,13 +3,13 @@ class Plants {
     this.generations = []
     this.boundaryBuffers = [0, 0]
     this.rules = {}
-    this._setInitialGeneration(initial)
     if (rules && rules.length > 0) {
       rules.forEach((rule) => {
         this.addRule(rule)
       })
     }
     this.boundaryBuffers = this.getBoundaryBuffers()
+    this._setInitialGeneration(initial)
   }
 
   _setInitialGeneration (initial) {
@@ -21,6 +21,7 @@ class Plants {
         }
       })
     )
+    this.generations[0] = this.pad(this.generations[0])
   }
 
   /**
@@ -51,7 +52,7 @@ class Plants {
     const prevGen = this.generations[this.generations.length - 1]
 
     // Loop through all pots in the last generation to create a new generation
-    const nextGen = prevGen.map((pot) => {
+    let nextGen = prevGen.map((pot) => {
       // Assemble pattern for the given pot
       let pattern = ''
       for (let x = pot.position - 2; x <= pot.position + 2; x++) {
@@ -66,15 +67,7 @@ class Plants {
       }
     })
 
-    // Padd the list to support future generation
-    for (let x = 1; x <= this.boundaryBuffers[0]; x++) {
-      const first = nextGen[0].position
-      nextGen.splice(0, 0, { position: first - 1, state: '.' })
-    }
-    for (let x = 1; x <= this.boundaryBuffers[1]; x++) {
-      const last = nextGen[nextGen.length - 1].position
-      nextGen.push({ position: last + 1, state: '.' })
-    }
+    nextGen = this.pad(nextGen)
 
     // Store the new generation
     this.generations.push(nextGen)
@@ -180,6 +173,23 @@ class Plants {
     return this.generations[generation].filter((p) => p.state === '#')
       .reduce((pacc, p) => pacc + p.position, 0)
   }
+
+  /**
+   * Pads the generation to the left and right so the pots are present to support future generations
+   * @param {Array} generation List of pots representing a generation
+   * @returns generation list with extra pots
+   */
+  pad (generation) {
+    for (let x = 1; x <= this.boundaryBuffers[0]; x++) {
+      const first = generation[0].position
+      generation.splice(0, 0, { position: first - 1, state: '.' })
+    }
+    for (let x = 1; x <= this.boundaryBuffers[1]; x++) {
+      const last = generation[generation.length - 1].position
+      generation.push({ position: last + 1, state: '.' })
+    }
+    return generation
+  }
 }
 
 module.exports = {
diff --git a/2018/day-12/solution.js b/2018/day-12/solution.js
@@ -11,12 +11,13 @@ const initialState = '#...#...##..####..##.####.#...#...#.#.#.#......##....#....
 const init = (data) => {
   const rules = data
   const plantTracker = new Plants(initialState, rules)
-  for (let gen = 1; gen <= 20; gen++) {
+  const generations = 20
+  for (let gen = 1; gen <= generations; gen++) {
     plantTracker.advance()
   }
-  console.log('Generating 20 generations from the input looks like this:')
+  console.log(`Generating ${generations} generations from the input looks like this:`)
   plantTracker.display()
-  const answer = plantTracker.getCheckSum(20)
+  const answer = plantTracker.getCheckSum(generations)
   const answer2 = ''
 
   console.log(`-- Part 1 --`)
