Не нравятся результаты поиска? Попробуйте другой поиск!

DLE FAQ » Все вопросы » Общие вопросы по PHP » Помогите редактировать запрос

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

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

вопрос

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


$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 комментарий

PHP-developer

3198

dj-avtosh - 2015-02-1728 сентября 2013 11:56 - #17147

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

https://elkhan.ru
По заказам пишем сюда: @Rud00y

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

Юзер

324

TheImDim - 2015-02-1728 сентября 2013 12:07 - #17149

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


$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 );
}

PHP-developer

3198

dj-avtosh - 2015-02-1728 сентября 2013 12:36 - #17151

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

PHP-developer

3198

dj-avtosh - 2015-02-1728 сентября 2013 12:40 - #17152

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

PHP-developer

3198

dj-avtosh - 2015-02-1728 сентября 2013 12:42 - #17153

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

PHP-developer

3198

dj-avtosh - 2015-02-1728 сентября 2013 12:45 - #17154

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

Админ

3457

ПафНутиЙ - 2015-02-1728 сентября 2013 22:52 - #17183

не в init.php а в системном кеше.

Каков вопрос - таков и ответ. Просто помните об этом.

PHP-developer

3198

dj-avtosh - 2015-02-1729 сентября 2013 04:11 - #17186

Я имел ввиду файл в котором контроллер написания кэша.

Админ

3457

ПафНутиЙ - 2015-02-1729 сентября 2013 11:16 - #17194

а вот это хз)

Каков вопрос - таков и ответ. Просто помните об этом.

Юзер

1599

FastFud - 2015-02-1728 сентября 2013 13:28 - #17156

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

PHP-developer

3198

dj-avtosh - 2015-02-1728 сентября 2013 14:09 - #17157

Ну ты был близок к истине))

Юзер

1599

FastFud - 2015-02-1728 сентября 2013 14:18 - #17158

так что ли


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

Юзер

324

TheImDim - 2015-02-1728 сентября 2013 14:21 - #17159

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

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

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

PHP-developer

3198

dj-avtosh - 2015-02-1728 сентября 2013 17:48 - #17167

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

PHP-developer

3198

dj-avtosh - 2015-02-1728 сентября 2013 17:59 - #17169

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

Юзер

324

TheImDim - 2015-02-177 октября 2013 14:03 - #17582

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

PHP-developer

3198

dj-avtosh - 2015-02-177 октября 2013 14:38 - #17586

http://ideer.ru

Юзер

324

TheImDim - 2015-02-1728 сентября 2013 18:12 - #17171

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

PHP-developer

3198

dj-avtosh - 2015-02-1728 сентября 2013 18:14 - #17172

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

Юзер

budda555 - 2015-02-1728 сентября 2013 19:49 - #17179

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

Юзер

1599

FastFud - 2015-02-1728 сентября 2013 14:24 - #17160

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 );
}

Юзер

1599

FastFud - 2015-02-1728 сентября 2013 14:29 - #17161

Есть 1 но

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

Юзер

1599

FastFud - 2015-02-1728 сентября 2013 14:46 - #17162

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


        $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 );
}

Но остается вопрос, правильно ли.

PHP-developer

3198

dj-avtosh - 2015-02-1728 сентября 2013 17:57 - #17168

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

Админ

3457

ПафНутиЙ - 2015-02-1728 сентября 2013 22:53 - #17184

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

Каков вопрос - таков и ответ. Просто помните об этом.

PHP-developer

3198

dj-avtosh - 2015-02-1729 сентября 2013 11:18 - #17195

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



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

      // Получим 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 );

                  }

              }

      }

Юзер

1599

FastFud - 2015-02-1729 сентября 2013 12:12 - #17199

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

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


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

Ну и теперь работает как надо.

PHP-developer

3198

dj-avtosh - 2015-02-1729 сентября 2013 14:17 - #17202

Молодец

Эксперт

Nagibaka - 2015-02-177 октября 2013 14:58 - #17587

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

Frontend MotherFucker=:^^

Юзер

324

TheImDim - 2015-02-177 октября 2013 15:52 - #17588

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

PHP-developer

3198

dj-avtosh - 2015-02-177 октября 2013 16:16 - #17590

Андрей, не спорь с программистом))) Он же крут))