Problem z prostym kodem do sprawdzania rekordu Opcje

[MalyKazio]

Chcialem przed wykonaniem zapytania sprawdzic czy istnieje juz odpowiedni wpis w bazie. Kod nie działa a wygląda tak:

(...)

[PHP] pobierz, plaintext 
<?php
$zapytanie = "SELECT * FROM hodowle WHERE hodowla=$_POST[hodowla]";
$wykonaj = mysql_query($zapytanie);
$znaleziono = mysql_num_rows($wynonaj);
echo $znaleziono;
 
if ($znaleziono == 0) {
$query = "INSERT INTO hodowle VALUES ('','$_POST[hodowla]')";
$wynik = mysql_query($query);
 
echo "Dodano do bazy danych hodowle $_POST[hodowla]";
echo "<BR><a href="modules.php?op=modload&name=Ranking&file=hodowle">Dodaj kolejna hodowle</a>";
} else {
echo "Hodowla istnieje!";
}
 
mysql_free_result($wykonaj);
 
mysql_free_result($wynik);
mysql_close($link);
?>
[PHP] pobierz, plaintext 

Niby banalna sprawa ale nie dziala.
Nie działa już w tym: echo $znaleziono; miejscu. Nie wyświetla nic i za każdym razem dodaje do bazy (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)
Probowalem też z COUNT(*) as ilosc ale to samo.
co robię zle?

Ten post edytował MalyKazio 25.12.2005, 00:47:16

[mikess]

Sprobuj takiego zapytania

[PHP] pobierz, plaintext 
<?php
 
$zapytanie = 'SELECT * FROM hodowle WHERE hodowla="'.$_POST['hodowla'].'"';
 
?>
[PHP] pobierz, plaintext 

i sprawdz czy przypadkiem zmienna $_POST['hodowla'] jest poprawnie napisana bo czasem duza litera moze narobic problemow (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif)

Ten post edytował mikess 25.12.2005, 01:11:34

[mike]

A ja proponuję zmienić podejście.
IMO sprawdzanie czy coś jest już w bazie danych tylko po to żeby to dadać lub nie jest niepotrzebne. Bezsensowna operacja.

1. Zakładasz na pola, które mają być unikalne odpowieni klucz (UNIQUE)
2. Dodajesz rekord.
3. Jeśli takiego jeszcze nie było to wszystko spoko.
4. Jeśli juz był to baza nie dada danych a zwróci komunikat o kodzie błądu 1062.
Banalne i nie trzeba się męczyć.

BTW już komuś tak polecałem: Ponowny zapis do bazy po odświeżeniu strony. Jest tam jakiś kawałek kodu, który naświetli Ci sprawę.

[MalyKazio]

Dzięki wielkie mikess
Ale skad taka różnica w cudzysłowach? Wszędzie indziej działa...kiedy dodaje pola do bazy, kiedy wyciągam z bazy...
Czym różni się (z punktu widzenia interpretatora php) kod:

[PHP] pobierz, plaintext 
<?php
$zapytanie = 'SELECT * FROM hodowle WHERE hodowla="'.$_POST['hodowla'].'"';
?>
[PHP] pobierz, plaintext 

Od:

[PHP] pobierz, plaintext 
<?php
$zapytanie = "SELECT * FROM hodowle WHERE hodowla=$_POST['hodowla']";
?>
[PHP] pobierz, plaintext 

(IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif)

[mikess]

podejrzewam ze zmienna $_POST['hodowla'] nie jest typu int wiec musi byc w cudzyslowiach bo baza danych przyjmuje wszystkie rekordy w cudzyslowiach oprocz integer-ów (chociaz int z cudzyslowiem tez dzialaja (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif) ) czyli wszystkie zmienne w zapytaniu do bazy musisz uzywac cudzysłowiów i mozesz stosowac cos takiego

[PHP] pobierz, plaintext 
<?php
 
$zapytanie = "SELECT * FROM hodowle WHERE hodowla='".$_POST['hodowla']."'";
 
?>
[PHP] pobierz, plaintext 

lub

[PHP] pobierz, plaintext 
<?php
 
