Задать вопрос
Не нравятся результаты поиска? Попробуйте другой поиск!
DLE FAQ » Все вопросы » Общие вопросы по PHP » Помогите редактировать запрос

Помогите редактировать запрос

     28.09.2013    Все вопросы » Общие вопросы по PHP    4182

вопрос
Здравствуйте, есть вот этот код


$rows = $db->super_query( "SELECT COUNT(*) as count FROM " . PREFIX . "_post WHERE category = '1' AND autor = '$user'");
    $games_num = $rows['count'];
    $tpl->set('{games-num}', $games_num);


в шаблоне я пишу {games-num} и выводит новости которые добавил пользователь в какой-то категории.

Проблема в том что у меня много категорий и делать много запросов не хорошо. Так вот помогите подредактировать этот код таким образом чтобы в шаблоне userinfo.tpl было так {games-num-1} где 1 это ID категории.

Ответил: ПафНутиЙ

Модуль
Вывод новостей текущего юзера:
{include file="engine/modules/user.news.counter.php?&userName=this&catId=1"}

Вывод новостей админа:
{include file="engine/modules/user.news.counter.php?&userName=admin&catId=5"}

Префикс кеша лучше оставить как есть, тогда кеш будет чиститься только при добавлении новостей.


UPD (dj-avtosh)

https://dle-faq.ru/faq/phpquest/6922-pomogite-redaktirovat-zapros.html#comment-id-17195
Отредактировал 29-09-2013, 14:18 - dj-avtosh
Причина: UPD BY DJ-AVTOSH

31 комментарий

dj-avtosh
PHP-developer

dj-avtosh - 28 сентября 2013 11:56 -

Приходит массив id категорий, юзаешь where category IN. Ну и далее в цикле запускаешь шаблонизатор регулярочкой.
По заказам пишем сюда: @Rud00y

ЯД: 41001679231462
Заказы в telegram (ремонт модулей, оптимизация нагрузок и т.п.):
В телегу писать сразу задачу и бюджет.
TheImDim
Юзер

TheImDim - 28 сентября 2013 12:07 -

В таких ситуациях лучше хранить значения отдельно, чем подсчитывать каждый раз.


$rows = $db->super_query( "SELECT `category` FROM " . PREFIX . "_post WHERE `autor` = '$user'");
$row = $db->get_row( $rows );

if( $row ) {
  do {
         ++$games_num[ $row['category'] ];
} while( $row = $db->get_row( $rows ) );
}

foreach( $games_num as $key => $val ) {
   $tpl->set( '{games-num-'.$key.'}', $val );
}

dj-avtosh
PHP-developer

dj-avtosh - 28 сентября 2013 12:36 -

Какой глупый код. Цикл пропускается как ресурс. Фу. Super_query с опущенным вторым параметром - вернет одно значение. Лучше не писать, без теста. Если плохо знаешь php.
По заказам пишем сюда: @Rud00y

ЯД: 41001679231462
Заказы в telegram (ремонт модулей, оптимизация нагрузок и т.п.):
В телегу писать сразу задачу и бюджет.
dj-avtosh
PHP-developer

dj-avtosh - 28 сентября 2013 12:40 -

Далее переменную в запросе пропускаешь как строку, что в 8 раз дольше обработается, еже ли по всем правилам разделить через точки.
По заказам пишем сюда: @Rud00y

ЯД: 41001679231462
Заказы в telegram (ремонт модулей, оптимизация нагрузок и т.п.):
В телегу писать сразу задачу и бюджет.
dj-avtosh
PHP-developer

dj-avtosh - 28 сентября 2013 12:42 -

Да и вообще не указаны типы переменных. Нет проверок, зато есть до и вайл. Понты.
По заказам пишем сюда: @Rud00y

ЯД: 41001679231462
Заказы в telegram (ремонт модулей, оптимизация нагрузок и т.п.):
В телегу писать сразу задачу и бюджет.
dj-avtosh
PHP-developer

