Допилил опенсорс проект, если есть желающие отревьювить - буду рад

@_Ivana · Регистрация: 01.03.2013

Студворк — интернет-сервис помощи студентам

Я не настоящий си-плюс-плюсник, просто ~~маску нашел~~ проект, который захотел доработать, оказался на С++

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

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

И т.п.

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

Собственно, мои доработки проекта https://github.com/Ivana-/kilogram-desktop

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

C++
1
// kg

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

C++
1
2
3
4
// kg begin
.......
.......
// kg end

@ablex · 02.07.2024, 18:53

Не по теме:

(Сейчас я тоже похоже попаду в чёрный список, если уже не), но...
автор, похоже, лукавит, что будет рад критике, судя по тому как он её воспринимает от Рыжего Лиса.

А между прочим все здраво Лис пишет... только вот поздновато конечно, дело-то уже сделано. Однако это НЕ повод, не повод, как по мне, так остро реагировать на хорошие замечания.

@_Ivana · 02.07.2024, 19:03 **[ТС]**

Не по теме:

0. Для попадания в ЧС надо постараться, поста выше явно недостаточно
1. Я тоже отметил ссылку от него на git blame как полезную и здравую
2. Стиль моих ответов в данном случае полностью определяется стилем его комментов, так что имхо все заслуженно

@ablex · 02.07.2024, 19:14

Не по теме:

Вот щас точно в ЧС попаду. Но я правду пишу, а правду говорить - благое дело :)
Так вот...
Автор сам же своим упрямством спровоцировал такой стиль комментов. По теме прослеживается. Сначала Лис нормально писал, а "ехидство" в ответы стал добавлять только после авторовых возражений. Как-то так.

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

@Royal_X · 02.07.2024, 19:22

Не по теме:

Сообщение от ablex

Вот щас точно в ЧС попаду.

ну и попадешь туда и что с того? я уже там больше года, може уже пару лет, а ведь всё у меня отлично)))

@_Ivana · 02.07.2024, 19:25 **[ТС]**

Не по теме:

Я только приветствую правду :) И обратную связь.

В ваших словах здравая мысль, признаЮ.
И, хотя мне не показалось, что "Сначала Лис нормально писал", но я действительно не хотел бы разбирать здесь сейчас кто что и как воспринял. Из конструктивного, Лис написал как надо было организовать и оформить проект, чтобы он (и, возможно, многие другие) стали смотреть его код. Ок, спасибо, принял к сведению.

Но тенденция радует - первые комменты были совсем ни о чем, потом уже с чем-то содержательным, и далее степень содержательности только росла :)

@KSergey9 · 03.07.2024, 07:50

Сообщение от _Ivana

Кто-то не умеет читать и видит слово "конкурс" на месте слова "курс"

Прошу прощения у тех, кого ввёл в заблуждение

@Catstail · 03.07.2024, 09:40

_Ivana, Приветствую! Давно Вас не было...

@_Ivana · 03.07.2024, 16:13 **[ТС]**

Catstail, встречно приветствую! Работал, не было повода и мотивации заходить на форум. А сейчас давно балду пинаю, со скуки Телеграмы напильником дорабатываю, а там оказался С++ и появился повод написать

ЗЫ меня больше Байт беспокоит своим отсутствием

@Catstail · 03.07.2024, 17:53

_Ivana, меня тоже.

