Question

我在做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
}

Answer 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;
}

你还没有注意你的分隔符参数。

全局变量不能分配两次

1 个答案: