Как задать сортировку списка по алфавиту?

Question

[there4]

Привет! Что-то я немного завис... есть у меня код, выводящий список закладок пользователя (заголовки записей):

/**
     * Выводим список закладок
     *
     * @param string $post_type
     * @param int $user_id
     * @param int $limit
     * @param bool $show_remove
     */
    function display_favlist( $post_type = 'all', $user_id = false, $limit = 10, $show_remove = true ) {

        $posts = $this->get_favorites( $post_type, $user_id, $limit );

        echo '<div class="favclass"><ul>';
        if ( $posts ) {

            $remove_title = __( 'Удалить из списка, 'wl1' );
            $remove_link = ' <a href="#" data-id="%s" title="%s" class="wl1-remove-favorite">x</a>';

            foreach ($posts as $item) {
                $extra = $show_remove ? sprintf( $remove_link, $item->post_id, $remove_title ) : '';
                printf( '<li><a href="%s">%s</a>%s</li>', get_permalink( $item->post_id ), get_the_title( $item->post_id ), $extra );
            }
        } else {
            printf( '<li>%s</li>', __( 'Нет закладок', 'wl1' ) );
        }
        echo '</ul></div>';
    }

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

Answer 1

[Max Medar]

Используйте функции сортировки. usort, например.

Answer 2

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

Сортировать лучше не на выводе в цикле, а еще при получении всех записей. В вашем случае за это отвечает вот этот код:

$posts = $this->get_favorites( $post_type, $user_id, $limit );

Покажите что там в методе get_favorites(). Судя по тому, что дальше вы работаете с массивом объектов, внутри этого метода вызывается get_posts(). Попросите его отсортировать для вас посты еще на этапе выборки из БД.

Comments

[Max Medar]

Зачем изменять поведения метода? В данном случае куда уместнее сделать банальную сортировку массива.

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

Max Medar, ну, во-первых потому что метод очевидно тупо проксирует на get_posts, формируя его из полученных параметров (тип поста, пользователь, сколько записей). Метод не является final и не вижу проблем почему вдруг нельзя менять его поведение. Можно захардкодить сортировку, можно передать ее дополнительным параметром. В чем проблема изменить метод и его сигнатуру при необходимости? Религия?

Во-вторых, лимит вполне возможно может использоваться для постраничного/порционного вывода, или даже для простого ограничения записей числом Х, но таким образом сортировка будет применяться к каждой порции записей независимо. То есть, вы на выходе будете получать не "все мои избранные по алфавиту", а Х моих избранных по дате, но отсортированы по алфавиту. Надеюсь, различие в получаемом списке просматривается. Как реально топикстартеру надо - это уже другой вопрос, но то что здесь имеет место неочевидный потенциальный баг - очевидно.

В третьих, идеологически правильнее делать сортировки максимально близко к источнику данных, тем более если он отлично умеет это делать. Зачем лишний оверхед на php?

В четвертых, просматривается еще один очень неочевидный момент - у топикстартера на выводе присутствует кнопка "удалить из избранного". После удаления не совсем понятно что происходит - подтягивается новый список целиком? Длина списка просто уменьшается? Подтягивается 1 дополнительный элемент? Или может есть буфер из нескольких элементов, который позволит просто дозаполнить список? Сам по себе этот неочевидный момент не критичен сейчас, но понимая его может появиться дополнительное понимание, можно ли обойтись простой сортировкой на выводе, или все же лучше сделать выборку правильно изначально?

Имхо, данные надо максимально правильно выбирать из БД изначально, а не строить костыли.

[Max Medar]

Игорь Воротнёв, это правильный подход. Но нужно ли в данном случае ТС-у делать правильно? Скорее всего нет.

В любом случае спасибо за развернутый ответ.

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

Но нужно ли в данном случае ТС-у делать правильно?

Может я конечно задрот (на самом деле да), но имхо правильно надо делать всегда, даже если конкретная задача / ситуация / проект не требуют этого прямо. Это вопрос личной гигиены и качественного роста собственных скилов.

[there4]

Вы предлагаете в этот метод usort вставить?

function get_favorites( $post_type = 'all', $user_id = 0, $count = 10, $offset = 0 ) {
        $where = 'WHERE user_id = ';
        $where .= $user_id ? $user_id : get_current_user_id();
        $where .= $post_type == 'all' ? '' : " AND post_type = '$post_type'";


        $sql = "SELECT post_id, post_type
                FROM {$this->table}
                $where
                GROUP BY post_id
                ORDER BY post_type
                LIMIT $offset, $count";

        $result = $this->db->get_results( $sql );

        return $result;
    }

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

Max Medar, ну вот, как я и предполагал, тут есть таки offset, а значит как минимум некое подобие пагинации/порционности. А значит, usort будет давать именно те проблемы, которые я описал в комментарии выше.

there4, у вас тут кастомный запрос к БД. Перепишите его чтобы сортировал по алфавиту, а не по post_type.

[vayo]

Игорь Воротнёв, привет, я похожий запрос себе на сайте пытался сделать, так и не получилось. Как по алфавиту записи выставить в итоге можно? На примере выше сможете объяснить?

[vayo]

Автор, поделитесь решением, если не сложно!

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

vayo, https://stackoverflow.com/questions/1066453/mysql-...
Либо JOIN, либо подзапрос. Почитайте по ссылке все комментарии к ответам, а также переходите по ссылкам на документацию оттуда. Там много полезной инфо которая объясняет как работает GROUP BY и ORDER BY, особенно в паре.