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

Question

[Button_k]

Помогите новичку оптимизировать запросы. Есть блок на главной странице с 3 запросами. Знаю что можно сделать вывод разных классов в одном запросе через переменные, а дальше if и else но не знаю как.

<section class="section_edit">
	<div class="wrapper_edit">
		<div class="container_edit">
			<div class="edit_left">
					<?php
					// параметры по умолчанию
					$my_posts = get_posts( array(
						'numberposts' => 1,
						'category_name'    => 'reviews',
						'orderby'     => 'rand',
						'order'       => 'DESC',
						'include'     => array(),
						'exclude'     => array(),
						'meta_key'    => '',
						'meta_value'  =>'',
						'post_type'   => 'post',
						'suppress_filters' => true, // подавление работы фильтров изменения SQL запроса
					) );

					global $post;

					foreach( $my_posts as $post ){
						setup_postdata( $post );
					?>
					<div class="edit_left_top">
						<div class="box_edit">
							<div class="box_img_edit">
								<a href="<?php the_permalink() ?>">
									<?php $thumb = wp_get_attachment_image_src( get_post_thumbnail_id($post->ID), 'full' );?>
									<img src="<?php echo $thumb['0'];?>" alt="" class="picture_edit">
								</a>
							</div>
							<div class="edit_meta">
								<div class="cat_block_game">
									<div class="cat_game"><?php the_category(' ') ?></div>
								</div>
								<div class="title_block_game">
									<a class="edit_left_top_title" href="<?php the_permalink() ?>"><?php trim_title_chars(60, '...'); ?></a>
									<p class="edit_left_top_title_description"><?php do_excerpt(get_the_excerpt(), 20); ?></p>
								</div>
								<div class="data_block_game">
									<div class="data_game"><?php echo get_the_date('j M. Y'); ?></div>
								</div>
							</div>
						</div>
					</div>
					<?php
					}

					wp_reset_postdata(); // сброс
					?>	
				<div class="edit_left_bottom">
				<?php
				// параметры по умолчанию
				$my_posts = get_posts( array(
					'numberposts' => 3,
					'category_name'    => '',
					'orderby'     => 'rand',
					'order'       => 'DESC',
					'include'     => array(),
					'exclude'     => array(),
					'meta_key'    => '',
					'meta_value'  =>'',
					'post_type'   => 'post',
					'suppress_filters' => true, // подавление работы фильтров изменения SQL запроса
				) );

				global $post;

				foreach( $my_posts as $post ){
					setup_postdata( $post );
				?>

				<div class="edit_left_bottom_item">
					<div class="edit_top_card">
						<div class="edit_top_card_img">
							<a href="<?php the_permalink() ?>">
								<?php $thumb = wp_get_attachment_image_src( get_post_thumbnail_id($post->ID), 'full' );?>
								<img src="<?php echo $thumb['0'];?>" alt="" class="picture_bottom_edit">
							</a>
						</div>
						<div class="meta_edit">
							<div class="cat_block_edit">
								<div class="cat_game"><?php the_category(' ') ?></div>
							</div>
						</div>
					</div>
					<div class="edit_bottom_card">
						<a class="item_edit_meta_title_left" href="<?php the_permalink() ?>"><?php trim_title_chars(40, '...'); ?></a>
						<div class="data_game"><?php echo get_the_date('j M. Y'); ?></div>
					</div>
				</div>

				<?php
				}

				wp_reset_postdata(); // сброс
				?>	
				</div>
			</div>
			<div class="edit_right">
				<?php
				// параметры по умолчанию
				$my_posts = get_posts( array(
					'numberposts' => 6,
					'category_name'    => '',
					'orderby'     => 'rand',
					'order'       => 'DESC',
					'include'     => array(),
					'exclude'     => array(),
					'meta_key'    => '',
					'meta_value'  =>'',
					'post_type'   => 'post',
					'suppress_filters' => true, // подавление работы фильтров изменения SQL запроса
				) );

				global $post;

				foreach( $my_posts as $post ){
					setup_postdata( $post );
				?>
				<div class="item_edit_right">
					<div class="item_edit_right_img">
							<a href="<?php the_permalink() ?>">
								<?php $thumb = wp_get_attachment_image_src( get_post_thumbnail_id($post->ID), 'full' );?>
								<img src="<?php echo $thumb['0'];?>" alt="" class="item_edit_right_images">
							</a>
					</div>
					<div class="item_edit_meta">
						<div class="item_edit_meta_data">
							<div class="data_game"><?php echo get_the_date('j M. Y'); ?></div>
						</div>
						<a href="<?php the_permalink() ?>" class="item_edit_meta_title"><?php trim_title_chars(46, '...'); ?></a>
					</div>
				</div>
				<?php
				}

				wp_reset_postdata(); // сброс
				?>	
			</div>
		</div>
	</div>
