Исходник игры "Машина"

Новые сообщения Правила форума Поиск

Страница 1 из 1 1
Модератор форума: ADMIN, Nextik

14.11.2011 в 15:34 Сообщение #1

HoldFast

Удаленные

Собстсвенно вот, держите

Прикрепления: mashina.zip (94.4 Kb)

Сообщение отредактировал HoldFast - Понедельник, 14.11.2011, 15:35

14.11.2011 в 19:42 Сообщение #2

PassCall

Знающий

Проверенные

Сообщений: 88

Награды: 0

Репутация: 4

Offline

Ну что можно сказать.
Пример неплохой, но

1. Полное отсутствие всякого форматирования нагоняет тоску. Серьезно.
2. Неиспользуемые (и несуществующие) файлы "left.PNG" и "right.PNG" только сбивают с толку новичков.
3. Ужасающая... нет, не так... УЖАСАЮЩАЯ оптимизация. Зачем присваивать четырем разным переменным ОДНУ И ТУ ЖЕ картинку?
4. Где комменты? Я программирую на МП около 3-х лет, но некоторые строки из кода просто вводят в ступор. Как вам vx:=w/2-16/2+16+random(rnd)? Просто понять, для чего это? А ведь первая (и главная) задача любого примера - это наибольшая простота. Да и вообще, почему нельзя написать "8" вместо "16/2"?
5. Некорректные для примера (так как использовать "короткие" названия для переменных в общем правильно) названия переменных. Они должны раскрывать свою суть лишь одним названием. Замени "w" на "width" и с первого раза станет ясно, что это такое и для чего нужно. А иначе лазишь по всему коду, чтобы посмотреть, где и как это используется.

В общем, чтобы понять, чему хочет научить пример, надо неплохо разбираться в МП. А тот, кто разбирается, напишет похожий пример сам. И гораздо лучше.

P.S. Я не хотел никого обидеть. Простите, если что.

Сообщение отредактировал PassCall - Понедельник, 14.11.2011, 20:18