0

我在做 TDD 时遇到了一些问题。我设计了这个功能:

String *subString;

String *getWordAndUpdate(String *line, char *delimiter) {
    String *word = malloc(sizeof(String));
    int i;

    stringLeftTrim(line);

    word->length = 0;
    word->startIndex = 0;

    for(i = line->startIndex; i < line->length; i++) {
        if(line->rawString[i] != ' ') {
            word->rawString[i] = line->rawString[i];
            line->startIndex++;
            word->length++;
        } else {
            break;
        }
    }

    line->length = line->length - i;

    return word;
}

使用我的 String 结构:

typedef struct {
    char *rawString;
    int startIndex;
    int length;
} String;

当我第一次调用这个 getWordAndUpdate 函数时它工作正常,但是当我第二次调用它时它说内存访问错误。

extern String *subString;
void test_getWordAndUpdate_should_get_the_first_word_from_a_line_of_instruction() {
    String oneLineString = {"movwf 0x10", 0, 10};

    subString = getWordAndUpdate(&oneLineString, " ,;"); // works fine

    TEST_ASSERT_EQUAL('m', subString->rawString[0]);
    TEST_ASSERT_EQUAL('o', subString->rawString[1]);
    TEST_ASSERT_EQUAL('v', subString->rawString[2]);
    TEST_ASSERT_EQUAL('w', subString->rawString[3]);
    TEST_ASSERT_EQUAL('f', subString->rawString[4]);
    TEST_ASSERT_NOT_EQUAL(' ', subString->rawString[5]);
    TEST_ASSERT_NOT_EQUAL('0', subString->rawString[6]);
    TEST_ASSERT_NOT_EQUAL('x', subString->rawString[7]);
    TEST_ASSERT_NOT_EQUAL('1', subString->rawString[8]);
    TEST_ASSERT_NOT_EQUAL('0', subString->rawString[9]);

    TEST_ASSERT_EQUAL(5, oneLineString.startIndex);
    TEST_ASSERT_EQUAL(5, oneLineString.length);
    TEST_ASSERT_EQUAL(0, subString->startIndex);
    TEST_ASSERT_EQUAL(5, subString->length);

    subString = getWordAndUpdate(&oneLineString, " ,;"); // bad memory access
}
4

1 回答 1

1

getWordAndUpdate在这些代码行中修改对象 oneLineString :

line->length--;
line->startIndex++;

因此,当您第二次调用它时,您的对象不一致。

尝试制作 oneLineString 的副本或找到不修改参数的算法。

也许,您可以使用这种功能:

String *subString;

String *getWordAndUpdate(String *line, char *delimiter) {
    String *word = malloc(sizeof(String));

    int i;
    for (i = 0; line->rawString[i] != ' '; ++i)
        word->rawString[i] = line->rawString[i];
    word->length = i;

    return word;
}

而且您还没有注意定界符参数。

于 2014-03-13T14:26:10.993 回答