Доброго времени суток. Это моя первая попытка по освоению ООП. Решил наверное сразу за сложно взяться. И вот в чем проблема, вроде бы все работает, но вот функция 



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

Собственно вот класс.

если запрос возвращает хотя бы одну строку, то в этой строке получается вечный цикл. Почему -- попробуйте разобраться сами.

да ладно зачем мучать человека  

каждый вызов mysql_fetch_array обрабатывает ряд результата запроса и сдвигает внутренний указатель на следующий ряд, и так пока ряды не закончатся.

mysql_query же получает эти ряды из базы и ставит внутренний указатель на начало.

смотрим  что у вас:


при первой итерации вызывается $this->m_query() (что есть ни что иное, как mysql_query), она производит запрос к базе и  возвращает результат запроса, с указателем на первом ряду,
mysql_fetch_array обрабатывает ряд и смещает указатель на следующий ряд, вроде пока все хорошо.
но вот на следующей итерации опять вызывается $this->m_query(), которая снова посылает запрос к базе и возвращает результат, снова устанавливая указатель на первый ряд, mysql_fetch_array обрабатывает ряд и смещает указатель на следующий ряд... чувствуете? все колеблется между первым и вторым рядом, и так до бесконености.

ps: тс, не обижайтесь, но пока здесь ООП нет, а есть функциональное программирование, только с помощью классов... 


Это сообщение отредактировал(а) K0T9I - 23.11.2009, 18:40

Кстати, мне кажется разумным дескрипторы запросов вообще в коде (в смысле — за пределами класса) никак не светить, т.е. чтобы они внутри объекта и оставались. Каждый очередной запрос клал бы в стек свой дескриптор:


Остальные функции (num_rows(), affected_rows()) вызывать надо будет на соответствующем уровне. И все результаты запроса должны быть обязательно высбождены.

К текущему треду это сильно не относится, понимаю, но хочется высказать свою мысль.

Это сообщение отредактировал(а) Pitlord - 23.11.2009, 19:23

Не понимаю, к чему все эти дескрипторы-шмескрипторы.
Три метода, которые получают, соответственно, скаляр, строку и массив. 
Все. 
В рамках стандартного пхп скрипта этого достаточно. 
Городить этот стек, чтобы использовать его в единственном скрипте на все приложение - вот это я называю го.. неправильной архитектурой

Ипатьев, я Вас вообще не понимаю, Вы про что?

Добавлено через 11 минут и 55 секунд
P.S. Правда с mysql_unbuffered_query() я, возможно, погорячился

1. Код ужасно оформлен.
2. Не соблюдены правила именования сущностей.
3. Переменные должны носить выразительные имена.
4. Класс должен быть переносим. Следовательно в корне неверно прописывать параметры конекта к СУБД внутри конструктора.
5. Кодировка! Используй исключительно UTF-8, иначе будет та же хрень, что и сейчас.
6. Уроаень абстракции. У тебя он нулевой - ты просто загнал функции в методы, при этом их реалдьно неудобно юзать. Т.е. тут нету смысла ООП. Исправляй.
7. Класс не несет смысла - является лишь НЕУДОБНОЙ оберткой над фйункциями. Исправляй.

Всем спасибо за ответы, есть к чему идти и стремится. Это мои первые шаги в ООП, и как оказались не в том направлении. Сегодня обязательно попробую закончить. Спасибо.