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

[eligantRU]

• [x] Добавить файлы .vsproj, .sln в git

• [x] Не пишем комментарии к заданию(тем более на русском)

• [x] Из сообщения о недостаточном числе аргументов не следует то, как правильно использовать программу

• [x] Страдает ридабилити

      ofstream output(argv[argc - 1], ios::binary);

• [x] Подправить сообщение об ошибке

          cout << "Output file dir not found" << endl;

• [x] cout - для результата работы, cerr - для ошибок

• [x] fstream в своём деструкторе вызывает метод close() - нет смысла это делать самостоятельно перед его гибелью

• [x] При завершении программы/функции с ошибкой, не следует возвращать код 0

  exit(0);
  return 0;

• [x] Использование в тестах программу, которую тестируешь - опасненько

• [x] В тестах нет сравнения ожидаемого результата с действительным

• [x] Соглашение о кодировании:

  char * ch = nullptr;
  main() - простая функция, не следует в ней делать какие-то сложные действия

• [x] Подумать, как обойтись без оператора new

[eligantRU]

• [x] boost::optional или std::optional

[eligantRU]

• [x] Неявные преобразования bool к int

[eligantRU]

• [ ] Транслит в тестах

• [ ] Начни пользоваться ConEmu(хотя бы FAR поставь)

• [ ] Для запуска тестов не надо копировать *.exe'шник, либо оформи в Post-Build Event + запуск тестов сразу после того

[eligantRU]

= 1.0 0.7 + 0.4 0.2

[eligantRU]

• [ ] Выправить английский язык
• [ ] numberOffArguments типа int, из Int-1 можем получит отрицательное число, с которым size_t не ладит
• [ ] Не обязательно не найден, может быть занят другим процессом

  throw invalid_argument("Input file '" + inputName + "' not found");

const size_t outputFileIndex = numberOffArguments - 1;

[eligantRU]

• [ ] Тесты недостаточны