Оцените сорцы glogg-а

RazrFalcon

или он полон говнокода/костылей/быдлокода? С поправкой на то что сейчас в крестах фичи новые есть моднявые.

Типичный легаси код, на который всем пох.

Например, минимальная версия:

GCC version 4.8.0 or later

А ещё есть костыль для gcc 4.6.3: https://github.com/nickbnf/glogg/blob/master/src/abstractlogview.cpp#L59-L95

Большинство issues просто проигнорировано. Нет ответа от мейтейнера, считай мёртвый проект.

Забыто обновление минимальной версии: https://github.com/nickbnf/glogg/issues/249

Хз что за программа и как она работает, может и хорошо, но проект заброшен.

https://github.com/nickbnf/glogg/blame/e191a6372ce0efc66787e543b1949479c1e79a59/src/abstractlogview.cpp#L59-L95

это было написано ДЕВЯТЬ! лет назад, если что.

То что заброшен это понятно, большинство issues это новые фичи народ просит. Костыль видел, но он на то и костыль. Вопрос именно в организации кода на крестах. Говнокод он даже не про баги, а про то как сделать марсианскую структуру проекта понятную только по полнолуниям и с определённой концентрацией алкоголя в крови.

Ну там большая часть кода вообще C++03. Возможно вас это и привлекло.

Единственный говнокод там, это факт того, что автор не следует Qt Code Style.

Из странного: раз и два (не ясно зачем тут chrono).

Никакого обилия комментариев я не увидел.

не следует Qt Code Style

а оно надо?

В Qt проекте - да.

Никакого обилия комментариев я не увидел.

посмотри на сорцы того же Inkscape или даже SDL2 (хотя он на си и чуть лучше) и поймешь как тут хорошо с комментариями (в этих проектах огромные блоки кода без комментариев вообще и местами переменные вида var1/a/b/temp). То что я видел в виде проприетарщины это вообще кровавые слёзы. Хорошо если на один файл есть 5 строчек комментов.

C++03

Да мне он нравился больше, чем 11 и тем более больше чем то что сейчас принято лепить, когда шаблон в шаблон и всякие unique_ptr без конца (хоть auto_ptr убрали в 17 версии и на том спасибо и вообще в 17 версии говно выкинули из языка, а в 20 опять больше завезли, чем выкинули), бесконечные cast-ы и ехал auto через auto. Для динамической типизации есть специально заточенные под неё языки, не надо портить читабельность в ЯП которые были изначально без auto, ИМХО.

поймешь как тут хорошо с комментариями

Ну по моим меркам очень мало.

всякие unique_ptr без конца

Ну как бы иначе никак.

Для динамической типизации есть специально заточенные под неё языки

auto

auto - это вывод типа, а не дин. типизация. Без auto жизни нет.

Для динамической типизации есть специально заточенные под неё языки, не надо портить читабельность в ЯП которые были изначально без auto, ИМХО.

auto - это не динамическая типизация.

auto - это вывод типа

Сложно int и double писать или название класса, которое помогает в читабельности кода? Правильно что софт на крестах стал гораздо чаще падать и всякие пивас-студии не помогают толком даже. Как раз когда там лагучий крайзис вышел 11 кресты и подъехали.

Ну по моим меркам очень мало.

А где много? Очень хочу посмотреть на это.

Ну как бы иначе никак.

Отчасти да.

Правильно что софт на крестах стал гораздо чаще падать и всякие пивас-студии не помогают толком даже.

Я хоть и C/C++-хейтер, но это полнейший тупняк. auto никак не влияет на стабильность работы проги. Да, есть edge cases, это же сишк, как ни как. Но ничего критичного. В том же Rust тот же самый auto и всё ок.

А где много?

Очевидный Qt. Ещё у Skia отличная дока. Там тоже есть простыни без комментариев, но много и такого кода.

Сложно int и double писать или название класса, которое помогает в читабельности кода?

IDE пишет все типы, их всегда видно. Пользователи блокнотов должны страдать.

auto защищает от неявных преобразований, и заставляет всегда инициализировать переменную. Этого достаточно, чтобы всегда писать auto.

auto защищает от неявных преобразований

Шта? Implicit cast ещё не выпилили.

Шта? Implicit cast ещё не выпилили.

и? как implicit cast будет задействован при auto?

https://gcc.godbolt.org/z/MhzP7s

а оно надо?

Там какой-то жуткий стиль ставить пробелы после открытой скобки и перед закрытой скобкой. Зачем так делать непонятно. Стиль Qt лучше.

std::optional<int> f1() { return 1; }
std::optional<double> f2() { return 2; }

int main()
{
    auto v0 = f1();
    std::cout << v0.value() << std::endl; // 1
    v0 = f2();
    std::cout << v0.value() << std::endl; // 2
}

PS: вы точно на плюсах пишите код, или просто на лоре вбрасываете?

v0 = f2();

тут нету auto, тут обычное присваивание.

Тогда потрудитесь объяснить что значит «auto защищает от неявных преобразований».

Тогда потрудитесь объяснить что значит «auto защищает от неявных преобразований».

Type a = что-то;// "что-то" может иметь тип не Type, но будет попытка преобразования "что-то" в Type, и если будет успешно, то не будет никакой ошибки, и компиляция пойдёт дальше.

auto b = что-то; // тип b равен типу std::decay_t<decltype("что-то")>

Сложно писать какой нить typical::iterator<Outer::Inner::value<double>, Policy<Lazy, Resumable<Coroutines<Stackfull>>> it = container.begin();. А ещё сложнее рефакторить.

раз тебе делать нехрен, погляди на код LibreOffice, тебе понравится

См. пример выше.

Можно написать свой wrapper, который будет пресекать неявные преобразования…

https://gcc.godbolt.org/z/16EYx3

А можно использовать нормальный яп.

Но зачем, когда auto отлично работает?

когда шаблон в шаблон

Шаблоны нужны для статического полиморфизма. Чем больше информации получится вынести в мету, тем лучше. Пока не было внятного constexpr и концептов, приходилось пользоваться шаблонами.

а в 20 опять больше завезли, чем выкинули

И правильно сделали. Завезли то, что нужно было завезти еще 10 лет назад. Будь это завезено тогда, не пришлось бы выкидывать то, что выкидывают сейчас – оно просто было бы не нужно.

бесконечные cast-ы и ехал auto через auto

Касты – моветон, если не считать move/forward. auto с одной стороны делает код более полиморфным, избавляет от ненужных приведений типов, и просто упрощает жизнь.