Быдлокод?

ViROSS

30 июл 2015 г., 15:56

4 8 15 16 23 42

ValekS, Но костюм же удаляется!
Я же проверил всё работает.

ValekS

30 июл 2015 г., 15:58

Ей 25

# ViROSS (30.07.2015 / 15:56)
ValekS, Но костюм же удаляется!
Я же проверил всё работает.

Это как ты пришёл с банкомата снимать деньги, он тебе сразу выдал чек "снято 100 рублей", а деньги потом может даст а может и нет...

ViROSS

30 июл 2015 г., 16:00

4 8 15 16 23 42

ValekS, Вообщем на сайте у меня есть магазин персонажей (костюмов)

Пользователи их могут надевать и старые костюмы я иногда удаляю и вот это и есть страница удаления.

Я удаляю костюм с магазина и он убирается у всех у кого уже надет после им приходит сообщение что такой то костюм удалён с магазина и снят с них а

`costume` = 0

Это ознаечает что им ставится радной костюм такой как был при регистрации тоесь 0 (стандартный)

ValekS

30 июл 2015 г., 16:01

Ей 25

ViROSS, и ещё хочу сказать про вот это:

WHERE `id` = '$id'

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

WHERE `id` = ".$id."

Или, если это конец запроса, вот так:

WHERE `id` = ".$id);

ViROSS

30 июл 2015 г., 16:07

4 8 15 16 23 42

# ValekS (30.07.2015 / 16:01)
ViROSS, Переменную выделить из тела запроса, что бы интерпретатору её было проще найти

Хорошо везде исправил.

ValekS

30 июл 2015 г., 16:08

Ей 25

# ViROSS (30.07.2015 / 16:00)
ValekS, Вообщем на сайте у меня есть магазин персонажей (костюмов)

Пользователи их могут надевать и старые костюмы я иногда удаляю и вот это и есть страница удаления.

Я удаляю костюм с магазина

Понятно. Немного не полная картинка конечно... Так и не пойму что такое эти $pid и $win. Там можно обойтись 1 запросом(что письма отправляет) в цикле.

ViROSS

30 июл 2015 г., 16:12

Изменено

4 8 15 16 23 42

Про win забудь, я его удалил , он лишний был.

$pid это как бы в базу заносится. номер костюма. тоесть имя костюма, потом я этот костюм в анкете вывожу кообщем когда костюм надевают он вот так записывается.

Надеваем костюм

mysql_query("UPDATE `users` SET `costume` = $pid WHERE `id` = ".$user_id);

Показываем костюм в анкете

<img src="../pers/images/'. $user['costume'].'.png" alt="Персонаж" />

ValekS

30 июл 2015 г., 16:35

Ей 25

ViROSS, теперь уже яснее. Значит $pid это ай-ди костюма. Ну так. Теперь можно, как я писал выше, сделать так, что бы в цикле был только 1 запрос.

ViROSS

30 июл 2015 г., 19:11

4 8 15 16 23 42

ValekS, Как это сделать? покажешь?