make only 1 wc call

1 files changed, 8 insertions, 6 deletions

diff --git a/entries/perryliao/fib.groovy b/entries/perryliao/fib.groovy
index af1c4df..168913f 100644
--- a/entries/perryliao/fib.groovy
+++ b/entries/perryliao/fib.groovy
@@ -9,8 +9,8 @@ pipeline {
         stage('Create Cache') {
             steps {
                 script {
-                    if (!fileExists("fibbonacciCache")) {
-                        sh 'echo "0\n1" > fibbonacciCache'
+                    if (!fileExists("fibCache")) {
+                        sh 'echo "0\n1" > fibCache'
                     }
                 }
             }
@@ -25,6 +25,7 @@ pipeline {
         stage('Get Fibbonacci') {
             steps {
                 script {
+                    nComputed = sh(returnStdout: true, script: "wc -l < fibCache").trim().toInteger()
                     result = fib(params.num.toInteger())
                     println("The result is F(${params.num}) = ${result}")
                 }
@@ -33,16 +34,17 @@ pipeline {
     }
 }
 
-int fib(int n, cache = 'fibbonacciCache') {
-    nComputed = sh(returnStdout: true, script: "wc -l < ${cache}").trim().toInteger()
+def nComputed
+
+int fib(int n, cache = 'fibCache') {
     if (n >= nComputed) {
         // Need to do more recursion
         result = fib(n - 2, cache) + fib(n - 1, cache)
+        nComputed++
         sh "echo '${result}' >> ${cache}"
         return result
     } else {
         // Already have what we need, so read from cache
-        return sh(returnStdout: true, script: "sed -n '${n + 1}p' ${cache}").trim().toInteger()
+        return sh(returnStdout: true, script: "sed -n '${n + 1}p' ${cache}").toInteger()
     }
 }
-