Finish the extraction that I was supposed to do in dbad7a.

Previous commit left the pc variable being used in Run method. This commit fixes that so that I have a commit to point to for blog.
michaelgwelch · Mar 22, 2012 · 8f6ee03 · 8f6ee03
1 parent dbad7a8
commit 8f6ee03
Showing 1 changed file with 10 additions and 9 deletions.
diff --git a/csharp/BrainmessShort/Main.cs b/csharp/BrainmessShort/Main.cs
@@ -23,25 +23,26 @@ public static void Main(string[] args)
 
         char Fetch()
         {
-            return program[pc];
+            var instruction = program[pc];
+            pc++;
+            return instruction;
         }
 
-        int JumpForward()
+        void JumpForward()
         {
-            return program.FindMatch(pc - 1) + 1;
+            pc = program.FindMatch(pc - 1) + 1;
         }
 
-        int JumpBackward()
+        void JumpBackward()
         {
-            return program.FindMatch(pc - 1);
+            pc = program.FindMatch(pc - 1);
         }
 
         public void Run() 
         {
             while(pc < program.Length)
             {
-                char instruction = Fetch ();
+                char instruction = Fetch();
-                pc++;
                 switch(instruction)
                 {
                 case '>': 
@@ -65,13 +66,13 @@ public void Run()
                 case '[':
                     if (tape[tc] == 0)
                     {
-                        pc = JumpForward ();
+                        JumpForward();
                     }
                         break;
                 case ']':
                     if (tape[tc] != 0)
                     {
-                        pc = JumpBackward ();
+                        JumpBackward();
                     }
                     break;
                 }