使用stdarg realloc

Question

我想連接字符串使用stdarg（庫）標題，但我做錯了什麼。 有一種更簡單的方法來使用realloc連接字符串？

#include <stdio.h> 
#include <stdlib.h> 
#include <string.h> 
#include <stdarg.h> 
void concatenaCodigo(char *string, char *format, ...){ 
    va_list args; 
    va_start(args, format); 
    int n = vsnprintf(NULL, 0, format, args); 
    string = (char*) realloc (string, n); 
    if (string != NULL) { 
     printf("Realloc OK!\n"); 
     vsprintf(string, format, args); 
     va_end(args); 
    } 
    else { 
     free (string); 
     puts ("Error (re)allocating memory"); 
     exit (1); 
    } 

} 
int main(){ 
    char *codigoC = NULL; 
    concatenaCodigo(codigoC, "Test%s%s","asd","asd"); 
    printf("%s\n", codigoC); 
} 

我做了更改。代碼應該如下所示？級聯尚未運行。

char* concatenaCodigo(char *format, ...){ 
    va_list args; 
    va_start(args, format); 
    int n = vsnprintf(NULL, 0, format, args); 
    char * newString; 
    newString = (char*) malloc(n); 
    vsprintf(newString, format, args); 
    va_end(args); 
    return newString; 
} 
int main(){ 
    char *codigoC = NULL; 
    codigoC = concatenaCodigo("Test%s%s", "asd", "asd"); 
    printf("%s\n", codigoC); 
} 

Answer 1

爲了避免丟失引用，我改變了接收指向char數組的指針的函數。

void concatenaCodigo(char **message, char *format, ...) 
{ 
    va_list args; 
    va_start(args, format); 
    int n = vsnprintf(NULL, 0, format, args) + 1; /* Note the +1! */ 
    va_end(args);         /* vsnprintf() 'uses up' args */ 
    char *newString = (char *) malloc(n); 
    if (newString != 0) 
    { 
     va_start(args, format);      /* Restart args */ 
     vsprintf(newString, format, args); 
     va_end(args); 
    } 
    *message = newString; 
} 

Answer 2

指針不按引用傳遞，所以，當你這樣做：

void concatenaCodigo(char *string, char *format, ...){ 
    // ... 
    string = (char*) realloc (string, n); 
} 

你覆蓋已傳遞的指針，永遠不會獲得新值該功能的。

concatenaCodigo應該返回新分配的字符串：

char* concatenaCodigo(char *format, ...) { 
    // ... 
    return string; 
} 

int main() { 
    char *codigoC = concatenaCodigo("Test%s%s","asd","asd"); 
    printf("%s\n", codigoC); 
} 

此外，你應該不呼叫free(string)如果realloc失敗。您不應將NULL指針傳遞給free()。

Answer 3

修改後的代碼中的問題是，您沒有正確重置va_list。您有：

char* concatenaCodigo(char *format, ...){ 
    va_list args; 
    va_start(args, format); 
    int n = vsnprintf(NULL, 0, format, args); 
    char * newString; 
    newString = (char*) malloc(n); 
    vsprintf(newString, format, args); 
    va_end(args); 
    return newString; 
} 
int main(){ 
    char *codigoC = NULL; 
    codigoC = concatenaCodigo("Test%s%s", "asd", "asd"); 
    printf("%s\n", codigoC); 
} 

您需要：

char *concatenaCodigo(char *format, ...) 
{ 
    va_list args; 
    va_start(args, format); 
    int n = vsnprintf(NULL, 0, format, args) + 1; /* Note the +1! */ 
    va_end(args);         /* vsnprintf() 'uses up' args */ 
    char *newString = (char *) malloc(n); 
    if (newString != 0) 
    { 
     va_start(args, format);      /* Restart args */ 
     vsprintf(newString, format, args); 
     va_end(args); 
    } 
    return newString; 
} 

int main(void) 
{ 
    char *codigoC = concatenaCodigo("Test%s%s", "asd", "asd"); 
    if (codigoC != 0) 
     printf("%s\n", codigoC); 
    free(codigoC); 
    return 0; 
} 

注意<stdarg.h>是標題，不是圖書館。