Relationer.. HJÄLP!!!!

SELECT * FROM artister as a, genre_music as gm, labels as l, lyrics as ls, music as m WHERE a.id = x AND a.id = gm.artist AND a.id = ls.artist AND a.id = m.artist AND l.musicid = m.id;

Ovanstående frågan borde hämta all information som gäller en viss artist. Observera att du måse ange artist id där jag har skrivit ett 'x' ( diret efter WHERE-satsen).

Tråden passar bättre i SQL-forumet, så tråden flyttas från PHP!

Om du vill säkerställa databasintegriteten (att rätt information lagras på rätt plats) i databasen rekommenderar jag dig att skapa främmandenycklar. Vilka tabeller som skall ha främmandenycklar beror helt och hållet på hur du har valt att designa din databas.

jme skrev:

SELECT * FROM artister as a, genre_music as gm, labels as l, lyrics as ls, music as m WHERE a.id = x AND a.id = gm.artist AND a.id = ls.artist AND a.id = m.artist AND l.musicid = m.id;

Ovanstående frågan borde hämta all information som gäller en viss artist. Observera att du måse ange artist id där jag har skrivit ett 'x' ( diret efter WHERE-satsen).

Tack för hjälpen, men nu fick jag...

Unknown column 'm.artist' in 'where clause'

:q

Guldstrand:

jag kikade lite på tavlan music och där är artist av typen char. Vad är artist i music? Ska den inte vara en int?

Om du ska ha främmandenycklar så måste de fälten som binds ihop vara av sama typ och av samma storlek. Du kör ibland med int(10) och ibland med int(11). Kör med samma storlek om dessa fält ska bindas ihop.

WHERE a.id = m.artist fungerar inte då a.id är av typen int och m.artist är av typen varchar.

Sen varför du får "Unknown column" är jag inte riktigt säker på för artist finns ju i tavlan music men det kan ha göra med att dessa två fält inte är av samma typ.

Sen om jag minns srätt så måste tavlorna vara av typen InnoDB för att man skall kunna använda främmandenycklar (foreign keys).

jme skrev:

Guldstrand:

jag kikade lite på tavlan music och där är artist av typen char. Vad är artist i music? Ska den inte vara en int?

Om du ska ha främmandenycklar så måste de fälten som binds ihop vara av sama typ och av samma storlek. Du kör ibland med int(10) och ibland med int(11). Kör med samma storlek om dessa fält ska bindas ihop.

WHERE a.id = m.artist fungerar inte då a.id är av typen int och m.artist är av typen varchar.

Sen varför du får "Unknown column" är jag inte riktigt säker på för artist finns ju i tavlan music men det kan ha göra med att dessa två fält inte är av samma typ.

Sen om jag minns srätt så måste tavlorna vara av typen InnoDB för att man skall kunna använda främmandenycklar (foreign keys).

Fältet "artist_id" (i music-tabellen) har int(10).
Har även kontrollat så att alla fält har samma "storlek" och "typ", men jag har fortfarande samma problem. :l

