search

jam231 / sia

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

Poprawiony błąd dezinformowania agentów #40

Closed kaiks closed 10 years ago

kaiks commented 11 years ago

Powiem tak:

QHash<qint32, QPair<qint32, qint32> > m_cachedBestBuyOrders

nie jest wierszem zbyt deskryptywnym.

Wartości te w rzeczywistości (przynajmniej wg nazewnictwa aplikacji) to: stockId, amount, price

natomiast baza danych dla pól 1 i 2 (przypominam:

            m_cachedBestBuyOrders.insert(stockId,
                                      qMakePair(query.value(0).toInt(),
                                                query.value(1).toInt()));

zwracała... price, amount

Błąd w sam raz na 2:40 dzień w oddania projektu ;)

kaiks commented 11 years ago

jeszcze zwrócę uwagę że dla agentów którzy nie aktualizują cen wg bestorder to nie ma znaczenia (są tacy?)

kaiks commented 11 years ago

jednocześnie zwracam uwagę że to nam niszczy konwencję getstockinfo, tj. zamiast [price] [amount] od teraz otrzymujemy [amount] [price] i trzeba bylo poprawić też last transaction

nie wiem jak się z tym czujecie ale ja na obecną chwilę idę spać na parę godzin i prezentację dokończę jutro

marimarek commented 11 years ago

Heh, ciezko oczekiwac dobrych wynikow od agentow jak dostaja podstawowe informacje na odwrot.

kaiks commented 11 years ago

no, ja się zgadzam, dlatego tym bardziej mnie dziwi że nikt tego wcześniej nie znalazł debugując własnych agentów