[PHP/SQL] mysql_numrows -> fout afhandelen

[Tijgertje84]

Ik heb op dit moment deze code:

PHP:

function zoekAlleUitkeringen()
{
    $zoekquery = mysql_query("  SELECT 
                                    DISTINCT(u.gsm_nr),a.debiteur_nr,a.debiteur_naam,a.abonvorm_naam
                                FROM 
                                    uitkering u, abonnementen a
                                WHERE 
                                    u.abonnement_id = ''
                                AND u.gsm_nr = a.gsm_nr"); 

    if(mysql_numrows($zoekquery))
    {
        $aReturn = array();
        while($aResult = mysql_fetch_assoc($zoekquery))
            array_push($aReturn, $aResult);
    
        return $aReturn;
    }
    else 
        return false;
}

maar het kan zo zijn dat de query geen resultaten krijgt en dat mag ook want deze functie is voor het genereren van een overzicht en als er geen resultaten zijn dan hoeft er geen overzicht te komen
dit werkt allemaal maar als je dan geen resultaten hebt dan krijg je deze melding:
PHP Warning: mysql_numrows(): supplied argument is not a valid bla bla bla....
Jah snap ik maar ik wil die error niet krijgen
hij komt in die if zodra er minimaal 1 result is en lijkt me dat ik het zo toch wel redelijk had opgelost

als ik er dit van maak:

PHP:

if(!empty($zoekquery))

werkt het wel en zonder foutmelding in de log ^^

maar nu wil ik dus weten of het dan uitmaakt wat ik gebruik?
en of mijn eerste manier fout is?

edit: het schiet me net te binnen ->

PHP:

if(@mysql_numrows($zoekquery))

zo werkt het dus ook
Maar is dit geen manier wat technisch gezien minder professioneel is??
want heb ooit keer opgevangen dat dit wel de makkelijke weg is om fouten te omzeilen of is dit gewoon goed?

hoofdreden is dat er geen of zeer weinig wordt weggeschreven in de log files

[ Voor 11% gewijzigd door Tijgertje84 op 03-08-2006 09:40 ]

[Tsjilp]

tis ook geen error he, maar een warning.
Wat betekent dat je code wel werkt, maar niet helemaal correct is.

[Tijgertje84]

Tsjilp schreef op donderdag 03 augustus 2006 @ 09:39:
tis ook geen error he, maar een warning.
Wat betekent dat je code wel werkt, maar niet helemaal correct is.

wil dus ook geen warnings hebben en deze gewoon netjes wil proberen op te lossen door het script goed te maken

dus de vraag is of de (@) de beste oplossing hier is?

[NMe]

Als een mysql_* functie een error opgooit omdat je geen valid resource hebt, dan is de resultset niet leeg maar klopt je query niet. Een lege resultset is ook gewoon valid. Debug je query maar eens.

@ is nooit een oplossing, eerder symptoombestrijding.

[MrNGm]

Het is toch mysql_num_rows?

edit:
Uit de PHP manual: Note: For downward compatibility, the following deprecated alias may be used: mysql_numrows()

Bovendien is die if-constructie niet helemaal lekker. Mysql_num_rows geeft niet standaard TRUE terug, waarom test je dan op TRUE en FALSE. Doe iets van

PHP:

if (mysql_num_rows($zoekquery) > 0) {
  // al die resultaten ophalen
} else {
  // geen resultaten
}

Ik moet eigenlijk nog uitzoeken of mysql_num_rows 0 kan returnen. Als dat zo is, zou ik in de else nog een if zetten die test op of mysql_num_rows 0 is of FALSE, in het geval van FALSE, return mysql_error, en anders zeggen dat er geen resultaten zijn gevonden.

[sQuarecoW]

mysql_num_rows kan idd 0 returnen, als er 0 rijen gevonden zijn.

Maar daarvoor moet je query wel correct zijn, en gezien de waarschuwing is ie dat niet. En dat kan dan dus liggen aan het niet bestaan van tabellen/velden die je opvraagt.

[MrNGm]

Dan is het een kwestie van mysql_error() toevoegen in de else clausule en klaar is kees. Zoals -NMe- al zei, even debuggen.

[Verwijderd]

en dit dan?

code:

<?php

function zoekAlleUitkeringen()
{
    $sql="SELECT  DISTINCT(u.gsm_nr),a.debiteur_nr,a.debiteur_naam,a.abonvorm_naam
                                FROM 
                                    uitkering u, abonnementen a
                                WHERE 
                                    u.abonnement_id = ''
                                AND u.gsm_nr = a.gsm_nr";
                                
                                
    if($zoekquery = mysql_query($sql))
    {
        if(mysql_num_rows($zoekquery)>0)
        {
            $aReturn = array();
            while($aResult = mysql_fetch_assoc($zoekquery))
                array_push($aReturn, $aResult);
        
            return $aReturn;
        }
    }
    
    return false;
    
}
?>