帮我调试这个 - 来自“const char*”的无效转换到“char*”

Question

我根本不明白为什么会出现这个错误。

Widget.cpp: In constructor 'Widget::Widget(Generic, char*, int, int, int, QObject*)':
Widget.cpp:13: error: invalid conversion from 'const char*' to 'char*'

就 Widget 的构造函数而言，我在任何地方都没有“const char*”。

class Widget: public QObject {
    Q_OBJECT
    Q_PROPERTY(char *col READ getCol WRITE setCol)
    Q_PROPERTY(char *row READ getRow WRITE setRow)
    Generic visitor;
    char *_name;
    char *_widget_base;
    int _row;
    int _col;
    int _type;
    public:
    Widget(Generic visitor, char *name, int row, int col, int type, QObject *parent);
    char* widgetBase() const;
    QString getCol() const;
    void setCol(const QString &col);
    QString getRow() const;
    void setRow(const QString &row);

};

Widget::Widget(Generic v, char *name, int row, int col, int type, 
    QObject *parent = 0 ) {
    visitor = v;
    std::string str(name);
    int pos1 = str.find(":");
    int pos2 = str.rfind(":");
    _widget_base = str.substr(pos1, pos2-pos1).c_str();
    _name = name;
    _row = row;
    _col = col;
    _type = type;
}

Answer 1

这是一个 const char * 值

str.substr(pos1, pos2-pos1).c_str();

Answer 2

除了已经得到解答的编译器错误之外，您的代码还有一些其他问题。 （至少，假设您发布的代码接近或完全是您在项目中实际使用的代码。）

即使您将 _widget_base 更改为 const 指针，这行代码也是一个问题：

_widget_base = str.substr(pos1, pos2-pos1).c_str();

substr 返回一个临时字符串对象。 c_str() 的结果仍然属于该临时字符串。而该行的写入方式，该行执行后临时字符串对象将被销毁。因此，问题在于 _widget_base 将保留指向已删除且可以随时重用的内存区域。

您可能还会遇到类似的 _name 问题，具体取决于您传递到 Widget 构造函数中的内容。

因此，您可以使用 _widget_base 和 _name 执行以下三件事之一：

1) 自己动态分配内存，使其不被任何其他对象管理。

std::string temp = str.substr(pos1, pos2-pos1);
_widget_base = new char[temp.length()+1];
strcpy(_widget_base, temp.c_str());

当然，您还需要在 Widget 的析构函数中管理该内存的删除。如果在 Widget 存在时可以更改该值，也可能会重新分配。

2）将这些成员变量设置为字符数组而不是指针。这样内存就成为 Widget 的永久部分，不需要进行管理。当然，您还必须确定数组有多大才足够大。

char _name[a big enough value];
char _widget_base[a big enough value];

然后：

std::string temp = str.substr(pos1, pos2-pos1);
strcpy(_widget_base, temp.c_str());

3) 将这些成员变量设为字符串对象。

std::string _name;
std::string _widget_base;

然后：

_widget_base = str.substr(pos1, pos2-pos1);

这种方式是最可取的，因为它是最稳健且最不容易出错的。您无需直接管理内存，也不用担心值太大而无法容纳。