</section>

Comments

[yarovikov]

Какая задача стоит?

[Button_k]

yarovikov, Оптимизировать код до одного запроса с разными выводом блоков

[yarovikov]

Button_k, в вашем случае это не имеет смысла. Но можете начать с разделения логики и верстки. html должен выглядеть примерно так

[Button_k]

yarovikov, Примерно как здесь. Видел похожую тему.

case "a":
?>
...the html...
<?php
break;

case "b":
...some code...
break;

}
?>

[Button_k]

yarovikov, Я думал что такой вывод с 3 запросами будет более затратный по ресурсам.

[yarovikov]

Button_k, в моем коде 1 запрос. Но в логике (часть, где вы получаете данные) при любом раскладе будет три, потому что разные условия в вашем массиве. Единственное, что вы можете сделать - не плодить говнокод, а делать красиво )

Answer 1

[Александр Варлаков]

Привет, для начала убери:

'include'     => array(),  - пустой
     'exclude'     => array(), - пустой
     'order'       => 'DESC', - по дефолту
     'meta_key'    => '',  - пустой
     'meta_value'  =>'',  - пустой

И будет что то вроде:

$my_posts = get_posts( array(
    'numberposts' => 1,
    'category_name' => 'reviews',
    'orderby' => 'rand',
    'suppress_filters' => true,
) );

Далее:
Это одно и тоже:

$my_posts = get_posts( array(
					'numberposts' => 3,
					'category_name'    => '',
					'orderby'     => 'rand',
					'order'       => 'DESC',
					'include'     => array(),
					'exclude'     => array(),
					'meta_key'    => '',
					'meta_value'  =>'',
					'post_type'   => 'post',
					'suppress_filters' => true, // подавление работы фильтров изменения SQL запроса
				) );


$my_posts = get_posts( array(
					'numberposts' => 6,
					'category_name'    => '',
					'orderby'     => 'rand',
					'order'       => 'DESC',
					'include'     => array(),
					'exclude'     => array(),
					'meta_key'    => '',
					'meta_value'  =>'',
					'post_type'   => 'post',
					'suppress_filters' => true, // подавление работы фильтров изменения SQL запроса
				) );

Сократи до одного:

$my_posts = get_posts( array(
					'numberposts' => 9,
					'orderby'     => 'rand',
					'post_type'   => 'post',
					'suppress_filters' => true, // подавление работы фильтров изменения SQL запроса
				) );

И выведи через for

$my_posts = get_posts( array(
    'numberposts' => 9,
    'orderby' => 'rand',
    'post_type' => 'post',
    'suppress_filters' => true,
) );

for ($i = 0; $i < 3 $i++) {
}
for ($i = 3; $i < count($my_posts);  $i++) {
}

Comments

[Button_k]

Да спасибо это все уберу конечно. Но вопрос в другом. Я как новичок не могу понять как сделать все правильно. Что бы в дальнейшем не переделывать и нагрузка от запроса в БД была минимальным. Если тот вариант который я написал выглядит не плохо, то я его оставлю. Но мне кажется что должен быть выход написания одного запроса с разным выводом стилей, а не копировать 3 запроса с изменение пару строк div

[Александр Варлаков]

Button_k, Я двумя частями отвечал

Опубликовал вторую

[Александр Варлаков]

Button_k,

1. Сократили количество запросов с 3 до 2
2. Упростили запросы убрав лишний код

Больше не знаю что тут оптимизировать

[Button_k]

Спасибо за ответ. А если 'category_name' => '', будут разными во всех запросах?

[Александр Варлаков]

Тогда это разные запросы и смысла объединять не будет

[Button_k]

Александр Варлаков, Еще раз спасибо что объяснили что к чему. В такой случае буду использовать свой код.