Почему скрипт неправильно складывает массив?

Question

[Александр Токмаков]

Есть код:

$summa=0;
$tech = mysql_query("SELECT id FROM tech WHERE tech_cat='$cat_id'   AND status='normal'");
while($techs = mysql_fetch_array($tech)){
    $tech_id=$techs['id'];
    $kolsite = mysql_query("SELECT SUM(sum) FROM remont_tech WHERE tech_id='$tech_id'");
    $kolsiter = mysql_fetch_array($kolsite);
    $summa+=$kolsiter[0];
}

Выводится сумма в зависимости от $cat_id!!

В программе 4 категорий (Компьютеры, принтеры, картриджи, мониторы)
В 3-ех категориях сумму проделанных работ вычисляет правильно, а в принтерах вместо 4070 руб, показывает 10003769 Руб.

В базе данных тип стоит INT.

Может сталкивались с такими казусами?

Answer 1

[Rsa97]

А если так? Полностью замените свою подпрограмму на:

$tech = mysql_query("SELECT SUM(`rt`.`sum`) ".
                    "    FROM `tech` AS `t` ".
                    "        INNER JOIN `remont_tech` AS `rt` ON `t`.`id` = `rt`.`tech_id` ".
                    "    WHERE `t`.`tech_cat` = '".mysql_real_escape_string($cat_id)."' ".
                    "        AND `t`.`status` = 'normal'");
if ($techs = mysql_fetch_array($tech))
    $summa = $techs[0];

Comments

[Александр Токмаков]

Правильно!!!
Короче у меня решение тоже правильно, просто для remont_tech тоже условие должно было быть `status` = 'normal'.
Поэтому у меня не билось))))

@Rsa97, а чем Ваш код лучше моего)? Я просто хочу выбрать лучший вариант

[Александр Токмаков]

@Rsa97 , можете не отвечать мой вариант: 0.062500 сек., Ваш:0.031250 сек

[Игорь Яцевич]

@calliko не только этим. То как Вы написали - простите, через жопу. Если есть возможность, то лучше всегда сделать 1 запрос чем N штук в цикле. как то так)

[Rsa97]

@calliko Добавлю, если у Вас ещё и внешний цикл по $cat_id присутствует, то опять таки можно объединить всё в один запрос, достаточно добавить в выборку `t`.`id` и начиная с переписать условие на "WHERE `t`.`status` = 'normal' GROUP BY `t`.`id`".

[Александр Токмаков]

@IgoNsk, @Rsa97, ОК. спасибо

[Евгений Кузнецов]

Да и если $cat_id - integer, то вместо mysql_real_escape_string($cat_id) можно так: (int)$cat_id

Answer 2

[Ilyas Masirov]

Попробуйте поменять запрос на:

"SELECT SUM(remont_tech.sum)
FROM tech
INNER JOIN remont_tech ON remont_tech.id = tech.tech_cat 
WHERE tech_cat= ".$cat_id."  AND status='normal'"

и посмотрите, что получится

Comments

[Александр Токмаков]

@IlyasMasirov Warning: mysql_fetch_array() expects parameter 1 to be resource, boolean given

Answer 3

[VeMax]

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

Так же хочу заметить, что вы используете устаревшее расширение (вроде начиная с 5.5) и возможно его когда то удалят. Обратите внимание на PDO_MySQL. Это позволите решить обе проблемы.