Question

我正在编写一个简单的XOR加密程序，然后我注意到函数的返回值不是我希望看到的那种。

我在代码中找不到问题，有人可以帮助我吗？

这是程序代码：

#include "windows.h"
#include "iostream"

using namespace std;


LPCSTR hasala(string original, char key){

  string changed;

  for (int temp = 0; temp < original.size(); temp++){
  changed += original[temp] ^ (int(key) + temp) % 255;
  }

  cout << changed.c_str()<<"\n\n";//works, output "acagaca"

  LPCSTR adart = changed.c_str();

  cout << adart<<"\n\n";//works, output "acagaca"

  return adart;

}

int main(){

    cout << hasala("abcdefg", 0);//doesn't work, output "||||@ER|||"...

    cout << "\n\n";

    Sleep(8000);

    return 0;
}

Answer 1

你拥有的指针是堆栈上std::string拥有的内存 - 当std::string超出范围时，它会释放它，然后你就读它了。您必须返回std::string，而不是LPCSTR。

Answer 2

您正在返回changed.c_str()，它只是指向由changed管理的缓冲区的指针。 changed是一个局部变量，当函数结束时会被销毁，所以你最终会返回一个不再存在的指针。

直接返回std :: string changed。

Answer 3

您正在返回在堆栈上分配的本地变量。

基本上你的代码转换为：

LPCSTR hasala(string original, char key){
string changed;
return changed.c_str();
}

分配内存（调用者必须释放它）或返回一个std :: string

LPCSTR函数的返回值变得无用

3 个答案: