Получение custom post type. Намудрил?

Question

[OlegSedoy]

Получаю кастомные записи, все работает, но что-то подсказывает, что намудрил. У кого будут замечания или комментарии?

<?php if ($res = \RestBuild\RestBuild::get_post('news', '4')): ?>
    <div class="news" id="news">
        <h2 class="news__title">Новости</h2>
        <div class="news__container">
            <?php
            foreach ($res as $post) {
                echo "<div class='news__item'>";
                echo "<h4>" . $post->post_title . "</h4>";
                echo get_the_post_thumbnail($post->ID, 'medium');;
                if (mb_strlen($post->post_content) >= 150) {
                    echo "<p>" . mb_strimwidth($post->post_content, 0, 150) . "</p>";
                    echo 'Подробнее';
                } else {
                    echo "<p>" . $post->post_content . "</p>";
                }
                echo "</div>";
            }
            ?>
        </div>
        <?php if ($res_load = \RestBuild\RestBuild::get_post('news', '', 4)): ?>
            <div class="news__load">Загрузить еще <?= count($res_load); ?></div>
        <?php endif; ?>
    </div>
<?php endif; ?>

метод

public static function get_post($post_type, $posts_per_page = -1, $offset = 0)
    {
        $my_posts = new \WP_Query;
        $myposts = $my_posts->query(array(
            'post_type'      => $post_type,
            'posts_per_page' => $posts_per_page,
            'offset'         => $offset,
            'order'          => 'ASC',
            'post_status'    => 'publish',
        ));
        wp_reset_postdata();
        return $myposts;
    }

Comments

[Модератор]

OlegSedoy дорогой пользователь, настоятельно рекомендуем еще раз обратить самое пристальное внимание на п. 3.1 регламента работы сервиса (и, в особенности, на его последний абзац).
В противном случае, ваши вопросы будут удаляться по причине тег-спама, а систематические нарушения приведут к блокировке учетной записи.

[Игорь Воротнёв]

А можно поинтересоваться зачем такие сложности? Потому что да, перемудрил.

Answer 1

[Дмитрий]

зачем усложнять простое? чем больше кода, тем больше шансов на ошибку.

коротко ревью (часть чисто мои хотелки, править не нужно):
1. весь вывод без фильтров (the_content, the_title и тп)
2. нет локализации (может и не нужна, но по стандартам надо)
3. <?= - короткие теги не по стандартам
4. get_post('news', '4') - второй параметр int или string?
5. $posts_per_page = -1 - плохо для производительности
6. два запроса к WP_Query, хотя можно было ограничится одним https://codex.wordpress.org/Plugin_API/Filter_Refe...
7. нет проверки/фолбэка для get_the_post_thumbnail()
8. mb_strlen($post->post_content) >= 150 можно заменить на стандартный https://developer.wordpress.org/reference/function...
9. ненужный wp_reset_postdata(). Функция нужна, если менялся глобальный $post при использовании стандартного лупа. Тут такого нет
10. собсно - нет стандартного лупа https://codex.wordpress.org/The_Loop

если следовать стандартам, код станет производительнее, проще, понятнее и не будет ломать работу других плагинов.

Comments

[Дмитрий]

а, ну и
get_post('news', '', 4)
не сделает ожидаемое get_post($post_type, $posts_per_page = -1, $offset = 0)

$posts_per_page в функции будет равна ''

[OlegSedoy]

Спс, за конструктивные замечания, частично исправляюсь

<?php
$visible_post = 4;
$query = \RestBuild\RestBuild::get_post_query('news', $visible_post);
if ($query->have_posts()) :?>
    <div class="news" id="news">
        <h2 class="news__title">Новости</h2>
        <div class="news__container">
            <?php
            while ($query->have_posts()) {
                $query->the_post();
                get_template_part('loop/loop', 'news');
            }
            ?>
        </div>
        <?php if (intval($query->found_posts) - $visible_post > 0) : ?>
            <div class="news__load">Загрузить еще <?php echo $query->found_posts - $visible_post; ?></div>
        <?php endif; ?>
    </div>
<?php endif; ?>

луп

<div class='news__item'>
    <h4><?php esc_html(the_title()); ?></h4>
    <?php if (has_post_thumbnail()) the_post_thumbnail('medium'); ?>
    <?php the_excerpt(); ?>
</div>

[Дмитрий]

OlegSedoy, еще можно поставить phpcs, будет подсвечивать, что не по стандартам