Как сделать выборку, чтобы совпали значения двух таблиц, и вывести результат?

Question

[Lorem Ipsum]

Имеются две таблицы mysql :
1) user_uploads

CREATE TABLE IF NOT EXISTS `user_uploads` (
  `upload_id` int(9) NOT NULL AUTO_INCREMENT,
  `image_name` text NOT NULL,
  `user_id_fk` int(11) NOT NULL,
  `created` int(11) NOT NULL,
  PRIMARY KEY (`upload_id`)
) ENGINE=InnoDB  DEFAULT CHARSET=utf8 AUTO_INCREMENT=0 ;

2) user_walls

CREATE TABLE IF NOT EXISTS `user_walls` (
  `id` int(10) unsigned NOT NULL AUTO_INCREMENT,
  `user_id` int(10) unsigned NOT NULL,
  `message` text COLLATE utf8_unicode_ci NOT NULL,
  `date` datetime NOT NULL,
  `images` varchar(255) COLLATE utf8_unicode_ci NOT NULL, 
  PRIMARY KEY (`id`)
) ENGINE=MyISAM  DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci AUTO_INCREMENT=0 ;

В таблицу user_walls в строку images записываются через запятую upload_id(из таблицы user_uploads)
Как сделать выборку из этих двух таблиц так, чтобы совпали значения:
upload_id(где upload_id = image_name) == images, и вывести значения таблицы user_walls вместе с загруженными файлами из таблицы user_uploads

Comments

[Lorem Ipsum]

таблица user_uploads

таблица user_walls

Answer 1

[Александр Литвинов]

Лучше изменить архитектуру

SELECT * FROM user_walls AS uw
JOIN user_uploads AS uu
ON uw.images REGEXP '[[:<:]]' || uu.upload_id || '[[:>:]]'
WHERE uw.id = 1

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

Comments

[Lorem Ipsum]

а что означает данная строка ? REGEXP '[[:<:]]' || uu.upload_id || '[[:>:]]'

[Lorem Ipsum]

если можно как вместе с текстом вывести эти файлы? у меня сейчас вот так: $result = mysql_query("SELECT `user_id`,`message`,`date`,`images` FROM `user_walls` LEFT JOIN `user_uploads` ON `user_walls`.`images`=`user_uploads`.`upload_id` WHERE `upload_id` = `image_name` ORDER BY `id` DESC");

[Lorem Ipsum]

но это не работает

[Александр Литвинов]

@GeorgeGeorge Это и не должно работать. В моем примере это регулярное выражение, сам не пробовал, но вроде должно сработать в MySQL

[Александр Литвинов]

@GeorgeGeorge upload_id я указал на абум, для примера

[Lorem Ipsum]

понял, спасибо, буду пробовать:)

[Александр Литвинов]

@GeorgeGeorge Если требуются делать подобные запросы, советую все таки создать табличку пересечения, конечно если это возможно. Это очень плохие запросы

[Александр Литвинов]

@GeorgeGeorge Хотелось бы узнать, сработал ли запрос :)

[Lorem Ipsum]

нет( не выводит ни сообщения, ни картинки, приложенные к ним:(

[Александр Литвинов]

@GeorgeGeorge Поправил, попробуй еще разок. У меня вроде работает

[Lorem Ipsum]

@Sander_Li действительно работает, но появляется повторение текста, если более 1 фотографии приложено к записи, но в ссылке на фотографию вместо имени выдает id, а надо имя

[Александр Литвинов]

@GeorgeGeorge Что за ссылке? еще одно поле?

[Lorem Ipsum]

@Sander_Li надо сравнить id user_uploads и images(это id user_uploads разделенные запятой в одной ячейки) и вывести images_name из user_uploads если id user_uploads и images(user_walls) совпадают

[Александр Литвинов]

@GeorgeGeorge У меня выглядит так

[Lorem Ipsum]

@Sander_Li только не понимаю как это записать

[Александр Литвинов]

@GeorgeGeorge Наверное я что-то недопонимаю( на моем скрине выводится image_name, при совпадениях images и upload_id. В условии WHERE uw.id это id строки user_walls. Вроде только по нему можно вывести.

[Lorem Ipsum]

@Sander_Li как тут картинки прикрепить?)

[Lorem Ipsum]

я свои варинты скину)

[Александр Литвинов]

@GeorgeGeorge добавь не к комментарию ответа, потом вырежи сюда :)

