Question

我的代码中的某些东西神秘地用垃圾值替换char（字符const char *内的upper）内的所有foo_add()。这似乎是在malloc中调用foo_add()之后发生的。我在函数内部的Xcode中设置断点，upper在malloc调用之前很好，但填充了随机值后跟。我是一个完整的C新手，所以我完全有可能做的事情有一些副作用，我不知道。我不知道它是否与malloc有关，我无法理解为什么会这样。代码如下。我添加了评论以显示问题发生的位置。

//in Foo.h
#include <stdlib.h>
#include <ctype.h>
#include <stdio.h>
#include <string.h>

typedef struct Foo {
    struct Foo * members[26] ;
} Foo ;

const char * toUpper(const char * in) ;
const char * getSubstring(const char * str, size_t startingIndex) ;

void foo_init(Foo * f) ;
void foo_add(Foo * f, const char * str) ;

//in Foo.c
#include "Foo.h"

const char * toUpper(const char * in) {

    size_t length = strlen(in) ;
    char temp[length + 1] ;

    for (size_t i = 0 ; i < length; i++) {
        temp[i] = toupper(in[i]) ;
    }
    temp[length] = '\0' ;
    char * out = temp ;
    return out ;
}

const char * getSubstring(const char * str, size_t startingIndex) {
    size_t size = (unsigned)(strlen(str) - startingIndex) ;
    char temp[size + 1] ; //+1 for '\0' (terminating char)
    for (size_t i = 0 ; i < size ; i++) {
        temp[i] = str[i + startingIndex] ;
    }
    temp[size] = '\0' ;
    const char * ret = temp ;
    return ret ;
}

void foo_init(Foo * f) {
    for (size_t i = 0 ; i < 26 ; i++) {
        f->members[i] = NULL ;
    }
}

void foo_add(Foo * f, const char * str) {

    const char * upper = toUpper(str) ;
    int index = ((int)upper[0]) - 65 ;

    size_t length = strlen(str) ;
    if (length > 0) {
        if (f->members[index] == NULL) {
            /* debugger says upper = "ABCXYZ" here, which is expected */
            f->members[index] = (Foo *)malloc(sizeof(Foo)) ;
            /* set another debugger breakpoint here - upper is now filled with garbage values */
        }

        /* take substr and call recursively until length == 0 */
        const char * next = getSubstring(upper, 1) ;
        foo_add(f->members[index], next) ;
    }
}

//in main.c
#include <stdio.h>

#include "Foo.h"

int main(int argc, const char * argv[])
{
    Foo f ;
    foo_init(&f) ;

    const char * str = "abcxyz" ;

    foo_add(&f, str) ;
    return 0;
}

Answer 1

罪魁祸首就是这段代码：

const char * function(const char * in) {
  char temp[..];
  char *out = temp;
  return out;
}

这里，temp在堆栈上分配，而不是在堆上。这意味着它的生命周期在函数范围退出时结束。您应该在堆上动态分配它，例如char *temp = calloc(length+1, 1)，或者在预先分配的空间（可能在堆栈上的那个时间点）从被调用的空间传递它。

什么是更改C字符串中的char值？

1 个答案: