интересное решение.
Проблема: в Qt есть конструктор QByteArray(int size, char c);, который создаёт "массив" размером size байт со значением c. Проблема в том, что регулярно находятся разработчики, которые путают аргументы и конструируют какие-нибудь QByteArray(0, 32).
Сначала я придумал предложить в Qt добавить
QByteArray(int size, int mustBeChar) = delete;
. (Понятно, что это source incompatible change и всё-такое.)
А сейчас у меня появилась (<подставьте прилагательное>) идея использования ключевого слова explicit в аргументах для запрета неявного преобразования.
Пример —
QByteArray(int size, explicit char c);
Что скажете?
на ревью разворачивать ?
кажется, тут уже кто-то имел такую идею
Тут надо сделать enum class byte_t и сделать перегрузку с (size_t , byte_t)
Увы, в Qt API иногда не так сильно продумано.
QByteArray(32, uunt8_t(’A’));
Задача не специфична для QByteArray. У std::string ровно та же проблема, но хорошего решения как починить не придумали. Есть https://clang.llvm.org/extra/clang-tidy/checks/bugprone-string-constructor.html
Почему size это инт
Но аргумент функции и так не допускает explicit
Обсуждают сегодня