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

@_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, меня тоже.

Новые блоги и статьи Все статьи Все блоги /
Доктрина интенционального знания - Доктрина для портала "Срез". Hrethgir 25.07.2026 Может найдётся кто захочет оценить доктрину. . . Написания правил участия для меня роскошь, требующая лимита времени, поэтому все сообщения не прошедшие модерацию будут видны только участникам портала,. . .	сукцессия 44. Решил подать на припринт в межународные сервисы препринтов. Но нужно одобрение от ученых anaschu 25.07.2026 Английский вариант. Пока кто то не одобрит мою личность, мне не получиться это опубликовать на препринте. Но заявку на публикацию статьи я сегодня подам.	сукцессия 43. Вторая научная статья за месяц- прайминг и гатгил anaschu 25.07.2026 две стороны одной монеты	Более приземисто - Эстафету хвоста в .cdl (деревья эстафеты в сад). Hrethgir 24.07.2026 В будущем, после написания блока инверсии обхода дерева (эстафеты хвоста), я планирую вернуться к нашему прошлому разговору о том, обладают ли знания целеполаганием. Тогда я пришел к выводу, что. . .
Вот представьте что вам дали бессмертие. kumehtar 24.07.2026 Вот представьте что вам дали бессмертие, ничего более не меняя. Вообще ничего, только бессмертие в нынешнем виде. Рады были бы? Что бы вы тут делали всё это время? Никакой пенсии. Никакого нового. . .	сукцессия 41 anaschu 24.07.2026 Численная верификация бифуркации в агентной модели лесной сукцессии: от одного параметра к ансамблю Автор: пользователь @Shumilov_AS \| Раздел: Прикладная математика / Численные методы Кратко. . .	сукцессия 40. Ансамблевая кластерная параметризаци, часть 1. anaschu 24.07.2026 Пр# Сопровождение научной статьи ИИ-ассистентом: подготовка публикации и калибровка агентно-ориентированной модели сукцессии микоризных систем Полевые заметки о двухнедельной совместной работе. . .	Теория всего 12. ВГК на планете в стратегической игре "терра" anaschu 21.07.2026 ### Главные семантические изменения и дешифровка новой физики 1. `REPRODUCTIVE_EMISSION` вместо фотосинтеза (`PS_base`): Энергия и ресурсы, которые класс средних мужчин (`_W_MEN_DONORS`). . .

@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 Супер-модератор 38210 / 21143 / 4313 Регистрация: 12.02.2012 Сообщений: 34,757 Записей в блоге: 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 Супер-модератор 38210 / 21143 / 4313 Регистрация: 12.02.2012 Сообщений: 34,757 Записей в блоге: 14
	03.07.2024, 17:53
	_Ivana, меня тоже. 0