оцените программу-тест

[Алексей_2012]

оцените программу по тестированию, если в коде можно что-то оптимизировать, то пишите исправленный код, или если нашли какие-то ошибки, то напишите какие

[Человек_Борща]

если в коде можно что-то оптимизировать, то пишите исправленный код

С чего бы это?
Сам исправлять и оптимизировать будешь.

если нашли какие-то ошибки, то напишите какие

Ок.

____________
Скачал. Увидел инсталятор - удалил и забил.

Не жирновато ли для тестировщика? 16 Мб.

[FaTaL]

Очень неприятный интерфейс. Цвета постоянно меняются и всё скачет, как будто прыгает. зачем в дистрибутив включили исходники? Да и плеер в тесте по моему совершенно ни к чему, он только отвлекает. Да и если кому то будет нужна музыка, то он её включит на своём любимом проигрывателе, но никак не в Вашей программе.
Блин и выйти не даёт из программы, пишет закончите сначала тест.

[Алексей_2012]

ну главное что интерфейс учителю понравился, но спасибо за критику, а что-нибудь еще про код скажете?

[саша40]

большая программа 7,6 мб в архиве. На Lazarus писал чтоли?

[phomm]

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

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

Теперь код, то чего Вы так долго желали. Тут уж я _не буду_ делать скидок на Ваш начальный уровень.

1 главное. Всё в кашу. Нет ни модулей, ни разделения по логическим блокам.
2 тоже оч важное. Всё построено чисто на обработчиках визуальных контролов, это не только сложно в поддержке и минимальная гибкость, это ещё и делает код непонятным ввиду что где и как делается, особенно учитывая, что имена всех вещей - либо стандартно что дельфи дала, либо адова смесь руслита с ингришем.
3. Нулевой кодстайл. Это и уже упомянутое именование идентификаторов и полное отсутствие идентации (отступов, переносов, выравнивания) и комментариев, и безудержная копипаста, и такие антипаттерны как Мейджик баттон и Мейджик намберс.
4. Копипаста - от неумения декомпозиции - надо как можно больше вводить методов/подпрограмм для работы через них, повышать им параметризацию, и в конечном итоге прийти к ООП, для инкапсуляции кода и данных, над которыми надо работать, вместе (в том числе и с целью уменьшения увеличившейся параметризованности подпрограмм)
5. Жуткая инкапсуляция в принципе, начиная от кучи глобально видимых вещей, заканчивая нелогичной обработкой данных, логически относящихся к одному блоку в другом блоке.
6. менее существенные вещи. Использование имени экземпляра в методах класса (form1 в методах tform1). Для такого проекта некритично, но плохо в любом случае - даже просто назвать по-новому экземпляр для повышения понятности (для правки вышеописанного плохого именования идентификаторов) - надо будет везде менять form1 , хотя на самом деле оно вообще не нужно в методах класса tform1, ну, а если очень надо - использовать Self.
7. нубское. Использование имейджей, лабелов поверх них, абсолютное позиционирование контролов, плохой воркфлоу ( в том число goto), неиспользование более лаконичных и грамотных средств языка и ртл, вроде in , destroy вместо free, Format и т.п. Использование где-то нагугленых винапи-примочек вместо более грамотной организации программы.


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

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

[Алексей_2012]

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

[Человек_Борща]

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

Совершенствую программу по уже выше сказанному.

[Алексей_2012]

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

[Человек_Борща]

Алексей_2012, инсталятор нужен тогда, когда перед установкой нужно подготовить ПК к программе(прописать настройки, установить драйвера, установить DLL в систему, прописать записи в реестр), кинуть ярлык на рабочий стол и потом перезагрузить ПК и только потом разрешить запуск ПО.

В вашей то ПО зачем инсталятор?