Merge pull request algorithm-visualizer#145 from archie94/ceasar-cipher-string-with-space

Ceasar cipher fix-Encrypt string with spaces

Author: nem035

algorithm/cryptography/caesar_cipher/basic/code.js

@@ -23,17 +23,20 @@ function cipher(str, rotation, direction, cipherTracer) {
     cipherTracer._wait();
 
     var currChar = str.charAt(i);
-    var r = rotation;
-
-    logger._print('Rotating ' + currChar + ' ' + direction + ' ' + rotation + ' times');
-    cipherTracer._select(i)._wait();
-
-    // perform given amount of rotations in the given direction
-    while (r-- > 0) {
-      currChar = getNextChar(currChar, direction);
-      cipherTracer._notify(i, currChar)._wait();
+    if (typeof alphabetMap[currChar] === 'number') { // don't encrpt/decrypt characters not in  alphabetMap
+      var r = rotation;
+
+      logger._print('Rotating ' + currChar + ' ' + direction + ' ' + rotation + ' times');
+      cipherTracer._select(i)._wait();
+
+      // perform given amount of rotations in the given direction
+      while (r-- > 0) {
+        currChar = getNextChar(currChar, direction);
+        cipherTracer._notify(i, currChar)._wait();
+      }
+    } else {
+      logger._print('Ignore this character');
     }
-
     str = str.substring(0, i) + currChar + str.substring(i + 1);
     logger._print('Current result: ' + str);
   }

algorithm/cryptography/caesar_cipher/basic/data.js

@@ -1,4 +1,4 @@
-var string = 'secret';
+var string = 'hello! how are you doing?';
 var rotation = 5;
 var alphabet = 'abcdefghijklmnopqrstuvwxyz';
 // create a map of char -> position to improve run time