你越界了

while (!isalpha(s[r]))

字符串
还有这里

while (!isalpha(s[l]))

型
r可以变为负值，l可以变为>= s.length()。
您应该添加一些检查，如

while (l < r && !isalpha(s[r]))

型
和/或

while (l < r && !isalpha(s[l]))

型
这一行同样的问题

if (tolower(s[r])!=tolower(s[l]))

型
这应该是

if (l < r && tolower(s[r])!=tolower(s[l]))

型

不同的方式（C++20）

一种不同的方法是使用以下命令从s中擦除所有非字母字符

std::erase_if(s, [](char c) { return !isalpha(c); });

型
并删除内部的while循环。

我认为你已经非常接近解决方案了。这里的陷阱是：

• 在循环中多次修改循环控制变量
• （因此）您在更改其值后使用循环控制变量，而无需进一步检查。

修复这类问题的简单方法是为每个迭代执行一个操作。你可以使用“else”来实现这一点。

class Solution {
public:
    bool isPalindrome(string s) {
        int l=0,r=s.length()-1;
        while(l<r)
        {
            if(!isalpha(s[r]))
            {
                --r;
            }
            else if(!isalpha(s[l]))
            {
                ++l;
            }
            else if (tolower(s[r])!=tolower(s[l]))
            {
                return false;
            }
            else
            {
                --r;
                ++l;
            }
        }
        return true;
    }
};

字符串

一个类似的修改方法可以解决这个问题，只要遇到任何非字母数字字符，就继续循环。这里：

bool isPalindrome(string s) {    
    int start=0,end=s.length()-1;
    
    while(start<=end){
        if(!isalpha(s[start])  && !isdigit(s[start])){
            start++;
            continue;
        }
        
        if(!isalpha(s[end]) && !isdigit(s[end])){
            end--;
            continue;
        }
        
        if(tolower(s[start])!=tolower(s[end])){
            return false;
        }
        
        start++;
        end--;
    }
    
    return true;

}

字符串

在某些情况下你会越界。所以你可以尝试下面的代码没有错误和内存泄漏。

bool isPalindrome(string s) {
  string a="";
  for(int i=0 ; i<s.length() ; i++)
  {
   if(s[i]>=65 && s[i]<=90)
        a+=(s[i]+32);
        else if(s[i]>=97 && s[i]<=122)
        a+=s[i];
        else if(s[i]>='0' && s[i]<='9')
        a+=s[i];
    }
    int start=0, end=a.length()-1;
    while(start<end)
    {
        if(a[start++]!=a[end--])
        return false;
    }
    return true;
}

字符串

正如其他人所说，你正在耗尽债券。下面是一个简化的解决方案，它不会耗尽债券，也没有if-then-else意大利面条代码：

bool IsPalindrome(const char *s)
{
    int forward, totLen, halfLen, backward;

    totLen = (int)strlen(s);    // Total length
    halfLen = totLen / 2;       // Half the length, if totLen is odd, will ignore the character in the middle

    for (forward = 0; forward < halfLen; forward++) // Index from front of string
    {
        backward = totLen - (forward + 1);          // Corresponding index from back of string
        if (s[forward] != s[backward])
            return false;                           // Not a palindrome
    }
    return true;    // Got one!
}

字符串