dj-avtosh - 28 сентября 2013 12:45 -

Да и вообще не понимаю момент, вопрос же в количестве новостей, при этом, однако, массив категорий приходит глобально как $cat_info, если не ошибаюсь и кэшируется в init.php
По заказам пишем сюда: @Rud00y

ЯД: 41001679231462
Заказы в telegram (ремонт модулей, оптимизация нагрузок и т.п.):
В телегу писать сразу задачу и бюджет.
ПафНутиЙ
Админ

ПафНутиЙ - 28 сентября 2013 22:52 -

не в init.php а в системном кеше.
Каков вопрос - таков и ответ. Просто помните об этом.
dj-avtosh
PHP-developer

dj-avtosh - 29 сентября 2013 04:11 -

Я имел ввиду файл в котором контроллер написания кэша.
По заказам пишем сюда: @Rud00y

ЯД: 41001679231462
Заказы в telegram (ремонт модулей, оптимизация нагрузок и т.п.):
В телегу писать сразу задачу и бюджет.
ПафНутиЙ
Админ

ПафНутиЙ - 29 сентября 2013 11:16 -

а вот это хз)
Каков вопрос - таков и ответ. Просто помните об этом.
FastFud
Юзер

FastFud - 28 сентября 2013 13:28 -

Было бы так просто для меня я бы не задал этот вопрос ).
Можите сделать пример как правильно будет а то я буду пробовать где то месяц.
dj-avtosh
PHP-developer

dj-avtosh - 28 сентября 2013 14:09 -

Ну ты был близок к истине))
По заказам пишем сюда: @Rud00y

ЯД: 41001679231462
Заказы в telegram (ремонт модулей, оптимизация нагрузок и т.п.):
В телегу писать сразу задачу и бюджет.
FastFud
Юзер

FastFud - 28 сентября 2013 14:18 -

так что ли 


$rows = $db->super_query( "SELECT COUNT(*) as count FROM " . PREFIX . "_post WHERE category IN ('" . implode ( "','", $cat_info[$category_id] ) . "') AND autor = '$user'");
TheImDim
Юзер

TheImDim - 28 сентября 2013 14:21 -

С super_query согласен, не заметил.

Да и вообще не указаны типы переменных. Нет проверок


Что за бред ты несешь? Я тут привел алгоритм каким образом можно подсчитать количество новостей у пользователя. Расписывать тут все я не собираюсь. Если есть желание и время, напиши скрипт, а не неси ерунду. А то на словах Лев Толстой, а на деле.
dj-avtosh
PHP-developer

dj-avtosh - 28 сентября 2013 17:48 -

Я знаю что я плохой программист)) просто любой язык кроме пэхэпэ :D не компилирует код без приведения типа. Это я тебе как Лев Толстой говорю. На деле я принимал участие в разработке сайта, на котором ты сейчас комментируешь, это так на заметку ;)
По заказам пишем сюда: @Rud00y

ЯД: 41001679231462
Заказы в telegram (ремонт модулей, оптимизация нагрузок и т.п.):
В телегу писать сразу задачу и бюджет.
dj-avtosh
PHP-developer

dj-avtosh - 28 сентября 2013 17:59 -

Я для тебя ещё внизу про двойное создание переменной написал. Это есть кул))) Оперативы же много нынче на серваках)))
По заказам пишем сюда: @Rud00y

ЯД: 41001679231462
Заказы в telegram (ремонт модулей, оптимизация нагрузок и т.п.):
В телегу писать сразу задачу и бюджет.
TheImDim
Юзер

TheImDim - 7 октября 2013 14:03 -

