jam231 / sia

Stock market server (part of stock market simulation system).
1 stars 0 forks source link

Chyba błędna obsługa pakietów SELL_STOCK_RESP, BUY_STOCK_RESP #12

Closed kaiks closed 11 years ago

kaiks commented 11 years ago
qint32 BuyStockRespMsg::length() const
{
    //8 bajtów na typ i dlugosc
    return 8 + sizeof(BuyStockRespMsg);
}

IOMessage::MessageType BuyStockRespMsg::type() const
{
    return BUY_STOCK_RESP;
}

void BuyStockRespMsg::send(QIODevice* connection)
{
    QDataStream tmpStream(connection);
    tmpStream<<length();
    tmpStream<<static_cast<qint32>(type());
    tmpStream<<m_transaction;
}

Coś tu poszło chyba nie tak :p Statycznie wyliczana długość pakietu na 8+sizeof klasa? która pewnie zajmuje kolejne 4 bajty

długość chyba ma wg naszej konwencji 2 nie 4 bajty, typ ma 1 nie 4 ps: co oznacza m_transaction?

no chyba że to jakaś mocno robocza wersja lub znów ja czegoś nie rozumiem :p

jam231 commented 11 years ago

Nie bug, tylko kawałek legacy kod zostawiony przez marka i który prawdopodobnie zostanie wyrzucony.