Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#323 closed улучшение (не будем делать)

Панель групповых каналов: кнопка "Выбрать новый"

Reported by: san Owned by: alx
Priority: средний Milestone: 2 очередь
Component: web-интерфейс (sw) Keywords:
Cc: andrei, Ivanmvtel

Description

Если исключить любой групповой канал(например G0) из таблицы коммутации,
а затем нажать "Выбрать новый"(G0 первый в очереди на новые) - все слагаемые G0 будут удалены.

Мне такое поведение кажется не правильным и не очевидным для пользователя, на мой взгляд эта кнопка не должна изменять(удалять) данные введённые пользователем.
Даже само название "Выбрать новый" звучит так что при нажатии этой кнопки точно никакого удаления данных происходить не будет, а будет лишь произведён выбор.

Change History (26)

in reply to:  description comment:1 by alx, 6 years ago

Саша, я не понял, с чем смысл этого тикета. Тип тикета - "улучшение", но ни в заголовке, ни в описании никакого улучшения не предложено...

Replying to san:

Мне такое поведение кажется не правильным

Почему? Аргументируй, пожалуйста.

и не очевидным для пользователя,

Опять же, аргументируй, пожалуйста.

на мой взгляд эта кнопка не должна изменять(удалять) данные введённые пользователем.

Попробую объяснить свою логику. Нажатие пользователем кнопки "Выбрать новый" - не самоцель. Пользователь кликает по этой надписи, потому что хочет занять какой-нибудь групповой канал для того чтобы использовать его для суммирования каких-то каналов. Поэтому мне кажется вполне логичным и правильным, что после нахождения неиспользуемого группового канала он приводится к определенному "начальному состоянию", а именно, к "очищенному", когда нет ни одного слагаемого. Групповой канал с таком состоянии готов к тому, что пользователь будет добавлять в групповой канал нужные ему слагаемые. Если бы это не делалось, пользователь мог бы получить групповой канал с уже назначенными слагаемыми, которые ему не нужны, что создало бы для него лишние неудобства.

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

Даже само название "Выбрать новый" звучит так что при нажатии этой кнопки точно никакого удаления данных происходить не будет, а будет лишь произведён выбор.

Лично я в названии "Выбрать новый" слышу только то, что будет выбрано нечто новое. :) Остальное - это ИМХО слуховые галлюцинации и фантазии слушателя. :)

comment:2 by san, 6 years ago

Аргументируй

На мой взгляд, отсутствие гк в таблице коммутации - это ещё недостаточное основание, чтобы считать данные в его строке мусором, а решение мусор это или нет должен принять пользователь, т.к. он точно знает а мы нет.

in reply to:  2 comment:3 by alx, 6 years ago

Replying to san:

На мой взгляд, отсутствие гк в таблице коммутации - это ещё недостаточное основание, чтобы считать данные в его строке мусором

Тут, наверное, следует дать определение. Под "информационным мусором" я подразумевал такую информацию, которая никак не влияет на работу аппаратуры. Конфигурация группового канала, не используемого в таблице коммутации, никак не влияет на работу аппаратуры. Поэтому изменение этой информации ничего "испортить" не может.

comment:4 by andrei, 6 years ago

Тогда давайте порты с неподнятым линком удалять из VLAN, тоже на работу в данный момент не повлияет...
И даже ничего не испортит.

in reply to:  4 comment:5 by alx, 6 years ago

Replying to andrei:

Тогда давайте порты с неподнятым линком удалять из VLAN, тоже на работу в данный момент не повлияет...
И даже ничего не испортит.

Так и удаляем! Если пользователь кликает отмеченный символом "Т" чекбокс в таблице VLAN, порт перестает быть членом VLAN'а.

comment:6 by andrei, 6 years ago

Да, действительно. Я ошибся.
А VLAN, в который не добавлен ни один порт тоже удаляется без желания пользователя?
По групповым каналам предлагаю не удалять слагаемые если пользователь не просит об этом.
В крайнем случае вывести уведомление.
А если хочется почистить конфигурацию, то можно добавить очередную кнопку, назвать ее "почистить мусор". При нажатии которой удалять всякие неадекватности.

