refactor: change rotateAndMultiply to higher order fn

Author: TheAlexLichter

src/affine.js

@@ -16,11 +16,12 @@ const transform = (input, options) => {
   if (!LEGAL_MULT_KEYS.includes(options.keys[0])) {
     throw new Error('Illegal keys')
   }
+  const rotateAndMultiplyWithOptions = rotateAndMultiply(getConfig(options))
 
-  return [...input].map(c => rotateAndMultiply(c, getRotateAndMultiplyConfig(options))).join('')
+  return [...input].map(rotateAndMultiplyWithOptions).join('')
 }
 
-const getRotateAndMultiplyConfig = options => ({
+const getConfig = options => ({
   types: [ROTATE_AND_MULTIPLY_TYPES.LOWERCASE, ROTATE_AND_MULTIPLY_TYPES.UPPERCASE],
   keys: options.keys,
   failOnUnknownCharacter: options.failOnUnknownCharacter,

src/helpers/rotateAndMultiply.js

@@ -1,6 +1,6 @@
 import { modInverse, throwOrSilent } from './index'
 
-export const rotateAndMultiply = (c, options) => {
+export const rotateAndMultiply = options => c => {
   options = { ...DEFAULT_ROTATE_AND_MULTIPLY_OPTIONS, ...options }
   const type = getType(c)
 

src/multiplicative.js

@@ -16,8 +16,9 @@ const transform = (input, options) => {
   if (!LEGAL_KEYS.includes(options.key)) {
     throw new Error('Illegal key')
   }
+  const rotateAndMultiplyWithOptions = rotateAndMultiply(getConfig(options))
 
-  return [...input].map(c => rotateAndMultiply(c, getRotateAndMultiplyConfig(options))).join('')
+  return [...input].map(rotateAndMultiplyWithOptions).join('')
 }
 
 const DEFAULT_OPTIONS = {
@@ -26,7 +27,7 @@ const DEFAULT_OPTIONS = {
   omitUnknownCharacter: false
 }
 
-const getRotateAndMultiplyConfig = options => ({
+const getConfig = options => ({
   types: [ROTATE_AND_MULTIPLY_TYPES.LOWERCASE, ROTATE_AND_MULTIPLY_TYPES.UPPERCASE],
   keys: [options.key, 0],
   failOnUnknownCharacter: options.failOnUnknownCharacter,

src/rot.js

@@ -17,7 +17,8 @@ import { ROTATE_AND_MULTIPLY_TYPES, rotateAndMultiply } from './helpers/rotateAn
 
 export default (input, options = {}) => {
   options = { ...DEFAULT_OPTIONS, ...options }
-  return [...input].map(c => rotateAndMultiply(c, getConfig(options))).join('')
+  const rotateAndMultiplyWithOptions = rotateAndMultiply(getConfig(options))
+  return [...input].map(rotateAndMultiplyWithOptions).join('')
 }
 
 const getConfig = options => ({