[php mysql] wyswietlanie tabli w przegladarce Opcje

[jjerry]

witam, jako ze jestem bardzo poczatkujacy moje pytanie pewnie wyda sie wam glupi i mimo iz ja mecze sie juz 2 dni nie moge sobie poradzic, w zwiazku z tym prosze o pomoc:

oto kod dodawania ksiazki do bazy magazyn (oczywiscie tabela i baza istnieje)
a przedstawione skrypty sa w 1 pliku

[HTML] pobierz, plaintext 
<form action="dodaj.php" method="post">
  <table>
<tr><td>tytul</td><td>	<input type="text" name="tytul" />			</td></tr>
<tr><td>autor</td><td>	<input type="text" name="autor" />			</td></tr>
<tr><td>ilosc</td><td>	<input type="text" name="ilosc" />			</td></tr>
<tr><td></td><td>	 <input type="submit" value="dodaj" />		</td></tr>
  </table>
</form>
<hr />
[HTML] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
	$laczenie = mysql_connect('127.0.0.1','root','krasnal');
	$wybierz = mysql_query("use magazyn");
	$tworz = "create table ksiazki (id int unsigned not null auto_increment, tytul varchar(24) not null, autor varchar(24) not null, ilosc int not null, primary key (id))";
	$dodaj = ("insert into ksiazki values('0', '$_POST[tytul]','$_POST[autor]','$_POST[ilosc]')");	
 
 
	if ($laczenie && $wybierz) {
		echo 'polaczyles sie z baza <br />';
		if (mysql_query($tworz)) {
			echo 'tabela stworzona';
		} else {
			echo 'tabela nie stworzona '.mysql_error();
		if ($_POST[tytul] <> NULL && $_POST[autor] <> NULL && $_POST[ilosc] > 0) {
			mysql_query($dodaj); # PROBELM WYSTEPUJE TUTAJ
				echo '<br />dodano rekord';
			} 
		}
	} else {
		echo 'brak polaczenia <br />';
	}
 
 
 
?>
[PHP] pobierz, plaintext 

chodzi mi o to zeby w 15 linijce przed zapytaniem mysql wprowadzic instrukcje if zeby sprawdzala czy dana ksiazka jest juz w magazynie a jesli tak zeby jej nie dodawala, i aby po kazdorazowym dodaniu ksiazki wyswietlony zostal stan magazynu, probowalem select * from ksiazki ale jakos nie moge sobie poradzic, przypominam ze jestem poczatkujacy wiec bede wdzieczny za objasnienia,
z gory dzieki

Ten post edytował jjerry 7.08.2006, 12:30:54

[Kicok]

[PHP] pobierz, plaintext 
<?php
$tworz = "create table ksiazki (id int unsigned not null auto_increment, tytul varchar(24) not null, autor varchar(24) not null, ilosc int not null, primary key (id))";
 
// (...)
 