in reply to:  6 ; comment:7 by alx, 6 years ago

Replying to andrei:

А VLAN, в который не добавлен ни один порт тоже удаляется без желания пользователя?

Почему "тоже"? И VLAN, и слагаемые группового канала, о которых говорил Саша, удаляются только по желанию пользователя. В первом случае пользователь выражает свое желание кликом иконки "Удалить", во втором - кликом "Выбрать новый". Самопроизвольно, без желания пользователя ничего не удаляется.

По групповым каналам предлагаю не удалять слагаемые если пользователь не просит об этом.

Повторяю: без просьбы пользователя ничего и не удаляется.

in reply to:  7 comment:8 by andrei, 6 years ago

кликом "Выбрать новый". Самопроизвольно, без желания пользователя ничего не удаляется.

Я как пользователь не желаю чтобы кнопка "выбрать новый" удаляла слагаемые из группового канала, которые я только что туда добавил, потратив на это время.

comment:9 by Ivanmvtel, 6 years ago

Может стоит считать неиспользуемым групповым каналом у которого нет ни одного слагаемого, даже если он не выведен в тдм. И при нажатии на выбрать новый, выбирается "полностью пустой" групповой канал.

in reply to:  9 comment:10 by andrei, 6 years ago

Replying to Ivanmvtel:

Может стоит считать неиспользуемым групповым каналом у которого нет ни одного слагаемого, даже если он не выведен в тдм. И при нажатии на выбрать новый, выбирается "полностью пустой" групповой канал.

Именно так мне это и видится.

comment:11 by san, 6 years ago

В первом случае пользователь выражает свое желание кликом иконки "Удалить", во втором - кликом "Выбрать новый".

В этом вся и разница: удалить при нажатии кнопки удалить - очевидное для пользователя поведение, удалить при нажатии "выбрать" - не очевидное.

ни в заголовке, ни в описании никакого улучшения не предложено...

более подходящего типа тикета не было :)

А вообще предложить я хотел один из вариантов:

  • убрать эту кнопку совсем
  • урезать функционал до выбора гк без слагаемых (что Ваня и Андрей предлагают)
  • оставить как есть, но перед удалением слагаемых предупредить пользователя: "Групповой канал Gx не участвует в коммутации. Удалить слагаемые ?" Да/Нет/Следующий. При нажатии "Нет" открывать Gx с имеющимися слагаемыми. При нажатии "Cледующий" выполнять поиск дальше/

in reply to:  9 comment:12 by alx, 6 years ago

Replying to Ivanmvtel:

Может стоит считать неиспользуемым групповым каналом у которого нет ни одного слагаемого, даже если он не выведен в тдм. И при нажатии на выбрать новый, выбирается "полностью пустой" групповой канал.

Если некий групповой канал не используется, какое имеет значение, как он сконфигурирован (какие каналы и как он складывает)?

comment:13 by Ivanmvtel, 6 years ago

Пока пользователь сам не удалит эти слагаемые, думаю не стоит их трогать

in reply to:  11 ; comment:14 by alx, 6 years ago

Replying to san:

А вообще предложить я хотел один из вариантов:

  • убрать эту кнопку совсем

Не согласен. Если кнопка не нужна - не нажимай ее. Это же самое отвечу и Андрею (сразу ответить не посчитал нужным в силу очвидности ответа, но раз уже второй человек это пишет, видимо, ответ очевиден не всем):

Replying to andrei:

Я как пользователь не желаю чтобы кнопка "выбрать новый" удаляла слагаемые из группового канала, которые я только что туда добавил, потратив на это время.

У тебя как пользователя, есть возможность и святое право не нажимать "выбрать новый". Если кнопку не нажимать - то и удалять слагаемые она не будет. Отмечу также, что твои действия как пользователя (потратить время на конфигурирование группового канала и не использовать его) нелогичны.

