Подскажите пожалуйста, в чем у меня ошибка php+mysql не хочет плюсовать одевание штанов, а одевание оружия плюсует.

Question

вот код:

//здесь одеваем оружие

$wea = mysql_query("SELECT * FROM inv WHERE IL_USER=\"$LOGIN\" AND IL_NO=\"1\" AND INV_CLASS=\"weapon\"");

if(isset($eq) AND mysql_num_rows($wea)<1){

$i1 =mysql_query("SET NAMES cp1251");

$i1 =mysql_query("SET NAMES utf8");

$i1= mysql_query("SELECT * FROM inv WHERE NUM_INV=\"$eq\" AND INV_CLASS=\"weapon\"");

$i2 = mysql_fetch_assoc($i1);

$u1 =mysql_query("SET NAMES cp1251");

$u1 =mysql_query("SET NAMES utf8");

$u1 = mysql_query("SELECT * FROM users WHERE nick_user=\"$LOGIN\"");

$u2 = mysql_fetch_assoc($u1);

$c = $u2[DMG] + $i2[DMG_INV];

$equip3 = mysql_query("UPDATE inv SET IL_NO=\"1\" WHERE IL_USER=\"$LOGIN\" AND NUM_INV=\"$eq\"");

$us1 = mysql_query("UPDATE users SET DMG=\"$c\" WHERE nick_user=\"$LOGIN\"");

}

//здесь одеваем штаны

$wea = mysql_query("SELECT * FROM inv WHERE IL_USER=\"$LOGIN\" AND IL_NO=\"1\" AND INV_CLASS=\"greaves\"");

if(isset($eq) AND mysql_num_rows($wea)<1){

$i1 =mysql_query("SET NAMES cp1251");

$i1 =mysql_query("SET NAMES utf8");

$i1= mysql_query("SELECT * FROM inv WHERE NUM_INV=\"$eq\" AND INV_CLASS=\"greaves\"");

$i2 = mysql_fetch_assoc($i1);

$u1 =mysql_query("SET NAMES cp1251");

$u1 =mysql_query("SET NAMES utf8");

$u1 = mysql_query("SELECT * FROM users WHERE nick_user=\"$LOGIN\"");

$u2 = mysql_fetch_assoc($u1);

$c = $u2[ARMOR] + $i2[ARM_INV];

$equip3 = mysql_query("UPDATE inv SET IL_NO=\"1\" WHERE IL_USER=\"$LOGIN\" AND NUM_INV=\"$eq\"");

$us1 = mysql_query("UPDATE users SET ARMOR=\"$c\" WHERE nick_user=\"$LOGIN\"");

}

Кристина Глинова · Accepted Answer

Во-первых. Нафига столько SET NAMES, причем противоречащих друг другу? Это делается только один раз, в начале скрипта!

Во-вторых, вам, батенька, недурно бы научиться осмысленно называть переменные. А не $u1, $u2. Код нечитабелен!

В-третьих, имена элементов именованного массива берутся в кавычки.
Пример: $u2['ARMOR']. Это не критичная ошибка, но замедляет исполнение кода (интерпретатор сначала будет искать константу с именем ARMOR, а только потом поймет что это строка) . И ухудшает читабельность. Если в настройках пхп поставить error_reporting E_ALL - вывалится море ошибок уровня Notice.

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

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

ВЕРДИКТ: Почистить код, привести его к читабельному виду и повторить вопрос. А может быть, и вопрос задавать не потребуется, потому что сразу все станет и так видно.

Ольга · Answer

а если сперва штаны и потом уже оружие?