Scripts - Tutorials - Forum - Downloads - Showcase - Contact

Artikels

VPN vergelijken

Algemeen

Beginpagina

FAQ

Grafische worm (243)

Links

Nieuwsartikels

Nieuwsarchief

Boeken programmeren

Overzicht

Samenwerken

Webhosting

Zoek op Sitemasters

Leden

Registreren

Ledenlijst

Ons team

Links

webhostingtop10.be

Sociale media

Follow @sitemasters

Sitemasters

Adverteren op Sitemasters?

Contacteer ons

RSS

Link naar ons

Donaties

Poll

Je moet ingelogd zijn om te stemmen.

Statistieken

Linkpartners

Forum

Categorieën > PHP & SQL

parser stopt onverwachts

cracker - 12/09/2005 00:54 (laatste wijziging 14/09/2005 19:20)

Onbekend

'lo.
't is misschien wel laat maar ik denk niet dat dit veel te maken zal hebben met de volgende fout.
Dus:
Ik ben bezig met mijn eigen CP/ACP en wil opvragen uit de database of en ook hoeveel nieuwe PM's er zijn.
Dit doe ik 2 keer maar ik begin met het eerste:
linksboven wordt de functie <?php unreadmail($username); ?> uitgevoerd. (admin.php)
Voor, tijdens of na stopt de parser en geeft hij de gebruiker slechts de code totaan die functie weer.
Dit resulteert uiteraard in een W3C fout en de hele site die niet werkt.
Hier is de code van de functie unreadmail() :

php code - Bekijk de code zonder highlighting - Klap code in

<?php
function unreadmail($ontvanger){
$ontvanger = strip_tags_sql(strip_tags(trim($ontvanger)));
$db = mysql_connect("localhost","ddbs2_webadmin","PASSWORD") or die(mysql_error());
mysql_select_db("ddbs2_server", $db) or die(mysql_error());
$aantal = mysql_query("SELECT * FROM pm WHERE map='inbox' AND gelezen='0' AND ontvanger='".$ontvanger."'") or die(mysql_error());
$rows = (int) mysql_num_rows($aantal) or die(mysql_error());
	if($rows == 1 && $rows != null && $rows != 0){
	echo("<font color=\"#ff0000\">".$rows." nieuw bericht</font>");
	}elseif($rows > 1 && $rows != null && $rows != 0){
	echo("<font color=\"#ff0000\">".$rows." nieuwe berichten</font>");
	}else{
	//Geen nieuwe berichten
	}
mysql_close();
}
?>

<?php
function unreadmail($ontvanger){
$ontvanger = strip_tags_sql(strip_tags(trim($ontvanger)));
$db = mysql_connect("localhost","ddbs2_webadmin","PASSWORD") or die(mysql_error());
mysql_select_db("ddbs2_server", $db) or die(mysql_error());
$aantal = mysql_query("SELECT * FROM pm WHERE map='inbox' AND gelezen='0' AND ontvanger='".$ontvanger."'") or die(mysql_error());
$rows = (int) mysql_num_rows($aantal) or die(mysql_error());
	if($rows == 1 && $rows != null && $rows != 0){
	echo("<font color=\"#ff0000\">".$rows." nieuw bericht</font>");
	}elseif($rows > 1 && $rows != null && $rows != 0){
	echo("<font color=\"#ff0000\">".$rows." nieuwe berichten</font>");
	}else{
	//Geen nieuwe berichten
	}
mysql_close();
}
?>

Bovenstaande functie staat in functies.php

In admin.php staat bovenaan require("functies.php");
Hierbij geen enkel probleem.

Of er nu PM's in de inbox zitten of niet, hierbij stopt de parser altijd.

tweede deel:
Ook kan je alle PM's opvragen. Dit gebeurt in een deel van pm.php.
code pm.php:

php code - Bekijk de code zonder highlighting - Klap code in

<?php
$sub = $_GET['sub'];
if(isset($sub) && strlen($sub) >= 2){
  switch($sub){
  case "outbox":
  display_outbox($username);
  break;
  case "deleted":
  display_recyclebin($username);
  break;
  default:
  display_inbox($username);
  }
}else{
display_inbox($username);
}
?>

<?php
$sub = $_GET['sub'];
if(isset($sub) && strlen($sub) >= 2){
  switch($sub){
  case "outbox":
  display_outbox($username);
  break;
  case "deleted":
  display_recyclebin($username);
  break;
  default:
  display_inbox($username);
  }
}else{
display_inbox($username);
}
?>

de functies display_outbox() en display_recyclebin() zijn nog niet gedefinieëerd en geven dus ook geen fouten.
Echter bij display_inbox() werkt het script enkel en alleen Als er PM's zijn voor de gebruiker.
Als de tabel pm geen rijen bevat met als ontvanger de gebruikersnaam van de huidige persoon stopt de parser voor het uitvoeren van het script.

(weer)

Er is geen parse error of niets.
Geen mySQL error...

Dit is display_inbox() (zit ook in functies.php) :

php code - Bekijk de code zonder highlighting - Klap code in

<?php
function display_inbox($ontvanger){
global $MSGnopminbox;
global $taal;
$prefix = "admin.php?section=pm&amp;sub=detail&amp;pmid=";
$ontvanger = strip_tags_sql(strip_tags(trim($ontvanger)));

$db = mysql_connect("localhost","ddbs2_webadmin","PASSWORD") or die(mysql_error());
mysql_select_db("ddbs2_server", $db) or die(mysql_error());
$aantal = mysql_query("SELECT * FROM pm WHERE map='inbox' AND ontvanger='".$ontvanger."'") or die(mysql_error());
$rows = (int) mysql_num_rows($aantal) or die(mysql_error());
	if($rows > 0 && $rows != null){
	echo("<table class=\"pmlist\" cellspacing=\"0\"><tr><th class=\"pmgelezen\">gelezen</th><th class=\"pmzender\">zender</th><th class=\"pmonderwerp\">onderwerp</th><th class=\"pmtijd\">tijd en datum</th></tr>");
	$ontvanger = strip_tags_sql(strip_tags(trim($ontvanger)));
	$resultb = mysql_query("SELECT ID, zender, onderwerp, gelezen, tijd FROM pm WHERE map='inbox' AND ontvanger='".$ontvanger."' ORDER BY gelezen, tijd DESC LIMIT 0,20")or die(mysql_error());
		while($data = mysql_fetch_assoc($resultb)){
			if($data['gelezen'] == 1){
			$data['gelezen'] = "<img src=\"../images/notepad%20blue.png\" alt=\"YES\" title=\"Dit bericht is al gelezen\" border=\"0\" width=\"20\" height=\"20\">";
			}else{
			$data['gelezen'] = "<img src=\"../images/notepad%20red.png\" alt=\"NO\" title=\"Dit bericht is nog niet gelezen\" border=\"0\" width=\"20\" height=\"20\">";
			}
		echo("\n<tr><td><a href=\"". $prefix . $data['ID'] ."\">".$data['gelezen']."</a></td><td>".$data['zender']."</td><td>".$data['onderwerp']."</td><td>".date("d-m-Y G:i", $data['tijd'])."</td></tr>");
		}
	echo("\n</table>");
	}else{
	//Geen berichten in de inbox
	echo($MSGnopminbox[$taal]);
	}
mysql_close();
}
?>

<?php
function display_inbox($ontvanger){
global $MSGnopminbox;
global $taal;
$prefix = "admin.php?section=pm&amp;sub=detail&amp;pmid=";
$ontvanger = strip_tags_sql(strip_tags(trim($ontvanger)));
 
$db = mysql_connect("localhost","ddbs2_webadmin","PASSWORD") or die(mysql_error());
mysql_select_db("ddbs2_server", $db) or die(mysql_error());
$aantal = mysql_query("SELECT * FROM pm WHERE map='inbox' AND ontvanger='".$ontvanger."'") or die(mysql_error());
$rows = (int) mysql_num_rows($aantal) or die(mysql_error());
	if($rows > 0 && $rows != null){
	echo("<table class=\"pmlist\" cellspacing=\"0\"><tr><th class=\"pmgelezen\">gelezen</th><th class=\"pmzender\">zender</th><th class=\"pmonderwerp\">onderwerp</th><th class=\"pmtijd\">tijd en datum</th></tr>");
	$ontvanger = strip_tags_sql(strip_tags(trim($ontvanger)));
	$resultb = mysql_query("SELECT ID, zender, onderwerp, gelezen, tijd FROM pm WHERE map='inbox' AND ontvanger='".$ontvanger."' ORDER BY gelezen, tijd DESC LIMIT 0,20")or die(mysql_error());
		while($data = mysql_fetch_assoc($resultb)){
			if($data['gelezen'] == 1){
			$data['gelezen'] = "<img src=\"../images/notepad%20blue.png\" alt=\"YES\" title=\"Dit bericht is al gelezen\" border=\"0\" width=\"20\" height=\"20\">";
			}else{
			$data['gelezen'] = "<img src=\"../images/notepad%20red.png\" alt=\"NO\" title=\"Dit bericht is nog niet gelezen\" border=\"0\" width=\"20\" height=\"20\">";
			}
		echo("\n<tr><td><a href=\"". $prefix . $data['ID'] ."\">".$data['gelezen']."</a></td><td>".$data['zender']."</td><td>".$data['onderwerp']."</td><td>".date("d-m-Y G:i", $data['tijd'])."</td></tr>");
		}
	echo("\n</table>");
	}else{
	//Geen berichten in de inbox
	echo($MSGnopminbox[$taal]);
	}
mysql_close();
}
?>

Nogmaals: enkel als er geen berichten zijn zit ik in de problemen met het tweede script.

Ik weet echt niet meer waar de fout zit.
Ik zit nu al drie dagen naar mijn scherm te kijken en telkens dingen weg te commentarieëren maar helaas...

Hopelijk zijn er andere (betere) pro's die mij een eindje vooruit kunnen helpen.
Daarvoor is Sitemasters er

Cracker.

10 antwoorden

Gesponsorde links

marten - 12/09/2005 23:10
Beheerder		ik zal het morgen eens aan mijn leraar laten zien Cracker Die heeft al wat jaren ervaring in php

Thomas - 13/09/2005 00:11
Moderator		Hehe. Als ik jouw parser was zou ik ook geen zin hebben in die code. </sarcasme> \:o/

cracker - 14/09/2005 07:23
Onbekend		marten: al resultaat ? FagorN: waarom?

Thomas - 14/09/2005 07:42
Moderator		- heb je al gekeken naar gereserveerde woorden (in MySQL) in je kolomnamen ? - mysql_num_rows retourneert altijd een integer, dus een expliciete typecast lijkt mij niet nodig - het if-statement om te controleren wat voor waarde $rows heeft kan echt stukken eenvoudiger - je hoeft alleen maar te controleren of deze 1 is, of niet - in het switch-statement in pm.php staat na de default-variant geen break - $prefix in display_inbox() begint met admin.php&, het lijkt mij dat hier een "?" moet staan in plaats van "&" - waarom maak je overal opnieuw een connectie met je database? dit hoef je in principe maar één keer te doen - heb je al geprobeerd de queries eens ergens anders gewoon in te voeren met een vaste waarde van $ontvanger? Is $ontvanger een string ?! Ik zou hier toch echt een ontvanger-id (dus een nummer) gebruiken, en geen naam o.i.d. Maak eerst je code eens schoon, en ga dan stapsgewijs wat zaken uitproberen. Je komt er vanzelf uit als je dingen systematisch aanpakt.

marten - 14/09/2005 10:45
Beheerder		Heb het aan mijn leraar laten zien.... Die zag ook geen fouten in de code en zei dat het nog aan bijvoorbeeld de server configuratie liggen. Mss kan je het script online zetten zodat ik hem kan testen? Samen met de tabellen natuurlijk

cracker - 14/09/2005 18:51
Onbekend		Citaat: - in het switch-statement in pm.php staat na de default-variant geen break niet nodig hier. Er kan dan ook maar 1 default zijn ae Citaat: - $prefix in display_inbox() begint met admin.php&, het lijkt mij dat hier een "?" moet staan in plaats van "&" => Niet HTML 4.01 Valid dan ;)

Maarten - 14/09/2005 18:53
Erelid		UUh.. bij default moet gewoon geen break staan, punt uit, dus het is idd niet nodig, dus haal het dan ook weg. En eenpagina.php?iets=dom is perfect valid, jij hebt staat eenpagina.php&iets=dom

cracker - 14/09/2005 19:13 (laatste wijziging 14/09/2005 20:07)
Onbekend		Citaat: dus haal het dan ook weg. Er staat er geen. (Quote FagorN) Citaat: - $prefix in display_inbox() begint met admin.php&, het lijkt mij dat hier een "?" moet staan in plaats van "&" & geeft geen ? maar wel een & dus ik ben correct. Murfy: Lees eens wat ik schreef voor ge naar dat van FagorN kijkt. edit: w00pz verkeerd gekopieerd die fout had ik al lang verwijderd. Sorry guys... zat te kijken naar de code in AceHTML, niet op deze pagina: daarom dat ik geen fout zag. In ieder geval, De parser blijft stoppen zonder reden. [u] EDIT:[/u] Ik had een pagina geïnclude en daarbij stopte de parser ook. Toen ik die pagina rechtstreeks opvroeg gaf de parser een parse error. Dus: Ik veronderstel dat daar ergens ook een parse error moet zitten. Weliswaar niet in functies.php want dan zou de parser stoppen bij iedere pagina.

Maarten - 14/09/2005 20:07
Erelid		Citaat: Murfy: Lees eens wat ik schreef voor ge naar dat van FagorN kijkt. Ten eerste ist FangorN en ten tweede heb ik het ook gelezen, jij zei dat blabla.php?iets=iets niet HTML valid is..

cracker - 14/09/2005 20:10
Onbekend		FangorN WH00ps 2nd: ik bedoelde ?moo=blaagh&bleh=fish & Dat bedoelde ik. Indien fout uitgelegd etc: sorry

Gesponsorde links

Dit onderwerp is gesloten.

Actiefste leden van de maand

Actieve forumberichten