Быдлокод?

187
.
4 8 15 16 23 42
Привет, читаю про всякий быдлокод тут и у меня вопрос вот мой код удаления костюма с магазина Мой код, считается быдлокодом? и как это исправить?
Как делать что бы небыло быдлокода??


<?php

define('_IN_JOHNCMS', 1);
$textl = 'Удаляем костюм с магазина';
$headmod = 'pers';
require_once ('../incfiles/core.php');
require_once ('../incfiles/head.php');
if ($rights < 7) {
   echo functions::display_error('У вас нет доступа.');
    require_once ('../incfiles/end.php');
    exit;
}
$text = mysql_result(mysql_query("SELECT `text` FROM `market` WHERE `id` = '$id'"), 0);
$win = mysql_result(mysql_query("SELECT `win` FROM `market` WHERE `id` = '$id'"), 0);
$pid = mysql_result(mysql_query("SELECT `pid` FROM `market` WHERE `id` = '$id'"), 0);
$eeq = $pid + 2;
if ($id) {
echo '<div class="smenu">Костюм <b>'.$text.'></b> успешно удален. <a href="index.php">вернуться</a> в магазин</div>';
$req = mysql_query("select * from `users` where `costume` >= $pid and `costume` <= $eeq");
while ($res = mysql_fetch_array($req)) {
$name = mysql_result(mysql_query("SELECT `name` FROM `users` WHERE `id` = '" . $res['id'] . "'"), 0);
echo '<div class="menu"><a href="/users/profile.php?user=' . $res['id'] . '">'.$name.'</a> Отправленно.</div>';
mysql_query("INSERT INTO `cms_mail` SET
`user_id` = '8',
`from_id` = '".$res['id']."',
`text` = 'Привет ".$name."
Твой костюм ".$text." был удалён с магазина, поэтому я снимаю его с твоего персонажа.
Надеть другой костюм 

 С Ув. Администрация!
',
`time` = '" . time() . "',
`them` = 'Костюмы'") or die(mysql_error());

mysql_query("UPDATE `users` SET `costume` = `costume` = 0 WHERE `id` = '" .$res['id']. "'");
}
mysql_query("DELETE FROM `market` WHERE `id` = '$id'");
} else {
echo '<div class="rmenu">ОШИБКА!<br />Такого костюма нет.</div>';
}
require('../incfiles/end.php');
?>
.
ValekS
Ей 25
ViROSS, а одним запросом нельзя обойтись:
$text = mysql_result(mysql_query("SELECT `text` FROM `market` WHERE `id` = '$id'"), 0);
$win = mysql_result(mysql_query("SELECT `win` FROM `market` WHERE `id` = '$id'"), 0);
$pid = mysql_result(mysql_query("SELECT `pid` FROM `market` WHERE `id` = '$id'"), 0);

?
Далее: запросы в цикле. Это зачем же такое делать то? И вообще я не пойму нафиг цикл и что ты там делаешь. Проверка
if ($id)
у тебя странная. То что надо писать isset($id) это уже ладно, но ты сразу используешь эту $id в запросах, а потом проверяешь есть она или нет.
.
4 8 15 16 23 42
ValekS, Можешь показать как?
.
Зелимхан
Отправленно
Отправлено
.
ValekS
Ей 25
# ViROSS (30.07.2015 / 15:40)
ValekS, Можешь показать как?
так же само почти:
$market = mysql_fetch_assoc(mysql_query("SELECT `text`, `win`, `pid` FROM `market` WHERE `id` = ".$id));
// $market['text'] - содержание поля text и т.д.
.
4 8 15 16 23 42
Scorpion, О, спасибо
.
4 8 15 16 23 42
ValekS, Вооо, спасибо, облегчил мне писанину, от души
.
Ей 25
# ViROSS (30.07.2015 / 15:48)
ValekS, Вооо, спасибо, облегчил мне писанину, от души
Там я только что подправил. Взял у тебя mysql_fetch_array, а оно немного не то. Есть mysql_fetch_assoc его лучше сразу и использовать.
.
ValekS
Ей 25
ViROSS, ещё такой вопрос: почему ты сразу выводишь на экран текст о завершённом действии, например "Удален", а само действие(удаляешь в данном случае) выполняешь потом? Вдруг произойдёт ошибка и твой "костюм" не удалится? А ты уже юзеру выдал "удален".
.
Ей 25
ViROSS, и расскажи всё таки что делает тот цикл с 3-мя запросами. А заодно и что это такое:
`costume` = `costume` = 0
Всего: 19