Рисование графиков в С++

Karim, ааа точно. В школе же проходил

Karim, вы сами на писали движок для рисования или где-то взяли? Если «где-то» — не поделитесь ссылочкой?

Unworldly, интересно узнать, в чем заключается жесть, т.к. я самоучка и опыта работе в команде программистов у меня почти нет.

elektroyar, первая жесть — это потенциально «размножающиеся» static-переменные в заголовочном файле, особенно mutex. Если я разобью свою программу на несколько модулей (файлов), каждый из которых включит заголовочный файл библиотеки, то у меня появится несколько mutex'ов, по одному на каждый модуль. Всё, защита от race condition сломана.

Добиться того, чтобы mutex все равно был только один, можно, вот пример с несколькими модулями, где показано, что из разных модулей в вашем коде получается разный адрес mutex'а, то есть, их там много.

А также дана правильная реализация, чтобы адрес был один на программу, то есть, чтобы mutex был единственным.

Вторая жесть — это то, что код не является exception-safe там, где это жизненно необходимо.

Вы защищаете mutex'ом изменения drawings. Перед изменением drawings соответствующий mutex lock'чится, после изменения — раз'lock'чивается назад. Как бы, всё хорошо. Но между этими двумя моментами вызываются функции push_back и make_shared, каждая из которых может выбросить исключение. Я, в коде своей программы, могу отловить исключения и продолжить работу дальше. Но, вот, только библиотечный код при этом не раз'lock'чит mutex, и при попытке повторно его за'lock'чить всё повиснет. Для избегания таких вещей в стандартной библиотеке специально имеется lock_guard, который раз'lock'чит mutex при любом развитии событий.

Третья жесть — вы вообще прогоняли свой же тест?
Он не падает где-то на 33-35 строке?

В функцию с переменным числом аргументов передаёте сами объекты, а в самой функции вычитываете их адреса. Но передали-то объекты, а не их адреса. Естественно, всё падает.

В функцию с переменным числом аргументов можно передавать только POD-типы, а вы аж вектор туда запихиваете. И LineSpec не является POD-типом. Чтобы он им стал, требуется оттуда выкинуть всё, что мешает скомпилировать его в pure C. Вам хочется использовать значения по умолчанию, но это можно сделать и с POD-типом, применив вспомогательную функцию. Вот набросок на эту тему. Значение последнего параметра берётся по умолчанию.

Вот здесь рекомендуют вместо функции с переменным числом аргументов использовать variadic templates или initializer_list. Но для этого, конечно, нужно владеть «шаблонной магией» хотя бы на среднем уровне.

Набросок решения с использованием variadic templates — здесь (общее количество пар заранее неизвестно).

Набросок решения с использованием initializer_list — здесь (общее количество пар в данном случае будет заранее известно).

В целом, остальное, навскидку, жестью не является, но очень много особенностей построения кода говорят о непонимании множества вещей.

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