if (mysql_query($tworz)) {
echo 'tabela stworzona';
} else {
echo 'tabela nie stworzona '.mysql_error();
?>
[PHP] pobierz, plaintext 

? Ty chcesz tworzyć tą tabelę przy każdym wywołaniu strony?

Po drugie nie wstawiaj wartości do kolumny z auto_increment. Poczytaj sobie jak inaczej można zapisać zapytanie INSERT: http://home.pl/pomoc/1699

Po trzecie najpierw sprawdź SELECT'em czy jest już książka o danym tytule i autorze w bazie - jeśli nie (mysql_num_rows" title="Zobacz w manualu php" target="_manual) to dopiero potem wstawiaj nowy wiersz do tabeli.

[-Gość-]

? Ty chcesz tworzyć tą tabelę przy każdym wywołaniu strony? - nie, jesli tabela istnieje zwracany jest blad i wykonywana instrukcja else (u mnie w kazdym razie to dziala)

Po trzecie najpierw sprawdź SELECT'em czy jest już książka o danym tytule i autorze w bazie - no wlasnie tego jak do tej pory nie potrafie zrobic, jak to wykonac? gdy select ma zwracac prawde lub fausz lub wyswietlic czy jest dana ksiazka to otzrymuje jedynie "resource id#3"

[Kicok]

mysql_fetch_row" title="Zobacz w manualu php" target="_manual
mysql_fetch_array" title="Zobacz w manualu php" target="_manual
mysql_fetch_assoc" title="Zobacz w manualu php" target="_manual

do wyboru do koloru


A co do tego błędu w przypadku nieznalezienia tabeli w bazie, to upewnij się, że jest ci to do szczęścia potrzebne...

[-Gość-]

no moze faktycznie nie jest to profesjonalne
dzieki za nfo

[jjerry]

zastosowalem sie do rad ale teraz mam :

Warning: mysql_fetch_row(): supplied argument is not a valid MySQL result resource in c:\usr\krasnal\www\P_H_P\sklep\dodaj.php on line 27
wierszy

dodam ze linia 27 to "$num_rows = mysql_fetch_row($result);"

[kossa]

Ten błąd oznacza że sknociłeś coś w zapytaniu SQL, np. odwołujesz się do nieistniejącego pola w tabeli (sprawdź literówki)

Łukasz

[=kokos=]

1. Hmm, niepotrzebnie za każdym razem gdy uruchamiasz skrypt tworzysz tabelę. Tworzenie baz, oraz strukturę tabel najlepiej wrzucić do jednego pliku np. o nazwie install.php i uruchamiasz go tylko raz.

2. Poleceniem SELECT sprawdź jakie ksiązki w bazie już istnieją.

3. Proponuję dodać jeszcze jedno pole do tabeli ksiązki a mianowicie kod ISBN. Jest to niepowtarzalny dziesięciocyfrowy identyfikator książki.

4. Sprawdzać przy pomocy ISBN'a czy taki wpis już istnieje czy nie. Porównaj to co sprawdziłeś poleceniem SELECT z $_POST['isbn']

5. Jeśli po dodaniu chcesz wyswietlić stan magazynu użyj funkcji header

P.S. Jeśli poda jakaś wartość do pola w tabeli które jest auto_increment to i tak wartość ta zostanie zmieniona na taką jaka powinna być, z tym że nie jestem pewien czy powinna ona być w apostrofach. Ale jednak lepiej omijać to pole

[MagnuM]

A Wam wszystkim ciężko opisać problem, tylko z błota w bagno kolege wprowadzacie?

Więc może ja spróbuje pomóc. Opiszmy więc problem.

[PHP] pobierz, plaintext 
<?php
	$laczenie = mysql_connect('127.0.0.1','root','krasnal');
	$wybierz = mysql_query("use magazyn");
	$tworz = "create table ksiazki (id int unsigned not null auto_increment, tytul 
varchar(24) not null, autor varchar(24) not null, ilosc int not null, primary key (id))";
	$dodaj = ("insert into ksiazki values('0', '$_POST[tytul]','$_POST[autor]','$_POST[ilosc]')");	
 
	if ($laczenie && $wybierz) {
		echo 'polaczyles sie z baza <br />';
		if (mysql_query($tworz)) {
			echo 'tabela stworzona';
		} else {
			echo 'tabela nie stworzona '.mysql_error();
		if ($_POST[tytul] <> NULL && $_POST[autor] <> NULL && $_POST[ilosc] > 0) {
			mysql_query($dodaj); # PROBELM WYSTEPUJE TUTAJ
				echo '<br />dodano rekord';
			} 
		}
	} else {
		echo 'brak polaczenia <br />';
	}
?>
[PHP] pobierz, plaintext 

Do linijki 2 jest dobrze, a w niej:

[PHP] pobierz, plaintext 
<?php
mysql_query("use magazyn");
?>
[PHP] pobierz, plaintext 

