c++ - 变量赋值覆盖其他变量的值

Question

我一直在努力学习一些有关 C++ 套接字编程的知识，因此一直在开发一个基本的 IRC 机器人。我已经连接并正常工作，但我在确定谁加入了 channel 时遇到了问题(即 JOIN 命令是针对机器人本身、随机用户还是我)。

我在 Windows 7 上使用 CLion。

我有以下代码:

//determine if message is JOIN command
if(regex_search(line, match, std::regex(R"(:([^!]*)!.*?JOIN)")))
{
    //get the nickname of the bot (details.getNickName() shown in next code segment)
    const char *trueNick = details.getNickName();
    //reformat nickname because the following is returned from the method alone
    //0x10042d0f9 <_ZStL6ignore+119> "CPlusPlusBotTest"
    const char *nick = string(details.getNickName()).c_str();
    //get the name of the user who joined
    const char *name = match.str(1).c_str();
    //debugging
    std::cout << name << " - name\n";
    std::cout << nick << " - nick\n";
    //might not be the correct way to compare the two? but I'll sort that out later
    if(name != nick)
    {
        //welcome the user
        char param[1024];
        sprintf(param, "%s :Hello, %s", details.getChannel(), name);
        sendData("PRIVMSG", param);
    }
}

我不确定为什么我从我的 getter 中得到多余的“东西”(我不知道它是什么)，因为这只是返回私有(private)变量的情况:

const char* BotDetails::getNickName() { return nickName; }

无论如何，这不是我的问题，因为我可以摆脱它(尽管它可能相当老套)。

我的问题是，当我连接到 channel 进行测试时，我在分配 trueNick 的行上设置了一个断点，这样我就可以看到在我单步执行程序时发生了什么，发生了以下情况:

1) trueNick 赋值:0x10042d0f9 <_ZStL6ignore+119> "CPlusPlusBotTest"

2) nick 被赋值:“CPlusPlusBotTest”

3) name 被赋值:"Seanharrs"并且 nick 被赋值:"Seanharrs"

这意味着当我的调试语句运行时，nick 和 name 是相同的值。我不确定为什么 nick 被重新分配为与 name 相同的值，这不应该发生。它也每次都会发生，不仅仅是为了我的名字。我尝试改用 char 数组和字符串，但无济于事。我也觉得奇怪的是 trueNick 从来没有受到影响，它只是这两个变量。感谢任何帮助(即使这只是检查此问题的替代/更好方法而不是修复方法，因为它很可能只是我这边没有其他人经历过的奇怪现象)。

Answer 1

这一行会导致未定义的行为:

const char *nick = string(details.getNickName()).c_str();

它将构造一个临时字符串对象并返回一个指向数据的指针。但是，临时意味着它会立即被破坏并且指针将无效。

EDIT:

It turned out that OP misunderstood the additional information displayed by the debugger and interpreted it as the value of the variable. After clarifying that, there is no need for the "conversion" which causes undefined behavior and the code can just be:

const char *nick = details.getNickName();
const char *name = match.str(1).c_str();
if( strcmp(name, nick) == 0 )
{
    //....
}

The example of undefined behavior is still shown below.

未定义行为的示例代码

考虑这段代码:

#include <iostream>
using namespace std;

int main() {
    const char t[] = "0x10042d0f9 <_ZStL6ignore+119> \"CPlusPlusBotTest\"";
    const char* pTrue = t;
    const char* p = std::string(t).c_str();
    std::cout << pTrue << std::endl;
    std::cout << p << std::endl;
    return 0;
}

它将输出(而不是:它可能输出):

0x10042d0f9 <_ZStL6ignore+119> "CPlusPlusBotTest"
0x10042d0f9 <_ZStL6ignore+119> "CPlusPlusBotTest"

(ideone.com 用于此示例并给出了以上输出)

因此，您可能会认为这没问题。 (但请注意:我没有得到 OP 提到的转换)。

现在考虑这段代码:

#include <iostream>
using namespace std;

int main() {
    const char tdemo[] = "Some demo text";   // Added this line
    const char t[] = "0x10042d0f9 <_ZStL6ignore+119> \"CPlusPlusBotTest\"";
    const char* pTrue = t;
    const char* p = std::string(t).c_str();
    const char* pdemo = std::string(tdemo).c_str();  // Added this line
    std::cout << pTrue << std::endl;
    std::cout << p << std::endl;
    return 0;
} 

它会输出(而不是:它可能会输出)

0x10042d0f9 <_ZStL6ignore+119> "CPlusPlusBotTest"
Some demo text

(ideone.com 用于此示例并给出了以上输出)

如您所见，*p 的值“意外”更改。它发生变化是因为指针在指向已经释放的内存的意义上是无效的。额外的行

const char* pdemo = std::string(tdemo).c_str();  

导致编译器重用该内存，因此值 *p 发生了变化。

换句话说 - 你有未定义的行为。

我猜你的问题出在details.getNickName();

在我看来，返回的指针并不是每次都指向同一个测试。可能它有一些初始化问题，所以它第一次返回错误的值，然后再更正值。

导致未定义行为的行不能进行 OP 声明的转换，因此它必须是发生更改的函数的返回值。