Question

我有以下代码：

int Class::set(const char *  id_tran_typ_)
{
        char *trimmed_id_tran_typ_ = NULL;
        trimString(id_tran_typ_, &trimmed_id_tran_typ_);
        int len = strlen(trimmed_id_tran_typ_);
        if (len < 1)
                _id_tran_typ = DEFAULT_STRING;
        else
                _id_tran_typ = id_tran_typ_;
        delete[] trimmed_id_tran_typ_;
        trimmed_id_tran_typ_ = NULL;
        return SUCCESS;
}

这里我在int len = strlen(trimmed_id_tran_typ_);

获得了一个SEGV信号

trimString函数是这样的：

char *trimString(const char *sP_, char **dP_)
{

        *dP_ = NULL;

        if (sP_ == NULL)
                return *dP_;

        const char *lastPos = (char *)&sP_[strlen(sP_)];
        const char *startPos = (char *)sP_;
        const char *endPos = lastPos;

        int len = strlen(sP_);

        for (; endPos != sP_; endPos--)
        {
                if (*(endPos - 1) != ' ')
                {
                        len = endPos - sP_; // *endPos = '\0';
                        break;
                }
        }
        if (len == 0) // if (*endPos != '\0')
        {
                *dP_ = new char[1];
                *dP_[0] = '\0';
                return *dP_;
        }
        for(; startPos < endPos; startPos++, len--)
        {
                if (*startPos != ' ')
                        break;
        }
        if (startPos == endPos)
        {
                *dP_ = new char[1];
                *dP_[0] = '\0';
                return *dP_;
        }

        *dP_ = STRDUP(startPos, len);
        return *dP_;
}

现在我的问题很明显，为什么我在这里遇到分段错误。

是不是因为我没有收到set()函数发送的trimString()函数中的字符串？

这就是STRDUP()的样子：

char *STRDUP(const char *sP_, const int len_)
{
        if (! sP_)
                return NULL;
        else if (len_ < 1)
                return NULL;

        char *newP = new char [ len_ + 1] ;
        memset(newP, '\0', len_+1);
        strncpy(newP, sP_, len_);
        return(newP);
}

Answer 1

您的STRDUP函数返回空指针或id_tran_typ_是空指针。

Answer 2

最可能的解释是，Class::set方法是在id_tran_type_设置为NULL的情况下调用的。您可以在检查长度之前检查trimmed_id_tran_typ_值是否为NULL来解决此问题。

int Class::set(const char *  id_tran_typ_)
{
    char *trimmed_id_tran_typ_ = NULL;
    trimString(id_tran_typ_, &trimmed_id_tran_typ_);
    if (!trimmed_id_tran_typ_ || strlen(trimmed_id_tran_typ_) < 1)
        _id_tran_typ = DEFAULT_STRING;
    else
        _id_tran_typ = id_tran_typ_;
    delete[] trimmed_id_tran_typ_;
    trimmed_id_tran_typ_ = NULL;
    return SUCCESS;
}

获得字符串长度时的SEGV信号

2 个答案: