[PHP][MYSQL] Funkcja maxid działa niepoprawnie Opcje

[thomson89]

Witam!
Natchniony, szybko napisałem sobie funkcję, która sprawdza (bynajmniej powinna) sprawdzić jakie jest największe id. Po kilku poprawkach parse errorów zobaczyłem ze funkcja działa. Niestety: podaje id o 2 większe niz jest maksymalne i nie wiem dlaczego. Np. w tabeli mamy 7 rekordów, a wd. funkcji jest ich 9. Na szczęście po kilku próbach zobaczyłem że jest to regularne. Dodałem id-2 i funkcja działa poprawnie.

Po kilku męczących chwilach doszedłem do wniosku ze nie wiem czym jest to spowodowane. Dlatego, proszę o pomoc:

[PHP] pobierz, plaintext 
<?php
function maxid($polaczenie, $serwer, $tabela, $nazwa = "id"){
        
        if($polaczenie == 'new'){
            
            //jak mamy otowrzyc nowe polaczenie, otwieramy je
            $polaczenie = lacz($serwer);    
        
        }
        else {
            
            //jak polaczenie bylo, sprawdzamy jego aktywnosc
            $polaczenie = mysql_ping($polaczenie);
            if(!$polaczenie)
                $polaczenie = lacz($serwer);      
            
        }
        
        $numer_id = 1;
        $zapytanie = "SELECT * FROM ".$tabela." WHERE ".$nazwa." = ".$numer_id;
        $wynik = mysql_query($zapytanie);
        $ilosc_wierszy = mysql_num_rows($wynik);
        if($ilosc_wierszy < 1){
            
            $id = $numer_id;
            return $id;
        
        }
        else {
        
        
            //glowne sprawdzenie maksymalnego id
            while($ilosc_wierszy >= 1){
                
                $zapytanie = "SELECT * FROM ".$tabela." WHERE ".$nazwa." = ".$numer_id;
                $wynik = mysql_query($zapytanie);
                $ilosc_wierszy = mysql_num_rows($wynik);
                $numer_id++;
                
            }
            
            $id = $numer_id-2;
            return $id;
        }
        
        
        rozlacz($polaczenie);
}
?>
[PHP] pobierz, plaintext 

Funckja korzysta z moich funkcji lacz() i rozlacz(). Lacz() ma duzo parametrów opcjonalnych, a tutaj wykorzystuję tylko zmienną do laczenia sie z odpowiednim serwerem. rozlacz() to po prostu mysql_close(). Na sam koniec przed return dałem -2.

Pozdrawiam!

Ten post edytował thomson89 22.03.2009, 15:43:42

[blooregard]

Wszystko możesz załatwić jednym zapytaniem SQL:

[SQL] pobierz, plaintext 
SELECT MAX(id) FORM tabela WHERE warunki;
[SQL] pobierz, plaintext 

[thomson89]

No, ale to co się nauczyłeś przy pisaniu skryptów, wpoiłeś ipt. to dobrze.

Ale się zdziwiłem że to tak można załatwić: no a mój problem?

[blooregard]

No, ale to co się nauczyłeś przy pisaniu skryptów, wpoiłeś ipt. to dobrze.

Tzn. JA się nauczyłem, czy Ty? (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Ale się zdziwiłem że to tak można załatwić: no a mój problem?

No widzisz, a Twój problem? Słaba znajomość języka SQL (SQL to nie tylko SELECT, INSERT, UPDATE, DELETE, to także kupa różnych przydatnych f-cji, między innymi MAX(), do tego transakcje, triggery, procedury składowane, pozdapytania i ich wykorzystanie np. w klauzulach WHERE, złączenia itp. itd.)
Poucz się trochę SQL-a, a wiele problemów, takich jak ten powyżej, rozwiążesz już na poziomie zapytania do bazy, bez konieczności pisania zbędnych funkcji.
Na początek np. to przejrzyj: http://dev.mysql.com/doc/refman/5.1/en/

[thomson89]

No dobra. Dzięki.