Simba-Avionic / srp

Simba rocket platform
0 stars 0 forks source link

nlohmann_json drop app if input file is not a valid json #98

Closed Mateusz-Krajewski closed 1 month ago

Mateusz-Krajewski commented 2 months ago

przed nlohmann::json::parse(file); trzeba sprawdzać poprawność json. nlohmann::json udostępnia metode accept która zwraca czy jest to poprawny json, jednak nie można do niej przekazać pliku gdyż json::parse nie wczyta potem z niego danych, chyba że da się jakoś zresetować wskaźnik w pliku.

att vxFFPWcXK-1b4-6TpDUFT5X25FVXkHa5sPpvBWCaTYg

do poprawy są wszystkie aplikacje wczytujące dane z pliku json i json_parser

bartoszsnieg commented 2 months ago

to można by przejść na jakąś warstwe abstrakcji żeby się przed tym zabezpieczyć . Przynajmniej fabryke trzeb aby było zrobić która będzie zwracać zwalidowany plik lub nie

Mateusz-Krajewski commented 2 months ago

+trzeba zrobić abstrakcje dla oczytu z pliku żeby się to zmockować dało

bartoszsnieg commented 2 months ago

Niekoniecznie abstrakcja jest konieczna. Da się to inaczej też ogarnąć

Mateusz-Krajewski commented 2 months ago

@bartoszsnieg jak?