@volodu,
соберу вместе , что тебе уже тут писали.
1. кавычки - местный стеб над не требуемой сверхоптимизацией .
2. использование устаревшего расширения - перейти на myqli_ - желательно , а скоро уже будет и обязательно . Но по большому счету разница для новичка с его тривиальными запросами не ощутима.
3. раскрывать все элементы массива в переменные , что дает самое большое кол-во строк. Вот это я совсем не понимаю. Зачем ? ведь можно просто работать с элементами массива. И потом по коду
будет сложно понять откуда взялась конкретная переменная , а если потребуется передать данные в функцию ?
4. И ХИТ маразма. ЗАПРАШИВАТЬ ВСЕ ДАННЫЕ из таблицы . У тебя что только один пользователь или в запросе забыл написать where id= ?