Как оптимизировать данный код?

Question

[Сергей Семенко]

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

public function findRegion(array $pos) {
		list($x, $y, $z) = $pos;
		
		$result = $this->database->query('
			SELECT COUNT(*) FROM `regions` WHERE 
			' . $x . ' >= LEAST(`Ax`, `Bx`) AND ' . $x . ' <= GREATEST(`Ax`, `Bx`) AND 
			' . $y . ' >= LEAST(`Ay`, `By`) AND ' . $y . ' <= GREATEST(`Ay`, `By`) AND 
			' . $z . ' >= LEAST(`Az`, `Bz`) AND ' . $z . ' <= GREATEST(`Az`, `Bz`)
		');
		
		if ($result->num_rows > 0) {
			return $this->database->query('
				SELECT * FROM `regions` WHERE 
				' . $x . ' >= LEAST(`Ax`, `Bx`) AND ' . $x . ' <= GREATEST(`Ax`, `Bx`) AND 
				' . $y . ' >= LEAST(`Ay`, `By`) AND ' . $y . ' <= GREATEST(`Ay`, `By`) AND 
				' . $z . ' >= LEAST(`Az`, `Bz`) AND ' . $z . ' <= GREATEST(`Az`, `Bz`)
			')->fetch_assoc();
		} else {
			return false;
		}
	}

В общем, я сделал так:

public function findRegion(array $pos) {
		list($x, $y, $z) = $pos;
		
		$result = $this->database->query('
			SELECT * FROM `regions` WHERE 
			' . $x . ' >= LEAST(`Ax`, `Bx`) AND ' . $x . ' <= GREATEST(`Ax`, `Bx`) AND 
			' . $y . ' >= LEAST(`Ay`, `By`) AND ' . $y . ' <= GREATEST(`Ay`, `By`) AND 
			' . $z . ' >= LEAST(`Az`, `Bz`) AND ' . $z . ' <= GREATEST(`Az`, `Bz`)
		');
		
		if ($result->num_rows > 0) {
			return $result->fetch_assoc();
		} else {
			return false;
		}
	}

Но так я получаю ошибку, если нет совпадений

Comments

[Максим Каракулов]

Anton
Добро пожаловать в мир подзапросов...

Сергей Семенко
Я отредактировал вопрос, так будет лучше? А по скорости нормально с подзапросами?

Дмитрий Энтелис
а при чем тут подзапросы?

Anton
Сергей Семенко: а сам как думаешь, можно ли дать четкий ответ на такой абстрактный вопрос? Дмитрий Энтелис: вопрос не в том "нужны ли 2 запроса?", вопрос в том "как сделать из двух один?"

Дмитрий Энтелис
Anton: я все таки считаю что давать нужно содержательные а не формальные ответы. Очевидно что подзапросы тут нафиг не нужны

Anton
Дмитрий Энтелис: Ну ка дети скажем громко кто у нас молодец? Правильно, Дима у нас молодец! Держи, Дима, конфету! P.S. Пишите исчо ваше мнение для нас очень важно..

Славка Великолепный
Anton: У него с юмором плохо

FanatPHP
Интересно, администрация будет принимать меры против макак, которые тяфкают на специалистов?

Славка Великолепный
FanatPHP: Интересно как администрация будет отличать макак от специалистов =))) А ещё интересно сколько специалистов останется если убрать отсюда всех макак и что эти специалисты тут будут делать без макак.

[Максим Каракулов]

Anton вам следовало вашу реплику написать в комментариях к вопроса, а не в качестве ответа. Перенес обсуждение в комментарии к вопросу, на будущее, будьте внимательней.

[Дмитрий Энтелис]

Anton Славка Великолепный Мне прямо интересно чем моё замечание вызвало настолько бурную реакцию )

Answer 1

[Дмитрий Энтелис]

Не очень понятно какую ошибку Вы получаете если нет совпадений.
Ваша функция должна просто возвращать false насколько понятно из кода.

PSДелать отдельный запрос select count имеет смысл только в том случае, когда вам например нужно получить количество результатов не получая самих результатов.

Answer 2

[Дмитрий Миронов]

Второй вариант должен работать, нужно смотреть ошибку (подозреваю она в php)

P.S. В первом варианте есть подозрительное место, возможно, что $result->num_rows будет всегда 1 возвращать для COUNT(*) запроса, в зависимости от библиотеки.

Answer 3

[FanatPHP]

Тут действительно можно выкинуть кучу бесмысленного кода.

$sql = "SELECT ...";
return $this->database->query($sql)->fetch_assoc();

- это ВЕСЬ код, который нужен.

При отсутствии совпадений ни этот код, ни код приведенный в запросе, никакой ошибки выдавать не будет. Автору померещилось.

А проблема, скорее, не в совпадениях, а в отсутствии исходных данных и неумении корректно добавляь данные в запрос.

Comments

[Сергей Семенко]

Я тоже так думал, но там действительно была ошиба. Сейчас же я уже перешел на SQLite3

[FanatPHP]

Сергей Семенко: Я не "думаю". Я ЗНАЮ.

[Сергей Семенко]

FanatPHP: значит я не прав. А можно спросить, как на SQLite3 такое провернуть?

[FanatPHP]

Используй PDO

[Сергей Семенко]

FanatPHP: мне именно SQLite3 нужно, нет поддержки PDO