[PHP][SQL][MySQL]Relacja pomiedzy dwoma tabelami i inserty Opcje

[viamarimar]

Witam mam strukture tabel jak ponizej. Problem w tym ze gdy probuje dodac jakiegos uzytkownika do tej struktury za pmoca insertow ponizej mam 0 w polach `dane_users_id` i `users_dane_id` czyli tam gdzie powinny byc powiazania relacjyjne? Powie mi ktos co robie zle lub wskaze jakis przyklad na ktorym moglbym sie zawzrorowac zeby wykonac cos podobnego czyli relacja pomiedzy dwiema tabelami i wpisanie do bazy.

[PHP] pobierz, plaintext 
mysql_query("SET FOREIGN_KEY_CHECKS=0"); 
 
$zapytanie="INSERT INTO ".$prefix."users ( `users_id` , `users_login` , `users_haslo` , `users_token` , `users_aktywny`)
VALUES ( '', '".$_POST['login']."', '$hash', '$s', '0')";
mysql_query($zapytanie) or die(mysql_error());
 
$zapytanie2="INSERT INTO ".$prefix."dane ( `dane_id` , `dane_imie` , `dane_nazwisko` , `dane_email`)
VALUES ( '', '".$_POST['imie']."', '".$_POST['nazwisko']."', '".$_POST['email']."')";
mysql_query($zapytanie2) or die(mysql_error());
[PHP] pobierz, plaintext 

[SQL] pobierz, plaintext 
-- phpMyAdmin SQL Dump
-- version 4.0.9
-- <a href="http://www.phpmyadmin.net" target="_blank">http://www.phpmyadmin.net</a>
--
-- Host: localhost
-- Czas wygenerowania: 12 Lut 2015, 14:49
-- Wersja serwera: 5.5.34
-- Wersja PHP: 5.3.27
 
SET SQL_MODE = "NO_AUTO_VALUE_ON_ZERO";
SET time_zone = "+00:00";
 
 
/*!40101 SET @OLD_CHARACTER_SET_CLIENT=@@CHARACTER_SET_CLIENT */;
/*!40101 SET @OLD_CHARACTER_SET_RESULTS=@@CHARACTER_SET_RESULTS */;
/*!40101 SET @OLD_COLLATION_CONNECTION=@@COLLATION_CONNECTION */;
/*!40101 SET NAMES utf8 */;
 
--
-- Baza danych: `projekt2`
--
 
-- --------------------------------------------------------
 
--
-- Struktura tabeli dla tabeli `wpl_dane`
--
 
CREATE TABLE IF NOT EXISTS `wpl_dane` (
  `dane_id` int(11) NOT NULL AUTO_INCREMENT,
  `dane_users_id` int(11) NOT NULL,
  `dane_imie` varchar(50) NOT NULL,
  `dane_nazwisko` varchar(50) NOT NULL,
  `dane_email` varchar(50) NOT NULL,
  PRIMARY KEY (`dane_id`),
  KEY `usersdane_users_id` (`dane_users_id`)
) ENGINE=InnoDB  DEFAULT CHARSET=latin1 AUTO_INCREMENT=2 ;
 
--
-- Zrzut danych tabeli `wpl_dane`
--
 
INSERT INTO `wpl_dane` (`dane_id`, `dane_users_id`, `dane_imie`, `dane_nazwisko`, `dane_email`) VALUES
(1, 1, 'ssssk', 'ssss', 'p.sssss@gmail.com');
 
-- --------------------------------------------------------
 
--
-- Struktura tabeli dla tabeli `wpl_users`
--
 
CREATE TABLE IF NOT EXISTS `wpl_users` (
  `users_id` int(11) NOT NULL AUTO_INCREMENT,
  `users_dane_id` int(11) NOT NULL,
  `users_login` varchar(50) NOT NULL,
  `users_haslo` varchar(50) NOT NULL,
  `users_token` varchar(10) NOT NULL,
  `users_aktywny` int(11) NOT NULL,
  PRIMARY KEY (`users_id`),
  KEY `users_usersdane_id` (`users_dane_id`)
) ENGINE=InnoDB  DEFAULT CHARSET=latin1 AUTO_INCREMENT=2 ;
 
