|
|
Регистрация Восстановить пароль |
Повторная активизация e-mail |
Регистрация | Задать вопрос |
Заплачу за решение |
Новые сообщения |
Сообщения за день |
Расширенный поиск |
Правила |
Всё прочитано |
|
|
Опции темы | Поиск в этой теме |
22.12.2012, 23:14 | #1 |
t45t
Участник клуба
Регистрация: 20.03.2012
Сообщений: 1,849
|
оцените программу-тест
оцените программу по тестированию, если в коде можно что-то оптимизировать, то пишите исправленный код, или если нашли какие-то ошибки, то напишите какие
from dark to light)
|
23.12.2012, 00:07 | #2 | ||
Старожил
Регистрация: 30.12.2009
Сообщений: 11,426
|
Цитата:
Сам исправлять и оптимизировать будешь. Цитата:
____________ Скачал. Увидел инсталятор - удалил и забил. Не жирновато ли для тестировщика? 16 Мб. |
||
23.12.2012, 10:04 | #3 |
Участник клуба
Регистрация: 09.11.2007
Сообщений: 1,762
|
Очень неприятный интерфейс. Цвета постоянно меняются и всё скачет, как будто прыгает. зачем в дистрибутив включили исходники? Да и плеер в тесте по моему совершенно ни к чему, он только отвлекает. Да и если кому то будет нужна музыка, то он её включит на своём любимом проигрывателе, но никак не в Вашей программе.
Блин и выйти не даёт из программы, пишет закончите сначала тест. Последний раз редактировалось FaTaL; 23.12.2012 в 10:06. |
23.12.2012, 15:57 | #4 |
t45t
Участник клуба
Регистрация: 20.03.2012
Сообщений: 1,849
|
ну главное что интерфейс учителю понравился, но спасибо за критику, а что-нибудь еще про код скажете?
from dark to light)
|
23.12.2012, 17:22 | #5 |
Участник клуба
Регистрация: 12.09.2012
Сообщений: 1,030
|
большая программа 7,6 мб в архиве. На Lazarus писал чтоли?
Что нужно программисту: Компьютер, Среда программирование, Воображение, Прямые руки, Мозги, Знания этой среды программирования.
Программист-это профессия, а программирование-это моё хобби. |
23.12.2012, 17:28 | #6 |
personality
Старожил
Регистрация: 28.04.2009
Сообщений: 2,882
|
Инсталятор - плохо, более того он просит перезагрузиться, кошмар.
Огромный размер - плохо, если уж так хочется изысков - (типа вон как проигрыватель сунуть) - то уж неужели на жепег не разжились. Проигрыватель как уже сказали - тоже эпик, причём фейл. Непонятные функции типа инструкции и подсказки (хотя, возможно, это требовалось по заданию) - подсказки в тесте неуместны, и так тесты проверяют только умение сдавать тесты но не знания, дак ещё и подсказки... нет слов. Чудовищный функционал. набор из непонятных файлов (которые, кстати, легко открываются и палятся), выбор только из 4 вариантов ответа с вопросами просто текстом. Нет никаких функций для составителя тестов, ни регистрации тестируемых, ни вменяемой навигации по рабочим окнам. Ничего короче... (а инсталятор и проигрыватель и картинки это конечно именно то , что называется "свистелки") Теперь код, то чего Вы так долго желали. Тут уж я _не буду_ делать скидок на Ваш начальный уровень. 1 главное. Всё в кашу. Нет ни модулей, ни разделения по логическим блокам. 2 тоже оч важное. Всё построено чисто на обработчиках визуальных контролов, это не только сложно в поддержке и минимальная гибкость, это ещё и делает код непонятным ввиду что где и как делается, особенно учитывая, что имена всех вещей - либо стандартно что дельфи дала, либо адова смесь руслита с ингришем. 3. Нулевой кодстайл. Это и уже упомянутое именование идентификаторов и полное отсутствие идентации (отступов, переносов, выравнивания) и комментариев, и безудержная копипаста, и такие антипаттерны как Мейджик баттон и Мейджик намберс. 4. Копипаста - от неумения декомпозиции - надо как можно больше вводить методов/подпрограмм для работы через них, повышать им параметризацию, и в конечном итоге прийти к ООП, для инкапсуляции кода и данных, над которыми надо работать, вместе (в том числе и с целью уменьшения увеличившейся параметризованности подпрограмм) 5. Жуткая инкапсуляция в принципе, начиная от кучи глобально видимых вещей, заканчивая нелогичной обработкой данных, логически относящихся к одному блоку в другом блоке. 6. менее существенные вещи. Использование имени экземпляра в методах класса (form1 в методах tform1). Для такого проекта некритично, но плохо в любом случае - даже просто назвать по-новому экземпляр для повышения понятности (для правки вышеописанного плохого именования идентификаторов) - надо будет везде менять form1 , хотя на самом деле оно вообще не нужно в методах класса tform1, ну, а если очень надо - использовать Self. 7. нубское. Использование имейджей, лабелов поверх них, абсолютное позиционирование контролов, плохой воркфлоу ( в том число goto), неиспользование более лаконичных и грамотных средств языка и ртл, вроде in , destroy вместо free, Format и т.п. Использование где-то нагугленых винапи-примочек вместо более грамотной организации программы. Да, я жесток. Уж извини, просто я писал систему тестирования, на заказ за денежку, и помимо того, что я смог ввиду изначально заложенной гибкости поменять программу без существенной переделки при изменении требований заказчика, я ещё смог для другого заказчика по дополнительному вороху заданий сделать хорошую работу через год. Всё из-за почти полного следования всему вышеперечисленному (ну, в смысле соблюдения в правильной манере, а не допущения описанных минусов кода). И, конечно, без свистелок, ибо сама система не нуждалась в них, просто потому что её спектр задач был довольно широк сам по себе, и её работа говорила за себя, а не мишура. Хочешь поговорить и внять - пиши, пообщаемся. Взыграет гордость - прими достойно критику и развивайся через "а я докажу" а не "ненавижу". |
24.12.2012, 03:52 | #7 |
t45t
Участник клуба
Регистрация: 20.03.2012
Сообщений: 1,849
|
спасибо вам ребят, если еще что-то нашли, то пишите, и кстати, я думаю что для новичка, изучающего делфи чуть больше трех месяцев совсем не плохо) если считаете что неплохо, то щелкайте на весы) а инструкция, помощь и список вопросов, а так же невозможность выхода если тест незакончен и отчет-это просьба учителя
from dark to light)
Последний раз редактировалось Алексей_2012; 24.12.2012 в 03:54. |
24.12.2012, 10:34 | #8 |
Старожил
Регистрация: 30.12.2009
Сообщений: 11,426
|
В целом очень не плохо для новичка, но все же репутацию ты ничем не заслужил. Писать программы тут может каждый.
Совершенствую программу по уже выше сказанному. |
25.12.2012, 00:02 | #9 |
t45t
Участник клуба
Регистрация: 20.03.2012
Сообщений: 1,849
|
и все-таки не пойму, почему инсталятор-плохо? Для нубов не понимающих в программировании очень даже ничего: поставил и работай, да, перезагрузка лишняя) а на счет плеера: так уж сложилось, что компы у нас в колледже старые, а программа расчитана на них, из проигрывателей только виндос медиа плеер, ито некоторые пк без кодеков, так что проигрыватель норм решение) правда он иногда выдает ошибку, с чем сейчас работаю, мерцание убрал, и кстати, алгоритм, определяющий правильность/неправильность ответа не очень конечно, но другого в голову не пришло, если знаете другой способ, то расскажите о нем)
from dark to light)
|
25.12.2012, 01:09 | #10 |
Старожил
Регистрация: 30.12.2009
Сообщений: 11,426
|
Алексей_2012, инсталятор нужен тогда, когда перед установкой нужно подготовить ПК к программе(прописать настройки, установить драйвера, установить DLL в систему, прописать записи в реестр), кинуть ярлык на рабочий стол и потом перезагрузить ПК и только потом разрешить запуск ПО.
В вашей то ПО зачем инсталятор? |
|
Похожие темы | ||||
Тема | Автор | Раздел | Ответов | Последнее сообщение |
Оцените программу | Pein95 | Софт | 2 | 02.03.2011 00:12 |
Оцените программу | Sprat | Софт | 5 | 14.12.2010 08:16 |
Пожалуйсто оцените программу | dinamid | Мультимедиа в Delphi | 11 | 13.10.2009 14:10 |
Оцените мою программу | kakawkin | Софт | 1 | 04.10.2009 15:18 |
Оцените программу | cheef | Софт | 11 | 30.04.2009 09:28 |