Где у меня двойное создание переменной? Ну да, этот процесс занимает целый гигабайт оперативной памяти в php. Вообще удивляют люди, которые выучат скриптовый язык и считают себя "программистами". А потом учат других жизни. Я не заявляю о себе как супер-пупер программист, и вам не советую заниматься балабольством, если конечно у Вас есть крупные проекты над которомы работали, покажите нам, чтобы мы знали с кем имеем дело, и не надо говорить что это dle-faq.ru.
dj-avtosh
PHP-developer

dj-avtosh - 7 октября 2013 14:38 -

По заказам пишем сюда: @Rud00y

ЯД: 41001679231462
Заказы в telegram (ремонт модулей, оптимизация нагрузок и т.п.):
В телегу писать сразу задачу и бюджет.
TheImDim
Юзер

TheImDim - 28 сентября 2013 18:12 -

Ну почему же любой язык, динамическая типизация есть и в ruby, python, perl и не только. Я не говорю, и не говорил что мой код оптимизирован и является обзразцовым, я лишь подсказываю как это можно сделать.
dj-avtosh
PHP-developer

dj-avtosh - 28 сентября 2013 18:14 -

No problem. Просто не стоит оценивать мои знания своими догадками. Тем более мой опыт работы. На личности переходить как то по юношески.
По заказам пишем сюда: @Rud00y

ЯД: 41001679231462
Заказы в telegram (ремонт модулей, оптимизация нагрузок и т.п.):
В телегу писать сразу задачу и бюджет.
budda555
Юзер

budda555 - 28 сентября 2013 19:49 -

Я вот наблюдаю за беседой и пришел к выводу, что Вы просто неадекватны, так как комментировать подобным образом мысли гениального программиста вроде dj-avtosh-а, просто бессмысленно.
Занялись бы делом )
FastFud
Юзер

FastFud - 28 сентября 2013 14:24 -

TheLmDim

вот так работает без super_ как сказали )


    $rows = $db->query( "SELECT `category` FROM " . PREFIX . "_post WHERE `autor` = '$user'");
$row = $db->get_row( $rows );
 
if( $row ) {
  do {
         ++$games_num[ $row['category'] ];
} while( $row = $db->get_row( $rows ) );
}
 
foreach( $games_num as $key => $val ) {
   $tpl->set( '{games-num-'.$key.'}', $val );
}
FastFud
Юзер

FastFud - 28 сентября 2013 14:29 -

Есть 1 но

Исчезает количество новостей в целом и комментариев
FastFud
Юзер

FastFud - 28 сентября 2013 14:46 -

вот так работает 


        $rows = $db->query( "SELECT `category` FROM " . PREFIX . "_post WHERE `autor` = '$user'");
        $rowz = $db->get_row( $rows );
 
if( $rowz ) {
  do {
         ++$m_num[ $rowz['category'] ];
}   while( $rowz = $db->get_row( $rows ) ); 
}
 
foreach( $m_num as $key => $val ) {
   $tpl->set( '{m-num-'.$key.'}', $val );
}


Но остается вопрос, правильно ли.
dj-avtosh
PHP-developer

dj-avtosh - 28 сентября 2013 17:57 -

Мне нравится два раза подряд создание переменной $rowz, Анатоле бы не одобрил :D
По заказам пишем сюда: @Rud00y

ЯД: 41001679231462
Заказы в telegram (ремонт модулей, оптимизация нагрузок и т.п.):
В телегу писать сразу задачу и бюджет.
ПафНутиЙ
Админ

ПафНутиЙ - 28 сентября 2013 22:53 -

А не проще ли оформить модулем отдельным, что-то многовато кода лишнего на вскидку, к том же без кеша.?
Каков вопрос - таков и ответ. Просто помните об этом.
dj-avtosh
PHP-developer

dj-avtosh - 29 сентября 2013 11:18 -

