KirillRodin / Metody_prog_RodinKirill_TKI-341

Бригада Родин Кирилл и Черкас Никита ТКИ-341
0 stars 0 forks source link

К заданию №2 #2

Closed FlashAnton closed 5 months ago

FlashAnton commented 6 months ago

Здесь запланировано обсуждение работы №2.

KirillRodin commented 5 months ago

Добрый вечер! Выложили задание №2 в основную ветку

FlashAnton commented 5 months ago

Здравствуйте.

К работе имеются замечания:

  1. В коде используются незначащие имена: переменных, проекта, решения, интерфейсных элементов и других компонент.
  2. Не вычищены от неиспользуемых библиотек классов соответствующие разделы экранных форм.
  3. В листинге присутствуют неиспользуемые обработчики событий.
  4. На Рисунке 4 цвет нумераторов для обозначения интерфейсных элементов управления подобран не эргономичный. Аналогично на Рисунках 5 и 6.
  5. Пункт 4 раздела 9 - необходимо показать, что курсор, действительно, отображается в правильном видел запущенном приложении.
  6. На Рисунке 13 показана неудачная и несовместная по дизайнерскому решению компоновка интерфейсных элементов управления.
  7. На Рисунке 13 речь про отрисовку какой-то модели, о которой не идёт речи в Техническом задании.
  8. "Дом с трубой" согласно постановке задания в Вашем варианте должен оказаться на 8-й позиции основного списка. Все остальные принятые решения по компоновке и отображению - авторские домыслы, связанные с некорректным восприятием текста индивидуального задания.
  9. Комбинированный список следует использовать в режиме DropDownList. Это мы ещё не успели рассмотреть - это материал, оставшийся к прочтению на предстоящей лекции, но так не эргономично, а также не эргономично демонстрировать пустые строки в списках. Номинально необходимо показывать что-то и это тоже материал к обсуждению на лекции.
  10. Не эргономично подобран цвет фигур, а вместе с ним и нажим пера. Некоторые фигуры с трудом распознаются на фоне Highlight.
  11. Вывод составлен очень кратко и не содержательно. Необходимо развить.
  12. Проиллюстрированное считается моветоном, с точки зрения изображения схем алгоритма: image Если у фигуры отсутствуют углы в её левой правой частях, то некорректно вводить и выводить связи из её левой и правой части. Как правило, блок "Процесс" связывается только снизу и сверху. Его боковины оставляются свободными под комментарии.
  13. Эти блоки, скорее всего, "Подпроцессы", нежели "Процессы" image Причём следует давать комментарии на схеме алгоритма о том, что нужно читать описание идеи компоновки в таком-то разделе Вашего отчёта.
  14. На сети Петри некорректно раскрыта ситуация с дочерней экранной формой. Либо "p4" должно называть иначе.
  15. Не все требования оформления ГПИ и приложения удачно подтверждены: про курсор уже сказано в замечании 5, к нему добавляется нераскрытие последовательности фигур, заданной по варианту.
KirillRodin commented 5 months ago

Здравствуйте! Выложили задание №2 с внесёнными исправлениями в основную ветку.

FlashAnton commented 5 months ago

Здравствуйте.

Посмотрел версию отчёта от 27 марта.

Сохраняются следующие замечания:

  1. В коде используются незначащие имена: переменных, проекта, решения, интерфейсных элементов и других компонент.

  2. На Рисунке 13 показана неудачная и несовместная по дизайнерскому решению компоновка интерфейсных элементов управления. Уточняю - нет единого дизайнерского решения в оформлении экранных кнопок. Их взаимное расположение неудачно.

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

  4. Вывод всё ещё краткий. Необходимо развить.

  5. Теперь на сети Петри и вовсе пропали упоминания об экранной форме со справкой, но на деле - она существует и является частью технологического процесса.

Nikitoxicc commented 5 months ago

Добрый вечер, перевыложили в основную ветку с исправлениями.

FlashAnton commented 5 months ago

Здравствуйте.

Не увидел правки по положениям, которые мы проговорили.

Проверьте, правильно ли всё прикрепилось.

FlashAnton commented 5 months ago

Здравствуйте.

С выполненными правками согласен.

Зачтено