0

假设我主要有一个功能a

a(c);

c声明为

char* c.

函数a其实是这样的:

void a(char* v)
{
    v[0] = 1;
    v[1] = 2;
}

这是错的吗?因为程序说c在这个函数中是未初始化的。

编辑:更多代码:

byte* msg;

insere_msg(t_msg, argv[2], msg);

void insere_msg(int size, char name[MAX], byte* v)
{
    FILE *fp;
    int i = 10;
    int n = 0;
    char aux;

    v[0] = (byte) 4;
    v[1] = (byte) 8;
    v[2] = (byte) 15;
    v[3] = (byte) 16;
    v[4] = (byte) 23;
    v[5] = (byte) 42;
    v[6] = (byte) size>>(3*8);
    v[7] = (byte) size>>(2*8);
    v[8] = (byte) size>>(1*8);
    v[9] = (byte) size;

    fp = fopen(name, "r");

    while(n < size)
    {
        fscanf(fp, "%c", &aux);
        v[i] = (byte) aux;
        i++;
        n++;
    }
}
4

1 回答 1

0

查看您的编辑:您需要在某处使用 malloc 命令分配内存。

msg = (char *)malloc(sizeof(char) * atLeastThree);

现在分配了 msg,它可以被推送到您的 insere_msg 函数和您的 a 函数中。

您还应该实现这个更“安全”的代码:

void a(char * v, unsigned int sizeofV){
    if(NULL != v && sizeofV >= 3){
        v[0] = 1;
        v[2] = 2;
    }else{
        printf("array v is either not allocated or not of proper size");
    }
}

在您操作指针指向的数据之前,您真的应该检查以确保点类型已被初始化并且它对于您尝试追求的功能的目的是有效的!!!

现在,如果您没有分配内存,这将无法解决问题,它只会防止它出错。

如果你想要更安全的代码,你将传入未分配的指针,让函数分配并返回大小。这里唯一的事情是,调用者负责释放内存。

unsigned int a(char * v){

    v = (char *)malloc(sizeof(char) * Necessity);

    if(NULL != v){
        v[0] = 1;
        v[2] = 2;
    }else{
        printf("array v is either not allocated or not of proper size");
    }

    return Necessity;
}
于 2012-07-01T04:26:14.883 回答