Попробуй, тестить негде было:



 /**
  * Подсчет количества постов по каждой категории у определенного юзера
  */

      // Получим category для данного автора

      $pUserArray = $db->super_query ( "SELECT category FROM " . PREFIX . "_post WHERE autor = '" . $user . "'", TRUE );

      // Т.к. возможна мультикатегорийность, то пройдемся по каждой полученной категории

      if ( ! empty ( $pUserArray ) )
      {

              $countFromCategory = array();

              foreach ( $pUserArray as $_pua )
              {

                  $categoryArr = explode ( ',', $_pua['category'] );

                  if ( ! empty ( $categoryArr ) )
                  {

                      if ( is_array ( $categoryArr ) )
                      {

                          foreach ( $categoryArr as $_ca )
                          {

                              ! isset ( $countFromCategory[$_ca] ) ? $countFromCategory[$_ca] = 1 : $countFromCategory[$_ca]++;

                          }

                      } else {

                          ! isset ( $countFromCategory[$categoryArr] ) ? $countFromCategory[$categoryArr] = 1 : $countFromCategory[$categoryArr]++;

                      }

                  } else {

                      continue;

                  }

              }


              if ( ! empty ( $countFromCategory ) )
              {

                  foreach ( $countFromCategory as $_cID => $_cfc )
                  {

                      $tpl->set( '{m-num-' . $_cID . '}' , $_cfc );

                  }

              }

      }

По заказам пишем сюда: @Rud00y

ЯД: 41001679231462
Заказы в telegram (ремонт модулей, оптимизация нагрузок и т.п.):
В телегу писать сразу задачу и бюджет.
FastFud
Юзер

FastFud - 29 сентября 2013 12:12 -

dj-avtosh
Да работает, спасибо но если пользователь ничего не добавлял то остаются только теги {m-num-1}

Я добавил еще:


$tpl->set_block( "'\\[unews-num\\](.*?)\\[/unews-num\\]'si", "" );
                        $tpl->set( '[not-unews-num]', "" );
                        $tpl->set( '[/not-unews-num]', "" );


Ну и теперь работает как надо.
dj-avtosh
PHP-developer

dj-avtosh - 29 сентября 2013 14:17 -

Молодец
По заказам пишем сюда: @Rud00y

ЯД: 41001679231462
Заказы в telegram (ремонт модулей, оптимизация нагрузок и т.п.):
В телегу писать сразу задачу и бюджет.
Nagibaka
Эксперт

Nagibaka - 7 октября 2013 14:58 -

После прочтения этого увлекательного топика, сразу захотелось добавить новую фичу на сайт - около автарки добавить счетчики Epic Fail(ну или баттхерт) и Epic Win для каждого пользователя.
Frontend MotherFucker=:^^
TheImDim
Юзер

TheImDim - 7 октября 2013 15:52 -

Не забудьте добавить кнопочку "Пофлудить" (ну или нести херню)
dj-avtosh
PHP-developer

dj-avtosh - 7 октября 2013 16:16 -

Андрей, не спорь с программистом))) Он же крут))
По заказам пишем сюда: @Rud00y

ЯД: 41001679231462
Заказы в telegram (ремонт модулей, оптимизация нагрузок и т.п.):
В телегу писать сразу задачу и бюджет.
Чтобы комментировать - войдите или зарегистрируйтесь на сайте

