【发布时间】:2020-02-10 15:54:40
【问题描述】:
我有以下构建器来构建Tokens:
// token_builder.h
class TokenBuilder
{
Keyword m_keyword;
TokenType m_type;
std::string m_symbol;
public:
TokenBuilder& set_keyword(Keyword);
TokenBuilder& set_toktype(TokenType);
TokenBuilder& set_symbol(std::string);
std::unique_ptr<Token>&& build();
};
// token_builder.cpp
TokenBuilder& TokenBuilder::set_keyword(Keyword k)
{
m_keyword = k;
return *this;
}
TokenBuilder& TokenBuilder::set_toktype(TokenType t)
{
m_type = t;
return *this;
}
TokenBuilder& TokenBuilder::set_symbol(std::string sym)
{
m_symbol = sym; // This is the point where it segfaults
return *this;
}
std::unique_ptr<Token>&& TokenBuilder::build()
{
std::unique_ptr<Token> token;
token->m_keyword = m_keyword;
token->m_type = m_type;
token->m_symbol = m_symbol;
return std::move(token);
}
程序在我试图通过调用TokenBuilder::set_symbol 初始化m_symbol 时以SIGSEGV 停止。这是main 的样子:
int main()
{
auto token = Token::builder()
.set_keyword(Keyword::IF)
.set_toktype(TokenType::ID)
.set_symbol("if")
.build();
std::cout << token->get_symbol() << std::endl;
}
如您所见,调用set_keyword 和set_toktype 不会导致任何错误。但是将文字“if”(或std::string 对象)传递给set_symbol 会导致程序崩溃。按值传递是故意的。进入set_symbol 并分析TokenBuilder 的状态并没有透露任何信息。正如预期的那样,m_keyword 设置为 Keyword::IF,m_type 设置为 TokenType::ID。这是崩溃的行:m_symbol = sym。
上面的程序可能还有其他重要的问题(我是否正确使用std::move?在很长一段时间后返回C++),但这种感觉就像一个菜鸟会犯的愚蠢错误。我在 Windows 10(64 位)上使用 MinGW-w64 编译器。
这是方便复制粘贴到您的机器的完整代码:
// src/include/token_builder.h
#ifndef TOKEN_BUILDER_H
#define TOKEN_BUILDER_H
#include <memory>
#include "token.h"
namespace funk
{
class Token;
class TokenBuilder
{
Keyword m_keyword;
TokenType m_type;
std::string m_symbol;
public:
TokenBuilder& set_keyword(Keyword);
TokenBuilder& set_toktype(TokenType);
TokenBuilder& set_symbol(std::string);
std::unique_ptr<Token>&& build();
};
}
#endif
// src/token_builder.cpp
#include "include/token.h"
#include "include/token_builder.h"
namespace funk
{
TokenBuilder& TokenBuilder::set_keyword(Keyword k)
{
m_keyword = k;
return *this;
}
TokenBuilder& TokenBuilder::set_toktype(TokenType t)
{
m_type = t;
return *this;
}
TokenBuilder& TokenBuilder::set_symbol(std::string sym)
{
m_symbol = sym;
return *this;
}
std::unique_ptr<Token>&& TokenBuilder::build()
{
std::unique_ptr<Token> token;
token->m_is_keyword = m_keyword != Keyword::_NONE_;
token->m_keyword = m_keyword;
token->m_type = m_type;
token->m_symbol = m_symbol;
return std::move(token);
}
}
// src/include/token.h
#ifndef TOKEN_H
#define TOKEN_H
#include <memory>
#include "keyword.h"
#include "token_type.h"
#include "token_builder.h"
namespace funk
{
class TokenBuilder;
class Token
{
friend class TokenBuilder;
Keyword m_keyword;
bool m_is_keyword;
std::string m_symbol;
TokenType m_type;
Token();
public:
Keyword& get_keyword ();
bool is_keyword();
std::string& get_symbol();
TokenType& get_type();
static TokenBuilder&& builder();
};
}
#endif // END TOKEN_H
// src/token.cpp
#include <string>
#include <memory>
#include "include/keyword.h"
#include "include/token_builder.h"
#include "include/token_type.h"
namespace funk
{
Keyword& Token::get_keyword()
{
return m_keyword;
}
bool Token::is_keyword()
{
return m_is_keyword;
}
std::string& Token::get_symbol()
{
return m_symbol;
}
TokenType& Token::get_type()
{
return m_type;
}
TokenBuilder&& Token::builder()
{
TokenBuilder builder;
return std::move(builder);
}
}
// src/main.cpp
#include <iostream>
#include "include/buffered_reader.h"
#include "include/token.h"
#include "include/token_builder.h"
int main()
{
using namespace funk;
auto token = Token::builder()
.set_keyword(Keyword::IF)
.set_toktype(TokenType::ID)
.set_symbol("if")
.build();
std::cout << token->get_symbol() << std::endl;
}
注意:我尽我所能在 StackOverflow 上查找了类似的问题,但没有找到与我的情况相符的问题。如果您认为它与您之前遇到的问题相似,请随时标记重复并将链接发布到原始问题。谢谢!
【问题讨论】:
-
在 gdb 下运行你的程序,然后运行回溯。它会告诉你你在哪条线上进行段错误。您提供的代码太多了,您最好学习如何调试自己的程序,而不是依赖 StackOverflow 为您调试它们。
-
Token::builder()返回对局部变量的引用。一旦函数返回,变量就会被销毁并且引用变得悬空。调用者使用该引用会表现出未定义的行为。 -
@Gillespie 我知道哪一行导致崩溃。阅读问题。
-
@DevashishJaiswal 抱歉,当我向下滚动时,它在您发布的代码墙中丢失了。我建议下次尝试制作stackoverflow.com/help/minimal-reproducible-example。
-
在
TokenBuilder::build中,您永远不会将token设置为引用有效的Token。它被有效地取消引用,同时仍然是一个空指针。
标签: c++ c++11 segmentation-fault stdstring