tin,gala377

Klasa package powinna być abstrakcyjna

TIN/inc/packages/Package.h

Line 17 in 3f949c6

Package() = default;

Nie powinniśmy mieć możliwość stworzenia istancji klasy Package. Po co komu tworzenies samego nagłówka?

Komunikacja

Rola suffixu _

TIN/inc/packages/Package.h

Lines 21 to 25 in 3f949c6

    
           uint32_t getTypeId_() const; 
        
           uint32_t getSize_() const; 
        
           uint32_t getId_() const;

suffix "_" dodajemy tylko przy polach klasy protected i privata. Nie ma sensu robić tego w metodach, a tym bardziej w public.

Using jako alias typu

TIN/inc/packages/PackageFactory.h

Line 49 in 3f949c6

    
           std::map<uint32_t, std::function<void(Package *basePackage, const std::string &serializationStr)>> deserializingFunctionsMap;

Użyj w klasie using alias = typ, żeby nie było takich ogromnych typów pól pls :)

Podstawowe przypadki użycia

Wybór lidera - Rafał Galczak

Sprawdzenie dokumentu dokumentacji pod względem literówek, itp.

Harmonogram prac (minimum 1, zalecane 2 punkty kontrolne dla odbioru częściowych funkcji/modułów projektu)

Napisać maila do prowadzącego z pytaniami:

Czy dobrze rozumiemy zakres projektowy?
Czy może być kilku slave'ów?

Wybrane środowisko sprzętowo-programowe (systemy operacyjne, biblioteki programistyczne) i narzędziowe (debugowanie, testowanie)

Podział prac w zespole

Wymagania funckjonalne

Kolejka i zapis do pamięci nieulotnej

Architekturę rozwiązania, tj. ilustrację i opis struktury logicznej systemu (koncepcyjnych bloków funkcjonalnych)

zapoznanie się Licklider Transmission Protocol

Brak wywołania base klasy

TIN/inc/packages/Acknowledge.h

Line 26 in 3f949c6

ar & acknowledgedPackedId_;

powinno być Package::serialize(archive)

Powinien być deleted

TIN/inc/packages/Acknowledge.h

Line 13 in 3f949c6

Acknowledge() = default;

Domyślny konstruktor nie ma sensu powinien być ustwiony na = deleted

Sposób testowania

Dokumentacja

Zrobić dokumentację na etap 1

Brakuje virtual

TIN/inc/packages/Package.h

Line 35 in 3f949c6

void serialize(Archive &ar, const unsigned int version) {

Nie wiem jak to jest z boostowym serialize. Więc popraw mnie jezeli się mylę, ale ta metoda powinna być virtual, żeby dało się ją przeciążać w klasach dziedziczących.

Wymagania niefunkcjonalne

Sposób demonstracji rezultatów, tj. scenariusze testów akceptacyjnych do zaprezentowania przy odbiorze projektu

złe accesor

TIN/inc/packages/Package.h

Line 27 in 3f949c6

uint32_t typeId_ = 0;

Powinno być w private

Do wyrzucenia

TIN/inc/packages/PackageFactory.h

Lines 29 to 43 in 3f949c6

    
           deserializingFunctionsMap[typeId] = [](Package *basePackage, const std::string &serialisationString) -> void { 
        
               boost::iostreams::basic_array_source<char> device(serialisationString.data(), serialisationString.size()); 
        
               boost::iostreams::stream<boost::iostreams::basic_array_source<char> > a(device); 
        
               boost::archive::binary_iarchive inputArchive(a); 
        
               inputArchive & *(basePackage); 
        
               inputArchive & *(static_cast<T *>(basePackage)); 
        
           }; 
        
           serializingFunctiosMap[typeId] = [](Package *basePackage) -> std::string { 
        
               std::string serialisationString; 
        
               boost::iostreams::back_insert_device<std::string> inserter(serialisationString); 
        
               boost::iostreams::stream<boost::iostreams::back_insert_device<std::string> > s(inserter); 
        
               boost::archive::binary_oarchive outputArchive(s); 
        
               outputArchive & *basePackage; 
        
               outputArchive & *(static_cast<T *>(basePackage)); 
        
               return serialisationString;

Te linijki wgl nie korzystują z tego, że ta funkcja jest templatowa. Nie ma sensu żeby to wgl tu było.

	uint32_t getTypeId_() const;

	uint32_t getSize_() const;

	uint32_t getId_() const;

	deserializingFunctionsMap[typeId] = [](Package *basePackage, const std::string &serialisationString) -> void {
	boost::iostreams::basic_array_source<char> device(serialisationString.data(), serialisationString.size());
	boost::iostreams::stream<boost::iostreams::basic_array_source<char> > a(device);
	boost::archive::binary_iarchive inputArchive(a);
	inputArchive & *(basePackage);
	inputArchive & (static_cast<T >(basePackage));
	};
	serializingFunctiosMap[typeId] = [](Package *basePackage) -> std::string {
	std::string serialisationString;
	boost::iostreams::back_insert_device<std::string> inserter(serialisationString);
	boost::iostreams::stream<boost::iostreams::back_insert_device<std::string> > s(inserter);
	boost::archive::binary_oarchive outputArchive(s);
	outputArchive & *basePackage;
	outputArchive & (static_cast<T >(basePackage));
	return serialisationString;

gala377 / tin Goto Github PK

tin's People

Contributors

Watchers

tin's Issues

Recommend Projects

Recommend Topics

Recommend Org