Теперь снова возвращаюсь к предложению san:

Никто пользователя нажимать ненужную ему кнопку не заставляет. Таким образом, в убирании кнопки никакого улучшения нет. А вот ухудшение - есть, так как тот пользователь, кому эта кнопка была нужна, потеряет возможность ее нажимать.

Replying to san:

  • урезать функционал до выбора гк без слагаемых (что Ваня и Андрей предлагают)

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

  • оставить как есть, но перед удалением слагаемых предупредить пользователя: "Групповой канал Gx не участвует в коммутации. Удалить слагаемые ?" Да/Нет/Следующий. При нажатии "Нет" открывать Gx с имеющимися слагаемыми. При нажатии "Cледующий" выполнять поиск дальше/

Не согласен. Не вижу смысла в отображении слагаемых группового канала, который нигде не используется. Я как пользователь нажимаю "Выбрать новый" для того чтобы сконфигурировать новый групповой канал. Что там в нем было когда-то раньше, меня не интересует. Повторю вопрос, который я задал Ivanmvtel: какое значение имеет конфигурация группового канала, если он не используется? Также напоминаю, что для того чтобы посмотреть конфигурацию группового канала, не меняя ее, у пользователя есть select в левой части панели групповых каналов.

Last edited 6 years ago by alx (previous) (diff)

in reply to:  13 comment:15 by alx, 6 years ago

Replying to Ivanmvtel:

Пока пользователь сам не удалит эти слагаемые, думаю не стоит их трогать

Никто их и не трогает. Пользователь сам нажимает "выбрать новый", тем самым давая указание компьютеру очистить конфигурацию группового канала. Если он делает это против своей воли, если кто-то держит пистолет у его виска - это не проблема аппаратуры MC04-DSL-3U. :) Пусть обращается в полицию...

comment:16 by san, 6 years ago

Вернусь к неочевидности:
Подходишь к кофемашине нажимаешь "Новая чашка кофе", из кофемашины вылезает молоток и расколачивает вдребезги старую чашку... Никто ведь не заставлял нажимать :)

Блин, из-за такой мелочи столько нафлудили.

Last edited 6 years ago by san (previous) (diff)

in reply to:  14 ; comment:17 by andrei, 6 years ago

Не согласен. Не вижу смысла в отображении слагаемых группового канала, который нигде не используется. Я как пользователь нажимаю "Выбрать новый" для того чтобы сконфигурировать новый групповой канал. Что там в нем было когда-то раньше, меня не интересует. Повторю вопрос, заданный Ivanmvtel: какое значение имеет конфигурация группового канала, если он не используется? Также напоминаю, что для того чтобы посмотреть конфигурацию группового канала, не меняя ее, у пользователя есть select в левой части панели групповых каналов.

Я хочу создать 28 (двадцать восемь) ГК. Открываю панель ГК, нажимаю "выбрать новый" добавляю в него слагаемые. Решаю сначала добавить все слагаемые и только потом скоммутировать созданные ГК. Нажимаю "выбрать новый", добавляю, повторяю. На автомате так... И на четырнадцатом повторе вижу что у меня G0 почему-то выбрался, глюк- думаю я. Ну а когда оказывается что не глюк а удаление начинаю негодовать. А учитывая что удаление моих трудов - недокументированная функция и нигде не описана...

comment:18 by andrei, 6 years ago

Про не заставляли нажимать. Можно еще добавить кнопку по центру экрана. На ней синий круг. При нажатии затирает весь конфиг, удаляет VLAN1, и паролит учетку админа.
И никто не просит нажимать синюю кнопку.

in reply to:  17 comment:19 by alx, 6 years ago

Replying to andrei:

Решаю сначала добавить все слагаемые и только потом скоммутировать созданные ГК.

Не решай так. Не вижу никаких объективных причин делать именно таким образом. Решай сразу коммутировать созданный групповой канал.

