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

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

Новые блоги и статьи Все статьи Все блоги /
Часы электронные Uhbif79 12.08.2026 Выкладываю программу часов. Программа позволяет: 1. Использовать системное время и дату, 2. Есть возможность вводить время и дату вручную. 3. Реализованы 2 будильника: начало и конец рабочего дня. . . .	Часы с будильником на основе класса QLCDNumber Uhbif79 12.08.2026 Всем добрый день, выкладываю программу часов с будильником на основе класса QLCDNumber. Здесь я пробовал самостоятельно создавал классы, впервые столкнулся с видимостью переменной одного класса из. . .	Установка MinGW GCC 16.2 и CMake 8Observer8 10.08.2026 VK Видео: https:/ / vkvideo. ru/ video-240781534_456239017 YouTube: eY5-5PyI9NM Текстовая версия	Неделя из жизни имитационной модели склада: мои кривые руки растут, откуда надо anaschu 10.08.2026 Неделя из жизни имитационной модели склада: как я почти написал неправильную логику и что с этим делать Работаю сейчас над учебно-рабочим проектом: строю в AnyLogic имитационную модель процессов. . .
Калькулятор для расчета родства russiannick 07.08.2026 1. Задача: Создать калькулятор для расчета родства. Родственных связей существует 8 ступеней, такие как: p - отец P - мать q - муж Q - жена b - брат B - сестра s - сын S - дочь	Мир по моей воле kumehtar 07.08.2026 Когда-то кажется, что всё просто. Ты весь такой светлый. Причиняешь добро. Борешься за справедливость в этом тёмном мире. Потом начинаешь замечать одну неприятную вещь. Почти каждый хороший. . .	Кредитный калькулятор Maks 05.08.2026 Решение задачи по прикладной информатике средствами 1С. Задача: Напишите приложение-калькулятор, которое помогает рассчитывать параметры кредита для аннуитетного и дифференцированного видов. . .	У нас сейчас поговорку "Опять 25" нужно переделать на "Опять +35". kumehtar 04.08.2026 С ностальгией вспоминаю времена моего детства, когда у нас и правда +25 - была максимальная температура летом. Раньше +25 °C реально казались вершиной жары, когда можно было весь день пропадать на. . .

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

Опции темы