[PHP][MYSQL]Zapis do bazy danych z listy wyboru - dalszy ciąg, temat aktualny Opcje

[Czapla]

Cześć

Założmy że mamy dwie tabele

przy_film

[SQL] pobierz, plaintext 
CREATE TABLE IF NOT EXISTS `przy_film` (
  `id_film` int(11) NOT NULL AUTO_INCREMENT,
  `tytul` varchar(80) character SET utf8 collate utf8_polish_ci NOT NULL,
  PRIMARY KEY  (`id_film`)
) ENGINE=MyISAM  DEFAULT CHARSET=utf8 AUTO_INCREMENT=203 ;
[SQL] pobierz, plaintext 

prz_gatunek

[SQL] pobierz, plaintext 
CREATE TABLE IF NOT EXISTS `przy_gatunek` (
  `id_gatunek` int(11) NOT NULL AUTO_INCREMENT,
  `gatunek` varchar(30) character SET utf8 collate utf8_polish_ci NOT NULL,
  PRIMARY KEY  (`id_gatunek`)
) ENGINE=MyISAM  DEFAULT CHARSET=utf8 AUTO_INCREMENT=5 ;
[SQL] pobierz, plaintext 

przy_film_has_przy_gatunek

[SQL] pobierz, plaintext 
CREATE TABLE IF NOT EXISTS `przy_film_has_przy_gatunek` (
  `id_film` int(11) NOT NULL,
  `id_gatunek` int(11) NOT NULL,
  PRIMARY KEY  (`id_film`,`id_gatunek`)
) ENGINE=MyISAM DEFAULT CHARSET=utf8
[SQL] pobierz, plaintext 

Każdemu filmowi przyporządkowany jest dokladnie jeden gatunek - jeden gatunek może być przyporządkowany wielu filmom - RELACJA 1:n

dla tabeli przy_film_has_przy_gatunek rekordy wyglądaj następująco

id_film/id_gatunku
1, 4
2, 4
3, 1
4, 2
5, 3

O ile w wypadku wyświetlania w formularzu rozwijanej listy z gatunkami wszystko jest OK to nie potrafie uporać się z zapisem do bazy.

Mała Legenda:

$aDBC - ustanawia połączenie do bazy
$SQL - zmienna przechowuje zapytanie SQL
FieldByName(); - pobiera rekordy z bazy
ExecSQL($SQL); - wykonuje zapytanie do bazy SQL
Next(); - zwraca kolejne rekordy

zapytanie SQL zostalo przerobione z zupełnie innego przykladu, a zmienne dostosowane do moich potrzeb...

Formularz wynik pliku php

[HTML] pobierz, plaintext 
<tr>
<td>Gatunek</td>
<td>
<select name="gatunek">
 <option value="4">aaa</option>
 <option value="2">fabularny</option>
 <option value="3">horror</option>
 <option value="1">komedia</option>
</select>   
</td>
</tr>
[HTML] pobierz, plaintext 

plik php tworzący formularz

[php]Gatunek
    

Ten post edytował Czapla 27.06.2008, 09:17:07

Powód edycji: dodałem tag (cysiaczek)

[MMX3]

Gatunek trzymany jest tu:

[PHP] pobierz, plaintext 
<?php
$_POST['gatunek']
?>
[PHP] pobierz, plaintext 

Chyba lepiej zamiast nazwy gatunku w value trzymać jego id

[PHP] pobierz, plaintext 
<?php
$lista .= '<option value="'.$aDBC->FieldByName ("gatunek") .'">'.$aDBC->FieldByName ("gatunek") .'</option>';
?>
[PHP] pobierz, plaintext 

na:

[PHP] pobierz, plaintext 
<?php
$lista .= '<option value="'.$aDBC->FieldByName ("id_gatunku") .'">'.$aDBC->FieldByName ("gatunek") .'</option>';
?>
[PHP] pobierz, plaintext 

Wtedy user wie co wybiera. A baza ma odrazu id.

[Czapla]

EDIT: PATRZ POST 1

Ten post edytował Czapla 16.06.2008, 10:51:48

[JoShiMa]

wyświetl sobie ten formularz w przeglądarce zobacz źródło strony, skopiuj ten kawałek, gdzie jest select/option i pokaż nam tu.

[Czapla]

EDIT: PATRZ POST 1

Ten post edytował Czapla 16.06.2008, 10:51:26

[JoShiMa]

