Лучший способ сохранить va_list для последующего использования в C / C ++ - PullRequest
7 голосов
/ 14 октября 2009

Я использую va_list для создания строки, которая отображается.

void Text2D::SetText(const char *szText, ...)

Это все хорошо, но теперь у пользователя есть возможность менять язык во время работы приложения. Мне нужно восстановить все текстовые строки и повторно кэшировать текстовые растровые изображения после инициализации. Я хотел бы сохранить va_list и использовать его всякий раз, когда необходимо сгенерировать текст.

Чтобы дать вам дополнительную информацию, это должно произойти в случае, когда в ключевой строке, которую я перевожу, содержится динамический фрагмент данных.

"Player Score:%d"

Это ключевая строка, которую мне нужно перевести. Я хотел бы сохранить число (числа), предоставленное в va_list для последующего использования (вне области функции, которая инициализирует текст), в случае, если его нужно будет повторно преобразовать после инициализации. Желательно, чтобы я держал копию va_list для использования с vsnprintf.

Я провел некоторые исследования в этой области и нашел несколько способов. Некоторые из которых я подвергаю сомнению, является ли это подходящим методом (с точки зрения стабильности и переносимости).

Ответы [ 5 ]

8 голосов
/ 14 октября 2009

Этот вопрос действительно вызвал у меня интерес. Также я столкнусь с подобной проблемой в моей собственной работе, поэтому решение, разработанное здесь, может мне тоже помочь.

Короче говоря, я написал проверочный код, который кэширует переменные аргументы для последующего использования - вы можете найти его ниже.

Мне удалось заставить приведенный ниже код корректно работать как в Windows, так и в Linux на базе Intel. Я скомпилировал с gcc на Linux и MSVC на Windows. Дважды повторяется предупреждение о злоупотреблении va_start () из gcc - это предупреждение вы можете отключить в вашем make-файле.

Я бы хотел знать, работает ли этот код на компиляторе Mac. Для компиляции может потребоваться небольшая настройка.

Я понимаю, что этот код:

  • Чрезвычайно злоупотребляет va_start () в соответствии со стандартом ANSI C.
  • Старая школа байтово-ориентированная C.
  • Теоретически непереносимо при использовании в качестве указателя переменной va_list.

Мое использование malloc () и free () было очень осознанным, поскольку макросы va_list взяты из стандарта C и не являются функциями C ++. Я понимаю, что в заголовке вашего вопроса упоминается C ++, но я попытался создать полностью C-совместимое решение, кроме использования некоторых комментариев в стиле C ++.

Этот код, без сомнения, также имеет некоторые ошибки или неточности при обработке строки формата. Я представляю это как подтверждение концепции, которую я взломал за два часа, а не готовый пример кода, готовый для профессионального использования.

Этот отказ от ответственности сказал: надеюсь, вы найдете результат столь же восхитительным, как и я! Это был прекрасный вопрос, который надо взломать. Болезненная и искаженная природа результата вызывает у меня глубокий восторженный смех. ;)

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

#define VERBOSE 0

#ifdef WINDOWS
#define strdup _strdup
#endif

/*
 * struct cached_printf_args
 *
 * This is used as the pointer type of the dynamically allocated
 * memory which holds a copy of variable arguments.  The struct
 * begins with a const char * which recieves a copy of the printf()
 * format string.
 *
 * The purpose of ending a struct with a zero-length array is to
 * allow the array name to be a symbol to the data which follows
 * that struct.  In this case, additional memory will always be
 * allocted to actually contain the variable args, and cached_printf_args->args
 * will name the start address of that additional buffer space.
 *
 */
struct cached_printf_args
{
    const char * fmt;
    char  args[0];
};


/*
 * copy_va_args -- Accepts a printf() format string and va_list
 *                 arguments.
 *
 *                 Advances the va_list pointer in *p_arg_src in
 *                 accord with the specification in the format string.
 *
 *                 If arg_dest provided is not NULL, each argument
 *                 is copied from *p_arg_src to arg_dest according
 *                 to the format string.
 *
 */
int copy_va_args(const char * fmt, va_list * p_arg_src, va_list arg_dest)
{
    const char * pch = fmt;

    int processing_format = 0;

    while (*pch)
    {
        if (processing_format)
        {
            switch (*pch)
            {
            //case '!': Could be legal in some implementations such as FormatMessage()
            case '0':
            case '1':
            case '2':
            case '3':
            case '4':
            case '5':
            case '6':
            case '7':
            case '8':
            case '9':
            case '.':
            case '-':

                // All the above characters are legal between the % and the type-specifier.
                // As the have no effect for caching the arguments, here they are simply
                // ignored.
                break;

            case 'l':
            case 'I':
            case 'h':
                printf("Size prefixes not supported yet.\n");
                exit(1);

            case 'c':
            case 'C':
                // the char was promoted to int when passed through '...'
            case 'x':
            case 'X':
            case 'd':
            case 'i':
            case 'o':
            case 'u':
                if (arg_dest)
                {
                     *((int *)arg_dest) = va_arg(*p_arg_src, int);
                     va_arg(arg_dest, int);
                }
                else
                    va_arg(*p_arg_src, int);
#if VERBOSE
                printf("va_arg(int), ap = %08X, &fmt = %08X\n", *p_arg_src, &fmt);
#endif
                processing_format = 0;
                break;

            case 's':
            case 'S':
            case 'n':
            case 'p':
                if (arg_dest)
                {
                    *((char **)arg_dest) = va_arg(*p_arg_src, char *);
                    va_arg(arg_dest, char *);
                }
                else
                    va_arg(*p_arg_src, char *);
#if VERBOSE
                printf("va_arg(char *), ap = %08X, &fmt = %08X\n", *p_arg_src, &fmt);
#endif
                processing_format = 0;
                break;

            case 'e':
            case 'E':
            case 'f':
            case 'F':
            case 'g':
            case 'G':
            case 'a':
            case 'A':
                if (arg_dest)
                {
                    *((double *)arg_dest) = va_arg(*p_arg_src, double);
                    va_arg(arg_dest, double);
                }
                else
                    va_arg(*p_arg_src, double);
#if VERBOSE
                printf("va_arg(double), ap = %08X, &fmt = %08X\n", *p_arg_src, &fmt);
#endif
                processing_format = 0;
                break;
            }
        }
        else if ('%' == *pch)
        {
            if (*(pch+1) == '%')
                pch ++;
            else
                processing_format = 1;
        }
        pch ++;
    }

    return 0;
}

/*
 * printf_later -- Accepts a printf() format string and variable
 *                 arguments.
 *
 *                 Returns NULL or a pointer to a struct which can
 *                 later be used with va_XXX() macros to retrieve
 *                 the cached arguments.
 *
 *                 Caller must free() the returned struct as well as
 *                 the fmt member within it.
 *
 */
struct cached_printf_args * printf_later(const char *fmt, ...)
{
    struct cached_printf_args * cache;
    va_list ap;
    va_list ap_dest;
    char * buf_begin, *buf_end;
    int buf_len;

    va_start(ap, fmt);
#if VERBOSE 
    printf("va_start, ap = %08X, &fmt = %08X\n", ap, &fmt);
#endif

    buf_begin = (char *)ap;

    // Make the 'copy' call with NULL destination.  This advances
    // the source point and allows us to calculate the required
    // cache buffer size.
    copy_va_args(fmt, &ap, NULL);

    buf_end = (char *)ap;

    va_end(ap);

    // Calculate the bytes required just for the arguments:
    buf_len = buf_end - buf_begin;

    if (buf_len)
    {
        // Add in the "header" bytes which will be used to fake
        // up the last non-variable argument.  A pointer to a
        // copy of the format string is needed anyway because
        // unpacking the arguments later requires that we remember
        // what type they are.
        buf_len += sizeof(struct cached_printf_args);

        cache = malloc(buf_len);
        if (cache)
        {
            memset(cache, 0, buf_len);
            va_start(ap, fmt);
            va_start(ap_dest, cache->fmt);

            // Actually copy the arguments from our stack to the buffer
            copy_va_args(fmt, &ap, ap_dest);

            va_end(ap);
            va_end(ap_dest);

            // Allocate a copy of the format string
            cache->fmt = strdup(fmt);

            // If failed to allocate the string, reverse allocations and
            // pointers
            if (!cache->fmt)
            {
                free(cache);
                cache = NULL;
            }
        }
    }

    return cache;
}

/*
 * free_printf_cache - frees the cache and any dynamic members
 *
 */
void free_printf_cache(struct cached_printf_args * cache)
{
    if (cache)
        free((char *)cache->fmt);
    free(cache);
}

/*
 * print_from_cache -- calls vprintf() with arguments stored in the
 *                     allocated argument cache
 *
 *
 * In order to compile on gcc, this function must be declared to
 * accept variable arguments.  Otherwise, use of the va_start()
 * macro is not allowed.  If additional arguments are passed to
 * this function, they will not be read.
 */
int print_from_cache(struct cached_printf_args * cache, ...)
{
    va_list arg;

    va_start(arg, cache->fmt);
    vprintf(cache->fmt, arg);
    va_end(arg);
}

int main(int argc, char *argv)
{
    struct cached_printf_args * cache;

    // Allocates a cache of the variable arguments and copy of the format string.
    cache = printf_later("All %d of these arguments will be %s fo%c later use, perhaps in %g seconds.", 10, "stored", 'r', 2.2);

    // Demonstrate the time-line with some commentary to the output.
    printf("This statement intervenes between creation of the cache and its journey to the display.\n"

    // THIS is the call which actually displays the output from the cached printf.
    print_from_cache(cache);

    // Don't forget to return dynamic memory to the free store
    free_printf_cache(cache);

    return 0;

}
6 голосов
/ 14 октября 2009

Хранение самого va_list не очень хорошая идея; стандарт требует только, чтобы аргумент va_list работал с va_start(), va_arg() и va_end(). Насколько я могу судить, va_list не гарантированно является копируемым.

Но вам не нужно хранить va_list. Скопируйте предоставленные аргументы в другую структуру данных, такую ​​как вектор (вероятно, void *), и получите их позже обычным способом Вы должны быть осторожны с типами, но это всегда относится к функциям в стиле printf в C ++.

3 голосов
/ 23 июня 2012

Вы можете использовать va_copy(), вот пример:

va_list ap;
va_list tmp;
va_copy(tmp, ap);
//do something with tmp
va_end(tmp);
2 голосов
/ 14 октября 2009

То, что вы описываете относительно «удержания числа (-ов), указанного в va_list», - это способ приблизиться к этому.

va_list поддерживает указатели на временную память в стеке (так называемое «автоматическое хранение» в стандарте C). После того, как функция с переменными args вернулась, это автоматическое хранилище исчезло, и его содержимое перестало использоваться. Из-за этого вы не можете просто сохранить саму копию va_list - память, на которую она ссылается, будет содержать непредсказуемый контент.

В приведенном вами примере вам нужно будет сохранить два целых числа, которые будут повторно использоваться при воссоздании этого сообщения. В зависимости от того, сколько строк формата вам приходится иметь дело, ваш подход может отличаться.

Для полностью общего типа подхода вам необходимо:

  • Напишите функцию «cache_arguments()», которая создает буфер динамической памяти значений, найденных в переменных аргументах.
  • В этом cache_arguments() будет использоваться строка формата printf() в стиле вместе с макросами va_start, va_arg, va_end. Вам нужно будет получить типы в соответствии с printf() спецификаторами типов, потому что sizeof(double) != sizeof(int).
  • Сохраните аргументы в кеше памяти с тем же выравниванием и заполнением, которые ожидаются va_arg() на вашей платформе. (Прочитайте ваш varargs.h файл.)
  • Получите ваши звонки на vsnprintf(), работая с этим буфером буферизованной памяти вместо указателя, созданного va_start().

Все вышеперечисленные элементы возможны на большинстве платформ, включая Linux и Windows.

Элемент, который вы можете рассмотреть при переводе, касается порядка слов. Что написано на английском языке как:

Игрок Сэм набрал 20 очков.

На некоторых (человеческих) языках может быть написан только бегло с порядком слов, аналогичным:

Игрок Сэм набрал 20 очков.

По этой причине в Win32 FormatMessage() API используется строка формата, подобная printf(), с функциональной разницей в том, что параметры нумеруются, как в:

Игрок% 1 набрал% 2! D! точек.
% 2! D! очки набраны игроком% 1.

(для каждого аргумента предполагается тип строки, поэтому %1 эквивалентно %1!s!)

Конечно, вы не можете использовать Win32 API, но функциональность изменения порядка слов форматированных аргументов - это то, что я пытаюсь представить как концепцию. Вы также можете реализовать это в своем программном обеспечении.

0 голосов
/ 20 ноября 2017

Способ сделать это в C состоит в том, чтобы вместо этого отправить структуру аргументов функции. Вы должны передать структуру по ссылке, а затем скопировать (memcpy) структуру в общее место, что позволит вам использовать ее позже. Вы распутываете структуру в месте назначения так же, как отправили ее. Вы сохраняете шаблон структуры для «установки и получения».

...