Вернуться   Форум Рубцовска > Городской Интернет > Программирование
+17°Давление 736
Ветер 3 м/с ЮЮЗ
Комфорт +16°

Ответ
 
Опции темы Опции просмотра
  #1  
Старый 18.04.2022, 19:09
Аватар для Neil
Neil Neil вне форума
Генерал-Полковник
 
Регистрация: 26.01.2008
Сообщений: 7,375
Спасибо: 2,775
Neil стоит на развилке
Лампочка

Вложение 91988

- добавил пояснения при ошибках в выборе IP/Маски
- https://github.com/Intern-box/Netute...lavnoe_Okno.cs
__________________
"любой вопрос имеет право на жизнь, любой человек может ошибаться или не знать каких-то простых вещей" © serr (lubuntu.ru)

Мальчик сказал маме: “Я хочу кушать”. Мама отправила его к папе.
Мальчик сказал папе: “Я хочу кушать”. Папа отправил его к маме.
Мальчик сказал маме: “Я хочу кушать”. Мама отправила его к папе.
И бегал так мальчик, пока в один момент не упал.
Что случилось с мальчиком? TTL кончился.


три слова - это два слова! © banda agency Киев

Последний раз редактировалось Neil; 03.05.2022 в 13:21.
Ответить с цитированием
Эти 3 пользователей(ля) поблагодарили Neil за это полезное сообщение:
BigO (19.04.2022), Capsaicin (18.04.2022), Копатель (18.04.2022)
  #2  
Старый 19.04.2022, 10:40
Аватар для Capsaicin
Capsaicin Capsaicin вне форума
Прапорщик
 
Регистрация: 13.11.2014
Сообщений: 308
Спасибо: 216
Capsaicin стоит на развилке
По умолчанию

Цитата:
Сообщение от Neil Посмотреть сообщение
Вложение 91988

- добавил пояснения при ошибках в выборе IP/Маски
- https://github.com/Intern-box/Netute...lavnoe_Okno.cs
Отлично! Молодец что учитываешь советы и стараешься развиваться.
Если хочешь улучшить написание кода и нужны комментарии, то вот, что заметил.

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

В качестве именования в шарпах в основном используют 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 и округлить.
__________________
Только ситхи всё возводят в абсолют.
Ответить с цитированием
Эти 2 пользователей(ля) поблагодарили Capsaicin за это полезное сообщение:
BigO (19.04.2022), Копатель (19.04.2022)
  #3  
Старый 19.04.2022, 12:07
Аватар для Neil
Neil Neil вне форума
Генерал-Полковник
 
Регистрация: 26.01.2008
Сообщений: 7,375
Спасибо: 2,775
Neil стоит на развилке
Лампочка

Цитата:
Сообщение от Capsaicin Посмотреть сообщение
...
Не храни всю логику в одном файле формы. Старайся разделять на логические блоки. На классы, так, чтобы каждый класс был вполне себе понятной логической единицей...
кода пока мало. поэтому пока так.
дальше гляну к чему приду...

Цитата:
Сообщение от Capsaicin Посмотреть сообщение
...
В качестве именования в шарпах в основном используют CamelCase нотацию (но не всегда конечно)...
посмотрю как-нибудь

Цитата:
Сообщение от Capsaicin Посмотреть сообщение
...
Отсутствие комментариев в коде это нормально, кто бы что ни говорил. Идеальный (которого не существует, кек) код понятно без комментариев благодаря стилю написания и именованию переменных и блоков...
кода мало. и так всё понятно комментировать нечего пока
методы старался делать не сложными и понятными.
что не понятно, пишите, отвечу

Цитата:
Сообщение от Capsaicin Посмотреть сообщение
...
Лучше бы вместо массив байт использовать класс с 4 октетами для описания айпи. И переопределить операции для работы с этим классом...
чё за класс такой?

Цитата:
Сообщение от Capsaicin Посмотреть сообщение
...
В методе byte[] Ocifrovka_IP(string stroka_s_ip) незачем использовать регулярку, тем более она в очень общих чертах написана. У тебя дальше по коду есть IPAddress.TryParse(stroka_s_ip, out ip_adres), как раз с помощью него лучше откинуть неподходящие варианты...
IPAddress отфильтровывает не всё. пришлось к регулярным обратиться.
на сайте MS об этом есть инфа, НО сходу я не нашёл.

Цитата:
Сообщение от Capsaicin Посмотреть сообщение
...
Не понятно, зачем столько раз делаешь битовый сдвиг вправо для позиции контрола с надписью.
Берёшь ширину групбокса, вычитаешь ширину надписи, делаешь битовый сдвиг вправо и это будет позицией надписи?
https://github.com/Intern-box/Netute...e_Okno.cs#L309...
поправлю. спешил уже не помню, НО на моменте инициализации
выравнивание почему то не сработало. решил так, временно

Цитата:
Сообщение от Capsaicin Посмотреть сообщение
...
Зачем такой способ вычисления квадратного корня (static uint Kvadratnii_Koren(uint kolvo_setei)). Можно же Math.Sqrt и округлить.
смотрел я его. почему то он мне не понравился. не помню почему
__________________
"любой вопрос имеет право на жизнь, любой человек может ошибаться или не знать каких-то простых вещей" © serr (lubuntu.ru)

Мальчик сказал маме: “Я хочу кушать”. Мама отправила его к папе.
Мальчик сказал папе: “Я хочу кушать”. Папа отправил его к маме.
Мальчик сказал маме: “Я хочу кушать”. Мама отправила его к папе.
И бегал так мальчик, пока в один момент не упал.
Что случилось с мальчиком? TTL кончился.


три слова - это два слова! © banda agency Киев
Ответить с цитированием
  #4  
Старый 19.04.2022, 13:08
Аватар для Дем
Дем Дем вне форума
Лейтенант
 
Регистрация: 13.11.2010
Адрес: алейскый
Сообщений: 791
Спасибо: 262
Дем стоит на развилке
По умолчанию

То что кода мало и всё понятно, расскажешь себе после пары месяцев не заглядывания в этот код)
__________________


Ответить с цитированием
  #5  
Старый 19.04.2022, 13:14
Аватар для Neil
Neil Neil вне форума
Генерал-Полковник
 
Регистрация: 26.01.2008
Сообщений: 7,375
Спасибо: 2,775
Neil стоит на развилке
Радость

Цитата:
Сообщение от Дем Посмотреть сообщение
То что кода мало и всё понятно, расскажешь себе после пары месяцев не заглядывания в этот код)
откуда в Вас столько негатива?
__________________
"любой вопрос имеет право на жизнь, любой человек может ошибаться или не знать каких-то простых вещей" © serr (lubuntu.ru)

Мальчик сказал маме: “Я хочу кушать”. Мама отправила его к папе.
Мальчик сказал папе: “Я хочу кушать”. Папа отправил его к маме.
Мальчик сказал маме: “Я хочу кушать”. Мама отправила его к папе.
И бегал так мальчик, пока в один момент не упал.
Что случилось с мальчиком? TTL кончился.


три слова - это два слова! © banda agency Киев
Ответить с цитированием
  #6  
Старый 19.04.2022, 16:38
Аватар для Копатель
Копатель Копатель вне форума
Майор
 
Регистрация: 16.11.2010
Сообщений: 2,741
Спасибо: 2,063
Копатель стоит на развилке
По умолчанию

Цитата:
Сообщение от Neil Посмотреть сообщение
откуда в Вас столько негатива?
Ответить с цитированием
Этот пользователь поблагодарил Копатель(а) за это полезное сообщение:
Чужой (12.08.2022)
Ответ


Ваши права в разделе
Вы не можете создавать новые темы
Вы не можете отвечать в темах
Вы не можете прикреплять вложения
Вы не можете редактировать свои сообщения

BB коды Вкл.
Смайлы Вкл.
[IMG] код Вкл.
HTML код Выкл.

Быстрый переход


Часовой пояс GMT +7, время: 05:49.


Powered by vBulletin® Version 3.8.4
Copyright ©2000 - 2024, Jelsoft Enterprises Ltd. Перевод: zCarot