Jeżeli nie korzystasz z mysql w formie okienkowej, tylko łączysz się za pomocą standardowych funkcji w php, to polecenie wygląda nieco inaczej:

[PHP] pobierz, plaintext 
<?php
$wybierz = mysql_select_db('magazyn');
?>
[PHP] pobierz, plaintext 

Linia nr. 3. Niby wszystko ok, ale zapewne jesteś nastawiony na przyjmowanie tego typu zgłoszeń, czyli książek, więc śmiało możesz ręcznie (poprzez np. phpMyAdmin'a) wstawić tabelę do bazy danych aby nie przejmować się tym później w skrypcie. Możesz śmiało wyrzucić tę linijkę.

Linia nr. 5. Zapytanie możesz śmiało zadeklarować w momencie w którym będzie ono potrzebne więc linijka jest do przeniesienia.

Przechodzimy do if'a. Dobrze jest sprawdzana pierwsze zmienna, czyli '$laczenie', druga zaś, tak jak wcześniej wspomniałem, jest zbędna.

Przejdźmy do pierwszego zagnieżdzonego if'a:

[PHP] pobierz, plaintext 
<?php
if (mysql_query($tworz))
?>
[PHP] pobierz, plaintext 

- to zapytanie spróbuje utworzyć tabelę w bazie danych, co wywoła niepotrzebnie zapytanie, to już omawialiśmy.

Dalej:

[PHP] pobierz, plaintext 
<?php
if ($_POST[tytul] <> NULL && $_POST[autor] <> NULL && $_POST[ilosc] > 0)
?>
[PHP] pobierz, plaintext 

Jeżeli chodzi o tablice asocjacyjne to należy nazwy pól ujmować w apostrofy lub cudzysowy, czyli ilustrując przykładem:

[PHP] pobierz, plaintext 
<?php
$_POST['tytul'] // a nie $_POST[tytul]
?>
[PHP] pobierz, plaintext 

.
I tutaj w tym miejscu należy wykonać zapytanie. Nie jest koneiczne najpierw umieszczanie treści zapytania w zmiennej a później deklarowanie w funkcji mysql_query(), bo to niepotrzebne zajmowanie pamięci.
Czyli teraz deklarujemy zapytanie:

[PHP] pobierz, plaintext 
<?php
$zapytanie = mysql_query("INSERT INTO ksiazki VALUES('". $_POST['tytul'] ."', '". $_POST['autor'] ."', '". $_POST['ilosc'] ."') ");
?>
[PHP] pobierz, plaintext 

W tym miejscu zanim wypiszemy treść: 'dodano rekord', przydałoby się sprawdzić czy zapytanie zostało poprawnie wykonane. Pomoże nam w tym zmienna $zapytanie, do której 'wrzucony' został efekt wykonywania zapytania.

[PHP] pobierz, plaintext 
<?php
if ( $zapytanie )
{
	echo 'dodano rekord <br />';
}
?>
[PHP] pobierz, plaintext 

Ok więc omówinie skryptu chyba zakończone. Przejdźmy do dalszych pytań.

chodzi mi o to zeby w 15 linijce przed zapytaniem mysql wprowadzic instrukcje if zeby sprawdzala czy dana ksiazka jest juz w magazynie a jesli tak zeby jej nie dodawala

Powinniśmy więc zaraz za sprawdzenie połączenia wywołać zapytanie które sprawdzi czy istnieje dana książka w bazie danych. Nie mamy tutaj neistety zabardzo jak zidentyfikować książki, np. po identyfikatorze ISBN, więc posłużymy się tytułem.

[PHP] pobierz, plaintext 
<?php
$zapytanie = mysql_query("SELECT tytul FROM ksiazki WHERE tytul = '". $_POST['tytul'] ."' ");
$rows = mysql_num_rows( $zapytanie ); // zmienna $tablica zawiera dane dotyczace ilosci rekordow z tytulem podanym w for
mularzu
// pozostaje sprawdzic czy liczba jest wieksza od 0
if ( $rows > 0 )
{
	echo 'książka o podanym tytule znajduje się już w bazie danych';
} else
{
	// dalsza część skryptu związana z dodaniem książki do bazy
}
?>
[PHP] pobierz, plaintext 

aby po kazdorazowym dodaniu ksiazki wyswietlony zostal stan magazynu

Więc zaraz za funkcją dodającą książkę należy wyświetlić stan:

[PHP] pobierz, plaintext 
<?php
$zapytanie = mysql_query("SELECT * FROM ksiazki");  // brak klauzuli WHERE oznacza że pobrane zostan szystkie rekordy z tabeli
while( $tablica = mysql_fetch_assoc( $zapytanie ) )
{
	echo 'ID: '. $tablica['id'] .' tytuł '. $tablica['id'] .'<br />';
}
?>
[PHP] pobierz, plaintext 

Ok. Troszkę namotałem więc przedstawię jak to powinno wyglądać w caej okazałości.

[PHP] pobierz, plaintext 
<?php
$laczenie = mysql_connect('127.0.0.1','root','krasnal');
$wybierz = mysql_select_db('magazyn');
	if ( $laczenie && $wybierz )
	{
		echo 'polaczyles sie z baza <br />';
 
		$zapytanie = mysql_query("SELECT tytul FROM ksiazki WHERE tytul = '". 
 
$_POST['tytul'] ."' ");
		$rows = mysql_num_rows( $zapytanie ); // zmienna $tablica zawiera dane dotyczace 
 
ilosci rekordow z tytulem podanym w formularzu
		 // pozostaje sprawdzic czy liczba jest wieksza od 0
		if ( $rows > 0 )
		{
			echo 'książka o podanym tytule znajduje się już w bazie danych';
		} else
		{
 
			if ( $_POST['tytul'] <> NULL && $_POST['autor'] <> NULL )
			{
				$zapytanie = mysql_query("INSERT INTO ksiazki VALUES('". $_POST['tytul'] 
 
."', '". $_POST['autor'] ."', '". $_POST['ilosc'] ."') ");
				if ( $zapytanie )
				{
					echo 'dodano rekord <br />';
 
					$zapytanie = mysql_query("SELECT * FROM ksiazki");  // brak klauzuli 
 
WHERE oznacza &#380;e pobrane zostan szystkie rekordy z tabeli
					while( $tablica = mysql_fetch_assoc( $zapytanie ) )
					{
						 echo 'ID: '. $tablica['id'] .' tytuł '. $tablica['id'] .'<br />';
					}
				}
			}
		}
	} else
	{
		echo 'brak polaczenia <br />';
}
?>
[PHP] pobierz, plaintext 

Uff... chyba wszystko jest ok

EDIT: oops, zapomnialem o wybraniu bazy, omg - juz dodałem do tego ostatniego kodu

Ten post edytował MagnuM 7.08.2006, 14:00:54

[jjerry]

magnum wielkie dzieki juz dziala, (musialem troszku zmienic) wszystko pieknie dziala,
wielkie dzieki za odpowiedzi

po przerobkach wyglada MNIEJ WIECEJ* to tak:

[HTML] pobierz, plaintext 
<form action="dodaj.php" method="post">		<!-- rozpoczecie formularza-->
 <table>
  <tr><td>tytul	</td><td>	<input type="text" name="tytul" />			</td></tr>
  <tr><td>autor	</td><td>	<input type="text" name="autor" />			</td></tr>
  <tr><td>ilosc	</td><td>	<input type="text" name="ilosc" />			</td></tr>
  <tr><td>isbn	</td><td>	<input type="text" name="isbn"  />			</td></tr>
  <tr><td>		</td><td>	<input type="submit" value="dodaj" />		</td></tr>
 </table>
</form>										<!-- koniec formularza-->
<hr />
[HTML] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
$laczenie = (@mysql_connect('127.0.0.1','root','krasnal'));		#zmienna laczenie posiada wartosc polaczenia z baza danych
$wybor_bazy = mysql_select_db('magazyn');		#zmienna wybor_bazy posiada wartosc wyboru bazy danych
 
$czy_ksiazka_istnieje = mysql_query		#zmienna ... sprawdza czy dany isbn powtarza sie w tabeli ->
("select isbn from ksiazki where isbn ='".$_POST['isbn']."'");	#<-
 
$dodaj = ("insert into ksiazki values	
('0', '$_POST[tytul]', '$_POST[autor]', '$_POST[ilosc]', '$_POST[isbn]')"); #zmienna dodaje dane do formularza
$sprawdz = mysql_num_rows($czy_ksiazka_istnieje);	#sprawdza czy zmienna czy ksiazka istnieje jest prawdiwa 
$wyswietl = mysql_query("select * from ksiazki");	#zaznacza * z tabeli ksiazka
 
if ($laczenie && $wybor_bazy) {		#jesli polaczyles sie z mysql i wybrales baza magazyn to:
	echo "polaczyles sie z baza mysql, wybrano tabele magazyn<br />";
		if ($sprawdz > 0) {		#..to sprawdz czy ksiazka (isbn) wystepuje w tabeli ksiazki
			echo "taka ksiazka juz wystepuje w bazie<hr />";
			while ($ck = mysql_fetch_assoc($wyswietl)) {	#jesli tak to wyswietl zawartosc magazynu 
echo $ck['id'].' '.$ck['autor'].' '.$ck['tytul'].' '.$ck['ilosc'].' '.$ck['isbn'].'<br />';
			}
	} else {	#jesli nie
		if ($_POST['tytul'] != NULL && $_POST['autor'] != NULL && $_POST['ilosc'] != NULL && $_POST['isbn'] != NULL) {	#wyswietl zawartosc magazynu sprawdzajac czy uzytkownik podal wszytskie dane
			mysql_query($dodaj);	#oraz dodaj ksiazke do tabeli
			echo "dodano nastepujaca pozycja: ".$_POST[isbn]."<hr />";	
			while ($ck = mysql_fetch_assoc($wyswietl)) {
echo $ck['id'].' '.$ck['autor'].' '.$ck['tytul'].' '.$ck['ilosc'].' '.$ck['isbn'].'<br />';
			}	# i na koniec znow wyswietl zawartosc magazynu
		} else {
			echo "nie podales pelnych informacji o produkcie";
		}
	}
} else {
	echo "nie udalo sie polaczyc z baza mysql lub wybrac tabeli magazyn<br />";
}
?>
[PHP] pobierz, plaintext 

i co najwazniejsze dziala
*mniej wiecej bo do forum wstawiaja sie znaczniki \ przed " itp

Ten post edytował jjerry 7.08.2006, 18:11:44

[kossa]

Dobrze że wstawia ukośniki :-) (to dla bezpieczeństwa)

Wyświetlając dane na stronie usuwaj je za pomocą funkcji: stripslashes

aby je dodawać stosuje się funkcję: addslashes

Łukasz

[jjerry]

Wyświetlając dane na stronie usuwaj je za pomocą funkcji: stripslashes

- usuwac z pamieci czy skad? mam rozumiec ze to takze dla bezpieczenstwa tak?

aby je dodawać stosuje się funkcję: addslashes

- dodawac, hmmm ale gdzie?

[kossa]

usuwasz ukośniki na stronie (stripslashes) aby ładnie wyglądało :-) nie są potrzebne przecież, tylko dodając do bazy robisz to dla bezpieczeństwa ale chyba na stronie nie chcesz mieć tytułu książki \"Sierotka Marysia\" tylko "Sierotka Marysia" bez ukośnikow :-)

Łukasz

[-Gość-]

aha
dzieki za nfo bede pamietal