运行 g++ 时出错应用程序。 （字符串加密）

Question

我正在尝试使用 C++ 加密和解密文件，使用以下代码：

#include <iostream>

void crypt(char* pData, unsigned int lenData, const char* pKey, unsigned int lenKey)
{
    for (unsigned int i = 0; i < lenData; i++)
        pData[i] = pData[i] ^ pKey[i % lenKey];
}

int main()
{
    char* data = (char*)"any binary string here";
    crypt(data, 22, "key", 3);
    std::cout << data;
}

我正在 Windows 6.1 (7) 上使用 g++ (tdm-1) 4.5.1 (MinGW) 进行编译，它使用没有错误或警告。当我尝试运行时，它会显示一个窗口，其中显示“app.exe 停止工作。Windows 可以在线检查是否有解决该问题的方法。” （类似这样，我的 Windows 不是英文的）。我不知道出了什么问题。

Answer 1

您正在尝试修改字符串常量。由于明显的原因（它是恒定的），这是行不通的。相反，请执行以下操作：

int main()
{
    char data[] = "any binary string here";
    crypt(data, 22, "key", 3);
    std::cout << data;
}

Answer 2

这句话是错误的：

char* data = (char*)"any binary string here";

首先，你不应该使用强制转换。接下来，字符串文字是一个常量。所以应该是：

const char* data = "any binary string here";

但是你想覆盖它。所以你需要一个不是常量的字符串。像这样：

char data[] = "any binary string here";

Answer 3

迈克很好地回答了这个问题。您不能修改常量字符串文字。 DOS时代已经快结束了。正确的最新生产级别 C++ 编译器应该发出带有适当标志的警告。只是为了在迈克的答案中添加一点内容，这里是常量字符串文字的一个很好的解释 - http://msdn.microsoft.com/en-us/library/69ze775t(v=vs.80).aspx

另外，这是更好的方法：

#include <iostream>

void crypt(char* pData, unsigned int lenData, const char* pKey, unsigned int lenKey)
{
    for (unsigned int i = 0; i < lenData; ++i)
        pData[i] ^= pKey[i % lenKey];
}

int main()
{
    char data[] = "any binary string here";
    const char key[] = "key";
    crypt (data, sizeof(data) - 1, key, sizeof (key) - 1);
    std::cout << data << std::endl;
}

注意后自增运算符、^= 和 sizeof 运算符。对于简单类型，编译器会为你做这种微优化，但养成一个好习惯是件好事。如果您有一个复杂的迭代器，那么使用后增量可能会损害性能关键路径。此外，硬编码字符串的大小很容易出错。稍后您或其他人可能会更改该字符串，但忘记更改其长度。更不用说每次你都得去数字符数。

快乐编码！