Товпеко Кирилл, 3530901/70201

[Ktlo]

Отчёт находится в ktlo/lab1/README.md.

[Ktlo]

Не получается поставить label. Возможно я что-то не так делаю, так как раньше их не ставил

[wrbbz]

Comments

• Overengineering -> плохая читабельность для того, что должно было быть простой программой
• Отсутствие кода, имеющего прямое отношение к предмету, в сдаваемой работе
• Использование самописных зависимостей (handtruth) для решения общих задач (логирование) -> страдающая поддержка/читабельность
• Важный код (в рамках данного предмета) вынесен в неправильно подключенные зависимости -> требует много времени на их поиск и чтение
• Относительное подключение git submodule -> страдающая воспроизводимость
• Исходный код не собирается. С помощью Docker - аналогично
• Dockerfile использует alpine:latest -> страдающая воспроизводимость
• Settings ответственен только за парсинг аргументов -> вводящие в заблуждение название

Need answers

• В описании протокола отсутствует описание того, что конфигурируется опцией pulse

To be continued ...

[Ktlo]

Я полагаю, что единственный вариант, как я могу исправить все комментарии -- это переписать всё. Вообще я ссылался на свою библиотеку и занимался оверинжинирингом, для того чтобы не наворотить ошибок связанных с асинхронным кодом. Также я надеялся, что код будет более понятный, если я буду использовать абстрактные connection, client, paket, gate, но видимо это только хуже сделало.

Сабмодули относительные были для того, чтобы по тем же путям на локальном сервере разработку вести, а потом просто зеркально разместить это на github. Про то, что я делаю мердж по другим путям я конечно же забыл и проверять не стал. Сборка вроде получается, если сабмодули на место поставить, если она даже так не происходит, то мне нужны подробности, так как у меня всё получилось на чистой системе.

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

Лучше тогда пока не смотрите вторую лабораторную, так как там все эти проблемы ещё хуже.

[Ktlo]

Опция pulse описана в аргументах программы. В описании протокола говорится о том, что необходимо переодически отправлять что-то, но слово pulse не было использовано.

[wrbbz]

Обращаю внимание, что это только комментарии.
Т.е. мы делимся наблюдениями, но конкретно по ним не требуем действий.
Очень хотелось бы увидеть программу поменьше, но переживать не стоит,
целиком переделывать эту не заставляем.

Про то, что требует реакции:
--pulse=PULSE период по окончанию которого подаётся следующий признак жизни
C нашей точки зрения "отправлять что-то" - часть сетевого взаимодействия.
То есть часть твоего протокола поверх TCP, нам это инересно,
так как эта лаба про то, что сразу поверх TCP.

Нужно описать, зачем пульс нужен и чем мотивируется его частота (ну и всегда ли он нужен)

[Ktlo]

Оказывается git submodules намного умнее чем я думал. Я могу сделать так, чтобы моя схема с локальным сервером продолжила работать и при этом проект собирался тут. Чудо не иначе. Поэтому я надеюсь, что он теперь будет собираться и у Вас без дополнительных действий.

Класс с аргументами переписан.

Dockerfile теперь ссылается на alpine:3.

Опция pulse описана подробнее.

Overengineering стал ещё хуже... Простите.

[wrbbz]

Overengineering - конечно, больно. И в данном случае он совсем не нужен. Но задание, в принципе сделано.
На будущее: не надо так