我的C字符串復制功能正在破壞其他變量的堆棧

Question

該函數在另一個程序中可以正常工作，但是在某些情況下，它會導致運行時錯誤，表明某個變量（有時甚至沒有發送到函數中）周圍的堆棧已損壞。

char *strCpy(char *dest, const char *source) {      // copy cstring
    char *origin = dest;                            // non iterated address
    while (*source || *dest) {
        *dest = *source;
        dest++;
        source++;
    }
    return origin;
}

功能的實現：

int main() {
    std::cout << "Will it crash?\n";
    char temp[255];
    char b[255];
    std::cin >> temp;
    strCpy(b, temp);
    std::cout << b;
    std::cout << "end\n";
    return 0;
}

在這種情況下，temp已損壞，但不一定是因為它已傳遞到函數中。 除此以外，我很難找到問題所在。

我編寫自己的復制功能的原因是由於對該項目的限制。 我也不允許使用[]索引數組

Answer 1

您在循環條件下的邏輯有缺陷，並會導致不確定的行為 。

目的地的內容尚未初始化 ，因此不確定 。 您不應該在所有情況下僅檢查源：

while (*source) { ... }

當然，您需要將終止符添加到目標。 這可以簡單地在循環之后完成，例如

*dest = '\0';