[Проблема]

Сломал мозг.

Так правильно составлен запрос к бд или нет?
need help...

PHP Code:

$query19 = mysql_query("SELECT *  FROM `schet` WHERE `id` = $inv_id");

$dow = mysql_fetch_array($query19);
$userid = $dow['userid'];
$credits = $dow['kredit'];

$query20 = $vbulletin->db->query_first("SELECT usergroupid FROM " . TABLE_PREFIX . "user WHERE `userid` = '".$userid."'");

if ($query20['usergroupid'] == 16)
{
$db = mysql_query("UPDATE `user` SET `credit_v` = `credit_v` + '".$credits."' WHERE `userid` = '".$userid."'");
} else {
$db = mysql_query("UPDATE `user` SET `credits` = `credits` + '".$credits."' WHERE `userid` = '".$userid."'");
}
echo "OK$inv_id\n";

} 


Больше волнует код, как начинается с $query20

Нужно чтоб через ID юзера узнать номер группы и далее условие для этой группы

несколько поправок...
если выбираются только значения из двух полей таблицы, зачем указывать * (астериск), т.е. все поля?
потом там ошибка синтаксиса в куске кода, не закрыта круглая скобка, а вообще я бы записал код так:

PHP Code:

$query19 = mysql_query("SELECT userid, kredit FROM `schet` WHERE `id` = " . intval($inv_id));

$dow = mysql_fetch_array($query19); 
$userid = $dow['userid']; 
$credits = $dow['kredit']; 

if($userid AND $credits)
{
      $query20 = $vbulletin->db->query_first("SELECT usergroupid FROM " . TABLE_PREFIX . "user WHERE `userid` = $userid "); 

      if($query20)
      {
            if($query20['usergroupid'] == 16) 
            {
                  $db = mysql_query("UPDATE `user` SET `credit_v` = `credit_v` + $credits WHERE `userid` = $userid "); 
            }else{
                  $db = mysql_query("UPDATE `user` SET `credits` = `credits` + $credits WHERE `userid` = $userid "); 
            }
            echo "OK$inv_id\n";
      }
} 


т.е. делать все возможные проверки

Как ни странно - теперь перестало условие выполняться.
А может оно и до этого не выполнялось((

Использую такой код

PHP Code:

$querys = mysql_query("SELECT *  FROM `credits_schet` WHERE `inveid` = $inv_id");
$dow = mysql_fetch_array($querys);
$userid = $dow['userid'];
$credits = $dow['kreditid'];
if($userid AND $credits)
{
      $query20 = mysql_query("SELECT usergroupid FROM `user` WHERE `userid` = $userid ");

      if($query20)
      {
            if($query20['usergroupid'] == 16)
            {
                  $db = mysql_query("UPDATE `user` SET `credit_v` = `credit_v` + $credits WHERE `userid` = $userid ");
            }else{
                  $db = mysql_query("UPDATE `user` SET `credits` = `credits` + $credits WHERE `userid` = $userid ");
            }
            echo "OK$inv_id\n";
      }
} 


т.к. на $vbulletin->db->query_first ругалось и выдавало ошибку.

Не зависимо от группы всегда пополняются credit_v
Такое ощущение что не видит usergroupid

Главное что пополнение проходит - значит все условия (кроме последнего) выполняются.

Натолкните на мысль, из-за чего такое может быть?

16 группа существует!

К скрипту вобловский глобал подключен?

PHP Code:

$r=$db->query_read_slave("SELECT userid,kreditid  FROM " . TABLE_PREFIX . "credits_schet WHERE inveid = '$inv_id'");
$dow=$db->fetch_array($r);
$userid = $dow['userid'];
$credits = $dow['kreditid'];
if($userid AND $credits)
{
$rs=$db->query_read_slave("SELECT usergroupid FROM " . TABLE_PREFIX . "user WHERE `userid` = $userid ");
$query20=$db->fetch_array($rs);
      if($query20)
      {
            if($query20['usergroupid'] == 16)
            {
                  $vbulletin->db->query_write("UPDATE `user` SET `credit_v` = `credit_v` + $credits WHERE `userid` = '$userid'");
            }else{
                  $vbulletin->db->query_write("UPDATE `user` SET `credits` = `credits` + $credits WHERE `userid` = $userid ");
            }
            echo "OK{$inv_id}\n";
      }
} 


ХЗ, возможно я просто параноик.


PS.

PHP Code:

 $query20 = mysql_query("SELECT usergroupid FROM " . TABLE_PREFIX . "user WHERE `userid` = $userid "); 


Ты тут запрос выполнил, а данные не за фечел из массива.

Т.к. в коде керка была дефолтная функция из класса воблы по работе с бд, которая тянула первое значение.

Нет не подключал.
Без него разве нельзя обойтись?

Таким образом?

PHP Code:

$query20  =  mysql_fetch_array(mysql_query("SELECT usergroupid FROM `user` WHERE `userid` = $userid ")); 


=======================

ЗАРАБОТАЛО... СПС StenLi

Originally Posted by spaxy

Нет не подключал.
Без него разве нельзя обойтись?

spaxy, Эм, ну как бы нет.
У тебя в скрипте определены такие дефайны как префикс и т.д, а собственно это уже вобловская информация.

Originally Posted by spaxy

Таким образом?

Да) если брать грубо, то таким.

С префиксом не работает уже, если ты про

PHP Code:

" . TABLE_PREFIX . " 


Этот файл у меня только записывает в БД, больше никак не влияет на работу булки.

Главное работает)))

spaxy, если полный код - не сильно большой по объему, и не является гос. тайной, то скинь пожалуйста его в тему, или мне в пм.
Хочу глянуть на реализацию.

PS. B дальнейшем, при вытягивании массивов через параметр where
Рекомендую фильтровать переменные, особенно если они идут реквестами.

Вся эта штуковина ни что иное как: vbcredits
А этот код взят из файла credits_robopayment.php
Просто допилил под свои нужды.
Элементы для шаблонов описывал тута: http://vbsupport.ru/forum/showthread.php?t=41422


сие добро лежит тута: http://vbsupport.ru/forum/showpost.p...&postcount=125