Не долго думая, скачал я «идеальные» менюшки (Jquerymenu) и увидел кучку «говнокода».
drupal.org/project/Jquerymenu
\\jquerymenu.module\\
function jquerymenu_block($op = 'list', $delta = 0, $edit = array()): 120
$edit — не известно для чего параметр, просто чтобы был, чисто по факту, чем больше параметров, тем более увесисто и значимо выглядит.
foreach ($enabledmenus as $menuname) { :142
if ($delta == $d) {
if ($menuname == 'navigation' && !empty($user->uid)) {
$title = $user->name;
}
else {
$title = db_result(db_query(«SELECT title FROM {menu_custom} WHERE menu_name = '%s'», $menuname));
}
Запросы в цикле ждут нас. Не вижу, что мешает сделать все в 1 запрос.
function jquerymenu_init() { :11
drupal_add_css(drupal_get_path('module', 'jquerymenu') .'/jquerymenu.css');
if (variable_get('jq_menu_animate', 0) == 0) {
drupal_add_js(drupal_get_path('module', 'jquerymenu') .'/jquerymenu.js');
}
if (variable_get('jq_menu_animate', 0) == 1) {
drupal_add_js(drupal_get_path('module', 'jquerymenu') .'/jquerymenu_no_animation.js');
}
}
Крутил, вертел, даже сквозь пальцы смотрел. Но эти шаманские уловки не открыли мне тайну 2х If подряд из которых может выполниться только 1 (=?: — в помощь).
хардкод и полное отсутствие ООП обсуждать бессмысленно, ибо это фишка Друпала и модуля в частности.
Заключающее Имхо, лучше потратить пару часов времени для изучения РНР и написания своего функционала, чем использовать подобный «идеальный» код, приобретая головную боль и ненужные нагрузки на проект — «бесплатно».
Я считаю, что «пошло» называть модулем то, что содержит в себе пару функций (РНР), большинство которых Вы и сами можете реализовать за 1-2ч, что в результате может быть на порядок лучше для Вашего проекта.
При установке настроек stright_hands.sys, большинство выше отмеченных модулей Вам уже не потребуется. Ибо Вы уже сами решите задачу более элегантно и удобно для Ваших нужд. =)
>он давно стал ей, AWARD уже дважды об этом объявил
Ииии? Есть так пиарить по миру — то почему бы и нет? Я знаю множество самописных решений, решающих узкий круг задач просто на 5+. Любая большая ЦМС, при решении подобных задач будет просто саксом, но никто не выдвинет на премию малоизвестную и специализированную разработку.
>а пресловутый битрикс — вот это точно говно, идеи тыреные
товарищ, в любой ЦМС 95% идей тыреные,
форум — стырили,
блоги — стырили,
и-магазин — стырили
Вот только не надо меланхоличного бреда про «Воровство» идей, оригинальных идей во всех ЦМС можно по пальцам посчитать.
>а их черный пиар друпала, когда заказные статьи против капли
Вас смешно читать. Заходим на форум друпала — читаем «Битрикс» говно, Заходим на форум Битрикса — «Друпал» сакс.
Это «Холивар», и здесь нет ПРАВЫХ и Виноватых. Каждый выбирает именно то, чем он умеет пользоваться и результат работы устраивает клиента\потребителя.
Меня вот всегда удивляло, неужто нельзя сказать, например, «Битрикс, Юми, Тупо3 неплохие системы, но я все же предпочитаю Друпал», так нет =) мы будем как в дет саду рвать тельняшки и кричать — «XXX» forever, «YYY» sux!
Не в этом дело. Есть вещи платные, есть бесплатные. Каждый сам уже определяет чем именно он хочет воспользоваться.
Также, автомобиль не сравним с жизненно-необходимыми вещами, однако мы можем купить и бентли и запорожец.
В общем я к чему, к тому что и бесплатное\платное может быть говном\хорошим.
Безполезный холивар.
На _каждую_ систему найдется свой _пользователь_ и\или _разработчик_
И говорить, что что-то сакс, а что-то рулит как-то по детски.
Не вижу большой пользы в использовании данного подхода. Объясняется очень просто:
1) Пользователь заходит на сайт, начинает грузиться текст\изображения
2) Видит, что ему не подходит и уходит с сайта.
В Вашем же случаи, Вы подарите пользователю 3+ мб ненужного трафика(ожидание) и соответствующую реакцию.
Особенно это будет заметно на сайте с большим объемом графической информации для пользователя с _НЕ_скоростным интернетом.
Идея конечно интересная, но еще над кодом надо поработать.
1) выделите все в класс, тяжело читать длиииииииинный код, да и вижу повторы в коде.
2) вместо require($_SERVER[«DOCUMENT_ROOT»]."/bitrix/header.php"); в данном случаи лучше писать require($_SERVER[«DOCUMENT_ROOT»]."/bitrix/modules/main/include/prolog_before.php");
3) было бы более замечательно если бы скрипт сам бы создавал ИБ и необходимые секции в нем
>А в общем — непонятный говнокод.
А у Вас самый понятный говнокод, несомненно. Человек старался, а Вы тут со своим кг\ам, некрасиво имхо.
>Ужас. Как и весь Битрикс. Коллеге, написавшему «include( $file );» мы бы всей конторой ритуально разбили голову об монитор.
Конструкции вида
for($i=0;$i<3;$i++)
include 'http://www.example.com/file.php?foo='.$i.'&bar='.$i;
До вашей конторы, видимо, никода не доходили.
ЗЫ. все ритуалите и поклоняетесь Ктулхе? Тогда мы идем к вам!
drupal.org/project/Jquerymenu
\\jquerymenu.module\\
function jquerymenu_block($op = 'list', $delta = 0, $edit = array()): 120
$edit — не известно для чего параметр, просто чтобы был, чисто по факту, чем больше параметров, тем более увесисто и значимо выглядит.
foreach ($enabledmenus as $menuname) { :142
if ($delta == $d) {
if ($menuname == 'navigation' && !empty($user->uid)) {
$title = $user->name;
}
else {
$title = db_result(db_query(«SELECT title FROM {menu_custom} WHERE menu_name = '%s'», $menuname));
}
Запросы в цикле ждут нас. Не вижу, что мешает сделать все в 1 запрос.
function jquerymenu_init() { :11
drupal_add_css(drupal_get_path('module', 'jquerymenu') .'/jquerymenu.css');
if (variable_get('jq_menu_animate', 0) == 0) {
drupal_add_js(drupal_get_path('module', 'jquerymenu') .'/jquerymenu.js');
}
if (variable_get('jq_menu_animate', 0) == 1) {
drupal_add_js(drupal_get_path('module', 'jquerymenu') .'/jquerymenu_no_animation.js');
}
}
Крутил, вертел, даже сквозь пальцы смотрел. Но эти шаманские уловки не открыли мне тайну 2х If подряд из которых может выполниться только 1 (=?: — в помощь).
хардкод и полное отсутствие ООП обсуждать бессмысленно, ибо это фишка Друпала и модуля в частности.
Заключающее Имхо, лучше потратить пару часов времени для изучения РНР и написания своего функционала, чем использовать подобный «идеальный» код, приобретая головную боль и ненужные нагрузки на проект — «бесплатно».
Ииии? Есть так пиарить по миру — то почему бы и нет? Я знаю множество самописных решений, решающих узкий круг задач просто на 5+. Любая большая ЦМС, при решении подобных задач будет просто саксом, но никто не выдвинет на премию малоизвестную и специализированную разработку.
>а пресловутый битрикс — вот это точно говно, идеи тыреные
товарищ, в любой ЦМС 95% идей тыреные,
форум — стырили,
блоги — стырили,
и-магазин — стырили
Вот только не надо меланхоличного бреда про «Воровство» идей, оригинальных идей во всех ЦМС можно по пальцам посчитать.
>а их черный пиар друпала, когда заказные статьи против капли
Вас смешно читать. Заходим на форум друпала — читаем «Битрикс» говно, Заходим на форум Битрикса — «Друпал» сакс.
Это «Холивар», и здесь нет ПРАВЫХ и Виноватых. Каждый выбирает именно то, чем он умеет пользоваться и результат работы устраивает клиента\потребителя.
Меня вот всегда удивляло, неужто нельзя сказать, например, «Битрикс, Юми, Тупо3 неплохие системы, но я все же предпочитаю Друпал», так нет =) мы будем как в дет саду рвать тельняшки и кричать — «XXX» forever, «YYY» sux!
Также, автомобиль не сравним с жизненно-необходимыми вещами, однако мы можем купить и бентли и запорожец.
В общем я к чему, к тому что и бесплатное\платное может быть говном\хорошим.
лишний пробел после:
>(не )?
На _каждую_ систему найдется свой _пользователь_ и\или _разработчик_
И говорить, что что-то сакс, а что-то рулит как-то по детски.
1) Пользователь заходит на сайт, начинает грузиться текст\изображения
2) Видит, что ему не подходит и уходит с сайта.
В Вашем же случаи, Вы подарите пользователю 3+ мб ненужного трафика(ожидание) и соответствующую реакцию.
Особенно это будет заметно на сайте с большим объемом графической информации для пользователя с _НЕ_скоростным интернетом.
>обиженых
>завидующих
Все ПО, Решения имеет права на существование, но вот хабростадонегатив, порою утомляет.
1) выделите все в класс, тяжело читать длиииииииинный код, да и вижу повторы в коде.
2) вместо require($_SERVER[«DOCUMENT_ROOT»]."/bitrix/header.php"); в данном случаи лучше писать require($_SERVER[«DOCUMENT_ROOT»]."/bitrix/modules/main/include/prolog_before.php");
3) было бы более замечательно если бы скрипт сам бы создавал ИБ и необходимые секции в нем
>А в общем — непонятный говнокод.
А у Вас самый понятный говнокод, несомненно. Человек старался, а Вы тут со своим кг\ам, некрасиво имхо.
>Ужас. Как и весь Битрикс. Коллеге, написавшему «include( $file );» мы бы всей конторой ритуально разбили голову об монитор.
Конструкции вида
for($i=0;$i<3;$i++)
include 'http://www.example.com/file.php?foo='.$i.'&bar='.$i;
До вашей конторы, видимо, никода не доходили.
ЗЫ. все ритуалите и поклоняетесь Ктулхе? Тогда мы идем к вам!