И даже хуже, байтоебим

даже с 1С программистами - ценятся и получают деньги те кто умеют хоть что-то

это не просто "выучил синтаксис и рубишь бабло"

И даже хуже, байтоебим

Байтоебить дома надо под одеялом

Чо не использовать null-safe языки-то?

Я, правда, всего три таких знаю

Или четыре

Из них без GC - один

Идти куда-то за деньгами вообще смысла мало. Не в деньгах счастье ж

Ну, сами виноваты

Байтоебить дома надо под одеялом

Стойка не помещается под одеялом

Я пытался, честно

@vladimirsuper 150к можно зарабатывать уйдя в менеджеры и там развиваясь, можно быть инженером не совсем в IT - даже с банальной электроникой можно в МСК рубить деньги.

с электроникой сложно, прям ваще. в хардваре зарплаты в разы ниже

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

хотя с 1с тоже нужно головой думать, да

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

чтоб программировать нужно с компьютером не на "Вы" быть

с электроникой сложно, прям ваще. в хардваре зарплаты в разы ниже

ну у меня из одногруппников из тех с кем общаюсь, как-то люди устроились и получают за электронику (цифровая схемотехника в чистом виде у одного, Verilog/VHDL у двух других) нормальные деньги (один порядка 250 в месяц, двое других больше 150)

то есть в общем реализуемо, но требует планирования

Братва

А есть люди со свободным временем?

Сишнички?

Не глянете, может есть какие идеи по улучшению кода?

И все такие: "Да, братан, давай свой код, мы посмотрим!"

Ну лан, чо Вот он https://github.com/alexclear/metrics-c-library/blob/master/prometheus.c

Одна идея у меня есть, и я записал ее в issues

Впрочем, там глаз выхватывает какие-то повторяющиеся шаблоны кода Надо попробовать заэкстрактить и поглядеть, насколько ухудшится читаемость

@demeliorator цели кода какие в производительности ж?

https://github.com/alexclear/metrics-c-library/blob/master/prometheus.c#L265-L274 кажется что лучше так fprintf'ы не разбивать, когда это не нужно (а тут не очень нужно). И это в целом про все твои fprintfы.

@demeliorator цели кода какие в производительности ж?

Ну - цель кода метрички собирать в программах на C Кроме того, это часть учебного курса "Программирование в среде UNIX"

Завтра в классе дадим нагрузку поглядим, че там на топе будет в флеймграфах

Ну - цель кода метрички собирать в программах на C Кроме того, это часть учебного курса "Программирование в среде UNIX"

если цели нет, то и так сойдет.

напоминает попытки сделать ООП на сях в очередной раз

напоминает попытки сделать ООП на сях в очередной раз

Так а чего его там делать - первый параметр это указатель на структуру с данными, и будем вместо "функция" говорить "метод"

Правда, не будет сокрытия данных - да и хер с ним

Так а чего его там делать - первый параметр это указатель на структуру с данными, и будем вместо "функция" говорить "метод"

семантически проще читать будет

с формальной точки зрения ты не проверяешь что память выделилась ни на одном маллоке

с формальной точки зрения ты не проверяешь что память выделилась ни на одном маллоке

Да, всё так

с тем чтоб сделать thread-safe будут проблемы.

return'ов в функциях нет

то есть в ифе есть, а так просто нету

с тем чтоб сделать thread-safe будут проблемы.

Здесь нам повезло - nginx в один тред работает Но - надо бы что-то придумать с этим, да

return'ов в функциях нет

Да - и тут меня крайне удивляет компилятор C, который такое разрешает - WTF?

Да - и тут меня крайне удивляет компилятор C, который такое разрешает - WTF?

собери с -Wall -Wextra в гцц )

и ваще возьми ради интереса clang 3.9 с scan-build сделай

О, это дело, спасибо

такое разрешают не все компиляторы, большинство ругнется а вот как отрабоатет функция без ретурна - как бы в гцц по одному, а вот что в шланге как ты хочешь - никакой гарантии

strncpy((*counter).label_names[i], label_names[i], strlen(label_names[i]) + 1); - во всяких таких местах - очень странно нафига

67 строка

И то верно

Потому что туда фигачатся статические константы

вопрос - strncpy vs memcpy в данном случаи это раз

Но это надо тогда завести doxygen-овский коммент

во вторых strcpy vs strncpy

