Как оптимизировать мою функцию?

Question

[okovalev]

function get_trank_num($mysqli, $client) {
		$sql = "SELECT * FROM `balance`
		WHERE `client` LIKE '$client'
		ORDER BY `min_left` DESC";
		
		if ($result = $mysqli->query($sql)) {
			while ($row = $result->fetch_assoc()) {
				$tel = $row['tel'];
				$busy_line = $row['busy_line'];
				$min_left = $row['min_left'];

				if($busy_line == 0 && $min_left > 500){
					return $tel;
				}
			}
			$result = $mysqli->query($sql);
			while ($row = $result->fetch_assoc()) {
				$tel = $row['tel'];
				$busy_line = $row['busy_line'];
				$min_left = $row['min_left'];

				if($busy_line == 1 && $min_left > 500){
					return $tel;
				}
			}
			$result = $mysqli->query($sql);
			while ($row = $result->fetch_assoc()) {
				$tel = $row['tel'];
				$busy_line = $row['busy_line'];
				$min_left = $row['min_left'];

				if($busy_line == 0){
					return $tel;
				}
			}
			$result = $mysqli->query($sql);
			while ($row = $result->fetch_assoc()) {
				$tel = $row['tel'];
				$busy_line = $row['busy_line'];
				$min_left = $row['min_left'];

				if($busy_line == 1){
					return $tel;
				}
			}
		$result->free();
		}

		return 0;
	}

Выглядит довольно убого. Особенно учитывая, что в последний момент я догадался, что после первого прохода в $result пусто)

Смысл в том, что по ряду параметров мы ищем самый приоритетный номер, но если его нет, то ищем более худщий критерий:
$busy_line == 0 && $min_left > 500
$busy_line == 1 && $min_left > 500
$busy_line == 0
$busy_line == 1
Если ни один из критериев не совпадает, то возращаем 0.
Как оптимизировать функцию?

Answer 1

[Артур К.]

Зачем столько раз делать один и тот же запрос? Почему бы все if'ы не сделать в одном цикле?

Comments

[okovalev]

Я согласен, что это глупость.
Просто я в последний момент сообразил, что fetch_assoc( ) извлекает результирующий ряд, а не перебирает.
Я допускаю, что можно сделать fetch_all() и допустим перебирать массив каждый раз.

Мне не нравиться, моя логика поиска "лучшего решения", по критериям:
$busy_line == 0 && $min_left > 500
$busy_line == 1 && $min_left > 500
$busy_line == 0
$busy_line == 1
Если не одно не совпало: 0

[Артур К.]

Это условие вообще можно в запрос перенести

[okovalev]

Артур Э., Тогда придется делать 4 запроса, а запрос к БД является одной из самых ресурсоемких операций.
Кроме того, при такой архитектуре, если успеть сделать второй запрос к скрипту, до момента, как завершился последний в цепочки, мы будет обращаться к одному телефону дважды.

[Артур К.]

okovalev, Приехали...
Так у вас уже 4 запроса делается! Каждая эта строка:
$result = $mysqli->query($sql);
— это запрос к базе

Вот абстрактный пример, когда выбираются значения по нескольким условиям в один результат:

select
  Name,
 (select sum(Quantity)  
    from Goods p1
    where
      p1.name = p1.name
      and p1.Data between @начало_периода_1 and @конец_периода_1
  ) as p1_sum,
(select sum(Quantity)  
    from Goods p2
    where
      p2.name = p2.name
      and p1.Data between @начало_периода_2 and @конец_периода_2
  ) as p2_sum
  , ..
from   Goods a

дальше сами разберетесь?

[Артур К.]

Если это сложно, то как минимум можно сделать первый шаг упрощения:

$ret = [];
 if ($result = $mysqli->query($sql)) {
      while ($row = $result->fetch_assoc()) {
        $tel = $row['tel'];
        $busy_line = $row['busy_line'];
        $min_left = $row['min_left'];

        if($busy_line == 0 && $min_left > 500){
          $ret['cond1'][] = $tel;
        }
        if($busy_line == 1 && $min_left > 500){
          $ret['cond2'][] = $tel;
        }
        if($busy_line == 0){
          $ret['cond3'][] = $tel;
        }
        if($busy_line == 1){
          $ret['cond4'][] = $tel;
        }
      }
    $result->free();
    }

var_dump($ret); // тут все телефоны, сгруппированные по указанным условиям

Который в результате затратит 1 запрос к базе, и 1 цикл перебора.

[Артур К.]

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

[Vitsliputsli]

okovalev, делайте на стороне БД, так даже проще. Запрос для MySQL будет примерно таким:

WITH q AS (
SELECT 
	balance.*,
	CASE 
    	WHEN busy_line=0 and min_left>500 THEN 
    		CASE WHEN @q>1 OR @q is null THEN @q:=1 ELSE 1 END
        WHEN busy_line=1 and min_left>500 THEN 
    		CASE WHEN @q>2 OR @q is null THEN @q:=2 ELSE 2 END
        WHEN busy_line=0 THEN 
    		CASE WHEN @q>3 OR @q is null THEN @q:=3 ELSE 3 END
        WHEN busy_line=1 THEN 
    		CASE WHEN @q>4 OR @q is null THEN @q:=4 ELSE 4 END
    END sel
	FROM balance 
	WHERE client LIKE ?
)
select * from q where sel=@q ORDER BY min_left DESC

[Артур К.]

Vitsliputsli, да, такой SQL даже красивее будет

[Vitsliputsli]

Артур Э., ага, мне очень нравятся переменные внутри запросов в MySQL, на других СУБД такого не видел, подобные задачи решаются через аналитические функции. Но мне кажется, что такой синтаксис понятнее и проще в разы, и для подобной задачи подходит как нельзя лучше. Но вот если понадобится портировать его в Postgre или Oracle, придется переписывать, хотя это касается любого мало-мальски сложного запроса.

[okovalev]

Артур Э., остановился на формирвоании массива с небольшими изменениями согласно конкретной задачи.
Решение от коллег с CASE в БД интересное и скажем лучшее, но я отказался от него т.к. сначала мне придется въехать, а потом еще и вспоминать т.к. подход неодчивидный)
Решение с вложенными SELECT мне тоже понравилось, в том смысле что мне кажется, что можно попробовать создать "представление" (кажется так называется) таблицы и обращаться к ней.