Forum PHP.pl

mam taką bazę danych:

[SQL] pobierz, plaintext 
CREATE TABLE `autor` (
  `id_autor` int(5) UNSIGNED NOT NULL AUTO_INCREMENT,
  `imie` varchar(20) DEFAULT NULL,
  `nazwisko` varchar(45) NOT NULL,
  PRIMARY KEY  (`id_autor`)
) ENGINE=MyISAM DEFAULT CHARSET=latin2;
 
CREATE TABLE `wydawn` (
  `wydawnictwo_id` int(5) UNSIGNED NOT NULL AUTO_INCREMENT,
  `wydawnictwo` varchar(255) DEFAULT NULL,
  PRIMARY KEY  (`wydawnictwo_id`)
) ENGINE=MyISAM  DEFAULT CHARSET=latin2;
 
CREATE TABLE `ksiazki` (
  `id` int(5) UNSIGNED NOT NULL AUTO_INCREMENT,
  `id_autor` int(5) UNSIGNED NOT NULL,
  `tytul` varchar(255) NOT NULL,
  `wydawnictwo_id` int(5) UNSIGNED NOT NULL,
  PRIMARY KEY  (`id`)
) ENGINE=MyISAM  DEFAULT CHARSET=latin2;
[SQL] pobierz, plaintext 

i skrypt php, który powinien dodawać rekordy do powyższej bazy danych:

[PHP] pobierz, plaintext 
<?php
$nazwisko=$_POST[nazwisko];
$imie=$_POST[imie];
$id_autor=$_POST[id_autor];
$tytul=$_POST[tytul];
$wydawnictwo_id=$_POST[wydawnictwo_id];
$wydawnictwo=$_POST[wydawnictwo];
 
if(empty($nazwisko))
{echo 'Wybierz autora!';
 exit;}
else{echo "Autor: ".$nazwisko;}
 
if(empty($tytul))
{echo '<br />Podaj tytuł!';
 exit;}
else{echo "<br />Tytuł: ".$tytul;}
 
if(empty($wydawnictwo))
{echo '<br>Wybierz wydawnictwo!';
 exit;}
else{echo "<br />Wydawnictwo: ".$wydawnictwo;}
 
 
$host="localhost";
$db_user="";
$db_password="";
$database="baza";
mysql_connect($host,$db_user,$db_password);
mysql_select_db($database);
 
$sql='insert into autor values (NULL, NULL,"'.$nazwisko.'")';
$id_autor=mysql_query("select id_autor from autor where nazwisko='$nazwisko'");
 
$sql='insert into autor values (NULL, "'.$wydawnictwo.'")';
$wydawnictwo_id=mysql_query("select wydawnictwo_id from  wydawn where wydawnictwo='$wydawnictwo_id'");
 
$sql='insert into ksiazki values (NULL,"'.$id_autor.'", "'.$tytul.'","'.$wydawnictwo_id.'")';
$rezultat=mysql_query($sql);
if($rezultat==true)
{echo 'Dane poprawnie zapisane w tabeli. <br />';}
else{echo 'dane nie zostaly wstawione';
exit;}
?>
[PHP] pobierz, plaintext 

niestety ten skrypt nie działa i nie potrafię sobie sama z nim poradzić, nie dodaje prawidłowo wartości do id_autor i wydawnictwo_id w tabeli książki no i nie chce dodać odpowiednich rekordów do tabel autor i wydawn :/

w tabeli ksiazki w wyniku dodania rekordu uzyskuje wartości:
id_autor: 0
tytuł: cos tam
wydawnictwo_id: 0


HELP!!!!

Najlepiej byłoby skorzystać z trigera, jeśli Twoja wersja bazy danych to dopuszcza.

A nie lepiej to wszystko zrobić w jednej tabeli? Mniej zapytani do bazy a efekt i możliwości większe w konstruowaniu zapytań...

[SQL] pobierz, plaintext 
CREATE TABLE `ksiazka` (
`id` int(5) UNSIGNED NOT NULL AUTO_INCREMENT,
`imie` varchar(20) DEFAULT NULL,
`nazwisko` varchar(45) NOT NULL,
`tytul` varchar(255) NOT NULL,
`wydawnictwo` varchar(255) DEFAULT NULL,
PRIMARY KEY (`id`)
) ENGINE=MyISAM DEFAULT CHARSET=latin2;
[SQL] pobierz, plaintext 

Nie, nie lepiej. Tak zbudowana baza jest nieefektywna.

Czyli mam przez to rozumieć że lepiej robić 3 zapytania niż jedno tak ? co mniej obciąży bazę?

Jeśli baza będzie skonstruowana dobrze, to w tym wypadku trzy zapytania są lepsze niż jedno. Mogłabym się tu rozwodzić na temat optymalizacji bazy, ale lepiej będzie jak poczytasz fachową literaturę

Zbyt płytko to oceniasz.
Co wykonujesz na bazie częściej? Zapis czy odczyt?

Masz rację, szybciej zapisac rekord do jednej tabeli niż do trzech. Choć różnica jest pomijalna i nie to jest najważniejsze. Najważniejsze jest optymalne przechowywanie dancyh i szybki dostęp do nich. A przy rozbiciu na tebele jest lepiej, szybciej i optymalniej.

Podana przez Ciebie konstrukcja jest zła. Tragiczna bym rzekł.
Jak sobie wyobrażasz dodanie 100 książek tego samego autora? 100 razy to samo w tabeli? Pomyłka.

No właśnie, teraz zastanawiam się nad tym aby skrypt sprawdzał czy dana pozycja już istnieje w bazie zanim ją doda, samo dodawanie już działa dzięki użytkownikowi golaod (bardzo dziękuję).