почему ты таки выбрал strncpy в таком виде?

В котором прописать, что память должна быть статически выделена, иначе пздц

почему ты таки выбрал strncpy в таком виде?

Ну, видимо, решил, раз строка - пусть будет strncpy

так почему ncpy а не просто cpy тогда?

все равно у тебя функция эта будет проверять налчие \0, хотя ты копируешь полностью строку

А это горе от ума в чистом виде

Я на примере этого объяснял коллегам, что важно чекать границы

Хотя - в данном конкретном случае это херня, я строкой выше все нормально выделяю

Ровно столько, сколько нужно

Ну лан, чо Вот он https://github.com/alexclear/metrics-c-library/blob/master/prometheus.c

мне, помимо "make this thread-safe", в глаз давит https://github.com/alexclear/metrics-c-library/blob/master/prometheus.c#L44 и после этого https://github.com/alexclear/metrics-c-library/blob/master/prometheus.c#L65. я не гуру, конечно, но зачем дважды присваивать переменной одно и то же значение? это же повторяется в new_histogram_vec, но не в initialize_buckets или observe_histogram.

мне, помимо "make this thread-safe", в глаз давит https://github.com/alexclear/metrics-c-library/blob/master/prometheus.c#L44 и после этого https://github.com/alexclear/metrics-c-library/blob/master/prometheus.c#L65. я не гуру, конечно, но зачем дважды присваивать переменной одно и то же значение? это же повторяется в new_histogram_vec, но не в initialize_buckets или observe_histogram.

Сишная идиоматика, надо в 44-й строке убрать иниациализацию, оставить только в цикле

тогда можно и в https://github.com/alexclear/metrics-c-library/blob/master/prometheus.c#L148 убрать

Хотя - в данном конкретном случае это херня, я строкой выше все нормально выделяю

это можно в функцию вытащить и сделать ее inline'ом (правда это потащит за собой -std=c99)

ERROR: S client not available

как минимум чтоб это читалось лучше

ну и не очень понятно нафига вот так вот, memcpy тут уместнее, ну либо strcpy без длины

coding style плавает слегка. "int *val", "int *", "int*" в буквально одной строчке - смотрится странно учитывая что только с указателями такая фигня периодически

где то ты маллоки кастуешь, где-то нет

Надо какой-нибудь линтер приделать

Пусть по рукам бьет

ИМАКС

ИМАКС

Не поверишь - с этого и начал

Но - в аудитории стоит прокся

А я чот не смог имакс настроить так, чтобы он аутентифицировался на ней

Туповат-с

@demeliorator таки ООП у тебя слегка напрашивается - хотя бы в ConcreteValue тот же, в котором у тебя value то один тип, то другой, смотря что происходит.

в export'е у тебя внутри цикла очень много чего напрашивается на разбиение на функции, кажется что ты все эти принты делаешь однотипно

Да - там надо посидеть подумать

в print_labeled_metric тоже кажется почти калькой с export'а

Как это перефигачить так, чтобы было попроще читать

только с fprintf вместо кастомной функции

в print_labeled_metric тоже кажется почти калькой с export'а

Да - на это есть issue #1

Сделать эту самую кастомную функцию параметром

это один и тот же issue

с тем что и цикл обобщить и упростить

Угу

В этой части, видимо, можно серьезно упростить

так в общем судить сложно - подчистить, а дальше под нагрузкой профилировать и смотреть в чем проблемы вылезут уже

Угу Так и сделаем Спасибо!

ПИТНИЦА !!

Угу Так и сделаем Спасибо!

Нормальный сишник что нибудь еще найдет вероятно

https://spb.hh.ru/vacancy/18806310

Да они там могут писать хоть 25круб. Но либо никто не придет, либо будут приходить олени.

Да они там могут писать хоть 25круб. Но либо никто не придет, либо будут приходить олени.

сбертех так не думает и набрал людей с сертами ccnp на 37-50к за пол года, заполнив отдел

сбертех так не думает и набрал людей с сертами ccnp на 37-50к за пол года, заполнив отдел

И? Ты реально думаешь что они там что-то знают, а не вызубрили?

И? Ты реально думаешь что они там что-то знают, а не вызубрили?

учитывая, что там был конкурс, и сдавали работы, да

Как это перефигачить так, чтобы было попроще читать

Я бы ещё многое в макросы вынес. Ту же аллокацию.