我在进行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
}

最佳答案

您可以在以下代码行中使用功能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;
}


而且,您不必担心定界符参数。

10-08 20:02