c++ - 结构尺寸太大，需要优化

Question

我的一个功能的代码有一个大问题，我为一款游戏编写了这个代码，将数据包从服务器发送到客户端，一切都很好，但问题在于服务器尺寸太大..尺寸更小，例如 100-300效果很好，但我的源代码有问题，因为它们有一个保护措施来检查数据包的缓冲区，如果太强大，将阻止发送功能，所以我需要我认为的其他功能或很多优化或其他结构..嗯

这是问题 - TPacket list[1000]；

typedef struct testa
{
    char    t_A[10 + 1];
    char    t_B[12 + 1];
    char    t_C[32 + 1];
    char    t_D[512 + 1];
    int     t_E;
    char    t_F[19 + 1];
    int     t_G;
} TPacket;

typedef struct testb
{
    BYTE header;
    TPacket list[1000]; // (If i put example 200 etc work) but when is so big = buffer mem_size overflow. memsize(131072) write_pos(32) iSize(598001)
} Test;

// FUNCTION TO SEND:
    Test p;
    p.header = HEADER_GC_T;

    SQLMsg *pMsg = DBManager::instance().DirectQuery("SELECT * FROM table.list ORDER BY date DESC LIMIT 1000");
    MYSQL_ROW row;
    int i = 0;

    if(pMsg->uiSQLErrno != 0)
        return;

    while ((row = mysql_fetch_row(pMsg->Get()->pSQLResult)))
    {
        p.list[i] = TPacket();
        strncpy(p.list[i].t_A, row[1], sizeof(p.list[i].t_A));
        strncpy(p.list[i].t_B, row[2], sizeof(p.list[i].t_B));
        strncpy(p.list[i].t_C, row[3], sizeof(p.list[i].t_C));
        strncpy(p.list[i].t_D, row[4], sizeof(p.list[i].t_D));
        str_to_number(p.list[i].t_E, row[5]);
        strncpy(p.list[i].t_F, row[6], sizeof(p.list[i].t_F) - 1);
        str_to_number(p.list[i].t_G, row[7]);
        i++;
    }

    if(pMsg->Get()->uiNumRows < 1000)
    {
        while (i < 1000)
        {
            p.list[i] = TPacket();
            strncpy(p.list[i].t_A, "", sizeof(p.list[i].t_A));
            strncpy(p.list[i].t_B, "", sizeof(p.list[i].t_B));
            strncpy(p.list[i].t_C, "", sizeof(p.list[i].t_C));
            strncpy(p.list[i].t_D, "", sizeof(p.list[i].t_D));          
            p.list[i].t_E = 0;          
            strncpy(p.list[i].t_F, "", sizeof(p.list[i].t_F) - 1);          
            p.list[i].t_G = 0;              
            i++;
        }
    }
    ch->GetDesc()->Packet(&p, sizeof(p));

Answer 1

您的 Test 结构非常大，为 588,000 字节，对于自动存储来说可能太大。使其静态应该可以解决问题，但会使您的代码不可重入并且绝对不是线程安全的。

如果问题出在最大数据包大小上，则必须将传输分成更小的数据包。在结构和 SQL SELECT 语句中使用较少数量的项目。

如果字符串比目标数组长，

strncpy 不会以 null 终止字符串。您绝对不应该使用此功能。阅读为什么你应该stop using strncpy already! 。您可以改为使用不同的函数，该函数通过截断进行复制，但不会以 null 终止目标。

假设 TPacket 的默认构造函数生成一个初始化为所有位 0 的 TPacket，则可以大大简化清除循环。如果没有，只需使用 memset 来完成此操作。

typedef struct testa {
    char t_A[10 + 1];
    char t_B[12 + 1];
    char t_C[32 + 1];
    char t_D[512 + 1];
    int  t_E;
    char t_F[19 + 1];
    int  t_G;
} TPacket;

typedef struct testb {
    BYTE header;
    TPacket list[200];
} Test;

// Utility function: copy with truncation, return source string length
// truncation occurred if return value >= size argument
size_t bstrcpy(char *dest, size_t size, const char *src) {
    size_t i;
    /* copy the portion that fits */
    for (i = 0; i + 1 < size && src[i] != '\0'; i++) {
         dest[i] = src[i];
    }
    /* null terminate destination if possible */
    if (i < size) {
        dest[i] = '\0';
    }
    /* compute necessary length to allow truncation detection */
    while (src[i] != '\0') {
        i++;
    }
    return i;
}

// FUNCTION TO SEND:
void myfunction() {
    Test p;
    p.header = HEADER_GC_T;

    SQLMsg *pMsg = DBManager::instance().DirectQuery("SELECT * FROM table.list ORDER BY date DESC LIMIT 200");
    MYSQL_ROW row;
    int i = 0;

    if (pMsg->uiSQLErrno != 0)
        return;

    while ((row = mysql_fetch_row(pMsg->Get()->pSQLResult))) {
        p.list[i] = TPacket();
        bstrcpy(p.list[i].t_A, sizeof(p.list[i].t_A), row[1]);
        bstrcpy(p.list[i].t_B, sizeof(p.list[i].t_B), row[2]);
        bstrcpy(p.list[i].t_C, sizeof(p.list[i].t_C), row[3]);
        bstrcpy(p.list[i].t_D, sizeof(p.list[i].t_D), row[4]);
        str_to_number(p.list[i].t_E, row[5]);
        bstrcpy(p.list[i].t_F, sizeof(p.list[i].t_F), row[6]);
        str_to_number(p.list[i].t_G, row[7]);
        i++;
    }

    if (i < 1000) {
        memset(&p.list[i], 0, (1000 - i) * sizeof(p.list[i]));
        //while (i < 1000) {
        //    p.list[i] = TPacket();
        //    i++;
        //}
    }
    ch->GetDesc()->Packet(&p, sizeof(p));