Форум MySQL

Пишу запрос

SELECT id, text FROM records WHERE author=".$this->info['id']." AND $viewer IN (can_see) ORDER BY add_date LIMIT $offset,$count

смысл в том, что в столбце can_see хранится список вида "1,2,3" id пользователей, которые могут видеть запись. Если так прямо и написать, "$viewer IN (1,2,3)", то все верно работает, если написать как выше, ничего не получается. я так понимаю, нужен вложенный SELECT запрос. Можно ли обойтись без него?

can_see - это константа?
или это поле?
или там $can_see должно быть?

can_see -- это поле в той же таблице, причем в той же строке. грубо говоря, нужно вытащить все записи какого-то пользователя (Вася), которые мог бы видеть Петя, т.е. Петя находится в списке видящих, который записан в поле can_see в формате 1,2,3 (id пользователей через запятую)

значения в поле через запятую - это прямое нарушение первого закона нормализации
при проектировании базы данных, лучше это переделать
нужно добавить таблицу связи can_see с полями id_user | id_user_see и примари кей на оба поля
___
кстати почему у вас $viewer переменная?
покажите поля таблицы

предлагаете мне создать новую таблицу can_see с полями id, record_id, user_id?

у вас Петя должен видеть все сообщения Васи или выборочно?
если все то ответил в предыдущем посте, если каждое сообщение имеет разрешение на просмотр то
record_id, user_id (id - тут не нужен, функцию первичного ключа будут выполнять оба поля - такой ключь называется составным, главное не забыть об этом сообщить базе данных)

Таблица

CREATE TABLE `records` (   `id` int(11) unsigned NOT NULL AUTO_INCREMENT,   `author` int(11) unsigned NOT NULL,   `text` varchar(500) NOT NULL,   `description` text NOT NULL,   `emotion` enum('none', 'happy', 'sad') NOT NULL DEFAULT 'none',   `location` int(11) NOT NULL,   `can_see` enum('all', 'friends', 'me', 'list') NOT NULL DEFAULT 'all',   `can_see_list` varchar(255) NOT NULL,   `cant_see` enum('none', 'all', 'list') NOT NULL DEFAULT 'none',   `cant_see_list` varchar(255) NOT NULL,   `count_comments` int(11) unsigned NOT NULL DEFAULT 0,   `add_date` int(11) unsigned NOT NULL,   PRIMARY KEY (`id`) ) ENGINE=MyISAM  DEFAULT CHARSET=utf8;

$viewer -- это переменная, которая передается в функцию, которая формирует список записей. Переменная -- это пользователь, который просматривает страницу.

теперь яснее

А имеет ли смысл делать таблицу с полями:
record_id, user_id, privacy ('show', 'hide') -- имеется в виду поле типа ENUM

или для show и hide лучше делать разные таблицы?

сегодня уже тыква не варит)) завтра подумаю и напишу (точнее уже сегодня)) )
но по моему лучше 2 таблицы
хотя они взаимоисключающие))
если разрешено, значит не запрещено и наоборот)
___
спать, спать, спать))) а то мозг лопнет))

спасибо за помощь)))

просто чую, теперь будут дикие джоины идти в запросе))

-------

получается, придется писать что-то типа:

SELECT r.id FROM records WHERE can_see='all' AND NOT EXISTS (SELECT * FROM cant_see_list WHERE user=$visitor AND record=r.id);

???

Такой запрос будет адекватен?

SELECT id FROM records WHERE author=$author AND can_see='all' OR (can_see='friends' AND EXISTS (SELECT * FROM friends WHERE user_from=$author AND user_to=$viewer AND status='approved')) OR (can_see='list' AND EXISTS (SELECT * FROM can_see_list WHERE user=$viewer AND record=id)) OR (can_see='me' AND author=$viewer) AND NOT EXISTS (SELECT * FROM cant_see_list WHERE user=$viewer AND record=id) ORDER BY add_date LIMIT 10;

Если работает и скорость работы устраивает - адекватен.

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

Для начала можно попытаться переписать запрос с использованием объединений таблиц, посмотреть при помощи оператора EXPLAIN схему выполнения и проиндексировать соответствующие ключи. Если скорость по прежнему будет не удовлетворительной следует думать в сторону нормализации таблиц или создания кэширующей таблицы, которая бы дублировала данные из нескольких таблиц (т.е. фактически хранить копию объединенных данных).