Question

我的代码有一个很大的问题，我为一个游戏编写代码，将数据包从服务器发送到客户端，一切正常，但问题是在服务器这么大的尺寸..更小的尺寸，如100-300工作得很好，但我的源码有问题，因为他们有保护从数据包检查缓冲区，如果很好将阻止发送功能，所以我需要其他功能我认为或多优化或其他结构..嗯

这是问题 - TPacket列表[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字节，对于自动存储而言可能太大了。让static解决问题，但会使你的代码不可重入，绝对不是线程安全的。

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

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

strncpy不会终止字符串。你永远不应该使用这个功能。阅读你应该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));

结构尺寸如此之大，需要优化

1 个答案: