Onderwerp: Problemen met PHP script - Krijf fout niet opgelost.

Ik krijg de volgende foutmelding op de site:

Wedstrijdschema Jeugd B1 seizoen 2012-2013

Warning: mysql_num_rows(): supplied argument is not a valid MySQL result resource in /home/schaesberg/domains/vvschaesberg.nl/public_html/wedstrijdschema.php on line 35

Er is nog geen wedstrijdschema beschikbaar.

Onderstaand het script: Line 35 heb ik dikgedrukt! Wie kan helpen? Dank jullie

<?php
$categorie = $_GET['p2'];
if($categorie == "junioren")
{
$categorie_q = "jeugd";
}
elseif($categorie == "pupillen")
{
$categorie_q = "jeugd";
}
else
{
$categorie_q = $categorie;
}
$team = $_GET['p3'];
$seizoen = "2012-2013";

if(substr($team,0,4) == "team")
{
$get_team = str_replace("team-", "", $team);
}
else
{
$get_team = $team;
}

$teamvolledig = "V.V. Schaesberg " . $get_team;


echo "<h1 style=\"padding:0px 0px 15px 0px;\">Wedstrijdschema " . ucfirst($categorie) . " " . ucfirst(str_replace("-", " ", $team)) . " seizoen " . $seizoen . "</h1>";

$query_wedstrijden = "SELECT *, DATE_FORMAT(datum, '%a %e %M %Y') as datum_format, DATE_FORMAT(tijd, '%H:%i') as tijd_format FROM wedstrijdschema WHERE categorie = '$categorie_q' AND (thuis = '$teamvolledig'OR uit = '$teamvolledig') AND seizoen '2012-2013' AND (doelpunten_thuis IS NULL AND doelpunten_uit IS NULL) ORDER BY datum ASC";

$result_wedstrijden = mysql_query($query_wedstrijden);
LINE 35 >>if(mysql_num_rows($result_wedstrijden) !=0)
{
echo "<table style=\"width:100%;\" cellpadding=\"5\" cellspacing=\"1\">";
while($record_wedstrijden = mysql_fetch_object($result_wedstrijden))
{
if($record_wedstrijden->status != "")
{
$status = "<div style=\"colored; font-weight:bold;\">" . $record_wedstrijden->status . "</div>";
}
else
{
$status = "-";
}


if($record_wedstrijden->thuis == $teamvolledig)
{
$thuis = "<b>" . $record_wedstrijden->thuis . "</b>";
}
else
{
$thuis = $record_wedstrijden->thuis;
}

if($record_wedstrijden->uit == $teamvolledig)
{
$uit = "<b>" . $record_wedstrijden->uit . "</b>";
}
else
{
$uit = $record_wedstrijden->uit;
}

$bg = ($a++ % 2) ? "********" : "#ebebeb";
echo "<tr style=\"background-color:" . $bg . ";\">
<td style=\"width:143px; font-size:11px;\">" . ucfirst(nlMonth($record_wedstrijden->datum_format)) . "</td>";

if($record_wedstrijden->thuis == "Geen team" or $record_wedstrijden->uit == "Geen team")
{
echo "<td colspan=\"5\" style=\"font-size:11px;\"><center><i>Geen wedstrijd</i></center></td>";
}
else
{
echo"
<td style=\"font-size:11px;\">" . $record_wedstrijden->tijd_format . "</td>
<td style=\"font-size:11px;\">" . $record_wedstrijden->type . "</td>
<td style=\"font-size:11px; text-alignight;\">" . $thuis . "</td>
<td style=\"font-size:11px; text-align:center;\">" . $status . "</td>
<td style=\"font-size:11px;\">" . $uit . "</td>
</tr>";
}
}
echo "</table>";
echo "<p>&nbsp;</p>";
}
else
{
echo "<p style=\"font-size:11px font-weight:bold; font-style:italic;\">Er is nog geen wedstrijdschema beschikbaar.</p>";
}

?>

Gebruik @mysql_num_rows($result_wedstrijden) en je bent van de warning af.

@t.bloo dat is niet de oplossing. Dat is gewoon de error message omzeilen.

Ik zie helemaal geen verbinding met de database?
$link = mysql_connect("localhost", "mysql_user", "mysql_password");
mysql_select_db("database", $link);

Daarnaast zou ik de mysql_num_rows in een variablen zetten:

$iAantalR = mysql_num_rows($result_wedstrijden);
En dan if($iAantalR > 0) { }

Je kan ook gewoon de MySQL error opvragen:

$result_wedstrijden = mysql_query($query_wedstrijden) or die (mysql_error());\

Erg basic Maar zoals BReady al aangeeft geen connect of whatsoever.

Ontbreekt hier geen spatie?
'$teamvolledig'OR

Aha het zit dus veel meer basic dan ik zo snel had bedacht, je gaat er toch niet van uit dat iemand überhaupt de connectie vergeet of de errormessages niet gecheckt heeft. Een query geeft niet altijd een resource af (bijvoorbeeld bij een insert query), dus je kunt niet altijd num_rows met het resultaat doen. Op de php website staat daar wat meer over verteld. Een oplossing *daarvoor* is om de warning te onderdrukken of uitgebreid te gaan checken met is_resource voordat je num_rows aanroept. Maar inderdaad, kijk eerst eens welke error de query teruggeeft.

Even leesbaar gemaakt :

Code:

<?php
$categorie = $_GET['p2'];
if($categorie == "junioren")
{
 $categorie_q = "jeugd"; 
}
elseif($categorie == "pupillen")
{
 $categorie_q = "jeugd"; 
}
else
{
 $categorie_q = $categorie; 
}
$team = $_GET['p3'];
$seizoen = "2012-2013";
 
if(substr($team,0,4) == "team")
{
 $get_team = str_replace("team-", "", $team);
}
else
{
 $get_team = $team;
}
 
$teamvolledig = "V.V. Schaesberg " . $get_team;
 

echo "<h1 style=\"padding:0px 0px 15px 0px;\">Wedstrijdschema " . ucfirst($categorie) . " " . ucfirst(str_replace("-", " ", $team)) . " seizoen " . $seizoen . "</h1>";
 
$query_wedstrijden = "SELECT *, DATE_FORMAT(datum, '%a %e %M %Y') as datum_format, DATE_FORMAT(tijd, '%H:%i') as tijd_format FROM wedstrijdschema WHERE categorie = '$categorie_q' AND (thuis = '$teamvolledig'OR uit = '$teamvolledig') AND seizoen '2012-2013' AND (doelpunten_thuis IS NULL AND doelpunten_uit IS NULL) ORDER BY datum ASC";
 
$result_wedstrijden = mysql_query($query_wedstrijden);
 LINE 35 >>if(mysql_num_rows($result_wedstrijden) !=0)
{
 echo "<table style=\"width:100%;\" cellpadding=\"5\" cellspacing=\"1\">";
 while($record_wedstrijden = mysql_fetch_object($result_wedstrijden))
 {
 if($record_wedstrijden->status != "")
 {
 $status = "<div style=\"colored; font-weight:bold;\">" . $record_wedstrijden->status . "</div>";
 }
 else
 {
 $status = "-";
 }
 

if($record_wedstrijden->thuis == $teamvolledig)
 {
 $thuis = "<b>" . $record_wedstrijden->thuis . "</b>"; 
}
 else
 {
 $thuis = $record_wedstrijden->thuis;
 }
 
if($record_wedstrijden->uit == $teamvolledig)
 {
 $uit = "<b>" . $record_wedstrijden->uit . "</b>"; 
}
 else
 {
 $uit = $record_wedstrijden->uit;
 }
 
$bg = ($a++ % 2) ? "********" : "#ebebeb";
 echo "<tr style=\"background-color:" . $bg . ";\">
 <td style=\"width:143px; font-size:11px;\">" . ucfirst(nlMonth($record_wedstrijden->datum_format)) . "</td>";
 
if($record_wedstrijden->thuis == "Geen team" or $record_wedstrijden->uit == "Geen team")
 {
 echo "<td colspan=\"5\" style=\"font-size:11px;\"><center><i>Geen wedstrijd</i></center></td>";
 }
 else
 {
 echo"
 <td style=\"font-size:11px;\">" . $record_wedstrijden->tijd_format . "</td>
 <td style=\"font-size:11px;\">" . $record_wedstrijden->type . "</td>
 <td style=\"font-size:11px; text-alignight;\">" . $thuis . "</td>
 <td style=\"font-size:11px; text-align:center;\">" . $status . "</td>
 <td style=\"font-size:11px;\">" . $uit . "</td>
 </tr>";
 }
 }
 echo "</table>";
 echo "<p>&nbsp;</p>";
}
else
{
 echo "<p style=\"font-size:11px font-weight:bold; font-style:italic;\">Er is nog geen wedstrijdschema beschikbaar.</p>"; 
}
 
?>

Ik vind manier van TS duidelijker dreamhost_nl ;-)
Dan zie je snel wat regel 35 is.

Als TS je code zou kopiëren krijgt hij minimaal een foutmelding (" LINE 35 >>") en de opmaak van een cel klopt niet ("text-alignight;" waar nu de smiley staat) :-)

Ieder zijn meug Yolknet. De oorspronkelijke code zal ook een foutmelding geven op "line 35 >>". Dat lijkt me nogal evident.
En om nog even betweterig te doen: De oorspronkelijke code gaf al een foutmelding op regel 35 en de verdwenen smileys zijn enkel voor de opmaak.

Ik heb die foutmelding al een hondertal keer gezien, meestal omdat ik een kleine typfout heb in de tabelnaam. Lang zoeken, omdat je hersens er blijven overlezen. Misschien iets gelijkaardigs? Of een lege variabele..

Er zijn meerdere oorzaken voor de foutmelding. Mysql geeft aan wat er precies mis is gegaan als je de oplossing van Yourwebhoster gebruikt.

http://www.pfz.nl/wiki/overzichtelijk-programmeren/ dit is een goede start, hoewel ik wel moet bekennen dat niet alles nog actueel is.

• { op de zelfde of niet regel
• , 2 of 4 spaties voor inspringen
• hungarian variables ($sUser ipv $user)

Zijn meer een persoonlijke voorkeur, en zelf volg ik nu de PSR-2 standaard.

En voor foutafhandeling http://www.pfz.nl/wiki/mysql-foutafhandeling-in-php/