Какой open source проект написан на труъ Си++?

Question

[SkyDatter]

Пол жизни писал для микроконтроллеров и на ассемблере. Сейчас перевели на кросс-платформенный проект, где всё "нужно писать на си++". И теперь сверху жалуются, что мой код "слишком сишный", то есть написан на Си без плюсов, хоть я и использую классы. Проблема в том, что никто из коллег на работе не может нормально объяснить, как, собственно должен выглядеть "плюсовый" код.

Посоветуйте пожалуйста какой-нибудь не очень сложный open source проект, написанный на "труъ с++"? "Не очень сложный" не обязательно - но как правило "большие" проекты смешивают в себе немало парадигм и стилей. Или посоветуйте библиотеку, на код и примеры использования которой можно посмотреть и сказать, что это - настоящий С++.

Comments

[Victor Bomberow]

как, собственно должен выглядеть "плюсовый" код

Красиво.

[Alex XYZ]

А почему вы спрашиваете про сторонний проект, хотя текущий проект на самом C++ написан? В чем отличие вашего кода от того, который в проекте?

[RabraBabr]

И теперь сверху жалуются, что мой код "слишком сишный", то есть написан на Си без плюсов, хоть я и использую классы. Проблема в том, что никто из коллег на работе не может нормально объяснить, как, собственно должен выглядеть "плюсовый" код.

Давайте так. C и C++ два совершенно разных языка между которыми нету ничего общего. Ну кроме синтаксиса. C++ язык общего назначения является по сути наследником Simula 67 с синтаксисом C. Базовый синтаксис C с незначительными расширениями образует kernel language. Используя его совместно с механизмом классов вы получаете возможность программировать с использованием абстрактных типов данных. Очевидно "плюсовый" код должен выглядеть как код использующий АТД.

Примеры.

Типичный c стайл код:

spoiler

int j = 0;
    char s[] = "hello\n";
    char ch;
    
    while (s[j])
    {
        ch = s[j];
        putchar(toupper(ch));
        j++;
    }

С++ код:

spoiler

std::string s("hello");
    std::transform(s.begin(), s.end(), s.begin(),
                   [](unsigned char c) -> unsigned char { return std::toupper(c); });

Answer 1

[Adamos]

Видимо, вы используете классы, но не используете ООП.
Пишете плоско-процедурно, без выделения абстракций.
Вам бы лучше на Java какой-нибудь опенсурс посмотреть. Ну, или почитать "Рефакторинг" Фаулера с примерами на той же джаве. Для смены парадигмы.

Answer 2

[Wataru]

Ну, посмотрите хотя бы хромиум. Там жесткий стиль и хоть качество кода не равномерное, в основном это точно не "Си с классами".

Но вряд ли просмотр стороннего кода поможет вам понять, что именно коллегам не нравится в вашем коде. Тут надо все-таки коллег допытываться. Смотреть на ими написанный код.

Если постоянные жалобы, то устройте код ревью в команде. Пусть ваш код хотя бы пару неделек кто-то постоянно будет проверять перед комитом и выдавать конкретные правки (а вообще код-ревью хорошо использовать всегда. Тот же github дает возможность это делать. Или есть опен сорсный gerrit).

Возможно, вы используете классы только как структуры максимум с какими-то тривиальными действиями (типа get_sum, get_value, set_value и т.д.). Когда как в ООП объекты должны инкапсулировать в себе логику и уметь делать нетривиальные вещи. Сама программа должна состоять из взаимодействующих объектов.

Еще использование stl. Стоит избегать массивов - используются std::vector. Так же вместо char* стоит использовать std::string. Ну и там куча алгоритмов есть: от выбора максимума в массиве до сортировки.

Answer 3

[rPman]

можно какой-нибудь ярко выраженный пример кода, на который жалуются коллеги?

могу предположить
вместе с c++ обычно идут библиотеки std, даже не так, можно printf а можно cout, можно со строками работать, ковыряясь с байтами самостоятельно, strccpy, while(*buf) и т.п. а можно пользоваться std:string, кучей готовых реализаций (и сильно неэффективного кода но с меньшими шансами на ошибки)

Answer 4

[Мирон]

Good example with templates and patterns.
One of a few projects where modules carefully encapsulate internals and expose a finely tuned publicly accessible interface
https://github.com/apache/arrow.git

Answer 5

[mayton2019]

Полностью согласен с необоснованностью претензий. Те кто делали код-ревью и отметили что код слишком сишный должны писать конкретные code-review points и аргументировать почему здесь надо затащить классы и ООП. Есть масса продуктов (git) написанных на С и ни у кого не возникает вопросов из серии почему мало ООП. Сколько надо ООП на квадратный метр? Килограмм?

Нет смысла также кидаться в гитхаб и искать там правильные TrueЪ примеры. Там тоже не боги горшки обжигают. Кроме того С++ - это не только ООП, это мультипарадигменный язык. Тоесть там будут где надо виртуальные вызовы а где надо лямбды или процессор шаблонов и только богу известно почему автор решил здесь так или эдак.

Comments

[Мирон]

не упоминайте всуе, простота си обманчива.
Типичный случай применения "template" тривиален, все множества с сочленениями через указатель имеют выбор между си - свободный указатель и указатель с жестким определением типа данных.
Каждый из этих частных случаев на си (без ухищрений с struct) требует полного описания, но со struct не дает транслятору полного доступа к инструкциям, что ведет к не запланированным прерываниям во время выполнения.
Template освобождает конструктора от повторения линий инструкций, а транслятор получает доступ ко всем деталям инструкций по, проверяя их на возможные ошибки во время сборки.

[mayton2019]

Мирон, этот забавный технический артефакт, который вы описали тем не менее не является основанием к переписыванию приложения. Поскольку вы ссылаетесь на performance issue (PI), то данный issue надо бенчмаркать и делать justification.

Если у вас есть ссылка на github или другой источник который моделирует проблему - то буду признателен.

[Мирон]

mayton2019,

может Вы имели ввиду другой комментарий?
у меня даже слово performance нет в комментарии, во всяком случае в нашей череде мнений

Answer 6

[Drno]

Если коллегам ненравится что то - пусть объяснят и покажут что.
В иных случая - неипёт