[Lorem Ipsum]

вот скинул) там где вопрос) выше)

[Lorem Ipsum]

и мне надо вывести все фотографии, вместе с этим текстом( то есть 4 фотографии и под ними текст)

[Александр Литвинов]

@GeorgeGeorge почти такую же создал табличку и выполнил запрос

[Александр Литвинов]

@GeorgeGeorge Запрос поправил на тот который исполнял

[Lorem Ipsum]

а как выборку сделать? чтобы вывести все фотографии и текст к ним)

[Александр Литвинов]

@GeorgeGeorge

[Александр Литвинов]

@GeorgeGeorge У меня не верно вывелось? вроде все верно

[Александр Литвинов]

@GeorgeGeorge Если слишком много результатов, можно перечислить что надо) Например так

SELECT  uu.upload_id, uw.message FROM user_walls AS uw
JOIN user_uploads AS uu
ON uw.images REGEXP '[[:<:]]' || uu.upload_id || '[[:>:]]'
WHERE uw.id = 1

[Александр Литвинов]

И вот результат запроса

[Lorem Ipsum]

@Sander_Li у меня тоже выводятся id) а мне надо как картинки это вывести) то есть у меня выводится вот так:

то есть выводятся id этих файлов, а не имена их) как вывести имена, а не id ?

[Lorem Ipsum]

<'img src="uploads/26,27,28,29" width="100" height="100"'>

[Lorem Ipsum]

именно выборка из бд и вывод на экран )

[Александр Литвинов]

@GeorgeGeorge Тут уже скорее дело не в запросе, а коде. В запросе можно указывать явно, что нуно выбирать. В первоначальном варианте выводится все, что нужно. Запрос работает верно

[Александр Литвинов]

@GeorgeGeorge Как я понял вот что нужно

SELECT image_name
FROM user_walls AS uw
JOIN user_uploads AS uu ON uw.images REGEXP  '[[:<:]]' || uu.upload_id ||  '[[:>:]]'
WHERE uw.id =1

[Lorem Ipsum]

@Sander_Li второй вариант ничего не выводит :(

[Александр Литвинов]

@GeorgeGeorge Как не выводит, у меня такая же таблица) Как проверяешь? в phpmyadmin?

[Александр Литвинов]

@GeorgeGeorge Если все верно понял, чтобы теперь все работало, надо править код.. foreach при выводе нужен, ведь раньше выводилась одна картинка.

[Lorem Ipsum]

@Sander_Li спасибо большое за помощь) вот нашел еще один варинт)) подскажи пожалуйста, как при выборке из таблицы, разделить значения ? у меня в images через запятую название фотографий)

[Lorem Ipsum]

[Lorem Ipsum]

$result = mysql_query("SELECT * FROM user_walls WHERE user_id = ".$user_id." ORDER BY `id` DESC");

[Александр Литвинов]

@GeorgeGeorge если с кодом туго, можно вот так попробывать

SELECT image_name as 'upload_id'
FROM user_walls AS uw
JOIN user_uploads AS uu ON uw.images REGEXP  '[[:<:]]' || uu.upload_id ||  '[[:>:]]'
WHERE uw.id =1

Должно выйти что-то вроде <'img src="uploads/имя1, имя2, имя3.." width="100" height="100"'> Последний вопрос недопонял:( На уровне бд?

[Lorem Ipsum]

на php) вот видишь скрин таблицы?) там в ячейке images два названия картинки, разделенные запятой, как их разделить при выборке ? сейчас у меня вот так получилось <'img src="uploads/имя1.jpg, имя2.jpg " width="100" height="100"'> , а надо вот
<'img src="uploads/имя1.jpg" width="100" height="100"'>
<'img src="uploads/имя2.jpg" width="100" height="100"'>
как именно в php коде(когда делаю выборку из таблицы) сделать?

