slavjan / monte_karlo Goto Github PK

Строка

логичней возвращать false
а ещё лучше:

virtual bool loadShape() =0;
virtual bool readFile(std::string path)=0;
virtual bool readParams()=0;

Строка

Думаю стоит проверять перед удалением на nullptr
cppcheck сюда ругается:
ошибка: Cppcheck: Mismatching allocation and deallocation: MonteKarloApp::control

https://github.com/Slavjan/Monte_Karlo/blob/master/ParametersReader.cpp#L38

Не факт что в 1 Байте 8 бит.
Это справедливо лишь для современной архитектуры, раньше были архитектуры и с 6 битами в байте, Pascal кстати разрабатывался под такую архитектуру из за чего был ужасно не эффективен и пришлось разрабатывать TurboPascal.

В общем и целом, просто на будущее знай что количество бит в 1 байте это не константа.

Не говоря уже о том что эта проверка не работает а 4*8 = 32 что как бы не максимальное значение int