Замечания по PrimeNumbers

alexey-malov commented 4 years ago

[ ] В отладочной конфигурации при помощи директив условной компиляции отключите тест, прогоняющий на верхней границе 100 миллионов. Он очень долго выполняется.

alexey-malov commented 4 years ago

https://github.com/RomanovAleksandr/oop/blob/6678fc6e9f79dc4c2b01bf017ba2aa9981bf8c4e/lab2/PrimeNumbers/PrimeNumbers/PrimeNumbersGenerator.cpp#L12-L14

[ ] isPrimeSize - неудачное имя для переменной типа int. Подберите получше

alexey-malov commented 4 years ago

https://github.com/RomanovAleksandr/oop/blob/6678fc6e9f79dc4c2b01bf017ba2aa9981bf8c4e/lab2/PrimeNumbers/PrimeNumbers/PrimeNumbersGenerator.cpp#L22

[ ] Тут вставка во множество происходит в порядке возрастания. Поэтому лучше воспользоваться версией метода insert, который принимал бы итератор-подсказку для позиции вставки: https://en.cppreference.com/w/cpp/container/set/insert

RomanovAleksandr / oop

Замечания по PrimeNumbers #12