comment:20 by Ivanmvtel, 6 years ago

Еще стараемся уходить от действий которые приводят к удалению/изменению без подтверждения, а тут получается как раз наоборот, да и почему-то выбрать новый включает в себя удаление чего-либо

comment:21 by san, 6 years ago

недокументированная функция и нигде не описана...

будет в описании, когда доделаешь, а пока в блоге было https://trac.adc-line.ru/sw-01/blog/sw_1.0-r1660#sw

in reply to:  20 comment:22 by alx, 6 years ago

Replying to Ivanmvtel:

Еще стараемся уходить от действий которые приводят к удалению/изменению без подтверждения, а тут получается как раз наоборот,

Я считаю, что запрос подтверждения имеет смысл тогда, когда действие изменяет что-то в работе аппаратуры, и тем самым может что-то испортить (нарушить работу связи). Как уже было неоднократно сказано выше, изменение режима неиспользуемого группового канала ничего в работе аппаратуры не меняет, и, следовательно, нарушить связь не может. Поэтому и подтверждение запрашивать не имеет смысла. Если пользователь кликнул по ошибке, он всегда может нажать кнопку "Отменить".

да и почему-то выбрать новый включает в себя удаление чего-либо

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

comment:23 by Ivanmvtel, 6 years ago

Были ситуации что определенное направление необходимо временно выключить из работы , т.е этот гк, допустим G2, становится не активным, слагаемые остаются. И создается новый групповой канал, для нового направления. Если убрать G2 из тдм, и нажать новый, то слагаемые G2 очистятся и G2 появится как новый. (GOи G1 заняты).
В принципе ничего страшного не происходит, но совсем не понятно зачем чистить слагаемые (а точнее предполагать что они более не нужны).

comment:24 by Ivanmvtel, 6 years ago

а так бы при нажатии Выбрать новый появился бы G3. А неактивный G2 со слагаемыми остался жив.

in reply to:  23 comment:25 by alx, 6 years ago

Replying to Ivanmvtel:

Были ситуации что определенное направление необходимо временно выключить из работы , т.е этот гк, допустим G2, становится не активным, слагаемые остаются. И создается новый групповой канал, для нового направления. Если убрать G2 из тдм, и нажать новый, то слагаемые G2 очистятся и G2 появится как новый. (GOи G1 заняты).

Пример понятен. В данном случае есть альтернативные варианты решения проблемы:

  • Перед внесением изменения в конфигурацию, сохранить текущую конфигурацию в файл, тогда будет возможность вернуть ее, что бы там позже не было наконфигурировано. ИМХО это вообще должно быть "золотым правилом" любого оператора на любой случай (без связи с групповыми каналами), поэтому ставлю этот вариант на первое место.
  • Временно выключив групповой канал из работы, "сохранить" его, назначив на любой неиспользуемый канал (трудно представить, что в таблице коммутации задейтсвованы все 1024 канала).
  • Наконец, если заранее известно, что групповой канал будет временно отключаться, можно выбрать канал с большим номером (31, 30, 29), тем самым снизив вероятность задествовать его для других целей.

Это только то, что сходу пришло в голову...

В принципе ничего страшного не происходит,

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

in reply to:  24 comment:26 by alx, 6 years ago

Resolution: не будем делать
Status: newclosed

Replying to Ivanmvtel:

а так бы при нажатии Выбрать новый появился бы G3. А неактивный G2 со слагаемыми остался жив.

См. мой comment:11. Сначала, пока есть "пустые" каналы, это работать будет. Потом при нажатии "Выбрать новый" пользователь будет получать сообщение, что чистых каналов больше нет. Получится абсурд: есть масса неиспользуемых групповых каналов, а выбрать для использования нечего...

Я смотрю, обсуждение пошло по второму кругу. Поэтому тикет закрываю. Появятся новые аргументы или предложения - переоткроем.

Last edited 6 years ago by alx (previous) (diff)
Note: See TracTickets for help on using tickets.