Comments 5
Проделана отличная работа, поддерживаю способ обучения через деланье, при наличии хотя бы минимальной теоретической базы. Препод попался, походу, крутой — это хорошо.
Что касается статьи, мне кажется, гораздо ценнее было бы обрисовать общую архитектуру, схему взаимодействия участников сети. Скриншотики работы программы? А также, почему было самым трудным написать серверную часть, в чем возникли самые большие сложности?
Не лучше ли приведенный код положить на GitHub и оставить ссылку? Класс сервера написан, а где сама логика, которая реализует работу сервера? На чем реализован клиент, про него ни слова, а жаль? Могу ли я у себя взять и запустить, придется до этого еще додуматься?
Слова нисколько не дополняют код и не помогают в нем разобраться. Кажется, словесное описание работы функций лучше положить в python-docstring и хранить прямо в репозитории, а статью полностью освободить от этого. Тем более, что код некоторых функций говорит за себя не менее красноречиво, чем любые слова. Как считаете?
По поводу кода из очевидного я бы отметил, что в классе Log функция read_and_return_list не к месту, она больше относится к конфигурации, чем к логгированию. А также, рекомендую обратить внимание на встроенный модуль logging, который умеет очень много чего и явно сюда просится.
Что касается статьи, мне кажется, гораздо ценнее было бы обрисовать общую архитектуру, схему взаимодействия участников сети. Скриншотики работы программы? А также, почему было самым трудным написать серверную часть, в чем возникли самые большие сложности?
Не лучше ли приведенный код положить на GitHub и оставить ссылку? Класс сервера написан, а где сама логика, которая реализует работу сервера? На чем реализован клиент, про него ни слова, а жаль? Могу ли я у себя взять и запустить, придется до этого еще додуматься?
Слова нисколько не дополняют код и не помогают в нем разобраться. Кажется, словесное описание работы функций лучше положить в python-docstring и хранить прямо в репозитории, а статью полностью освободить от этого. Тем более, что код некоторых функций говорит за себя не менее красноречиво, чем любые слова. Как считаете?
По поводу кода из очевидного я бы отметил, что в классе Log функция read_and_return_list не к месту, она больше относится к конфигурации, чем к логгированию. А также, рекомендую обратить внимание на встроенный модуль logging, который умеет очень много чего и явно сюда просится.
Спасибо огромное за такой детальный разбор каждого элемента. Давно хотел разобраться в теме, и только благодаря подобным постам это получается сделать эффективнее всего.
Поддерживаю! Выложите на github, пожалуйста.
UFO just landed and posted this here
Sign up to leave a comment.
Простой P2P сервер на python