Новые блоги и статьи Все статьи Все блоги /
Nekobox - outbounds[0].transport: unknown transport type: raw damix 01.10.2026 Фикс ошибки Правым кликом по серверу -> отладочная информация -> edit Заменить "net": "raw", на "net": "tcp", Нажать кнопку reload.	Программный домашний кинотеатр russiannick 27.09.2026 Сподобился на программный домашний кинотеатр. В качестве ЯВУ по традиции выбрал js. В помощники взял Яндекс-Алису. Было создано три зала на разные интересы. исторические и ретро сериал Хичкок. . .	Беседа с ИИ о программистах, недопускающих к созданию и правке кода генеративные ИИ и причины этого zorxor 21.09.2026 Раньше я радовался или получал некоторые эмоции, пусть небольшие, но всё же, от самого процесса написания кода, рекомпиляции и запуска, видя постепенное развитие программы и прочее. А теперь лень. . .	Мобильное приложение ColorStep pavlinmavlin 17.09.2026 Реализовал приложение Красный, Зеленый, Синий в Unity3d + c#. Название изменил на ColorStep. Приложение прошло модерацию и теперь доступно для скачивания. Делал его сам, шаг за шагом — и вот,. . .
Запрет дублирования строк в табличной части Maks 13.09.2026 Реализация из решения ниже выполнена на нетиповом справочнике "Нормы ТО" с табличной часть "Виды ТО", разработанного в КА2, со следующими реквизитами: - ВидТО (СправочникСсылка. ВидыТО); - ВидГСМ. . .	Скрипты Tampermonkey для CyberForum, ChatGPT, Claude и пр. Jin X 06.09.2026 Скрипты Tampermonkey для CyberForum, ChatGPT, Claude и пр. Работая с форумом и нейросетями в браузере часто хочется что-то подкорректировать или добавить какого-то функционала. Ниже прикреплён. . .	Программа опроса у.з. расходомера SLS-720F Argus19 02.09.2026 Программа опроса у. з. расходомера SLS-720F Программа опрашивает один раз в минуту три ультразвуковых расходомера SLS-720F через интерфейс RS-485 по протоколу Modbus RTU. Опрашиваются регистры. . .	Hyper-V: Компьютер должен поддерживать доверенный платформенный модуль 2.0. Maks 31.08.2026 При установке Windows 11 на виртуальную машину Hyper-V 2-го поколения вылезла такая ошибка: Решение: в параметрах виртуальной машины, в разделе "Безопасность" (Security) активировать флаг. . .

@ablex
	02.07.2024, 18:53
	Не по теме: (Сейчас я тоже похоже попаду в чёрный список, если уже не), но... автор, похоже, лукавит, что будет рад критике, судя по тому как он её воспринимает от Рыжего Лиса. А между прочим все здраво Лис пишет... только вот поздновато конечно, дело-то уже сделано. Однако это НЕ повод, не повод, как по мне, так остро реагировать на хорошие замечания. 0

@_Ivana
	02.07.2024, 19:03 [ТС]
	Не по теме: 0. Для попадания в ЧС надо постараться, поста выше явно недостаточно 1. Я тоже отметил ссылку от него на git blame как полезную и здравую 2. Стиль моих ответов в данном случае полностью определяется стилем его комментов, так что имхо все заслуженно 0

@ablex
	02.07.2024, 19:14
	Не по теме: Вот щас точно в ЧС попаду. Но я правду пишу, а правду говорить - благое дело :) Так вот... Автор сам же своим упрямством спровоцировал такой стиль комментов. По теме прослеживается. Сначала Лис нормально писал, а "ехидство" в ответы стал добавлять только после авторовых возражений. Как-то так. Понятно, что автор взрослый мальчик и так просто не поведется на такое. Но все-таки. Если пришел просить критики, то эго надо подальше спрятать, на время хотябы, пока не станет понятна суть критики. 0

@_Ivana
	02.07.2024, 19:25 [ТС]
	Не по теме: Я только приветствую правду :) И обратную связь. В ваших словах здравая мысль, признаЮ. И, хотя мне не показалось, что "Сначала Лис нормально писал", но я действительно не хотел бы разбирать здесь сейчас кто что и как воспринял. Из конструктивного, Лис написал как надо было организовать и оформить проект, чтобы он (и, возможно, многие другие) стали смотреть его код. Ок, спасибо, принял к сведению. Но тенденция радует - первые комменты были совсем ни о чем, потом уже с чем-то содержательным, и далее степень содержательности только росла :) 0

@Catstail Супер-модератор 38223 / 21155 / 4314 Регистрация: 12.02.2012 Сообщений: 34,769 Записей в блоге: 14
	03.07.2024, 09:40
	_Ivana, Приветствую! Давно Вас не было... 0

@_Ivana 4949 / 2289 / 287 Регистрация: 01.03.2013 Сообщений: 5,991 Записей в блоге: 32
	03.07.2024, 16:13 [ТС]
	Catstail, встречно приветствую! Работал, не было повода и мотивации заходить на форум. А сейчас давно балду пинаю, со скуки Телеграмы напильником дорабатываю, а там оказался С++ и появился повод написать ЗЫ меня больше Байт беспокоит своим отсутствием 0

@Catstail Супер-модератор 38223 / 21155 / 4314 Регистрация: 12.02.2012 Сообщений: 34,769 Записей в блоге: 14
	03.07.2024, 17:53
	_Ivana, меня тоже. 0