[c] C 형식의 문자열 만들기 (인쇄하지 않음)

POSIX-2008 호환 시스템 (최신 Linux)을 사용하는 경우 안전하고 편리한 asprintf()기능을 사용할 수 있습니다. malloc()메모리 가 충분하므로 최대 문자열 크기에 대해 걱정할 필요가 없습니다. 다음과 같이 사용하십시오.

char* string;
if(0 > asprintf(&string, "Formatting a number: %d\n", 42)) return error;
log_out(string);
free(string);

이것은 안전한 방식으로 문자열을 구성하기 위해 얻을 수있는 최소한의 노력입니다. sprintf()당신이 질문에 준 코드는 깊이 결함이 :

• 포인터 뒤에 할당 된 메모리가 없습니다. 메모리의 임의의 위치에 문자열을 쓰고 있습니다!

• 썼더라도

  char s[42];

  괄호 안에 어떤 숫자를 넣어야할지 모르기 때문에 심각한 문제에 봉착 할 것입니다.

• “안전한”변형을 사용 했더라도 snprintf()문자열이 잘릴 위험이 있습니다. 로그 파일에 쓸 때 이는 비교적 사소한 문제이지만 유용했을 정보를 정확하게 잘라낼 수있는 가능성이 있습니다. 또한 후행 끝줄 문자를 잘라내어 실패한 줄 끝에 다음 로그 줄을 붙입니다.

• 당신의 조합을 사용하려고하면 malloc()하고 snprintf()모든 경우에 올바른 동작을 생산하기 위해, 당신은 약 두 배 많은 코드로 내가 주어진 것보다 함께 결국 asprintf(), 기본적으로의 기능을 재 프로그램 asprintf().

스타일 매개 변수 목록 자체를 log_out()취할 수 있는 래퍼를 제공하려는 경우를 인수로 취하는 printf()변형 vasprintf()을 사용할 수 있습니다 va_list. 다음은 이러한 래퍼의 완벽하게 안전한 구현입니다.

//Tell gcc that we are defining a printf-style function so that it can do type checking.
//Obviously, this should go into a header.
void log_out_wrapper(const char *format, ...) __attribute__ ((format (printf, 1, 2)));

void log_out_wrapper(const char *format, ...) {
    char* string;
    va_list args;

    va_start(args, format);
    if(0 > vasprintf(&string, format, args)) string = NULL;    //this is for logging, so failed allocation is not fatal
    va_end(args);

    if(string) {
        log_out(string);
        free(string);
    } else {
        log_out("Error while logging a message: Memory allocation failed.\n");
    }
}

답변

#include <stdarg.h>
#include <stdio.h>

// a function that accepts a string:

void foo( char* s);

// You'd like to call a function that takes a format string 
//  and then calls foo():

void foofmt( char* fmt, ...)
{
    char buf[100];     // this should really be sized appropriately
                       // possibly in response to a call to vsnprintf()
    va_list vl;
    va_start(vl, fmt);

    vsnprintf( buf, sizeof( buf), fmt, vl);

    va_end( vl);

    foo( buf);
}



int main()
{
    int val = 42;

    foofmt( "Some value: %d\n", val);
    return 0;
}

static FILE *logfp = ...;

void log_out(const char *fmt, ...)
{
    va_list args;

    va_start(args, fmt);
    vfprintf(logfp, fmt, args);
    va_end(args);
}

void log_out_wrapper(const char *fmt, ...)
{
    va_list args;
    size_t  len;
    char   *space;

    va_start(args, fmt);
    len = vsnprintf(0, 0, fmt, args);
    va_end(args);
    if ((space = malloc(len + 1)) != 0)
    {
         va_start(args, fmt);
         vsnprintf(space, len+1, fmt, args);
         va_end(args);
         log_out(space);
         free(space);
    }
    /* else - what to do if memory allocation fails? */
}

 #include <stdio.h>
 #include <stdarg.h>

 void
 eprintf (const char *template, ...)
 {
   va_list ap;
   extern char *program_invocation_short_name;

   fprintf (stderr, "%s: ", program_invocation_short_name);
   va_start (ap, template);
   vfprintf (stderr, template, ap);
   va_end (ap);
 }