Похожие вопросы

 28.06.2012 promax  Общие вопросы по PHP, MySQL
 07.05.2013 onneuha  Общие вопросы по PHP, Модули, Ламерские вопросы
 03.06.2013 FastFud  Общие вопросы по PHP
 19.08.2013 KDenis  Общие вопросы по PHP, Модули, MySQL
 23.09.2013 FastFud  Общие вопросы по PHP
 10.01.2014 FastFud  Общие вопросы по PHP
 08.06.2014 Amilo130  Общие вопросы по PHP
 11.06.2014 blocbaster  Общие вопросы по PHP
 29.01.2015 kent  Общие вопросы по PHP
 17.02.2015 Goot  Общие вопросы по PHP, Модули, MySQL
  • mondolfo | Шаблон waitlogin при авторизации через с ...
    Фото mondolfo
    mondolfo
    Шаблон waitlogin при авторизации через соцсеть?
    Вчера, 01:31
    /engine/modules/social.php
  • TeraMoune | Как изменить вывод custom по нескольким ... 11
    Фото TeraMoune
    TeraMoune
    Как изменить вывод custom по нескольким тегам?
    28-10-2024, 01:05
    Я же написал, продублирую: нужно использовать [[:punct:]]

    И проблема в условии проверяющее версию mysql, можно поменять местами true и false, изменить знак меньше на больше или вовсе убрать код строки с version_compare и оставить ту строку кода $tempArray[] где используется [[:punct:]].

    В статье также указано, что важно использовать экранирование, собственно, что и указано в методе по исправлению ошибки.
    Статья по решению проблемы

    Там у человека совершенно другое условие поиска. Он не теги ищет которые строго разделены запятыми, он просто слова ищет и ему как раз подойдёт граница слова. И это когда-то давно поиск был разделяя по границе слова, теперь есть character_class и используется другой метод. Но это не отменяет того, что вы нашли вопрос и решение которое никак не связано с темой новости поиска новостей с наличием тегов.
  • homa152 | Как изменить вывод custom по нескольким ... 11
    Фото homa152
    homa152
    Как изменить вывод custom по нескольким тегам?
    27-10-2024, 23:33
    Если вы готовы предложить какое-то решение для исправление проблемы 3685, отлично. Иначе к сожалению на последних версиях MySQL хак не рабочий, согласно вашему трактату.
  • homa152 | Как изменить вывод custom по нескольким ... 11
    Фото homa152
    homa152
    Как изменить вывод custom по нескольким тегам?
    27-10-2024, 23:20
    Я буквально 2 часа назад столкнулся с проблемами при установке этого хака, проблема была именно в ошибке 3685, сайт не просто переставал работать, использовалась стандартная заглушка при критических ошибках MySQL исправить получилось методом описанным выше, для версия до 8 хак работает исправно, но выше, требуется доработка, так как синтаксис считается ошибочным, вы можете проверить самостоятельно.

    В статье также указано, что важно использовать экранирование, собственно, что и указано в методе по исправлению ошибки.
    Статья по решению проблемы
  • TeraMoune | Как изменить вывод custom по нескольким ... 11
    Фото TeraMoune
    TeraMoune
    Как изменить вывод custom по нескольким тегам?
    27-10-2024, 22:32
    Напротив нужно использовать [[:punct:]] вместо скобок, про это написано в документации на которую вы ссылаетесь. А \b означает границу слова и не способен правильно отображать результаты. Граница слова это когда выражение будет находить отдельные слова. Например новость имеет теги (Социальная сеть, Блокировка) и tags REGEXP '\\b(сеть)\\b' найдёт новость с тегом Социальная сеть, а так же Рыбацкая сеть и в принципе найдёт все новости где присутствует слово "Сеть" и будет много лишних результатов, а не только нужные.

    В хаке как раз присутствует проверка на старую версию и новую mysql и в каждом случае используется тот вариант который является валидным для используемой версии.
  • homa152 | Как изменить вывод custom по нескольким ... 11
    Фото homa152
    homa152
    Как изменить вывод custom по нескольким тегам?
    27-10-2024, 21:30
    Добавлю касаемо обновления архитектуры MySQL
    При добавлении хака с версии MySQL 8.0, возможно появление ошибки: 3685. Неверный аргумент для регулярного выражения.

    Можно пофиксить самостоятельно внеся корректировки в сам хак из панели админа.

    Строку:
    $tempArray[] = "tags REGEXP '[[:<:]](" . $value  . ")[[:>:]]'";


    Заменить на: 
    $tempArray[] = "tags REGEXP '\\\\b(" . $value  . ")\\\\b'";


    Согласно справочному руководству MySQL 8.0 / ... / Регулярные выражения, «MySQL поддерживает регулярные выражения с помощью международных компонентов для Unicode (ICU)».

    То есть, вместо "[[:<:]] и [[:>:]]", нужно использовать: "\b"
  • vitnet | Есть ли возможность изменить язык сайта? 1
    Фото vitnet
    vitnet
    Есть ли возможность изменить язык сайта?
    25-10-2024, 16:22
    Ну как вариант можно использовать тег {lang text=***} подробнее


    {lang text="LANG_PARAMETER"}

    Данный тег предназначен для вывода информации из языкового файла website.tpl, который в данный момент используется, где LANG_PARAMETER это значение параметра, который необходимо вывести. Например, тег {lang text="attach_images"} выведет надпись “Прикрепить изображения” из языкового пакета, в случае использования русского языка и если используется английский язык, то “Attach Images”
  • vitnet | Есть готовый модуль рейтинга комментарие ... 1
    Фото vitnet
    vitnet
    Есть готовый модуль рейтинга комментариев?
    25-10-2024, 13:12
    Вариант 1 (вручную)

    1.0 Открываете файл engine/modules/show.full.php находите в нем
    $where_approve . " ORDER BY " . PREFIX . "_comments.id " . $comm_msort;

    И меняете здесь id на rating
    Пример:
    $where_approve . " ORDER BY " . PREFIX . "_comments.rating " . $comm_msort;

    2.0 Затем переходите в админку /admin.php?mod=options&action=syscon --> Настройки комментариев --> Порядок сортировки комментариев --> По убыванию

    2.1 Далее --> Тип рейтинга комментариев выставляете "Нравится или Не нравится"

    Готово!

    Вариант 2 (плагин)
    Для просмотра содержимого спойлера, перейдите к выбранному событию.
  • Graf | Как сделать аякс кнопку "загрузить еще"? 2
    Фото Graf
    Graf
    Как сделать аякс кнопку "загрузить еще"?
    23-10-2024, 16:03
    Вот рабочий пример со скрытие пагинатора и кнопки.
    Полный код файла navigation.tpl
    [not-available=userinfo]
