1

我在将结构写入共享内存时遇到问题,因为当我这样做时,代码只写入结构的第一部分。

这是我的数据结构:

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++;
        }
4

2 回答 2

4

传递给 snprintf 的 4 指定要写入的最大字符数。不是写的数量。snprtinf 返回它确实写入的字符数。所以你只打印 1 个字符,在它之后留下 3 个空值。

做就是了:

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

这样,它将尝试打印字符串后跟 4 个空格,并打印最多 4 个字符。

于 2013-01-24T05:53:53.860 回答
1

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", ' ');

哇——这比我想象的要努力得多。

于 2013-01-24T08:04:59.267 回答