Сравнение значений между двумя массивами

Кажется, ваша проблема связана с изменениями reverse() фактический массив. Так поступаем

var reverse = noSpace.reverse();

Обратит noSpace и назначит ссылку на него в переменной reverse. То есть оба массива будут одним и тем же (перевернутым) массивом.

Чтобы обойти это, я использовал .slice() для создания копии исходного массива, а затем вызвал .reverse() для этого нового массива , избавляя вас от любых конфликтов.

Вот рабочий фрагмент того, как это выглядит:

function palindrome(str) {
    var str_array = str.toLowerCase().split("");
    var no_space = str_array.filter(function(val) {
        return val !== " ";
    });

    // By applying '.slice()', we create a new array
    // reference which can then be reversed and assigned
    // to the 'reverse' variable
    var reverse = no_space.slice().reverse();

    function check(a, b) {
        var partial;
        var result = 1;
        for(var i=0; i < a.length; i++) {
            if(a[i] !== b[i]) {
                // We don't need to keep
                // comparing the two, it
                // already failed
                return 0;
            } else {
                // I've kept this part even though
                // I don't really know what it is
                // intended for
                partial = 1;
                result *= partial;
            }
        }
        return result;
    }
    return check(no_space, reverse) === 1;
}

console.log(palindrome("a b a"));
console.log(palindrome("r y e"));

То, как вы закодировали палиндром, слишком сложно.

Но с вашим кодом есть одна проблема: когда вы выполняете reverse(), он также изменяет исходный массив.

Поэтому вам нужно будет убедиться, что вы скопировали его через slice().

Также вы можете напрямую отправить логический результат, а не выполнять 1 и 0.

В result *= partial; 1 * 1 всегда будет равно 1

Я не исправлял ваш код, но вот вам оптимизированное решение.

function palindrom(string) {
    var arr = string.split("");

    var lengthToCheck = Math.floor(arr.length / 2);
    for (var i = 0; i < lengthToCheck; i++) {
        if (arr[i] != arr[arr.length - (1 + i)]) {
            return false;
        }
    }

    return true;
}

Сначала я разбиваю массив после каждого символа переданной строки. После этого я получаю половину длины массива, так как достаточно проверить только половину. С помощью цикла for я сравниваю первую половину со второй половиной. Как только я обнаружил два не совпадающих символа, я возвращаю false. В случае, если вся первая половина соответствует второй половине массива, цикл for будет завершен, и после этого будет возвращено значение true.

На самом деле происходит .reverse() реверсирование массива на месте, а затем сохранение ссылки на этот массив, который не является тем, что вы вызываете в своем check() методе.

Простое исправление - изменить оператор if:

if (a[i] !== b.reverse()[i])