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

Question

[Ruslan Ruslanov]

yii2, рендерю view записи, также нужно добавлять кнопки "вперед" и "назад", для навигации по записям.

все сделал, выглядит так:

public function actionView($id)
        {
    //        $model = Images::findOne(['id' => $id]);
            $next = $id;
            $prev = $id;
            $all = Images::find()->all();
            foreach ($all as $key => $item) {
                if ($item['id'] == $id){
                    $model = $item;
                    if (array_key_exists($key + 1, $all)){
                        $next = $all[$key + 1]['id'];
                    }
                    if (array_key_exists($key - 1, $all)){
                        $prev = $all[$key - 1]['id'];
                    }
                    break;
                }
            }
            if (!$model || !$id){
                throw new HttpException(404, '404');
            }
            return $this->render('view', [
                'model' => $model,
                'next' => $next,
                'prev' => $prev,
            ]);
        }

это работает, но при большом количестве записей страницы грузятся по 10+ секунд, а это неприемлимо.

как переписать данный кусок кода? по сути мне нужно просто узнать id соседних элементов, но пока я ничего другого не придумал.

Comments

[Дмитрий Ларин]

Использовать кеширование

[Ruslan Ruslanov]

Дмитрий Ларин, приведите пример кода, пожалуйста

[Дмитрий Ларин]

public function actionView($id)
{
	$next = $id;
	$prev = $id;
	
	$data = $cache->getOrSet($id, function () {
		$all = Images::find()->all();
		foreach ($all as $key => $item) {
			if ($item['id'] == $id){
				$out['model'] = $item;
				if (array_key_exists($key + 1, $all)){
					$out['next'] = $all[$key + 1]['id'];
				}
				if (array_key_exists($key - 1, $all)){
					$out['prev'] = $all[$key - 1]['id'];
				}
				break;
			}
		}
		return out;
	});
	$next = $data['next'];
	$prev = $data['prev'];
	$model = $data['model'];
	
	if (!$model || !$id){
		throw new HttpException(404, '404');
	}
	return $this->render('view', [
		'model' => $model,
		'next' => $next,
		'prev' => $prev,
	]);
}

Подробнее в документации

Answer 1

[Antony Tkachenko]

Сделайте просто нормальные запросы, незачем перебирать всю таблицу.

//next
SELECT * FROM foo WHERE id > <your_id> ORDER BY id LIMIT 1;
//prev
SELECT * FROM foo WHERE id < <your_id> ORDER BY id LIMIT 1;

Решение в лоб, если подумать можно найти варианты получше (но не суть важно).
П.С. прежде чем работать с orm советую ознакомиться с mysql как минимум, чтобы понимать какой запрос нужен и как организовать грамотную структуру бд.

Comments

[Ruslan Ruslanov]

ага, прекрасное решение. прежде чем советовать что-то, советую оттестировать код, если вы не понимаете, что пишете.
для id = 2: next = 3, prev = 1;
для id = 3: next = 4, prev = 1;
и т.д.

[vyachin]

Ruslan Ruslanov, вам предложили подход, а вы уже накинулись на человека. И кто захочет вам потом помогать

//next
SELECT * FROM foo WHERE id > ORDER BY id ASC LIMIT 1;
//prev
SELECT * FROM foo WHERE id < ORDER BY id DESC LIMIT 1;

[Ruslan Ruslanov]

vyachin, если бы не его "p.s.", то я бы отреагировал спокойнее. а за ваш вариант спасибо, он сработал.

[Ruslan Ruslanov]

vyachin, добавьте ваш комментиарий в ответ, чтобы я мог отметить его решением.