Open oMystique opened 8 years ago
rem Проверим пустой файл
%PROGRAM% emptystring.txt %TEMP%\empty.txt "some string" "some another string"
if %ERRORLEVEL% NEQ 1 goto err
Этот тест упадёт сразу же, ибо программа вернет нулевой код возврата. Следует доработать тесты. Ссылки на материалы выше. Пока k=0.7
.gitignore
продукты сборки C++ проектов. Правильный.gitignore
можно найти в репозитории у Алексея Малова.c++ unsigned int currPosition = 0;
Следует более внимательно подбирать сокращения для имен переменных. В данном случае curr == ток? Наврятли именно это имелось в виду. :) Так же, следует объявлять переменные как можно ближе к месту использования. Да, и для хранения позиции индекса строки лучше использоватьsize_t
.std::string(argv[3]).length()
вполне можно использоватьstrlen(argv[3])
textPosition
лучше подобрать другое название, из которого было бы понятно позицию чего конкретно сохраняют в эту переменную.Лучше выделить поиск и замену строки в отдельную функцию.
Тесты
Следует сравнивать полученный результат с ожидаемым результатом, а не входным файлом.
set PROGRAM="OOP lab1 replace.exe"
Не стоит в автоматических тестах явно присваивать переменнойPROGRAM
путь к exe-шнику. Лучше этот путь передать с первым параметром командной строки.SET program="%1"
В репозитории Алексея Малова располагается пример с автоматическими bat-текстами для одного из заданий 1 лабораторной работы. Так же, нужно ознакомиться с пояснениями, доступными по ссылке - https://github.com/alexey-malov/oop/wiki/%D0%9F%D0%BE%D1%8F%D1%81%D0%BD%D0%B5%D0%BD%D0%B8%D1%8F-%D0%BF%D0%BE-1-%D0%BB%D0%B0%D0%B1%D0%BE%D1%80%D0%B0%D1%82%D0%BE%D1%80%D0%BD%D0%BE%D0%B9-%D1%80%D0%B0%D0%B1%D0%BE%D1%82%D0%B5На будущее - проектам не стоит давать имена, содержащие пробельные символы.