Spory refactoring protokołu.

jam231 / sia

Stock market server (part of stock market simulation system).

1 stars 0 forks source link

Spory refactoring protokołu. #48

Closed jam231 closed 10 years ago

jam231 commented 10 years ago

Protokół jest w kiepskim stanie (jak cala reszta :8ball: ), wiekszość jest zrobiona byle jak.

Zakładam, że ten issue będzie zakończony po tym jak protokół będzie doprowadzony do jako takiego stanu, a to znaczy, że oprócz refactoringu kodu musi być:

dokumentacja/okomentowanie odpowiednich fragmentów kodu,
spec'sy - uzupelniona wiki

Postęp prac będzie można obserwować na branch Issue#47.

jam231 commented 10 years ago

Pozostało jeszcze trochę zmienić sposób używania loggera przez protokół. To czy będą dodawane nowe rzeczy jeszcze się okaże - zmienić się już nic raczej nie powinno.

Nie jestem pewien jak będzie trzeba rozwiązać sprawę z utrzymywaniem połączenia tcp, podobno da się ustawić KEEP_ALIVE.

jam231 commented 10 years ago

Trochę się pospieszyłem ze stwierdzeniem że zmienić się już nic nie powinno.

jam231 commented 10 years ago

Ok, refactoring wydaje sie skonczony. W tych wiadomosciach ktore sa nie powinno sie juz nic zmieniac. Testy dla wszystkich wiadomosci rowniez juz sa, pozostaje tylko dobrze przetestowac funkcje do deserializacji i #47 będzie skończony.