通过多个函数运行一个值（Javascript）

Question

我想创建一个函数“palindromes()”来检查一个值是否是回文（向前和向后拼写相同）。 为此，我创建了 4 个函数，其中：

1. 使所有字母变小
2. 删除所有非字母字符
3. 反转随后的数组，最后...
4. 检查该数组是否为回文。

查看以下函数：

  function makeSmall(input) {
    lowerCase = input.toLowerCase();
    return lowerCase;
  }

  function keepOnlyLetters(input) {
    var patt1 = /[a-z]/g;
    var onlyLetters = input.match(patt1);
    return onlyLetters;
  }

  function reverseArray(array) {
    var reversedArray = array.slice().reverse();
    return reversedArray;
  }

  function checkPalindromes(array) {
    var reversedArray = array.slice().reverse();
    for (let i = 0; i <= array.length; i++) {
      if (array[i] != reversedArray[i]) {
        return false;
      }
    }
    return true;
  }

如何确保函数“palindromes()”采用一个值并在所有这些函数中运行它以最终给出该值是否为回文的答案（真或假）？

最好的问候， 贝尼

Answer 1

函数有一个收益递减点。当调用函数与使用内联函数的主体一样短时，您可能已经达到了这一点。例如，makeSmall(input) 与仅使用 input.toLowerCase() inline 相比并没有任何改进，而且会更慢且更难理解。 input.toLowerCase() 已经是一个函数；将它包装在另一个函数中只是浪费工作。

话虽如此，回答你的问题，因为你所有的函数都返回输入到下一个的值，你可以把你的函数放在一个数组中并调用reduce()：

function palindromes(input) {
    return [makeSmall, keepOnlyLetters, reverseArray, checkPalindromes].reduce((a, c) => c(a), input)
}

Answer 2

因此，在开始尝试进行组合之前，有时最好按顺序进行，以确保您理解问题。随着你在作曲上的进步，你最终会知道使用什么工具。

function checkPalindrome(string){
 return string
  .toLowerCase()
  .match(/[a-z]/g)
  .reverse()
  .reduce(function ( acc, letter, index ) {
    return acc && string[index] == letter
  })
}

checkPalindrome('test') // false
checkPalindrome('tet') // true

好的，我们从程序上理解它并且知道有四个步骤。我们可以将这四个步骤拆分出来，但是由于两个步骤需要事先了解数组状态，并且我们不想引入converge 或lift，我们应该只使用pipe function 并结合以下步骤需要以前的状态。这样做的原因是函数最终会失去你可以使它们变得更小的多少，并且尝试将这些步骤分开不仅会损害可读性，还会损害可维护性。对于为该部分制作两个功能所付出的努力，这些并不是很好的回报！

function pipe (...fns){
  return fns.reduce( function (f, g){
    return function (...args){
      return g(
        f(...args)
      )
    }
  }
}

所有这个函数都是预先加载（组合）一堆函数以使一个函数的输出以left to right顺序（也称为数组顺序）应用于下一个函数的输入)。

现在我们只需要输出三个函数来管道：

function bringDown(string){ return string.toLowerCase() } // ussually called toLower, see note

function onlyLetters(string){ return string.match(/[a-z]/g) }

function flipItAndReverseItCompare(arrayLike){  // I like missy elliot... ok?
  let original = Array.from(arrayLike)
  return original
    .slice()
    .reverse()
    .reduce(function (acc, val, ind){
      return acc && val == original[ind]
    })
}

现在我们可以通过管道处理它们

let palindrome = pipe(
  bringDown,
  onlyLetters,
  flipItAndReverseItCompare
)

!palindrome('Missy Elliot') // true... and I never will be
palindrome('Te t') // true

现在您已经开始学习函数组合了！

Answer 3

您可以像这样将函数调用串在一起......

var input = 'Racecar';

if (checkPalindromes(reverseArray(keepOnlyLetters(makeSmall(input))))) {
   alert("It's a palindrome");
}

Answer 4

您可以以嵌套方式调用它们并在回文函数中返回最终结果。

示例代码：（在 cmets 中有更改）

function makeSmall(input) {
    // Added var to prevent it from being a global
    var lowerCase = input.toLowerCase();
    return lowerCase;
}

function keepOnlyLetters(input) {
    var patt1 = /[a-z]/g;
    var onlyLetters = input.match(patt1);
    return onlyLetters;
}

// This function is not really needed and is unused
/*function reverseArray(array) {
    var reversedArray = array.slice().reverse();
    return reversedArray;
}*/

function checkPalindromes(array) {
    var reversedArray = array.slice().reverse();
    for (let i = 0; i <= array.length; i++) {
        if (array[i] != reversedArray[i]) {
            return false;
        }
    }
    return true;
}

// New Palindromes function
function palindromes(input){
    return checkPalindromes(keepOnlyLetters(makeSmall(input)));
}

注意：

你真的不需要这么多的功能来做到这一点。我把它放在这里作为对你确切问题的严格回答。此处的其他答案显示了如何以更短（更好？）的方式解决这个问题

Answer 5

尝试以下 sn-p。

    function makeSmall(input) {
        lowerCase = input.toLowerCase();
        return lowerCase;
    }

    function keepOnlyLetters(input) {
        var patt1 = /[a-z]/g;
        var onlyLetters = input.match(patt1);
        return onlyLetters;
    }

    function reverseArray(array) {
        var reversedArray = array.slice().reverse();
        return reversedArray;
    }

    function checkPalindromes(array) {
        var reversedArray = array.slice().reverse();
        for (let i = 0; i <= array.length; i++) {
          if (array[i] != reversedArray[i]) {
            return false;
          }
        }
        return true;
    }
    
    var result = checkPalindromes(reverseArray(keepOnlyLetters(makeSmall("Eva, Can I Stab Bats In A Cave"))));
    
    console.log(result);

注意函数是如何在一行中一个接一个地调用的。