--
-- Zrzut danych tabeli `wpl_users`
--
 
INSERT INTO `wpl_users` (`users_id`, `users_dane_id`, `users_login`, `users_haslo`, `users_token`, `users_aktywny`) VALUES
(1, 1, 'root', '6758e143593bd7ffbc9c9996fed95890ab7db3debae19b1a9f', '4cEKyhKtoC', 0);
 
--
-- Ograniczenia dla zrzutów tabel
--
 
--
-- Ograniczenia dla tabeli `wpl_dane`
--
ALTER TABLE `wpl_dane`
  ADD CONSTRAINT `fk_users` FOREIGN KEY (`dane_users_id`) REFERENCES `wpl_users` (`users_id`);
 
--
-- Ograniczenia dla tabeli `wpl_users`
--
ALTER TABLE `wpl_users`
  ADD CONSTRAINT `wpl_users_ibfk_1` FOREIGN KEY (`users_dane_id`) REFERENCES `wpl_dane` (`dane_id`);
 
/*!40101 SET CHARACTER_SET_CLIENT=@OLD_CHARACTER_SET_CLIENT */;
/*!40101 SET CHARACTER_SET_RESULTS=@OLD_CHARACTER_SET_RESULTS */;
/*!40101 SET COLLATION_CONNECTION=@OLD_COLLATION_CONNECTION */;
[SQL] pobierz, plaintext 

Ten post edytował viamarimar 12.02.2015, 15:54:06

[robertpiaty]

Po pierwsze masz małą ilość kolumn w jednej i drugiej tabeli więc ja bym to wszystko w jednej tabeli.

Po drugie jak już robisz relację to po co tak?

ALTER TABLE `wpl_dane`
ADD CONSTRAINT `fk_users` FOREIGN KEY (`dane_users_id`) REFERENCES `wpl_users` (`users_id`);

--
-- Ograniczenia dla tabeli `wpl_users`
--
ALTER TABLE `wpl_users`
ADD CONSTRAINT `wpl_users_ibfk_1` FOREIGN KEY (`users_dane_id`) REFERENCES `wpl_dane` (`dane_id`);

Robisz relację z pierwszej tabeli do drugiej a potem z drugiej do pierwszej

Jak się uparłeś że to ma być w dwóch tabelach to moim zdaniem powinna tu być relacja 1:1 i wystarczy tylko jeden Foreign Key

Wracając do pytania To proponuję zrobić tak:
Dodaj na kluczach głównych UNSIGNED co sprawi że będą mogły być tam tylko liczby dodatnie

W insertach nie podawaj kolumny która jest FK (mają przecież auto_increment).

Wywal z tabeli z users kolumnę i klucz do tabeli dane

Zakładam że login i hasło jest unikalne.

[SQL] pobierz, plaintext 
$zapytanie="INSERT INTO ".$prefix."users ( `users_login` , `users_haslo` , `users_token` , `users_aktywny`)
VALUES ( '".$_POST['login']."', '$hash', '$s', '0')";
mysql_query($zapytanie) OR die(mysql_error());
 
$zapytanie2="INSERT INTO ".$prefix."dane (`dane_users_id`, `dane_id` , `dane_imie` , `dane_nazwisko` , `dane_email`)
SELECT users_id, '".$_POST['imie']."', '".$_POST['nazwisko']."', '".$_POST['email']." FROM ".$prefix."users WHERE `users_login` =  '".$_POST['login']."' AND `users_haslo` = '$hash' LIMIT 1";
mysql_query($zapytanie2) OR die(mysql_error());
[SQL] pobierz, plaintext 

I taka rada na koniec - nie dodawaj żmienne do SQL bezpośrednio z formularza bo ktoś Ci bazę danych zniszczy. Poczytaj o PDO w manualu PHP