Nie rozumiem tego:

[PHP] pobierz, plaintext 
<?php
$_REQUEST[$lista] == "gatunek";
?>
[PHP] pobierz, plaintext 

[Czapla]

bump, problem wciąż aktualny


V

ok poprawiłem ale wciąż nie działa

Ten post edytował Czapla 15.06.2008, 16:50:59

[matx132]

jedyne co tu widze i nie rozumiem to to:)

$POST['gatunek'];

czy nie powinno być
$_POST['gatunek'];

[Czapla]

W dalszym ciągu czekam na pomoc, napisałem trochę jaśniej i wytłuściłem caly problem, a kod który jest nie aktualny wyrzuciłem, aktualny kod w pierwszym poście.

UWAGA! SZCZEGÓŁY W PIERWSZYM POŚCIE

Ten post edytował Czapla 16.06.2008, 10:55:31

[erix]

W rezultacie skrypt zapisze tylko pierwsze pole do bazy danych w liście rozwijanej, a ma zapisać wybrane.

Ale pole które? W bazie (a.k.a. kolumna), inny select, czy może masz na myśli wartość selecta, a może po prostu jednego select, bo po kodzie można wywnioskować, że pól wyboru jest trochę więcej.

Napisz jaśniej, co chcesz mieć wstawione do bazy, a co zapisuje obecnie (konkretniej!).

[Czapla]

Przerobiłem troche ten skrypt i pozbyłem się tabeli przy_film_has_przy_gatunek wrzuciłem do tabeli przy_film klucz obcy z tabeli przy_gatunek. Być może jeszcze zapytanie jest źle skonstruowane ponieważ skrypt teraz wogujle nie działa wciąż pracuje nad tym .

ZAPYTANIA DO TABEL

przy_film

[SQL] pobierz, plaintext 
CREATE TABLE IF NOT EXISTS `przy_film` (
  `id_film` int(11) NOT NULL AUTO_INCREMENT,
  `id_gatunek` int(11) NOT NULL,
  `tytul` varchar(30) collate utf8_polish_ci NOT NULL,
  PRIMARY KEY  (`id_film`),
  KEY `id_gatunek` (`id_gatunek`)
) ENGINE=MyISAM  DEFAULT CHARSET=utf8 COLLATE=utf8_polish_ci COMMENT='przechowuje dane filmów' AUTO_INCREMENT=1 ;
[SQL] pobierz, plaintext 

przy_gatunek

[SQL] pobierz, plaintext 
CREATE TABLE IF NOT EXISTS `przy_gatunek` (
`id_gatunek` int(11) NOT NULL AUTO_INCREMENT,
`gatunek` varchar(30) character SET utf8 collate utf8_polish_ci NOT NULL,
PRIMARY KEY (`id_gatunek`)
) ENGINE=MyISAM DEFAULT CHARSET=utf8 AUTO_INCREMENT=1 ;
[SQL] pobierz, plaintext 

NOWE ZAPYTANIE DO LĄCZENIA TABEL WYGLĄDA TAK (proszę o sprawdzenie czy jest poprawne):

[SQL] pobierz, plaintext 
SELECT  * FROM przy_film LEFT JOIN przy_gatunek ON przy_gatunek.id=przy_film.id_gatunek AND przy_film.id_gatunek = ".$ID_GATUNEK ."
[SQL] pobierz, plaintext 

Skrypt zapisujący do bazy danych

[PHP] pobierz, plaintext 
<?php
//(..)
$SQL = "delete from przy_film where id_gatunek=". $ID_GATUNEK;
$aDBC->ExecSQL ($SQL);
$SQL = "SELECT  * FROM przy_film LEFT JOIN przy_gatunek ON przy_gatunek.id=przy_film.id_gatunek 
AND przy_film.id_gatunek = ".$ID_GATUNEK ." ";
$SQL .= " order by gatunek";
$aDBC ->ExecSQL ($SQL);
 
		do{
						$SQL = "insert into przy_film (id_gatunek) values (";
						//$SQL .= $aDBC->FieldByName ("id");
						$SQL .=  $ID_GATUNEK;
						$SQL .= ")";
						$aDBC->ExecSQL ($SQL);
				}while ($aDBC->Next());
?>
[PHP] pobierz, plaintext 

Skrypt tworzący formularz

[PHP] pobierz, plaintext 
<?php
//(..)
		$SQL = "SELECT id_gatunek FROM przy_film LEFT JOIN przy_gatunek ON przy_gatunek.id=
przy_film.id_gatunek AND przy_film.id_gatunek = ".				$ID_GATUNEK ." ";
		//$SQL = "SELECT * FROM przy_gatunek";
		$SQL .= " order by gatunek";
		$aDBC ->ExecSQL ($SQL);
		do{
		 $lista .= '<option value="'.$aDBC->FieldByName ("id_gatunek") .'">'.$aDBC->FieldByName ("gatunek") .'</option>';
		}while ($aDBC->Next());
		echo '<select name="gatunek">'.$lista.'</select>';
?>
[PHP] pobierz, plaintext 

Ale pole które? W bazie (a.k.a. kolumna), inny select, czy może masz na myśli wartość selecta, a może po prostu jednego select, bo po kodzie można wywnioskować, że pól wyboru jest trochę więcej.

Tak pól wyboru było więcej ponieważ tamto zapytanie było złożone ale już je przerobiłem.
[quote]
Napisz jaśniej, co chcesz mieć wstawione do bazy, a co zapisuje obecnie (konkretniej!).
[,/quote]
Chce zapisywać do tabeli przy_film, id_gatunek, wartość selecta id_gatunek z tabeli przy_gatunek, następnie wyświetlać go po tym właśnie id w polu edycji w selectie jako gatunek [ rekord w tabeli przy_gatunek ] oraz w przeglądzie.


Edit a to zapytanie jest źle, po wrzuceniu do phpMyAdmin wypisuje mi że występuje duplikat id_gatunku w dwóch tabelach

Ktoś może mi udzielić odpowiedzi co mam nie tak?

EDIT: Naniesione poprawki

Ten post edytował Czapla 18.06.2008, 12:16:55

[erix]

A nie masz przypadkiem jakiegoś babola w tym miejscu?

[PHP] pobierz, plaintext 
<?php
$SQL = "insert into przy_film (id_gatunek) values (";
						$SQL .= $aDBC->FieldByName ("id");
						$SQL .= "," . $ID_GATUNEK;
?>
[PHP] pobierz, plaintext 

Chcesz wstawić dwa pola do bazy, a definiujesz tylko jedno...

[Czapla]

racja zarymowałem tą pierwszą linijkę

byłbym wdzięczny gdyby ktoś rzucil okiem na to zapytanie, kod i powiedzial mi co jest nie tak, bo męcze to już od "patrz data pierwszego postu "...

[erix]

Przyjrzyj się polom, jakie deklarujesz w INSERT i wartościom, jakie podajesz. Jaka jest składnia INSERT?

[Czapla]

teraz to ja nie czaje o.O co jest źle?

składnia insert jest taka

INSERT INTO nazwa tabeli (nazwa kolumny1) values (wartości kolumny1)

Ten post edytował Czapla 18.06.2008, 20:37:31

[erix]

No właśnie. A w tamtym Twoim jest: nazwaKolumny1 => wartość1, wartość2.

Ta kluczowa jest pomijana.

[-eKuba-]

masz 3 kolumny
|NOT NULL auto_increment | NOT NULL | NOT NULL |
a zapytanie wstawia jedną wartość ID_GATUNEK
wydaje mi się ze powinno być

[SQL] pobierz, plaintext 
INSERT INTO przy_film VALUES(NULL, '$id_gatunek', '$tytul');
[SQL] pobierz, plaintext 

a najlepiej zrób echo z tego inserta echo $SQL;
i wynik zapytania wykonaj bezpośrednio w bazie, tam wywali ci jaki to błąd.

[Czapla]

Tak?

[PHP] pobierz, plaintext 
<?php
$SQL = "delete from przy_film where id_gatunek=". $ID_GATUNEK;
$aDBC->ExecSQL ($SQL);
$SQL = "SELECT  * FROM przy_film LEFT JOIN przy_gatunek ON przy_gatunek.id=przy_film.id_gatunek 
AND przy_film.id_gatunek = ".$ID_GATUNEK ." ";
$SQL .= " order by gatunek";
$aDBC ->ExecSQL ($SQL);
 
	do{
	$SQL = "insert into przy_film (id_gatunek, tytul) values ('";
	$SQL .= $ID_GATUNEK;
	$SQL .= "','";
	$SQL .= $TYTUL;
	$SQL .= "'";
	$SQL .= ")";
	$aDBC->ExecSQL ($SQL);	
	$ID_FILM = mysql_insert_id ();	
		}while ($aDBC->Next());
?>
[PHP] pobierz, plaintext 

Wyświetlanie mi w dalszym ciągu nie działa...

Ktoś może mi napisać jak powinno wyglądać poprawnie skonstruowane zapytanie i skrypt, Jedna satysfakcjonująca odpowiedź To by mi bardzo pomogło i mogłbym skupić się na inne części projektu....

Ten post edytował Czapla 19.06.2008, 07:49:27

[erix]

Cały czas ktoś próbuje. ;]

Wyświetlanie mi w dalszym ciągu nie działa...

A masz tu wyświetlanie? Uporządkuj wszystko w całość (mam na myśli tłumaczenie) logicznie, to wtedy będzie można szybciej dojść do rozwiązania.

Poza tym, IMHO, trochę jesteś niekonsekwentny; w jednym miejscu używasz funkcji ze swojej klasy abstrakcji DB, a w drugim już funkcji dotyczących obsługi konkretnej bazy.

[PHP] pobierz, plaintext 
<?php
do{
	$SQL = "insert into przy_film (id_gatunek, tytul) values ('";
	$SQL .= $ID_GATUNEK;
	$SQL .= "','";
	$SQL .= $TYTUL;
	$SQL .= "'";
	$SQL .= ")";
	$aDBC->ExecSQL ($SQL);	
	$ID_FILM = mysql_insert_id ();	
		}while ($aDBC->Next());
?>
[PHP] pobierz, plaintext 

Nie rozumiem tu np. sensu pobierania ID ostatnio wstawionego rekordu, skoro go nigdzie nie używasz oraz w ogóle wstawiania wielokrotnie tych samych (!) danych; w końcu nie zmieniasz za każdym przebiegiem pętli zmiennych $ID_GATUNEK i $TYTUL. No chyba, że ta Twoja klasa tworzy takie zmienne, to by się zgadzało, ale tak, czy tak - trochę chaotycznie.

[Czapla]

To jeszcze raz:

mamy dwie tabele:
przy_film z koumnami id_film [PK], tytul [VARCHAR], id_gatunek [FK]
przy_gatunek z kolumnami id_gatunek [PK], gatunek

teraz chce aby id_gatunek z przy_film przechowywał wartość id_gatunek z przy_gatunek i na podstawie tego id wyświetlał gatunek z tabeli przy_gatunek w liście rozwijanej.

Na podstawie wszystkich informacji można to zrobić tak że opcje z rozwijanej listy wyświetlam selectem z tabeli przy_gatunek, każdej opcji nadaje wartość po id. W pliku set zapisuje to id do id_gatunku w tabeli przy_film.
Tylko co jeśli bede chciał ponownie zedytować rekord filmu - będe musial wykonać odwrotne zapytanie?

Kod jakim dysponuje:

Formularz

[PHP] pobierz, plaintext 
<tr><td>Gatunek</td><td>
	<?php
	$SQL  = " ]ZAPYTANIE";
	$aDBC->ExecSQL ($SQL);
	do{
	 $lista .= '<option value="'.$aDBC->FieldByName ("id_gatunek") .'">'.$aDBC->FieldByName ("gatunek") .'</option>';
	}while ($aDBC->Next());
	echo '<select name="gatunek">'.$lista.'</select>';
	?> 
		</td></tr>
[PHP] pobierz, plaintext 

Plik zapisu set_

[PHP] pobierz, plaintext 
<?php
$SQL = "delete from przy_film where id_gatunek=". $ID_GATUNEK;
$aDBC->ExecSQL ($SQL);
$SQL = "ZAPYTANIE[";
$aDBC ->ExecSQL ($SQL);
	do{
	$SQL = "insert into przy_film (id_gatunek) values ('";
	$SQL .= $ID_GATUNEK;
	$SQL .= "'";
	$SQL .= ")";
	$aDBC->ExecSQL ($SQL);	
	$ID_FILM = mysql_insert_id ();	
	}while ($aDBC->Next());
?>
[PHP] pobierz, plaintext 

Czyli właściwie cały problem sprowadza się do skonstruowania prawidlowych ZAPYTAŃ do bazy danych...
Mam nadzieje że wszystko jasno napisałem

Pozdrawiam

Ten post edytował Czapla 20.06.2008, 07:28:14