Задать вопрос

Насколько отвратителен данный код?

Здравствуйте. Насколько отвратителен данный код? На работоспособность пока что не проверял, просто хотелось бы узнать Ваше мнение.
bool update_uwsgi_config() {
         const char *source = fmt::format("/tmp/{}.ini", unid).c_str();
         const char *on_restart = "service uwsgi restart";
         const char *dest = "/etc/uwsgi/apps-available/www.ini";
         const char *on_update = fmt::format("lxc file push {} {} {}", source,contname,dest).c_str();
         system(on_update);
         system(on_restart);
         remove(dest);
         return true;
     }
  • Вопрос задан
  • 238 просмотров
Подписаться 1 Простой Комментировать
Ответ пользователя devalone К ответам на вопрос (3)
devalone
@devalone
̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻̻
- Не кросплатформерно(хотя вряд ли она тут нужна, если используется lxc)
- если contname или unid принимаются от пользователя, то это серьёзная уязвимость
- зачем лишняя зависимость, если форматирование есть в std::?
- константы лучше выносить в конфиг, удобнее редактировать потом

Да и такое обычно пишут на python/bash, если конечно это не часть проекта, который уже на C++
Ответ написан
Комментировать