[Александр Литвинов]

@GeorgeGeorge Сейчас напишу) просто

[Lorem Ipsum]

@Sander_Li спасибо большое)

[Александр Литвинов]

@GeorgeGeorge Что-то в таком духе. Толлько советую использовать Mysqli или что-то вроде того. Ваш код не безопасен.

while ($row = mysql_fetch_assoc($result)) {
  $images = implode(',', result['images']);
  foreach($images as $image){
        echo '<img src = "'.$image.'">';
  }
}

[Александр Литвинов]

Собственно сам запрос $result = mysql_query("SELECT images FROM user_walls WHERE user_id = ".$user_id." ORDER BY `id` DESC");

[Lorem Ipsum]

@Sander_Li спасибо большое) от души) спасибо!)
не безопасен в плане sql инъекций ?

[Александр Литвинов]

@GeorgeGeorge Да, именно. Может в этом конкретном случае, она и не случится. Но в целом. Да и функция устаревшей является.

[Lorem Ipsum]

@Sander_Li понял)
php учу месяца 2) этот проект делаю и одновременно учусь)

[Александр Литвинов]

@GeorgeGeorge Для работы с базой удобно использовать ActiveRecord, Dao, Datamapper. Советую посмотреть, но скорее всего понадобится больше знаний.

[Lorem Ipsum]

понял, посмотрю обязательно.
пока ты тут еще) что-то ошибок мне он навыдовал)
Warning: implode() [function.implode]: Invalid arguments passed in строка 2
Warning: Invalid argument supplied for foreach() in строка 3
Warning: implode() [function.implode]: Invalid arguments passed in строка 2
Warning: Invalid argument supplied for foreach() in строка 3

[Александр Литвинов]

@GeorgeGeorge я там опечался, забыл доллар поставить на второй строке)

[Александр Литвинов]

@GeorgeGeorge и еще не $result['image'], a $row['image'] Засыпаю уже :)

[Александр Литвинов]

$row['images']*

[Александр Литвинов]

while ($row = mysql_fetch_assoc($result)) {
  $images = implode(',', $row['images']);
  foreach($images as $image){
        echo '<img src = "'.$image.'">';
  }
}

[Lorem Ipsum]

@Sander_Li да все увидел)) спасибо большое)) огромное спасибо!)

[Lorem Ipsum]

@Sander_Li вы тут ?:) просто ошибки выдает что-то( исправленный варинат тоже( целый день не могу разобраться в чем может быть дело

[Александр Литвинов]

@GeorgeGeorge Давай в скайп лучше? Тут уже не удобно. Sander_Li.

[Lorem Ipsum]

давай) добавил)

Answer 2

[t0di]

По-правильному вариант с такой архитектурой таблицы не приемлем. Если конечно таблицы маленькие и рост не планируется, то можно закрыть а это глаза. А вообще есть 2 варианта:
1. вводить третью связующую таблицу.
2. в таблице user_uploads добавить поле wall_id
Лично я бы выбрал второй вариант, тогда получение всех картинок делается одним left join.

Comments

[Lorem Ipsum]

а если планируется рост?

[t0di]

вариант номер 2. добавляешь поле wall_id в таблицу uploads. в итоге у тебя у каждой картинки есть запись о том к какому сообщению она привязана.

[Lorem Ipsum]

понял, буду пробовать этот способ, спасибо :)

[t0di]

соответственно в таблице можно убрать поле user_id_fk. Если я правильно понимаю, что таблица предназначена только хранения картинок со стены.

[Lorem Ipsum]

@t0di именно так, пытаюсь сделать что-то подобное стены "вконтакте", а именно прикрипление фотографий к постам, как "вконтакте"