Add Hill Cipher.

README.md

@@ -141,6 +141,7 @@ a set of rules that precisely define a sequence of operations.
 * **Cryptography**
   * `B` [Polynomial Hash](src/algorithms/cryptography/polynomial-hash) - rolling hash function based on polynomial
   * `B` [Caesar Cipher](src/algorithms/cryptography/caesar-cipher) - simple substitution cipher
+  * `B` [Hill Cipher](src/algorithms/cryptography/hill-cipher) - polygraphic substitution cipher
 * **Machine Learning**
   * `B` [NanoNeuron](https://github.com/trekhleb/nano-neuron) - 7 simple JS functions that illustrate how machines can actually learn (forward/backward propagation)
   * `B` [k-NN](src/algorithms/ml/knn) - k-nearest neighbors classification algorithm

src/algorithms/cryptography/hill-cipher/_test_/hillCipher.test.js

@@ -1,13 +1,10 @@
-import hillCipherEncrypt from '../hillCipher';
+import { hillCipherEncrypt, hillCipherDecrypt } from '../hillCipher';
 
 describe('hillCipher', () => {
-  it('should throw an error when the length of the keyString does not equal to the power of length of the message ', () => {
+  it('should throw an exception when trying to decipher', () => {
-    const invalidLenghOfkeyString = () => {
+    expect(hillCipherDecrypt).toThrowError('This method is not implemented yet');
-      hillCipherEncrypt('hello', 'helloworld');
-    };
-
-    expect(invalidLenghOfkeyString).toThrowError();
   });
+
   it('should throw an error when message or keyString contains none letter character', () => {
     const invalidCharacterInMessage = () => {
       hillCipherEncrypt('hell3', 'helloworld');
@@ -15,11 +12,35 @@ describe('hillCipher', () => {
     const invalidCharacterInKeyString = () => {
       hillCipherEncrypt('hello', 'hel12world');
     };
-    expect(invalidCharacterInMessage).toThrowError();
+    expect(invalidCharacterInMessage).toThrowError(
-    expect(invalidCharacterInKeyString).toThrowError();
+      'The message and key string can only contain letters',
+    );
+    expect(invalidCharacterInKeyString).toThrowError(
+      'The message and key string can only contain letters',
+    );
   });
+
+  it('should throw an error when the length of the keyString has a square root which is not integer', () => {
+    const invalidLengthOfKeyString = () => {
+      hillCipherEncrypt('ab', 'ab');
+    };
+    expect(invalidLengthOfKeyString).toThrowError(
+      'Invalid key string length. The square root of the key string must be an integer',
+    );
+  });
+
+  it('should throw an error when the length of the keyString does not equal to the power of length of the message', () => {
+    const invalidLengthOfKeyString = () => {
+      hillCipherEncrypt('ab', 'aaabbbccc');
+    };
+    expect(invalidLengthOfKeyString).toThrowError(
+      'Invalid key string length. The key length must be a square of message length',
+    );
+  });
+
   it('should encrypt passed message using Hill Cipher', () => {
     expect(hillCipherEncrypt('ACT', 'GYBNQKURP')).toBe('POH');
+    expect(hillCipherEncrypt('CAT', 'GYBNQKURP')).toBe('FIN');
     expect(hillCipherEncrypt('GFG', 'HILLMAGIC')).toBe('SWK');
   });
 });

src/algorithms/cryptography/hill-cipher/hillCipher.js

@@ -1,18 +1,28 @@
+// The code of an 'A' character (equals to 65).
+const alphabetCodeShift = 'A'.codePointAt(0);
+const englishAlphabetSize = 26;
 
 /**
- * generate key matrix from given keyString
+ * Generates key matrix from given keyString.
  *
- * @param {integer} length
+ * @param {string} keyString - a string to build a key matrix (must be of matrixSize^2 length).
- * @param {string} keyString
+ * @return {number[][]} keyMatrix
- * @return {Array[][]} keyMatrix
  */
-const generateKeyMatrix = (length, keyString) => {
+const generateKeyMatrix = (keyString) => {
+  const matrixSize = Math.sqrt(keyString.length);
+  if (!Number.isInteger(matrixSize)) {
+    throw new Error(
+      'Invalid key string length. The square root of the key string must be an integer',
+    );
+  }
   const keyMatrix = [];
   let keyStringIndex = 0;
-  for (let i = 0; i < length; i += 1) {
+  for (let i = 0; i < matrixSize; i += 1) {
     const keyMatrixRow = [];
-    for (let j = 0; j < length; j += 1) {
+    for (let j = 0; j < matrixSize; j += 1) {
-      keyMatrixRow.push((keyString.codePointAt(keyStringIndex)) % 65);
+      // A → 0, B → 1, ..., a → 32, b → 33, ...
+      const charCodeShifted = (keyString.codePointAt(keyStringIndex)) % alphabetCodeShift;
+      keyMatrixRow.push(charCodeShifted);
       keyStringIndex += 1;
     }
     keyMatrix.push(keyMatrixRow);
@@ -21,48 +31,54 @@ const generateKeyMatrix = (length, keyString) => {
 };
 
 /**
- * generate message vector from given message
+ * Generates a message vector from a given message.
  *
- * @param {*} message
+ * @param {string} message - the message to encrypt.
- * @return {Array} messageVector
+ * @return {number[]} messageVector
  */
 const generateMessageVector = (message) => {
   const messageVector = [];
   for (let i = 0; i < message.length; i += 1) {
-    messageVector.push(message.codePointAt(i) % 65);
+    messageVector.push(message.codePointAt(i) % alphabetCodeShift);
   }
   return messageVector;
 };
 
 /**
- * validate data and encrypt message from given message and keyString
+ * Encrypts the given message using Hill Cipher.
  *
  * @param {string} message plaintext
  * @param {string} keyString
  * @return {string} cipherString
- *
  */
+export function hillCipherEncrypt(message, keyString) {
+  // The keyString and message can only contain letters.
+  const onlyLettersRegExp = /^[a-zA-Z]+$/;
+  if (!onlyLettersRegExp.test(message) || !onlyLettersRegExp.test(keyString)) {
+    throw new Error('The message and key string can only contain letters');
+  }
+
+  const keyMatrix = generateKeyMatrix(keyString);
 
-export default function hillCipherEncrypt(message, keyString) {
-  const length = keyString.length ** (0.5);
   // keyString.length must equal to square of message.length
-  if (!Number.isInteger(length) && length !== message.length) {
+  if (keyMatrix.length !== message.length) {
-    throw new Error('invalid key string length');
+    throw new Error('Invalid key string length. The key length must be a square of message length');
-  }
-  // keyString and messange can only contain letters
-  if (!(/^[a-zA-Z]+$/.test(message)) || !(/^[A-Za-z]+$/.test(keyString))) {
-    throw new Error('messange and key string can only contain letters');
   }
 
-  const keyMatrix = generateKeyMatrix(length, keyString);
   const messageVector = generateMessageVector(message);
-  let ciperString = '';
+  let cipherString = '';
-  for (let row = 0; row < length; row += 1) {
+  for (let row = 0; row < keyMatrix.length; row += 1) {
     let item = 0;
-    for (let column = 0; column < length; column += 1) {
+    for (let column = 0; column < keyMatrix.length; column += 1) {
       item += keyMatrix[row][column] * messageVector[column];
     }
-    ciperString += String.fromCharCode((item % 26) + 65);
+    cipherString += String.fromCharCode((item % englishAlphabetSize) + alphabetCodeShift);
   }
-  return ciperString;
+
+  return cipherString;
 }
+
+// @TODO: Implement this method.
+export const hillCipherDecrypt = () => {
+  throw new Error('This method is not implemented yet');
+};