ASmMegane / MLiTA

0 stars 0 forks source link

Замечания по 13.10 #2

Open pechischev opened 8 years ago

pechischev commented 8 years ago

репозиторий проекта не должен содержать файлы сборки(Debug), и к тому же нет файла проекта с расширением .sln необходимый для запуска на других устройствах

pechischev commented 8 years ago

нет тестов для проверки граничных условий:

на пустом файле когда размерность превышает максимально допустимые значения 2-3 теста на нетривиальные случаи

pechischev commented 8 years ago

те же замечания что и в https://github.com/ASmMegane/MLiTA/issues/1

pechischev commented 8 years ago

нет проверки на существование файла

pechischev commented 8 years ago

в функциях int CountSuprefikWord(std::vector & allWord, int startPosition, std::vector & suprefiks) void OutputResult(std::vector & allWord, int countSuprefiks) std::vector & allWord лучше принимать по константной ссылке, меньше шанс что-нибудь заменить в процессе работы функции

pechischev commented 8 years ago

тоже самое и в void CreateWord(std::vectorstd::string & words, std::vector & allWord, int & counterAllWord, bool isSupref) для "std::vectorstd::string & words"

pechischev commented 8 years ago

дублирование кода, выдели в функцию image

в size_t CountSuperfikWord(const std::vector & allWord, size_t startPosition, std::vector & superfiks) слишком длинное условие while не соблюден стиль кодирования также вместо "for (size_t i = 0; i < words.size(); i++) " можно использовать "for (auto & word : words)" для простого пробегания по строке image

опечатка в имени функции image

принято