Сергей В.
244 сообщения
#17 лет назад
DIMM_on, возможно. Я сам не реализовывал вап, но можно наконец определять клиент сам
Дмитрий Р.
23 сообщения
#17 лет назад
Это так.. но тут никак не решить этого. просто выбирать веб тему для веб браузеров. и xhtml для телов
Николай Т.
205 сообщений
#17 лет назад
DIMM_on: еще раз повторяю что смена темы происходит произвольно
Николай Т.
205 сообщений
#17 лет назад
Zionit: кеширование нужно убрать на уровень шаблонизатота, шаблонизатор тоже лучше сменить на смарти, работу с бд лучше спрятать в орм, и имхо конфиг лучше выглядит когда он обьект
Дмитрий Р.
23 сообщения
#17 лет назад
e1it3, то есть вы не меняете7 она сама??
Николай Т.
205 сообщений
#17 лет назад
DIMM_on: да конечно)
Андрей Д.
1267 сообщений
#17 лет назад
Цитата ("e1it3"):
zionit: кеширование нужно убрать на уровень шаблонизатота, шаблонизатор тоже лучше сменить на смарти, работу с бд лучше спрятать в орм, и имхо конфиг лучше выглядит когда он обьект


Я лучше знаю, что мне нужно и как мне делать.
И лучше не нужно мне давать установки...
Николай Т.
205 сообщений
#17 лет назад
Zionit:
Цитата:
Я лучше знаю, что мне нужно и как мне делать.
И лучше не нужно мне давать установки...

воспринимайте критику адекватно, через некоторое время вы все-равно придете к тому что для построения простых sql запросов придется писать свой sql конструктор и кеширование вывода сместиться в сторону шаблонизатора, а в модули придет кеширование данных (может даже на mvc перейдёте). а ещё вы вероятно откажитесть от типизации блоков контента... так что все у вас ещё будет впереди.
Дмитрий Р.
23 сообщения
#17 лет назад
e1it3, это очень странно. при тестинге таких багов не было замечено..
Андрей Д.
1267 сообщений
#17 лет назад
Цитата ("e1it3"):
Zionit:
Цитата:
Я лучше знаю, что мне нужно и как мне делать.
И лучше не нужно мне давать установки...

воспринимайте критику адекватно, через некоторое время вы все-равно придете к тому что для построения простых sql запросов придется писать свой sql конструктор и кеширование вывода сместиться в сторону шаблонизатора, а в модули придет кеширование данных (может даже на mvc перейдёте). а ещё вы вероятно откажитесть от типизации блоков контента... так что все у вас ещё будет впереди.


Я сомневаюсь в вашем профессиональном уровне и верю в свой, а ваша критики - скорее указания.
Если я к этому перейду то уж точно не благодаря вам...
Если вам не будет нравиться мой продукт, то просто не используйте, а лучше сделайте своё, тк. всё что я увидел у вас в портфолио стоит мелочь, некоторое вообще основано на чужих наработках
Николай Т.
205 сообщений
#17 лет назад
>Я сомневаюсь в вашем профессиональном уровне и верю в свой, а ваша критики - скорее указания.


if($cache->is_chached("mini_news.tpl"))
{
$tpl->result = $cache->get_page("mini_news.tpl");
}

чесно говоря целесобразность нахождения этого куска кода тут вызывает сомнения, причин несколько
1) явное указание short-news-block и mini_news.tpl
2) плохая маштабируемость (например вам придется поставить костыли к коду при проверках на доступность контента для пользователя в зависимостях от его прав)
резюме: легко убирается в конструктор класса module