Faktycznie po przemyśleniu tego zagadnienia optymalniej jest przechowywać jest raz jedno nazwisko niż tyle samo ile wydał książek. Burza mózgów czasem wskazana

Tu nie ma co się zastanawiać nad optymalnością. Czemu ludzie wymyślili programowanie obiektowe ? Bo wszystko da się dobrze zilustrować dzięki OOP można nawet skonstruować w banalny sposób człowieka (tzn imitacje)
Czemu zatem nie ma co się zastanawiać nad konstrukcją tabel ? Bo istnieje coś takiego jak schemat drzewa który jest po prostu wszechmocny do czegoś takiego. Zatem oczywiste jest, że baz powinna wyglądać tak: lista_autorow->lista_ksiazek(lista_autorow_id,gatunek_id)<-gatunek(lista_autorow_id)

aj, teraz sobie nie daje rady z tym aby sprawdzało przed wstawieniem rekordu do bazy czy np. dana osoba lub wydawnictwo już czasem nie istnieje kombinowałam z zapytaniem select a potem z if-ami, ale nie wychodzi mi nic

[PHP] pobierz, plaintext 
<?php
$id_autor=mysql_query("select id_autor from autor where nazwisko='$nazwisko'"); 
if(empty($id_autor))
{
$wynik = mysql_query('SELECT 0 FROM autor WHERE id_autor='.$nazwisko');
if (mysql_num_rows($wynik)>0) {
    echo 'autor istnieje';
} 
else {
    $sql="insert into autor values (NULL, NULL,'".$nazwisko.")";
    mysql_query($sql) or die(mysql_error());
    $id_autor=mysql_insert_id();
    $rezultat=mysql_query($sql);
}
}
?>
[PHP] pobierz, plaintext 

zamiast:
$wynik = mysql_query('SELECT 0 FROM autor WHERE id_autor='.$nazwisko');
daj
$wynik = mysql_num_rows(mysql_query('SELECT 0 FROM autor WHERE id_autor='.$nazwisko.'));

lipa, bo nadal nie sprawdza czy autor istnieje, dodaje go na nowo:( takim sposobem mam dwóch Kwiatkowskich

na podstawie tego co napisałem już bys sobie sam mógł to zrobić ....

[PHP] pobierz, plaintext 
<?php
$id_autor = mysql_num_rows(mysql_query('SELECT id_autor FROM autor WHERE nazwisko='.$nazwisko.));
if($id_autor<1){
//reszta
}
?>
[PHP] pobierz, plaintext 

Ten post edytował Spawnm 21.10.2008, 16:55:24

jestem za tępa na to:/ nie dodaje dwa razy tego samego autora, ale w tabeli ksiazki wywala 0 zamiast pobierać id_autor

[PHP] pobierz, plaintext 
<?php
$id_autor = mysql_num_rows(mysql_query("SELECT id_autor FROM autor WHERE nazwisko='.$nazwisko.'"));
if($id_autor<1)
{
$wynik = mysql_num_rows(mysql_query("SELECT 0 FROM autor WHERE nazwisko='".$nazwisko."'"));
if (mysql_num_rows($wynik)>=0) {
    echo 'autor istnieje';
    $id_autor=mysql_insert_id();
} else {
$sql="insert into autor values (NULL, NULL,'".$nazwisko."')";
mysql_query($sql) or die(mysql_error());
$id_autor=mysql_insert_id();
$rezultat=mysql_query($sql);
}
}
?>
[PHP] pobierz, plaintext 

aaa no i jeszcze błąd: Warning: mysql_num_rows(): supplied argument is not a valid MySQL result resource in C:\Program Files\WebServ\httpd\cos\dod.php on line 48 (linia 48 to: if (mysql_num_rows($wynik)>=0) { )

Ten post edytował unnamed 22.10.2008, 15:47:22

$id_autor = mysql_num_rows(mysql_query("SELECT id_autor FROM autor WHERE nazwisko='.$nazwisko.'"));
powinno być ... where nazwisko='".$nazwisko."' "));


dalej

$wynik = mysql_num_rows(mysql_query("SELECT 0 FROM autor WHERE nazwisko='".$nazwisko."'"));
if (mysql_num_rows($wynik)>=0) {

2 razy num_rows ?

a da się raz? wydawało mi sie ze tak jak chciałam zrobić powinno być poprawnie, ale ja się nie znam :] powoli się uczę

W ogóle nie rozumiem tej dziwnej i pogmatwanej konstrukcji a ty Spawnm zamiast ją nakierować na dobrą to dajesz słabo ukierunkowane podpowiedzi.

[PHP] pobierz, plaintext 
<?php
$id_autora = mysql_query("SELECT id_autor FROM autor WHERE nazwisko='.$nazwisko.'") or die(mysql_error()); // Stworzenie zapytania
// po co tworzyc drugie ktore selectuje 0 dla znalezionych nazwisk ?
if(mysql_num_rows($id_autora)==0) // sprawdzenie czy ilosc znalezionych rozna od zera
{
   // jesli tak
  $insert = "INSERT...";
   mysql_query($insert) or die(mysql_error());
  
}
else
{
  $pobierz = mysql_fetch_array($id_autora,MYSQL_ASSOC);
}
?>
[PHP] pobierz, plaintext 

Ten post edytował batman 22.10.2008, 18:22:31

ja się chyba potnę, drugi dzień myślę nad tym i mam dość:/ teraz ponownie poprawnie pobiera id_autor ale za to tym razem dodaje autora pomimo, ze jest on już w bazie