Как получить список из массива без дублей?

Question

[Artemka1903]

Всем привет! Не могу разобраться, как получить список категорий из массива без дублей. Вот код

public static function getRightCategories()
    {
        $db = Db::getConnection();
        $sql = $db->query('SELECT category FROM news');
        $i = 0;
        $rightCategories = array();
        while ($row = $sql->fetch()) {
            $rightCategories[$i]['category'] = $row['category'];
            $i++;
        }
        return $rightCategories;
    }

Answer 1

[roswell]

SELECT DISTINCT category FROM news

Answer 2

[Slava Rozhnev]

CREATE TABLE categories (
	id int auto_increment primary key,
  	name varchar(64)
);

<?php
function getRightCategories($db)
{
	$result = $db->query('SELECT id, name FROM categories');
	return $result->fetchAll(PDO::FETCH_KEY_PAIR);
}

$categories = getRightCategories($pdo);

print_r($categories);

Test PDO fetch online

Answer 3

[Sergey]

Категории перенести в отдельную таблицу с уникальным id, а в новостях добавить связь.

Answer 4

[ThunderCat]

Не могу разобраться, как получить список категорий из массива без дублей.

Получать нужно из базы нормальный список, без дублей. В вашем случае это будет запрос с дистинкт, как написал roswell, однако:
1) Запросы с дистинкт будут перебирать всю таблицу, что плохо и медленно
2) Если у вас категории пишутся в таблицу ньюс, значит у вас лажа со структурой бд. Категории должны лежать отдельной справочной таблицей, выборка из которой будет во первых в разы быстрее, а во вторых будет удовлетворять третьей нормальной форме (3NF).
3) Текущая структура не позволяет задать более 1 категории на новость.

Есть еще куча проблем с таким хранением, но это уже другой вопрос.

Comments

[Artemka1903]

Я пока только учусь, все придет со временем)

[Ипатьев]

Artemka1903, не придет, если будешь продолжать говногодить после того как показал правильное решение

[Artemka1903]

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

[ThunderCat]

Лентюй, В случае дистинкт в любом случае будет медленнее чем из справочника. И нормальные формы тоже никто не отменял.
Artemka1903, Никто не требует от новичка идеальный код, хотя в данном конкретном случае речь больше про архитектуру и структуру. Но если это проект для обучения (хотя о чем я, если это реальный проект то это даже важнее!), то надо стараться применить все адекватные советы, как минимум для того чтобы понимать как оно должно быть, а то так и застрянешь на говнокоде.

В программировании нельзя бояться переделывать большие куски только по тому что "ну и так же работает, пофиг что из г-на и веток", потому что буквально завтра все может перестать работать, тут го*но подсохло, тут веточка отвалилась, и все, на вас смотрят как на говнбракодела, а нафига такой нужен в команде?

Короче, чем раньше заболеешь "перфекционизмом кода", тем быстрее вырастешь как специалист.

[ThunderCat]

Лентюй,

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

А то я думал что это хитрый набор слов конечно. Любая группировка добавляет работы движку, какой бы дешевый/продвинутый он не был.

причем тут вообще нормальные формы?

При том что таблица денормализованная, и сущность "категория" должна лежать в отдельной таблице, со связью один ко многим или многие ко многим через пивот? И тогда выборка из плоской таблицы не потребует группировок или других потных телодвижений?