function palindrome(str) {
let regex = /\W/g;
str = str.toLowerCase().replace(regex, '');
function reverseString(str) {
let reverseString = "";
for (let i = str.length - 1; i >= 0; i--) {
reverseString += str[i];
}
return reverseString;
}
if (str === reverseString(str)) {
return true;
}
return false;
}
const result = palindrome("eye");
console.log(result)
但是，不需要该内部功能
function palindrome(str) {
const regex = /\W/g;
let reverseString = "";
str = str.toLowerCase().replace(regex, '');
for (let i = str.length - 1; i >= 0; i--) {
reverseString += str[i];
}
if (str === reverseString) {
return true;
}
return false;
}
const result = palindrome("eye");
console.log(result)
虽然我可能会像
function palindrome(str) {
str = str.toLowerCase().replace(/\W/g, '');
return str === str.split('').reverse().join('');
}
const result = palindrome("eye");
console.log(result);
here's the problems with your code
function palindrome(str) {{ remove the second {
you declare regex but never use it
/\W/g.remove; ... what does this do? a regex doesn't have a remove property
toLowerCase is a function, that returns a new value - what you do doesn't call the function, nor does it store the result of calling the function
remove this line
OK
don't make this a const because you can't change const's - also, naming a variable the same as the function it is in can cause confusion - as you'll see
this will cause an infinite loop, you need i-- to decrement i on each loop
OK
OK
OK
OK
You need to call the function - it's confusing because your variable name
OK
OK
OK
OK
This calls the function, but does nothing with the result
So, the resulting fixed code is:
function palindrome(str) {
let regex = /\W/g;
str = str.toLowerCase().replace(regex, '');
function reverseString(str) {
let reverseString = "";
for (let i = str.length - 1; i >= 0; i--) {
reverseString += str[i];
}
return reverseString;
}
if (str === reverseString(str)) {
return true;
}
return false;
}
const result = palindrome("eye");
console.log(result)
But, no need for that inner function
function palindrome(str) {
const regex = /\W/g;
let reverseString = "";
str = str.toLowerCase().replace(regex, '');
for (let i = str.length - 1; i >= 0; i--) {
reverseString += str[i];
}
if (str === reverseString) {
return true;
}
return false;
}
const result = palindrome("eye");
console.log(result)
Though I'd probably write it like
function palindrome(str) {
str = str.toLowerCase().replace(/\W/g, '');
return str === str.split('').reverse().join('');
}
const result = palindrome("eye");
console.log(result);
发布评论
评论（2）
这是您的代码
函数palindrome（str）{{删除第二个{REGEX，但切勿使用它/\ w/g.remove;...这有什么作用？ REGEX没有删除属性tolowercase是一个函数，它返回一个新值 - 您所做的不称呼该函数，也不会存储调用该函数的结果const，因为您不能更改const的const - 另外，命名变量与所使用的函数相同可能会引起混乱 - 因为您会看到i -循环是：
但是，不需要该内部功能
虽然我可能会像
here's the problems with your code
function palindrome(str) {{remove the second{regexbut never use it/\W/g.remove;... what does this do? a regex doesn't have a remove propertytoLowerCaseis a function, that returns a new value - what you do doesn't call the function, nor does it store the result of calling the functionconstbecause you can't change const's - also, naming a variable the same as the function it is in can cause confusion - as you'll seei--to decremention each loopSo, the resulting fixed code is:
But, no need for that inner function
Though I'd probably write it like
您可以执行以下操作：
You can do the following: