C - Трудности записи всей структуры в общую память

У меня проблемы с записью структуры в разделяемую память, потому что когда я это делаю, код записывает только первую часть структуры.

Вот моя структура данных:

struct shmData{
    char number[4];
    char description[1020];
};

Вот мой код:

//Structure Pointer - placed on SHM
struct shmstruct *ptr;
char description_to_write[1024] = {0};
struct shmData *data_ptr;
data_ptr = (struct shmData*) description_to_write;

//Shared Memory init
create_shm(&ptr);
printf("Started\n");

//Define Offsets
//init_shm(ptr);

//#########################

//Check if number exist
    if (!(strcmp("NO SUCH CODE\0", get_description(ptr, 6)))) {
        //-> If not, get next offset (nput) where to write new data
//      sem_wait(&ptr->mutex);
//      long offset = ptr->msgoff[ptr->nput];
//      printf("OFFSET: %li\n", offset);
//      if (++(ptr->nput) > NMESG)
//          ptr->nput = 0; /* circular buffer */
//      sem_post(&ptr->mutex);
//      printf("Buffer: %s", argv[2]);

    snprintf(data_ptr->number, 4, "%s", "6");
    snprintf(data_ptr->description, 1020, "%s\n", argv[2]);

    printf("DATA: %s\n", data_ptr->number);
    printf("DATA: %s\n", data_ptr->description);

    printf("description_to_write: %i\n", description_to_write);

    //strcpy(&ptr->msgdata[offset], );
    //sem_post(&ptr->nstored);
    //printf("Wrote...\n\n");
} else {
    //-> Else get offset of this number and put data to it
}

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

Мне нужно, чтобы он был записан в общую память следующим образом:

Number(space)(space)(space)Description

Общий дамп памяти выглядит так:

00003b0: 3120 2020 496e 7075 7420 4572 726f 720a  1   Input Error.
00003c0: 0000 0000 0000 0000 0000 0000 0000 0000  ................
*
00007b0: 3220 2020 4f75 7470 7574 2045 7272 6f72  2   Output Error
00007c0: 0a00 0000 0000 0000 0000 0000 0000 0000  ................
00007d0: 0000 0000 0000 0000 0000 0000 0000 0000  ................

От отладчика:

description_to_write[0] char    54 '6'  
description_to_write[1] char    0 '\000'    
description_to_write[2] char    0 '\000'    
description_to_write[3] char    0 '\000'
description_to_write[4] char    83 'S'  
description_to_write[5] char    111 'o' 
description_to_write[6] char    109 'm' 
description_to_write[7] char    101 'e' 
description_to_write[8] char    84 'T'  
description_to_write[9] char    101 'e' 
description_to_write[10]char    120 'x' 
description_to_write[11]char    116 't' 
description_to_write[12]char    10 '\n'

Уродливое решение:

int i = 0;
        while(i < 4){
            if(data_ptr->number[i] == '\000'){
                data_ptr->number[i] = 32;
            }
            i++;
        }
Источник

2 ответа

Решение

4, переданные snprintf, задают МАКСИМАЛЬНОЕ количество символов для записи. Не сумма, чтобы написать. snprtinf retursn число символов, которые он написал. Таким образом, вы печатаете только 1 символ, который оставляет 3 нуля после него.

Просто делать:

snprintf(data->number,4,"%s    ","6");

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

Источник

Ваше уродливое решение для использования snprintf() и затем переписать символы '\0', поскольку пробелы имеют некоторые недостатки:

  • snprintf() никогда не будет писать цифру в data_ptr->number[3], Для строк длиннее 3 символов он усекает и пишет '\0' вместо копирования 4-го символа в исходной строке.

  • если вы не уверены, что структура будет инициализирована нулем перед вызовом snprintf() затем символы после '\0', которые snprintf() требуется, чтобы написать не будет установлен на что-либо snprintf(), Это может не быть проблемой в вашем коде после публикации, но это хрупко перед лицом изменений в структуре data_ptr указывает, что выделено и инициализировано.

Поскольку поле очень маленькое, вы можете рассмотреть что-то вроде:

memset( data_ptr->number, ' ', sizeof(data_ptr->number));
memcpy( data_ptr->number, "6", strlen("6"));

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

Вот слегка протестированная функция, которую вы можете использовать вместо этого (хотя я признаю, что ненавижу название):

size_t str2mem_padded( char* dest, size_t dest_size, char const* src, char fill)
{
    size_t result = 0;

    while (dest_size && *src) {
        *dest++ = *src++;
        ++result;
        --dest_size;
    }
    while (dest_size) {
        *dest++ = fill;
        --dest_size;
    }

    return result; // returns number of chars copied from src so you 
                   //   can determine if there was truncation
}

В вашем коде вы могли бы использовать следующее вместо вызова snprintf(data_ptr->number, 4, "%s", "6"):

str2mem_padded( data_ptr->number, sizeof(data_ptr->number), "6", ' ');

Вау - это было намного больше усилий, чем я думал.

Источник
Другие вопросы по тегам