$zapytanie = 'SELECT * FROM hodowle WHERE hodowla="'.$_POST['hodowla'].'"';
 
?>
[PHP] pobierz, plaintext 

oba powiny dzialac

[MalyKazio]

Dzięki wielkie za wyjaśnienie. Faktycznie, zmienna hodowla jest typu text. czyli wszedzie, gdzie dodaje do bazy danych i pobieram z niej to powinieniem ujac zmienne w takie skomplikowane cudzysłowy?

[mikess]

własnie tak wszystko procz int musi byc w cudzysłowiach (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[qbuz]

w apostrofach nie w cudzyslowach, najlepiej tak

[PHP] pobierz, plaintext 
<?php
 
$zapytanie = 'SELECT * FROM hodowle WHERE hodowla=''.$_POST['hodowla'].''';
 
?>
[PHP] pobierz, plaintext 

to tak jak w php - sa to po prostu stringi, nie moga byc niczym nie 'polaczone', w koncu skad MySQL by "wiedzialo", czy jakies kolejne slowo to jakies polecenie/funkcja, czy tylko slowo do wstawienia do bazy?

Ten post edytował qbuz 28.12.2005, 05:48:39

[miedzna]

A może chodzi po prostu o literówkę w zmiennej w 4 linii:

[PHP] pobierz, plaintext 
<?php
$znaleziono = mysql_num_rows($wynonaj);
?>
[PHP] pobierz, plaintext 

przed tym używasz zmiennej $wykonaj a tu używasz $wynonaj

pozdrawiam

[SongoQ]

własnie tak wszystko procz int musi byc w cudzysłowiach

Ale i tak bazka sobie rzutuje na format pola (przewaznie tak jest), wiec najlepiej posylac wszystko jako string.

[mikess]

w apostrofach nie w cudzyslowach, najlepiej tak

[PHP] pobierz, plaintext 
<?php
 
$zapytanie = 'SELECT * FROM hodowle WHERE hodowla=''.$_POST['hodowla'].''';
 
?>
[PHP] pobierz, plaintext 

to nie ma az takie roznicy czy w cudzyslowiach czy w apostrofach bo mozna to zapisac na 4 sposoby

[PHP] pobierz, plaintext 
<?php
 
$zapytanie = 'SELECT * FROM hodowle WHERE hodowla=''.$_POST['hodowla'].''';
 
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
 
$zapytanie = "SELECT * FROM hodowle WHERE hodowla="".$_POST['hodowla'].""";
 
?>
[PHP] pobierz, plaintext 

i te dwa sposoby co wyrzej napisalem

w sumie jak kto woli ale chyba gdzies czytalem ze apostrofy są szybciej interpretowane przez php (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

[qbuz]

ale chyba gdzies czytalem ze apostrofy są szybciej interpretowane przez php

no to dobrze czytales i wlasnie dlatego napisalem, ze "tak najlepiej" ;]

w przypadku apostrofow ' ' php zwraca po prostu danego stringa, a w przypadku cudzyslowow " " parser php poszukuje jeszcze zmiennych itp. (parsuje ten kod), stad nieco dluzsze wykonywanie

Ten post edytował qbuz 29.12.2005, 03:54:05

[SongoQ]

$zapytanie = "SELECT * FROM hodowle WHERE hodowla=\"".$_POST['hodowla']."\"";

Ten kod jest niepoprawny, poniewaz w SQLu " sie daje w nazwach tabel, pol itd a nie do stringu. Do stringu uzywa sie '

[mikess]

$zapytanie = "SELECT * FROM hodowle WHERE hodowla=\"".$_POST['hodowla']."\"";

Ten kod jest niepoprawny, poniewaz w SQLu " sie daje w nazwach tabel, pol itd a nie do stringu. Do stringu uzywa sie '

jezeli to co napisales jest niezgodne ze standardem SQL to sie z Tobą zgodze ale jezeli nie to niestety musze Cie zmartwic ale ten kod równiez dziala (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

[SongoQ]

Lepiej pisac cos co na kazdej bazie bedzie dzialac niz pozniej meczyc sie ze cos jest nie tak.