Цитата:
Сообщение от Neil
заметите косяк, если не сложно, отпишите.
|
Не то чтоб косяки, но все таки, если хочется чтобы приложение не ощущалось как из начала двухтысячных:
1. Маска. Зачем показывать ошибку выбора маски, когда можно показывать только те варианты, которые работают? Неподходящие опции можно либо удалить из списка, либо сделать не активными
2. Опять же маска. Маску и префикс (бит в маске) было бы неплохо объединить при выборе, ибо во входных условиях может быть именно префикс (аля 192.168.0.1/24) и кастовать туда-сюда в голове не каждый может/хочет.
3. Число подсетей. Видно только 2 знака у выбранного значения, остальные потеряны.
4. Снова число подсетей. Аналогично пункту 1 - недоступные варианты стоит исключить из выбора.
5. Кнопка очистить. Она рядом с разделением подсетей, но чистит и вкладку "рассчитанные данные", что не очевидно. Собственно, зачем она вообще нужна тоже не понятно.
6. Общая композиция интерфейса. Кнопки подсчитать и разделить доступны сразу в основном окне, но их результат в разных вкладках. Да, они переключаются автоматически (за что плюс), но таки лучше эти вещи фундаментально разделить между теми же вкладками. Выбрали что хотим - сделали. Почему IP адрес - это входные данные, а число подсетей - нет? Тоже не понятно.
7. Интерактивность. В дополнение к пунктам 1 и 4. Кнопки подсчитать и разделить тоже лишние. Пересчитывать таблицы можно сразу при изменении входных данных. Если уж очень хочется сохранять состояния в процессе выбора (или между выбором) - это лучше сделать в явном виде, как историю операций (например, как у того же калькулятора начиная с десятки).
8. Последствия отсутствия интерактивности. Входные данные имеют значения по умолчанию, но рассчитанные данные пустые. Имеет смысл синхронизировать на старте, хотя бы как пример работы. Кнопка "разделить" недоступна пока не нажмешь "посчитать", но остается доступной при изменении входных данных. Если результат деления на подсети зависит от рассчитанных данных, то это ошибка. Если не зависит, то почему кнопка не доступна сразу?
Понимаю, что программа создана для локального использования конкретным человеком, и технически она работает, но не считаю подход "в одной ситуации делаем хорошо, а в другой как повезет" правильным, никто не сможет провести черту когда одно перетекает в другое без ошибок. В конце концов эта программа ведь выложена тут, надеюсь, для того чтобы сделать её лучше.
Цитата:
Сообщение от Копатель
Тебе пора уже на гитхабе репозиторий заводить.
|
С этого начинать стоит. Это и дисциплинирует, ибо стыдно в вечную память интернета выкладывать совсем уж плохой код, как минимум будешь его перечитывать перед тем как отправить. Сам по себе навык работы с гитом так же важен и ценен (мое откровение последних лет, раньше не представлял, что такого может не быть). Да и посмотреть код куда интересней, чем сидеть и тыкать в приложение.