Jag har lyckats att komma ett steg längre med att hämta rätt information till rätt ID, men det finns fortfarande problem med att hämta ALL information. :(

Hur se din nuvarande sqlfråga ut?

jme skrev:

Hur se din nuvarande sqlfråga ut?

Den ser ut som följer...

$sql = mysql_query("
SELECT  * 
FROM artister, countries, music, genre_music 
WHERE 
artister.id = ".$_GET['id']." AND 
countries.artist_id AND 
music.artist_id AND 
genre_music.artist_id AND
artister.kat_id
") or die(mysql_error());

:r

1. du har:

WHERE
artister.id = ".$_GET['id']." AND
countries.artist_id AND ...

du måste ha typ:

WHERE
artister = xxx AND countries.artist_id = xxx osv

2. Placera ALDRIG värden ur GET och POST direkt i sql-värden. Du kan utsätta din kod för s k sql-injections, dvs en illasinnad person kan placera en sql-fråga i parametern id som sedan exekveras.

Googla för med infromation om sql-injections.

I tavlorna music och genre_music så har du inte artist_id.

I tavlan music har du fältet label som är av typen varchar men ska den inte vara en int och referera fältet id i tavlan labels (foreign key)?

jme skrev:

I tavlorna music och genre_music så har du inte artist_id.

I tavlan music har du fältet label som är av typen varchar men ska den inte vara en int och referera fältet id i tavlan labels (foreign key)?

Såhär ser mina tabeller ut nu.

CREATE TABLE `artister` (
  `id` int(11) NOT NULL auto_increment,
  `kat_id` int(10) unsigned NOT NULL default '0',
  `musicid` int(10) unsigned NOT NULL default '0',
  `datum` datetime NOT NULL default '0000-00-00 00:00:00',
  `anamn` varchar(80) NOT NULL default '',
  `fnamn` varchar(80) NOT NULL default '',
  `birthday` date NOT NULL default '0000-00-00',
  `formed` date NOT NULL default '0000-00-00',
  `land` int(10) unsigned NOT NULL default '0',
  `medlemmar` varchar(100) NOT NULL default '',
  `artist_foto` varchar(100) NOT NULL default '',
  `info` text NOT NULL,
  `webb` varchar(100) NOT NULL default '',
  `lyrics` int(10) unsigned NOT NULL default '0',
  PRIMARY KEY  (`id`),
  UNIQUE KEY `anamn` (`anamn`),
  KEY `land` (`land`),
  KEY `musicid` (`musicid`,`lyrics`)
) ENGINE=MyISAM DEFAULT CHARSET=latin1 AUTO_INCREMENT=253 ;

CREATE TABLE `genre_music` (
  `id` int(11) NOT NULL auto_increment,
  `genre` varchar(80) NOT NULL default '',
  `datum` datetime NOT NULL default '0000-00-00 00:00:00',
  `kat_ikon` varchar(50) NOT NULL default '',
  `g_info` varchar(255) NOT NULL default '',
  `artist_id` int(10) unsigned NOT NULL default '0',
  `musicid` int(10) unsigned NOT NULL default '0',
  PRIMARY KEY  (`id`),
  KEY `artist` (`artist_id`)
) ENGINE=MyISAM DEFAULT CHARSET=latin1 AUTO_INCREMENT=24 ;

-- 
-- Dumping data for table `genre_music`
-- 

INSERT INTO `genre_music` VALUES (1, 'Blues', '0000-00-00 00:00:00', 'kat_ikon_1.png', '', 0, 0);
INSERT INTO `genre_music` VALUES (2, 'Humor', '0000-00-00 00:00:00', 'kat_ikon_2.png', '', 0, 0);
INSERT INTO `genre_music` VALUES (3, 'Country', '0000-00-00 00:00:00', 'kat_ikon_3.png', '', 0, 0);
INSERT INTO `genre_music` VALUES (4, 'Folkmusik', '0000-00-00 00:00:00', 'kat_ikon_4.png', '', 0, 0);
INSERT INTO `genre_music` VALUES (5, 'Gospel', '0000-00-00 00:00:00', 'kat_ikon_5.png', '', 0, 0);
INSERT INTO `genre_music` VALUES (6, 'Jazz', '0000-00-00 00:00:00', '', '', 0, 0);
INSERT INTO `genre_music` VALUES (7, 'Rap', '0000-00-00 00:00:00', '', '', 0, 0);
INSERT INTO `genre_music` VALUES (8, 'Reggae', '0000-00-00 00:00:00', '', '', 0, 0);
INSERT INTO `genre_music` VALUES (9, 'Rock', '0000-00-00 00:00:00', 'kat_ikon_9.png', '', 0, 0);
INSERT INTO `genre_music` VALUES (10, 'Filmmusik', '0000-00-00 00:00:00', 'kat_ikon_10.png', '', 0, 0);
INSERT INTO `genre_music` VALUES (11, 'Barn', '0000-00-00 00:00:00', 'kat_ikon_11.png', '', 0, 0);
INSERT INTO `genre_music` VALUES (12, 'Dansband', '0000-00-00 00:00:00', 'kat_ikon_12.png', '', 252, 0);
INSERT INTO `genre_music` VALUES (13, 'Pop', '0000-00-00 00:00:00', 'kat_ikon_13.png', '', 66, 0);
INSERT INTO `genre_music` VALUES (14, 'Techno', '0000-00-00 00:00:00', '', '', 0, 0);
INSERT INTO `genre_music` VALUES (15, 'Electronica', '0000-00-00 00:00:00', 'kat_ikon_15.png', '', 0, 0);
INSERT INTO `genre_music` VALUES (16, 'Soul & R&B', '0000-00-00 00:00:00', '', '', 0, 0);
INSERT INTO `genre_music` VALUES (17, 'Hip Hop', '0000-00-00 00:00:00', 'kat_ikon_17.png', '', 0, 0);
INSERT INTO `genre_music` VALUES (18, 'HÃ¥rdrock', '0000-00-00 00:00:00', '', '', 0, 0);
INSERT INTO `genre_music` VALUES (19, 'Klassiskt', '0000-00-00 00:00:00', '', '', 0, 0);
INSERT INTO `genre_music` VALUES (20, 'Trance & Dance', '0000-00-00 00:00:00', '', '', 0, 0);
INSERT INTO `genre_music` VALUES (21, 'Metal', '0000-00-00 00:00:00', '', '', 0, 0);

CREATE TABLE `labels` (
  `id` tinyint(3) NOT NULL auto_increment,
  `lnamn` varchar(100) NOT NULL default '',
  `logo` varchar(100) NOT NULL default '',
  `info` text NOT NULL,
  `webb` varchar(50) NOT NULL default '',
  `musicid` int(10) unsigned NOT NULL default '0',
  `artist_id` int(10) unsigned NOT NULL default '0',
  PRIMARY KEY  (`id`),
  KEY `artist_id` (`artist_id`)
) ENGINE=MyISAM DEFAULT CHARSET=latin1 AUTO_INCREMENT=41 ;

CREATE TABLE `lyrics` (
  `id` int(10) NOT NULL auto_increment,
  `artist_id` tinyint(10) unsigned NOT NULL default '0',
  `musicid` int(10) unsigned NOT NULL default '0',
  `titel` varchar(50) NOT NULL default '',
  `artistnamn` varchar(50) NOT NULL default '',
  `text` text NOT NULL,
  `datum` datetime NOT NULL default '0000-00-00 00:00:00',
  PRIMARY KEY  (`id`)
) ENGINE=MyISAM DEFAULT CHARSET=latin1 AUTO_INCREMENT=127 ;

CREATE TABLE `music` (
  `id` int(10) unsigned NOT NULL auto_increment,
  `artist_id` int(10) unsigned NOT NULL default '0',
  `kat_id` int(10) unsigned NOT NULL default '0',
  `media` enum('album','singel','dvd','ej angivet') NOT NULL default 'album',
  `label` int(10) unsigned NOT NULL default '0',
  `lyric` int(10) unsigned NOT NULL default '0',
  `tid` varchar(10) NOT NULL default '',
  `added` datetime NOT NULL default '0000-00-00 00:00:00',
  `titel` varchar(100) NOT NULL default '',
  `m_covers` varchar(100) NOT NULL default '',
  `m_info` text NOT NULL,
  `ean` varchar(50) NOT NULL default '',
  `buylink` varchar(255) NOT NULL default '',
  PRIMARY KEY  (`id`),
  KEY `artist` (`artist_id`,`label`,`lyric`)
) ENGINE=MyISAM DEFAULT CHARSET=latin1 AUTO_INCREMENT=90 ;

CREATE TABLE `songs` (
  `id` int(10) NOT NULL auto_increment,
  `artist_id` int(10) unsigned NOT NULL default '0',
  `musicid` int(10) unsigned NOT NULL default '0',
  `kat_id` int(10) unsigned NOT NULL default '0',
  `namn` varchar(50) NOT NULL default '',
  PRIMARY KEY  (`id`),
  UNIQUE KEY `musicid` (`musicid`),
  KEY `artistid` (`artist_id`)
) ENGINE=MyISAM DEFAULT CHARSET=latin1 AUTO_INCREMENT=1 ;

Jag använder följande kod för att hämta all information från tabellerna ovan.

<?php }elseif ($_GET['id'] != ""){

$sql = mysql_query("
SELECT  * 
FROM artister, countries, music, genre_music 
WHERE 
artister.id = ".$_GET['id']." AND 
countries.artist_id AND 
music.artist_id AND 
genre_music.artist_id AND
artister.kat_id
") or die(mysql_error());

?>

  <table width="<?=$width?>" align="center" class="text_2">
  <?php if (mysql_affected_rows() == 0){?>
    <tr>
      <td>Information saknas.</td>
    </tr>
  <?php }else{
  $RS = mysql_fetch_array($sql);?>
    <tr>
      <?php if ($RS['artist_foto'] != ""){?>
        <td valign="top"><img src=../artist_foto/<?=$RS['artist_foto'];?> border=1 width=100 alt=<?=$RS['namn'];?>></td>
      <?php }?>
      <td valign="top">
      <!--
	  [ <a href=favoriter.php?id=<?=$RS['id'];?> title=Lägg&nbsp;till&nbsp;<?=$RS['namn'];?> class="link_2"><b>Lägg till i favoriter</b></a> ]<br>
      <br>
	  -->
      <b>Artist/Grupp):</b> <?=$RS['anamn'];?><br>
      <?php if ($RS['fnamn'] != ""){?>
      <b>Fullständigt namn:</b> <?=$RS['fnamn'];?><?php }?>
	  <br>
	  
      <?php if ($RS['birthday'] != ""){?>
      <b>Född:</b> <?=$RS['birthday'];?><?php } ?>
	  <br>
	  
	  <?php if ($RS['land'] != ""){?>
      <b>Land:</b> <img src=../bilder/<?=$RS['flagpic'];?> alt=<?=$RS['name'];?>><?php }?>
	  <br>
	  
	  <?php if ($RS['bor'] != ""){?>
      <b>Bor:</b> <?=$RS['bor'];?><?php }?>
	  <br><br>
      
	  <?php if ($RS['info'] != ""){?>
      <b>Information:</b><br>
      <?=nl2br($RS['info']);?><?php }?>
	  <br><br>
      <b>Kategori:</b><br>
      <a href=kategorier.php?id=<?=$RS['id'];?>><?=$RS['genre'];?></a><br><br>
	  
	  <?php if ($RS['medlemmar'] != ""){?>
      <b>Medlemmar i bandet:</b><br>    
      <?=nl2br($RS['medlemmar']);?><br><br>
      <?php }?>
	  
	  <?php if ($RS['musicid'] != ""){?>
      <b>Diskografi:</b><br>
      <a href=musik.php?id=<?=$RS['musicid'];?>><?=$RS['titel'];?></a><?php }?>
	  <br><br>

      <?php if ($RS['lyrics'] != ""){?>
	  <b>Sångtexter/Lyrics:</b><br>
      <?=$RS['lyrics'];?><?php }?>
	  <br><br>

      <?php if ($RS['webb'] != ""){?>
      <b>Officiell webbplats:</b><br>
      <a href=http://<?=$RS['webb'];?> class=link_2 target=_blank><?=$RS['webb'];?></a><br><br>
      <?php }?>

Hade verkligen uppskattat ifall NÅGON kunde hjälpa mig med koden så att den hämtar RÄTT info till resp. ID, samt lite optimering. :r

Du har följande i tavlan lyrics:

artist_id tinyint(10) unsigned NOT NULL default '0',

tinyint kan endast lagra värden -128 - 127 men då du kör med unsigned så kan du endast lagra 0-127.

Jag förmodar att artist_id i tavlan lyrics refererar till en viss post i tavlan artiester?

Om du senare tänker köra med foreign keys så kommer du få problem pga att i tavlan lyrics är artist_id unsigned men i tavlan artister så är id inte unsigned och om jag minns rätt så kommer du då inte kunna skapa en foreign key relation mellan dessa två.

I tavlan lyrics har du också artistnamn. Varför, artist_id pekar väl mot en artist?

jme skrev:

I tavlan lyrics har du också artistnamn. Varför, artist_id pekar väl mot en artist?

Orsaken till detta är att (lyrics) är ett annat script som jag har användt mig av tidigare på ett lite annorlunda sätt. ;)

Jag kommer ändra alla "värden" på artist_id till samma för att förhindra komplikationer.
Vad mer måste jag göra?

Som det ser ut nu, så sparar jag...

artist_id (id:et på artisten i tabellen artister)
kat_id (id:et på kategorin i tabellen genre_music)
label_id (id:et på label i tabellen labels)

..i tabellen "music" när jag lägger till nya album/singlar, men jag vet inte riktigt hur jag ska hämta/visa alla information till rätt artist. :(
(Hoppas någon förstår vad jag menar.)

Eller...
Måste jag spara music id:et i tabellen "artister", samt liknande för övrig info som ska sparas?

Lite klurigt detta ju...