Разрушаем мифы: гк в MobileCMS

Печать RSS
395

A
Автор
Чатланин
0
Меня очень зацепили посты в теме про MobileCMS о якобы присутствии в ней говнокода и, соответственно, плохом качестве моих работ.
Высказывания о гк, как я заметил, были в основном от кодеров недоучек и завистников малолеток, нормальные программисты обычно писали положительные отзывы (ну, может быть слегка поправляли, за это им спасибо).
Поэтому хотелось бы чтоб вы высказали критику в этом топике, но предварительно представившись (работы, стаж, код), чтобы ваши высказывания не были очередным балабольством.
Начну с себя, дам своё резюме как программиста (не для понтов, а для решения проблемы, так сказать, отстоять честь).
Имею опыт работы около 4х лет, не полный список работ: http://www.free-lance.ru/users/Ant0ha2 , поглядите на топ 100 рейтинга ваплог.нет, админы проектов #7 и #19 (sever.ru и waprik.ru) одни из моих постоянных (!довольных) заказчиков.
Согласен по поводу большого количества @, я даже не знал, что сейчас ставить их уже считается не корректным, буду работать над этим (мало у кого код идеален, если вы считаете, что это не так, буду рад взглянуть на ваш код).
Есть ли какие либо существенные замечания по коду, кроме споров о том что быстрее echo или print и о "вреде" кавычек? Очень прошу, пишите без тупых выкриков, только сторого по делу.
A
Автор
Чатланин
0
Архитектура скрипта заимствована от очень популярного в среде вэб программистов "поджигателя" - CodeIgniter ( http://codeigniter.com , на русском http://code-igniter.ru ) имя ей HMVC (хелпер, модель, представление/отображение, контроллер), лучшей архитектуры я еще не встречал, если вы знаете что нить лучше, буду рад на это глянуть.
Принципы работы и некоторые библиотеки так же оттуда. Почему я просто не воспользовался поджигателем? Имхо, его существенные недостатки - не гибкость (которая заставляет писать по определенным правилам) и универсальность (фреймворк заточен практически под все задачи, это его очень утяжеляет). Поэтому я решил написать свое, более легкое и более гибкое ядро, не считаю его качественней CI, да и глупо это, и к CI отношусь очень респектабельно.
Так же кое какие принципы были заимствованы от Kohana - набирающий обороты, производный от поджигателя фреймворк. От кохана позаимстована структура хелперов - статические классы (в поджигателе это обычные пользовательские функции).
Изменил: Ant0ha (19.07.2010 / 01:36)

ツ ツ ツ
0
Начали к мелочам придираться изо высокой цены (как для этой местности) и изо того что ты написал про высокое качество кода, того и цеплялись к соплямь, не обращай внимания, на визави любят потролить, любую тему посмотри где идет речь о продаже с высокой ценой, точно такое же как и у тебя в теме, не поддавайся на провокации и не обращай внимания на "умников" smile
Структура у движка отличная, мелкие недочеты можно найти у каждого, да и это всего лишь первая версия, все впереди ещё =)
Изменил: Lugaro (19.07.2010 / 01:44)

Дух
0
1, забей, глупо, ))) что ты этим докажешь? Живи пиши коди работай))) нет предела совершенству, а так на критику реагировать, cms найдет своих пользователей, а эта тема будет еще одним продолжением той)
С

Малиновые штаны
0
Надо было плюнуть на добродетельность и продавать движок хоть по минимальной цене. А то многие просто из зависти зажопились на автора. На все придирки я бы отвечал так "Умный блин? Напиши лучше и выложи в паблик!"
A
Автор
Чатланин
0
> и изо того что ты написал про высокое качество кода
да, высокого, и от слов не отказываюсь, но я же не написал, что прям идеального качества.
З.Ы. спс за отзывы, вот только критики куда то делись...
N

Голубые штаны
0
Прям ажиотаж развели.
Скачал, поковырял не много. Нормальный, прокомментированный, удобочитаемый код, разобраться не составит труда, было бы желание.

Что могу заметить, следовало бы проверять, поддерживает ли браузер application/xhtml+xml, а лишь затем отдавать заголовок. Несколько ошибок валидности в разметке (html атрибуты в xhtml используются).

На счет шаблонов, зачем столько php кода в них? Тут нужно спросить, для кого шаблоны, для себя или для дизайнера? Если для себя - вполне приемлемо, избавляет я думаю от лишнего кода, который потребовался для полного разделения на дизайн и код. Если для дизайнера - может в какой то степени неудобно, но если учесть что это не веб и тут нет особых заморочек с разметкой, да и вообще разметки не много - вполне приемлемо.
A
Автор
Чатланин
0
Для дизайнеров (для создания тем) я думаю оставить только основные шаблоны (header.tpl, footer.tpl, style.tpl) этого хватит чтобы организовать вполне оригинальную тему оформления вап версии сайта.
Кстати, в ядре есть поддержка смарти, т.е. можно просто шаблоны сделать на смарти, при этом в код контроллеров вообще не соваться, только указать что нужно работать не со стандартными пхп шаблонами, а со смарти.
Но лично меня смарти не устраивает - лишняя нагрузка, лишний код в контроллерах. Поддержку смарти добавил для гибкости ядра, один из моих постоянных заказчиков привык к смарти, иногда приходится использовать этот шаблонизатор.
Изменил: Ant0ha (19.07.2010 / 10:28)
A

Оранжевые штаны
0
я сразу писал

14. Azzido (08.07.10 / 20:49)
Чатланин [On]
я в шоке от кода : klass
(Opera/9.80, 94.51.202.74)

я пожалуй первый раз такое качественное творение вижу в wap (из того что мне попадалось) с применением модульности, правда я не совсем осилил зачем там некоторые классы, например с mime типами, но уверен что не просто так, поэтому для многих этот код делается совершенно непонятным и запутанным, пока что wap держится около процедурных методов, проще для понимания, я сам тоже сторонник этого..
М

Малиновые штаны
0
#1, забей. Ты так только троллей кормишь
Стикеры / Теги / Правила / Топ тем / Топ постов / Поиск