Не работает mysqli_data_seek, что не правильно?

Question

[CeBePHblY]

есть такой самописный класс по работе с бд

//класс для работы с БД
	class DB
	{
        var $mysqliCnn='', $resQuery='', $rows='';

		//подключение к БД
		function connectOpen()
		{
			//подключение
			$this->mysqliCnn=mysqli_connect(CFG_DB_SERVER_IP, CFG_DB_USER_NAME, CFG_DB_USER_PASS, CFG_DB_NAME);
			//выбор кодировки
            mysqli_query($this->mysqliCnn,'SET NAMES "'.CFG_DB_CODE.'"');
		}

        //выполнение запроса к бд
        function query($query)
        {
            $this->resQuery=mysqli_query($this->mysqliCnn, $query);
            $this->rows = mysqli_fetch_assoc($this->resQuery);
            return $this->rows;
        }

        //закрытие подключения
        function connectClose()
        {
            mysqli_close($this->mysqliCnn);
        }

        //количество строк в результате
        function numRows()
        {
            return mysqli_num_rows($this->resQuery);
        }

        //переход в указанную строку результата
        function setRow($numRow)
        {
            mysqli_data_seek($this->resQuery,$numRow);
        }
	}

пробую перебрать результаты так

for ($indexMod=0;$indexMod<$db->numRows();$indexMod++)
		{
            //какую строку обрабатываем
            $db->setRow($indexMod);
			//ид подключения
			$idInclude=$allModule['id_rang'];
			//ид подключенного модуля в ранге
            $idModule=$allModule['rang'];
}

$db->numRows() возвращает правильное количество строк, цикл тоже выполняется это же количество раз, но при $db->setRow($indexMod); не происходит перемещения указателя. В чем причина?

Answer 1

[FanatPHP]

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

В текущем виде этот класс не несет ни малейшего смысла. Надо его выкинуть и работать с mysqli напрямую. Для этого надо изучить примеры из документации.
После того как получится освоить базовые операции и понять назначение основных функций, можно будет потихоньку начинать пробовать писать свой класс.

Comments

[CeBePHblY]

все работало до этого момента :) прислушаюсь к совету. а все-таки, подскажите почему не происходит перемещение, чтобы я понял свою ошибку.

[FanatPHP]

потому что mysqli_data_seek вообще никогда не нужно для работы с mysqli. Это бессмысленная и ненужная функция. попробуй всё-таки посмотреть в документации, как получают данные из БД с помощью mysqli

[CeBePHblY]

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

[FanatPHP]

Зря переписал. Тот код, который у тебя - он ХУЖЕ, чем чистый mysqli без оберток. И ты еще снова прибежишь сюда с вопросом, почему у тебя все не работает.

[FanatPHP]

Впрочем, нет. Ты прибежишь ещё раньше, из-за другой ошибки

[CeBePHblY]

спасибо, буду убирать этот класс, и работать с mysqli напрямую.

[CeBePHblY]

Плюс хранение данных результата в классе - это классические же грабли.

вот эта цитата мне особенно понравилась. значит я что-то не понимаю по классам. к этому выводу вы пришли в процессе практики? или может есть где почитать по таким нюансам?

[FanatPHP]

Ну слава богу, уговорил. Твоя функция query() возвращает не то что ты думаешь. И твой класс никогда не сможет выполнять запросы в цикле. И это еще не все проблемы

[FanatPHP]

Это не проблема классов вообще, это проблема класса для работы с БД. Ты написал класс, воображая, что в каждый моменть времени будешь работать только с одним запросом. А это, разумеется, не так. И подумай, что станет с твоими $resQuery и $rows, когда ты выполнишь второй запрос, не закончив работу с первым.

[CeBePHblY]

Да да, я с этим столкнулся, поэтому и решил прислушаться к вашему первому совету

Answer 2

[KorsaR-ZN]

Почему Вы не хотите использовать PDO, а строите свой велосипед?

Answer 3

[Иван Гонтаренко]

а где вы получаете данные эти ?

//ид подключения
$idInclude=$allModule['id_rang'];
//ид подключенного модуля в ранге
$idModule=$allModule['rang'];

как я вижу, вы только курсор меняете в setRow
и зачем вам for ?
раз уж на то пошло, добавьте метод себе в класс

function fetchRow() {
            return mysqli_fetch_row($this->resQuery);
}

и получайте через foreach или while данные

foreach( $db->fetchRow() as $row ){}
// или
while( $row = $db->fetchRow() ) {}

Comments

[FanatPHP]

через foreach? Вот чем мне нравятся писатели собственных классов - это тем, что они все поголовно не понимают, что делают :)

[Иван Гонтаренко]

FanatPHP: да, согласен :) не подумал, без \Iterator ничего не получится.