Как ускорить foreach?

Question

[nevesomostjke]

Привет, есть вот у меня foreach в приложении laravel, точнее foreach в foreach.
Все что происходит, проверяется наличие товара по прайсу в базе. Из за этого, медленно очень загружается страница.

view:

@foreach ($parts as $part)
			<?php $aval = 0; ?>
			@foreach ($prices as $k=>$v)
			<?php
				
				// Поиск наличия
				$coem = str_replace(' ', '', $part->part_number);
				$poem = str_replace(' ', '', $v->oem);

				$coem = str_replace('-', '', $coem);
				$poem = str_replace('-', '', $poem);

				$coem = str_replace('-', '.', $coem);
				$poem = str_replace('-', '.', $poem);

				$coem = str_replace('(', '.', $coem);
				$poem = str_replace('(', '.', $poem);

				$coem = str_replace(')', '.', $coem);
				$poem = str_replace(')', '.', $poem);	

				$sim = similar_text($coem, $poem, $perc);
				if ($perc==100) { 
					if ($v->count >= 1) {
						$aval = 1;
						break;
					}
				}
				?>
			@endforeach

Вот контроллер:

// Получаем характеристики продукта
                    $attributes = DB::table('article_attributes')
                    ->where('datasupplierarticlenumber', '=', $request->part_number)
                    ->where('supplierid', '=', $supplier->id)
                    ->get();

                    // Название изделия (есть еще статус изделия и не только см. таблицу articles)
                    $article = DB::table('articles')
                    ->select('NormalizedDescription', 'ArticleStateDisplayValue')
                    ->where('DataSupplierArticleNumber', '=', $request->part_number)
                    ->where('supplierid', '=', $supplier->id)
                    ->first();

                    // Файлы продукта
                    $files = DB::table('article_images')
                    ->select('Description', 'PictureName')
                    ->where('DataSupplierArticleNumber', '=', $request->part_number)
                    ->where('supplierId', '=', $supplier->id)
                    ->get();

                    // Применимость изделия
                    $result = [];
                    $rows = DB::table('article_li')
                    ->select('linkageTypeId', 'linkageId')
                    ->where('DataSupplierArticleNumber', '=', $request->part_number)
                    ->where('supplierId', '=', $supplier->id)
                    ->get();
                    
                    // разбор
                    foreach ($rows as &$row) {
                        $result[]= DB::select("SELECT DISTINCT p.id, mm.description make, m.description model, p.constructioninterval, p.description FROM passanger_cars p 
                        JOIN models m ON m.id=p.modelid
                        JOIN manufacturers mm ON mm.id=m.manufacturerid
                        WHERE p.id=" . $row->linkageId);
                    }

                    // Массив с информацией о товаре
                    $info = array(
                        "part_number" => $request->part_number, // OEM номер
                        "supplier_name" => $request->supplier_name, // Имя производителя
                        "supplier_id" => $supplier->id, // ID производителя
                    );

                    // Выгружаем прайсы
                    $prices = DB::table('prices')
                    ->select('oem', 'price', 'count', 'stock')
                    ->get();

Comments

[Роми]

Вы бы просто приложили структуру таблицы в БД и описание того что нужно вывести, потому что лично мне из этого кода вообще ничего не понятно

[Dark_Dante]

$coem = str_replace(' ', '', $part->part_number);
        $poem = str_replace(' ', '', $v->oem);

        $coem = str_replace('-', '', $coem);
        $poem = str_replace('-', '', $poem);

        $coem = str_replace('-', '.', $coem);
        $poem = str_replace('-', '.', $poem);

        $coem = str_replace('(', '.', $coem);
        $poem = str_replace('(', '.', $poem);

        $coem = str_replace(')', '.', $coem);
        $poem = str_replace(')', '.', $poem);

В чем глобальный смысл делать так? Почему бы не сделать

$coem = str_replace(
        [')', '(', ' ', '-', '_'],
        ['.', '.', '', '.', ''], 
        $coem,
    );

А не городить десяток вызовов str_replace.
С переменной $poem то же самое. А еще оптимальнее это вынести в отдельную функцию, что то вида

function replacementToRules(string $string): string
{
    return str_replace(
        [')', '(', ' ', '-', '_'],
        ['.', '.', '', '.', ''], 
        $string,
    );
}

//и потом в цикле 
        $coem = replacementToRules($part->part_number);
        $poem = replacementToRules($v->oem);

Вот это что такое?

// разбор
                    foreach ($rows as &$row) {
                        $result[]= DB::select("SELECT DISTINCT p.id, mm.description make, m.description model, p.constructioninterval, p.description FROM passanger_cars p 
                        JOIN models m ON m.id=p.modelid
                        JOIN manufacturers mm ON mm.id=m.manufacturerid
                        WHERE p.id=" . $row->linkageId);
                    }

Зачем тут цикл, зачем передача $row по ссылке? Почему не сделать выбор через IN, типа

// разбор
//не знаю как правильно делается подготовленный запрос в ларавеле
$result= DB::prepare("SELECT DISTINCT p.id, mm.description make, m.description model, p.constructioninterval, p.description FROM passanger_cars p 
                        JOIN models m ON m.id=p.modelid
                        JOIN manufacturers mm ON mm.id=m.manufacturerid
                        WHERE p.id IN (:Ids)");
$iDs = array_map(
    fn ($r): int => $r->linkageId, 
    $rows,
);

//а дальше скармливаешь подготовленному запросу массив с адишниками

И да, это не контроллер, это максимум модель. Так что тормоза тут не в форыче, а вообще в целом. Вообще удивлен что этот "код" работает

[JhaoDa]

Какая жуть...

[Lynn «Кофеман»]

Чего только не понапишут лишь бы с sql не разбираться

[Rsa97]

А зачем каждый раз перевычислять $coem во внутреннем цикле, если он там не меняется и зависит только от внешнего цикла?

[N]

Помимо всего, что уже сказали...similar_text не из быстрых функций...

[N]

К тому же...это дерзко - гонять по кругу цикл каждый раз сравнивая таким образом все записи в БД...
O_o

Answer 1

[Timur Maslov]

Проверять наличие товара по прайсу в базе нужно через SQL запрос. В этом примере тянутся данные из нескольких таблиц БД и потом через php производятся вычисления. Нужно что бы вычисления делала СУБД, а не PHP.