Цитата:
Сообщение от Neil
|
Отлично! Молодец что учитываешь советы и стараешься развиваться.
Если хочешь улучшить написание кода и нужны комментарии, то вот, что заметил.
Не храни всю логику в одном файле формы. Старайся разделять на логические блоки. На классы, так, чтобы каждый класс был вполне себе понятной логической единицей.
В качестве именования в шарпах в основном используют CamelCase нотацию (но не всегда конечно).
Отсутствие комментариев в коде это нормально, кто бы что ни говорил. Идеальный (которого не существует, кек) код понятно без комментариев благодаря стилю написания и именованию переменных и блоков.
Лучше бы вместо массив байт использовать класс с 4 октетами для описания айпи. И переопределить операции для работы с этим классом.
В методе
byte[] Ocifrovka_IP(string stroka_s_ip) незачем использовать регулярку, тем более она в очень общих чертах написана. У тебя дальше по коду есть
IPAddress.TryParse(stroka_s_ip, out ip_adres), как раз с помощью него лучше откинуть неподходящие варианты.
Что вызывает вопросы в коде без ответов:
Не понятно, зачем столько раз делаешь битовый сдвиг вправо для позиции контрола с надписью.
Берёшь ширину групбокса, вычитаешь ширину надписи, делаешь битовый сдвиг вправо и это будет позицией надписи?
https://github.com/Intern-box/Netute...e_Okno.cs#L309
Зачем такой способ вычисления квадратного корня (
static uint Kvadratnii_Koren(uint kolvo_setei)). Можно же Math.Sqrt и округлить.