Как я могу сделать этот цикл быстрее?

Question

Как я могу сделать этот цикл быстрее?

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

foreach ($purchaseYears as $purchaseYear){

$totalInventory = \App\Inventory::where('purchase_year', $purchaseYear)->count();

if ($purchaseYear < 1960) { $purchaseYear = 'Pre 1960'; }
else if ($purchaseYear >= 1960 && $purchaseYear < 1965) { $purchaseYear = '1960-64'; }
else if ($purchaseYear >= 1965 && $purchaseYear < 1970) { $purchaseYear = '1965-69'; }
else if ($purchaseYear >= 1970 && $purchaseYear < 1975) { $purchaseYear = '1970-74'; }
else if ($purchaseYear >= 1975 && $purchaseYear < 1980) { $purchaseYear = '1975-79'; }
else if ($purchaseYear >= 1980 && $purchaseYear < 1985) { $purchaseYear = '1980-84'; }
else if ($purchaseYear >= 1985 && $purchaseYear < 1990) { $purchaseYear = '1985-89'; }
else if ($purchaseYear >= 1990 && $purchaseYear < 1995) { $purchaseYear = '1990-94'; }
else if ($purchaseYear >= 1995 && $purchaseYear < 2000) { $purchaseYear = '1995-99'; }
else if ($purchaseYear >= 2000 && $purchaseYear < 2005) { $purchaseYear = '2000-04'; }
else if ($purchaseYear >= 2005 && $purchaseYear < 2010) { $purchaseYear = '2005-09'; }
else if ($purchaseYear >= 2010 && $purchaseYear < 2015) { $purchaseYear = '2010-14'; }
else if ($purchaseYear >= 2015 && $purchaseYear < 2019) { $purchaseYear = '2015-19'; }

if (isset($chartData[$purchaseYear])) {
$prev = $chartData[$purchaseYear];
$chartData[$purchaseYear] = $totalInventory + $prev;
$prev = null;
} else {
$chartData[$purchaseYear] = $totalInventory;
}

}

0

laravel-5.2 php query-performance

Решение

Другие решения

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

if ($purchaseYear < 1960) {
$purchaseYear = 'Pre 1960';
} else {
$prefix = substr($purchaseYear,0,2);
$decade = substr($purchaseYear,1,1);
$suffix = substr($purchaseYear,-1);
$purchYear = $prefix . $decade
if ($suffix<5) {
$purchYear .= '0-' . $decade . '4';
} else {
$purchYear .= '5-' . $decade . '9'
}

Не проверял это, но я думаю, что он сделает то, что вы хотите.

0

Источник

Accepted Answer

Я считаю, что наиболее трудоемкой частью должно быть выполнение запроса. Если вы используете whereInТогда я думаю, что вы можете выполнить запрос только один раз, а затем перебрать результаты. Я не проверял это, но это должно быть что-то вроде этого:

$purchasesByYear = \App\Inventory::select('purchase_year',
DB::raw('count(*) as purchase_count'))
->whereIn('purchase_year', $purchaseYears)
->groupBy('purchase_year')->get();

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

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

foreach ($purchasesByYear as $purchaseYear) {
$year = $purchaseYear->purchase_year;
if ($year < 1960) {
$yearRange = 'Pre 1960';
} else {
// subtract one until the year is a multiple of five
while ($year % 5) { $year--; }
// then construct the range string using the starting number
$yearRange = $year.'-'.($year+4);
}
if (isset($chartData[$yearRange])) {
$chartData[$yearRange] += $purchaseYear->purchase_count;
} else {
$chartData[$yearRange] = $purchaseYear->purchase_count;
}
}

0