$mini_news = $db->query("SELECT `".PREFIX_MAIN."post`.`id` as `id`,
`".PREFIX_MAIN."post`.`date` as `date`,
`".PREFIX_MAIN."post`.`alt_name` as `alt_name`,
`".PREFIX_MAIN."contents`.`text` as `text`
FROM `".PREFIX_MAIN."post`, `".PREFIX_MAIN."contents`
WHERE `".PREFIX_MAIN."post`.`id_site` = '".ID_SITE."'
AND `".PREFIX_MAIN."post`.`show` = '1'
AND `".PREFIX_MAIN."post`.`type` = 'shortnews'
AND `".PREFIX_MAIN."contents`.`lang` = '".LANGUAGE."'
AND `".PREFIX_MAIN."contents`.`type` = 'mininews_s'
AND `".PREFIX_MAIN."contents`.`item_id` = `".PREFIX_MAIN."post`.`id`
ORDER BY `date` DESC
LIMIT 0, ".$config, false, false);

вы не заметили что рябит от константы PREFIX_MAIN ? (нормальные хостеры дают несколько баз, но это исключение чем правило)
пробнем переписать к ЧПВ (человечески понятный вид)

$db->select(array('post_id'=>'id', 'post_date'=>'date' ...));
$db->setWhere()
$db->setOrder()
$db->setLimit()
$mini_news = $db->exec()

гибкость кода возрастает в разы, не нужен лимит, комментим и так далее


$tpl->load_module_template('mini.tpl', "news");
$tpl->compile('short-news-block');
$cache->save_page("mini_news.tpl", $tpl->result);

вот зачем это писать будет в каждом модуле каждый раз ?

$tpl->set("{full_link}", is_cpu(HOME_URL."index.php?do=news&action=".$result, 
HOME_URL."news/".$result.".html"));

is_cpu - довольно говорящее имя функции (проверка на наличие процессора ???)


>Если я к этому перейду то уж точно не благодаря вам...
Цитата:

"Есть три пути у человека, чтобы поступать разумно:
первый, самый благородный - размышление,
второй, самый легкий - подражание,
третий, самый горький - опыт.
Конфуций, V век до н.э."

тогда рекомендую изучить limb-project.com и symfony-project.com


>Если вам не будет нравиться мой продукт, то просто не используйте, а лучше сделайте своё, тк.
я очень надеюсь что он мне будет нравиться, но как истинный велосипедописатель я буду писать свою реализацию
>всё что я увидел у вас в портфолио стоит мелочь, некоторое вообще основано на чужих наработках
даты файлов 2006 год, залил новенькое , конечно все самое вкусное стоит денег, поэтому этим я не могу поделиться с кем либо...
Андрей Д.
1267 сообщений
#17 лет назад
Цитата:
is_cpu - довольно говорящее имя функции (проверка на наличие процессора ???)

лол... ЧПУ (обычно ничего умного в 4 ночи в голову уже не лезет)

Цитата:
вы не заметили что рябит от константы PREFIX_MAIN

Мне не рябит, превык... Люблю делать такие запросы самостоятельно...

Оффтопик
вот зачем это писать будет в каждом модуле каждый раз ?

не каждый модуль требует кеширования, но совет учту

Цитата:
легко убирается в конструктор класса module

файлы кеширования распределяються вручную, оспорений быть не может, тк в каждом модуле может быть несколько кешей... Кеш может иметь своё имя и т.д.

Цитата:
залил новенькое , конечно все самое вкусное стоит денег, поэтому этим я не могу поделиться с кем либо...

6 Кб кода тоже не очень много ))) Делиться не нужно, покажите...

В любом случае - что-то не так? -сделайте лучше...
Андрей Д.
1267 сообщений
#17 лет назад
Оффтопик
limb-project.com и symfony-project.com

хочеться всё своё... Разве-что веб редактор будет чужой...
Александр С.
13 сообщений
#17 лет назад
Николай Т.
205 сообщений
#17 лет назад
>лол... ЧПУ (обычно ничего умного в 4 ночи в голову уже не лезет)
>Мне не рябит, превык... Люблю делать такие запросы самостоятельно...
а тот кто будет это поддердивать или дописывать их не любит
>не каждый модуль требует кеширования, но совет учту
конфиг нас спасет )
>файлы кеширования распределяються вручную, оспорений быть не может, тк в каждом модуле может быть несколько кешей... Кеш может иметь своё имя и т.д.
значит модули стоит дробить на более мелкие кусочки...
>хочеться всё своё... Разве-что веб редактор будет чужой...
возмите хорошие идеи и включите их в свое

>6 Кб кода тоже не очень много))) Делиться не нужно, покажите...
ваше портфолио размерами кодовой базы тоже не изобилует, а код написан полность мной, без подглядываний в чужие наработки с использованием мана и спецификаци. UI пока не готов, поэтому и показывать нечего, всему свое время...
Андрей Д.
1267 сообщений
#17 лет назад
Моя кмс как раз и построена на хороших идеях, потому то, что вы предлагаете, уже наверняка отвергли сторонние разработчики (и не просто так)
Андрей Д.
1267 сообщений
#17 лет назад
Цитата:
а тот кто будет это поддердивать или дописывать их не любит

рассчитано, что основную массу будет делать наша студия
Николай Т.
205 сообщений
#17 лет назад
Zionit: вы главное биатрикс не напишите очередной, а разработчиков хороших я привел и у них есть чему поучиться
Андрей Д.
1267 сообщений
#17 лет назад
Цитата ("e1it3"):
zionit: вы главное биатрикс не напишите очередной, а разработчиков хороших я привел и у них есть чему поучиться


Цель - быстрота, простота, мин нагрузка... О изменении шаблонизатора подумаю, но мне эта идея не оч нравиться
Николай Т.
205 сообщений
#17 лет назад
Ну что поделать если автор qicky (wp from phpclub) не донес его с форума до сорсфоржа, тогдаб я вам смарти не советовал, смарти хорош тем что имеет хорошую документацию и кучу плагинов, но за это приходится мириться со скоростью