Показать сообщение отдельно

  #18  
Старый 18.07.2009, 15:55
eLWAux
Постоянный
Регистрация: 15.06.2008
Сообщений: 941
Провел на форуме:
5111568

Репутация: 2399


По умолчанию

Цитата:
Сообщение от FireFenix  
Двиг поставил, посмотрел....
Хотелось бы отметить несколько минусов:

1) Оформеление
а) При запуске скрипта, с настроенным php по дефолту (Apache 2.2.11 + PHP 5.2.10), половину кода отобразилось как текст из-за тега "<?"
Думаю лучше было бы использовать "<?php" на всякий случай =)
б) эффект смены скрипта немного сбивает с толку, т.е. расположение эелментов меняется и резкий взрыв цветов => использование html шаблонов было бы полезно
в) не плохо было бы добавить управление квестами + задниями квестов, где можно было бы добавить квест или изменить ответ, последовательность и т.д.

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

3) Бегло просмотрел код
а) function query($whot) - улыбнуло
б) думаю для дальнейшего развития и правил хорошего кода - все конфиги типа $this->name, $this->host определить к примеру в $this->cfg->db['name'], т.е. переменные назначить в свою рабочую область, что думаю повысит понимание кода =)
в) также правило хорошего кода - html отдельно от исполняемого кода => желательно использовать бы html шаблоны
г) кое-где для читабильности лучше бы юзать switch вместо if и передавать одним параметров (например сортировку) =)
б) для универсальности, лучше вынести код обработки url отдельно, и через инклюды вызывать функции, которые тоже желательно бы разбить на ещё более составляющие группы файлов
е) в некоторых местах бы не помешала оптимизация кода и выкидывание лишнего кода
ж) Отсудствие комментариев и не сгруппированности переменных, немного сложно уловить суть того или иногда кусочка кода

Это было сугубо моё мнение, поэтому не обязательно брать его на заметку и кидать в меня помидорами =)

P.S. Готов помочь в создании и тестировании движка ^_^
1.a) уже изменяю
1.б) не понял )
1.в) спасибо. тоже в процессе
2.а) зачем? как на меня - все нормально..
2.б) думаю нет. процесс создания квеста надо сделать максимально простим и динамическим.
2.в) поподробней пожалуйста )
3.а) банально и просто
3.б) вообще-то я не рассчитывал на то, чтоб другие смотрели в код ))
$this->eLng, $this->UInfo, $this->QInfo, ... я делал все так, что в первую очерь, МНЕ было удобно модифицировать и менять код.
3.в) html-щаблоны у меня - это темы )
3.г) изменю.., хотя это на качество\etc не повлияет )
3.д) мой стиль - все одном месте )
3.е) 3.г
3.ж) стоп-стоп. архив не для девелоперов )
все кто, хочет изменить код - мне в аську пишут )

Цитата:
P.S. Если так судить, то квесты нужно делать как HP. Официальный квест для Админов и как реклама серваков HP Blade
Всё как положено, анимация, картинки, русская озвучка, сюжет, консоль
постараюсь до завтра сделать такой же на своем движке
ну не 1 в 1, но максимально похож

Последний раз редактировалось eLWAux; 18.07.2009 в 15:58..
 
Ответить с цитированием