Вот этот код почему то не работает mysql_query("UPDATE user SET all = '".($user['all']-10)."', `arena` = '".$user['arena_max']."' WHERE id = '".$user['id']."'"); потом сделал так mysql_query("UPDATE user SET `arena` = '".$user['arena_max']."' WHERE id = '".$user['id']."'"); и стало работать вижу где ошибка но надо чтоб код работал полностью

all = all - 10




mysql_query("UPDATE user SET all = all - 10 , `arena` = arena_max WHERE id = '".$user['id']."'");


________
посл. ред. 20.01.2019 в 09:31; всего 2 раз(а); by Trec80

mysql_query("UPDATE `user` SET `all` = '".$user['all']."' - '10', `arena` = '".$user['arena_max']."' WHERE id = '".$user['id']."'");

говнокод

ну дай пример не говнокода ))

ExcLuSiVe , all= '".($user['all']-10)."', `arena` не че не забыл поставить?

Сибирский , #2
________
посл. ред. 20.01.2019 в 13:10; всего 1 раз(а); by Trec80

Trec80 , одно и то же )) Пример не гавнокода это где sql запрос не присутствует в основном коде, а вычисления идут методами экземпляра класса а не напрямую в запросе

как пример

// авторизация пользователя
$person = new person( autorize() );

$person->arena_up();

Ты для начала спросил бы , у него MVC или нет , Есть класс обработки или нет.
Ты спросил что в данной теме не говно код , я указал.
Да и запускать в конструктор функцию убого

Он ведёт к тому, что подобные операции надо инкапсулировать, чтобы в случае изменения структуры базы/ещё чего нибудь, можно было просто переписать сеттер. ООП здесь не причём.