Оптимизация функций.

[Dexes]

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

PHP код:

function check_category_by_name ($name) {
    global $link, $mc_array;
    $id = 0;
    
    foreach ($mc_array as $key => $value) {
        $pos = strpos($key, $name);
        if ($pos !== false) {
            return $mc_array[$key];
        }
    }
    
    return $id;
} 


$mc_array выглядит примерно так

PHP код:

Array
(
    [Мобильные телефоны; сотовый телефон] => 3063
    [Мобильные телефоны; коммуникатор] => 3064
    [Телефония, факсы; телефон PANASONIC KX-TG] => 3065
    [Телефония, факсы; телефон PHILIPS CD] => 3065
    [Телефония, факсы; телефон TEXET TX-D] => 3065
    [Телефония, факсы; телефон VOXTEL Combo] => 3065
    [Телефония, факсы; телефон VOXTEL Elegant] => 3065
    [Телефония, факсы; телефон VOXTEL Profi] => 3065
    [Телефония, факсы; телефон VOXTEL Select] => 3065
    [Телефония, факсы; телефон VOXTEL Symphony] => 3065
    [Телефония, факсы; телефон VOXTEL Wall] => 3065
    [Телефония, факсы; доп.трубка] => 3066
    [Телефония, факсы; радиостанция] => 3067
    [Телефония, факсы; телефон] => 3068
    [Телефония, факсы; телефонный адаптер] => 3068
    [Телефония, факсы; телефонный коннектор] => 3068
    [Телефония, факсы; телефонный переходник] => 3068
    [Телефония, факсы; Телефонная гарнитура] => 3068
    [Аксессуары к сотовым; Bluetooth] => 3069
    [Телефония, факсы; GSM-шлюз] => 3070
    [Телефония, факсы; мини АТС] => 3071
    [Телефония, факсы; Термо-пленка] => 3072
    [Телефония, факсы; Тонер] => 3072
) 


Только там пара сотен элементов
Мне не важно что в нем после ";". Важно совпадение данных до точки с заяптой с входящей в функцию переменной $name.
Как только нашли такое совпадение функция возвращает value и финита.

PHP код:

function check_category_by_name ($name) {
    global $link, $mc_array;
    $id = 0;
    
    foreach ($mc_array as $key => $value) {
        $pos = explode(';', $key);
        if ($pos[0] == $name) {
            return $mc_array[$key];
        }
    }
    
    return $id;
} 


Такой вариант обрабатывает дольше.

вторая функция, самая дикая

PHP код:

function check_product_by_name ($name, $category_id = 0) {
    global $link;
    
    $row = array('product_id' => 0);
    $sql = 'select product_id, model, model2, model3, model4, model5 from product where category_id = '.$category_id;
    $query = mysql_query ($sql, $link) or sql_error ($sql, $link, __FILE__, __LINE__);
    while ($row = mysql_fetch_assoc($query)) {
        $pos = strpos($row['model'], $name);
        if ($pos === false) {
            $pos = strpos($row['model2'], $name);
        }
        else {
            return $row;
        }
        if ($pos === false) {
            $pos = strpos($row['model3'], $name);
        }
        else {
            return $row;
        }
        if ($pos === false) {
            $pos = strpos($row['model4'], $name);
        }
            else {
            return $row;
        }
        if ($pos === false) {
            $pos = strpos($row['model5'], $name);
        }
        else {
            return $row;
        }
        if ($pos !== false) {
            return $row;
        }
    }
    return $row;
} 


Запросили из базы все модели товаров из раздела, если нашли вхождение модели (или альтернативного названия модели) в входящем значении $name вылетаем из функции возвращая product_id.
Как можно оптимизировать эти две дико жрущие серверные ресурсы вещи?

[Cronos20]

Это все очень просто решается средствами самой базы данных
1. Неправильное проектирование БД - нужно все ваши model1, model2 ... 4,5 вынести в отдельную таблицу ( TABLE models (product_id, model_name) )
2. Расставить индексы
3. Простой поиск по моделям с JOIN и like

select ... from product JOIN models WHERE models.model_name like '%....%'

Может я не до конца понял код, ввиду очень плохой читабельньности ваших функций, я не вдавался в подробности ваших циклов перебора. Но мне кажется, все решается вполне просто.

[Dexes]

like - есть плохо.
Если при "=" mysql работает с индексами, то с like скула начинает работать уже со значениями, и врят ли многим отличается от strpos.

-- ADD --

С базой я к сожалению ничего не могу сделать, с какой дали работать, с той и приходится.

[ADSoft]

а может как вариант

select product_id, model, model2, model3, model4, model5 from product where category_id = '.$category_id && (model like %$name% || model2 like %$name% || model3 like %$name% || model4 like %$name% || model5 like %$name%)

или

select product_id, model, model2, model3, model4, model5 from product where category_id = '.$category_id && concat(model, model1, model2, model3, model4,model5) like %$name%

а вообще - лучше реально поменять структуру БД
like - не всегда есть плохо
отличаться будет тем что нагрузка будет на БД ... снижая скорость.. во втором случае - нагрузка на PHP код ... то есть память и процессор

[ADSoft]

в вашем случае вытаскивать ВСЮ базу и конопатить её - ужасно!
а если записей тысячи? миллионы?

[Dexes]

Здесь не like нужен, а ему обратный)))
не
model like %$name%
а
$name like %model%
Если можно так выразиться)

Сообщение от ADSoft

в вашем случае вытаскивать ВСЮ базу и конопатить её - ужасно!
а если записей тысячи? миллионы?

Записей может быть до тысячи, всё же ограничить вывод через category_id было неплохим решением)))
про вытаскивание согласен, но и поменять структуру БД не в моей компетенции...

[ADSoft]

ну поменяй на обратный.... пусть $name like %model%
только тогда второй способ не подойдет точно )

[Dexes]

Не знал что так можно... (чего по молодости только не узнаешь)

[Cronos20]

Никогда не понимал логика заказчика, который просит чего-то там наоптимизировать в скриптах, когда БД изначально сделана через одно место и ее трогать почему-то нельзя )
Хотя скрипт конвертации данных из одной таблицы в 3 другие пишется за пару минут. Думаю самый простой способ - убеждать заказчиков в неизбежности нормальной оптимизации начиная с корня проблемы.

P.S. Советую для скорости это все выполнять в хранимой процедуре, потому что их вызов хорошо кэшируется

[Dexes]

Я не общаюсь напрямую с заказчиком. Мне ставят задачу написать/оптимизировать модуль, я делаю) Не более того.