The OpenNET Project / Index page

[ новости /+++ | форум | теги | ]



"Выпуск p2p-мессенджера Communist 1.4"
Версия для распечатки Пред. тема | След. тема
Форум Разговоры, обсуждение новостей
Исходное сообщение [ Отслеживать ]
Присылайте удачные настройки в раздел примеров файлов конфигурации на WIKI.opennet.ru.
. "Выпуск p2p-мессенджера Communist 1.4" +1 +/
Сообщение от Аноним (-), 25-Июн-22, 17:00 
> Мои эмоции роли не играют.

Я намекал что патч с перепаханого по всей площади сорца - штука так себе. Вы сразу на старте дали челу рецепт надежного зарубания совместной работы над сорцом. А мне стало интересно зачем :)

> в том числе для того и выложен в открытый доступ.

Мне казалось что основная фича опенсорса это возможность совместно что-то делать. А если в один фэйс все фигарить, смысл открытия сорца получается не очень понятный если честно.

Впрочем, со своей стороны чего я вам мозг клюю? Чисто практически - я не вижу док на протокол и общий overview идеи, а вштырить в концепции и идеи по такому коду мне несколько напряжно. Хотя против форматирования я на самом деле не сильно возражаю, предпочитая чуть иной стиль но для меня вот именно это - переживаемо (как для оригинального вопрошающего хз). А вот общее структурирование и отсутствие док на то какой был план - уже таки проблема.

Ну вот например:

if (Netmode == "1" && ownips.size () == 0)

Как вы думаете, очевидно ли из этого кода что за Netmode равный 1? Вообще, даже сишники давно догадались что магических чисел быть не должно. Должны быть константы в человекочитаемом виде. Потому что искать 100500 килобайт назад где вон то определено... ух... нет, а даже эдитор в IDE не найдет в таком виде где это определено и что означает.

А чтобы было не совсем скучно, NetworkOperations::commOps () если не ошибаюсь более 1000 строк кода. Поэтому не сойти с ума не будучи его кодером... ну... лично в мой маленький мозг СТОЛЬКО кода в одной сущности не лезет. Я конечно понимаю что иногда кодишь на одном дыхании, поймав волну, но остальные понятия не имеют что у вас в голове - и без должного партиционирования задачи на более простые субзадачи врубиться становится просто малореально. И по-моему это говорит что подобную цацу давно надо было отрефакторить до чего-то человекочитаемого. Разбив на штук пять субкомпонентов, атомарных и с говорящими названиями.

Ответить | Правка | Наверх | Cообщить модератору

Оглавление
Выпуск p2p-мессенджера Communist 1.4, opennews, 25-Июн-22, 09:38  [смотреть все]
Форумы | Темы | Пред. тема | След. тема



Партнёры:
PostgresPro
Inferno Solutions
Hosting by Hoster.ru
Хостинг:

Закладки на сайте
Проследить за страницей
Created 1996-2024 by Maxim Chirkov
Добавить, Поддержать, Вебмастеру