当它甚至不执行函数时超出了 C++ 时间限制

Question

当我在 LeetCode 中解决一个问题时，我发现了一些很奇怪的东西。

我有这条线，我认为它给了我一个超出时间限制的错误：

s.erase(i-k, k);

当我评论（//）这一行时，它没有显示时间超过错误，但奇怪的是，即使我没有评论它也从未执行过。

下面是整个代码。 Here 是问题链接。

class Solution {
public:
    string removeDuplicates(string s, int k) {
        char prev = s[0];
        int cnt = 1;
        cnt = 1;

        for(int i = 1; i < s.size() + 1; i++){
            if(s[i] == prev){
                cnt++;
            } else {
                if(cnt == k){
                    // when input is "abcd" it never comes to this scope
                    // which is impossible to run erase function. 
                    s.erase(i-k, k);

                    i = 0;
                }
                
                if(i >= s.size()) break;
                
                cnt = 1;
                prev = s[i];
            }
        }
        
        return s;
    }
};

当 Input 为 "abcd" 时，它甚至不会进入 'erase' 函数所在的 if 范围。 虽然 'erase' 函数从未运行过，但它仍然会影响时间复杂度，我不明白原因。

有人能解释一下吗？还是这只是 LeetCode 的问题？

Answer 1

当程序遇到严重错误（编码错误）和/或崩溃时，许多在线比赛服务器会报告 Time Exceeding。

例如读取超出数组范围的错误。或取消引用坏（垃圾）指针。

为什么超时。因为严重错误程序可能会挂起和/或崩溃。这意味着它也不会及时提供结果。

所以我认为你必须调试你的程序来发现所有的编码错误，而不是花时间优化算法。

关于这一行s.erase(i-k, k); - 它可能会在i < k 时崩溃/挂起，然后你有负值，这是.erase() 方法不允许的。例如，当您获得 i - k 等于 -1 时，size_t 类型（擦除的第一个参数的类型）将溢出（环绕）到值 18446744073709551615，这绝对超出范围，超出内存边界，因此您的程序可能会崩溃和/或挂起。当删除的字符太多时，擦除也会崩溃，即擦除s.erase(a, b)你必须注意a + b <= s.size()，它不受擦除功能的控制。

参见擦除方法的documentation，不要将负值作为该方法的参数。检查你的算法永远不会有负值，即在调用s.erase(i-k, k);时永远不会有i < k，也永远不会有i-k + k > s.size()。为确保没有程序崩溃，您可以执行以下操作：

int start = std::min(std::max(0, i-k), int(s.size()));
int num = std::min(k, std::max(0, int(s.size()) - start));
s.erase(start, num);