<div class="nav-load" id="nav-load">[next-link]Загрузить еще[/next-link]</div>

<nav class="navigation"[available=lastcomments] style="padding: 4% 8%;"[/available]>
	<div class="page_next-prev">
		<span class="page_prev" title="Предыдущая страница">
			[prev-link]<svg class="icon icon-left"><use xlink:href="#icon-left"></use></svg>[/prev-link]
		</span>
		<span class="page_next" title="Следующая страница">
			[next-link]<svg class="icon icon-right"><use xlink:href="#icon-right"></use></svg>[/next-link]
		</span>
	</div>
	<ul class="pagination">
		{pages}
	</ul>
</nav>
<script type="text/javascript">
$(document).ready(function(){
    
         $('body').on('click','#nav-load a',function(){
        var urlNext = $(this).attr('href');
        var scrollNext = $(this).offset().top - 200;
        if (urlNext !== undefined) {
            $.ajax({
                url: urlNext,
                beforeSend: function() {
                    ShowLoading('');
                },            
                success: function(data) {
                    $('.navigation').remove();
                    $('#nav-load').remove();
                    $('#dle-content').append($('#dle-content', data).html());
                    $('#dle-content').after($('#bottom-nav'));
                    window.history.pushState("", "", urlNext);
                    $('html, body').animate({scrollTop:scrollNext}, 800);    
                    HideLoading('');
                },
                  error: function() {                
                    HideLoading('');
                    alert('что-то пошло не так');
                  }
            });
        };
        return false;
    });
    
});
</script>
[/not-available]
  • vitnet | Как перенести категории в DLE? 2
    Фото vitnet
    vitnet
    Как перенести категории в DLE?
    22-10-2024, 11:30
    Для начала проверьте наличие файла /engine/cache/system/category.php и если он был до замены БД то удалите его а ещё лучше очистить кеш.
наверх