比較產品與一段時間和for循環

Question

我正在一個腳本，將找到任何兩個3位數字是迴文的所有產品，這裏是我正在努力確定一個數字是否是一個腳本回文並比較3位數字的產品 -

var isPalindrome = function(number) { 
var number = number.toString(); 
var length = number.length; 
var x = 0; 
var y = 1; 
while (x<=(length/2)) { 
    if(number[x]==number[length-y]) { 
     x++; 
     y++; 
    } else { 
     return false; 
    } 
    return true; 
} 
}; 

var counter = function() { 
var palindromeProducts = []; 
var x = i; 
var y = 100; 
var product = x*y; 
while (y<1000) { 
    for (i=100; i<1000; i++) { 
     if (isPalindrome(product)) { 
      palindromeProducts.push(x*y); 
     } 
    } 
    y++; 
} 
console.log(palindromeProducts); 
}; 

counter(); 

的isPalindrome功能似乎就回文不到17位長的工作，但它也將一些數字不是迴文，我可以返回true不知道爲什麼。

palinProduct函數的思想是在while循環中for循環將遍歷x的所有值並比較產品，然後while循環將增加y然後再次運行for循環，但它不會似乎沒有工作。

Answer 1

在isPalindrome功能，「返回true」語句應在while循環之外。這是由Quantastical指出的。

在計數器功能中，有幾個問題。我在初始化之前已分配給x。 x在循環之外被分配一次值，然後永遠不會改變。產品在循環之外被賦值一次，永遠不會改變。代碼可以通過使用兩個循環迭代x和y來簡化。

固定碼可能看起來像......

var isPalindrome = function(number) { 
    var number = number.toString(); 
    var length = number.length; 
    var x = 0; 
    var y = 1; 
    while (x<=(length/2)) { 
     if(number[x]==number[length-y]) { 
      x++; 
      y++; 
     } else { 
      return false; 
     } 
    } 
    return true; 
}; 

var counter = function() { 
    var palindromeProducts = []; 
    for (var x = 100; x < 1000; x++) { 
     for (var y = 100; y < 1000; y++) { 
      var product = x*y; 
      if (isPalindrome(product)) { 
       palindromeProducts.push(product); 
      } 
     } 
    } 
    console.log(palindromeProducts); 
}; 

Answer 2

您的誤報可能是由於您的isPalindrome函數中存在錯誤的while循環造成的。該功能在循環的第一次迭代返回true ...

while (x<=(length/2)) { 
    if(number[x]==number[length-y]) { 
     x++; 
     y++; 
    } else { 
     return false; 
    } 
    return true; // this is in the wrong place, I believe 
} 

我想你究竟想要做的是：

while (x<=(length/2)) { 
    if(number[x]==number[length-y]) { 
     x++; 
     y++; 
    } else { 
     return false; 
    } 
} 
return true;