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

ik kom dr echt neit uit!!

burdy92 - 07/05/2005 16:11

HTML interesse

heey... ik heb dus weer eens een fout.. (misschien een kleine maarjah)
ik heb een script (een personal message systeem) en nu geeftie altijd ook al heb ik de username goed getypt: deze username bestaat niet ofzo:P en dan verstuurd hij hem dus niet... ik heb al veel geprobeerd maar werkt neit...:

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

 
<?php 
if(!isset($_SESSION["gebruikersnaam"])){  echo "Je bent niet ingelogd en kunt daarom geen persoonlijk bericht versturen!"; } else {?><?
if($_SERVER['REQUEST_METHOD'] == 'POST'){
$queryu = mysql_query("SELECT * FROM `bz_gebruikers`"); 
$resultu = mysql_num_rows($queryu);
$objectu = mysql_fetch_object($queryu);
if(!$van_wie == $naar_wie) {
if(!$objectu->gebruikersnaam == $naar_wie) {
$naar_wie = $_POST['naar_wie'];
$titel = $_POST['titel'];
$bericht = $_POST['bericht'];
$datum = date("d-m-Y");
$tijd = date("G:i");
$van_wie = ($_SESSION["gebruikersnaam"]);

mysql_query ("INSERT into `bz_pb` (id, naar_wie, titel, bericht, datum, tijd, van_wie) values (NULL,'$naar_wie','$titel','$bericht','$datum','$tijd','$van_wie')");
echo "Het bericht is verzonden naar: $naar_wie!";
} else { 
echo "Deze gebruiker bestaat niet<br> en daarom kun je hem geen persoonlijk bericht versturen!";

}
} else { 
$van_wie = ($_SESSION["gebruikersnaam"]);
echo "Je gaat toch geen pb naar jezelf versturen?!"; }
} else {
?>
hier een form 
<? }} ?>

<?php 
if(!isset($_SESSION["gebruikersnaam"])){  echo "Je bent niet ingelogd en kunt daarom geen persoonlijk bericht versturen!"; } else {?><?
if($_SERVER['REQUEST_METHOD'] == 'POST'){
$queryu = mysql_query("SELECT * FROM `bz_gebruikers`"); 
$resultu = mysql_num_rows($queryu);
$objectu = mysql_fetch_object($queryu);
if(!$van_wie == $naar_wie) {
if(!$objectu->gebruikersnaam == $naar_wie) {
$naar_wie = $_POST['naar_wie'];
$titel = $_POST['titel'];
$bericht = $_POST['bericht'];
$datum = date("d-m-Y");
$tijd = date("G:i");
$van_wie = ($_SESSION["gebruikersnaam"]);
 
mysql_query ("INSERT into `bz_pb` (id, naar_wie, titel, bericht, datum, tijd, van_wie) values (NULL,'$naar_wie','$titel','$bericht','$datum','$tijd','$van_wie')");
echo "Het bericht is verzonden naar: $naar_wie!";
} else { 
echo "Deze gebruiker bestaat niet<br> en daarom kun je hem geen persoonlijk bericht versturen!";
 
}
} else { 
$van_wie = ($_SESSION["gebruikersnaam"]);
echo "Je gaat toch geen pb naar jezelf versturen?!"; }
} else {
?>
hier een form 
<? }} ?>

kan iemand me helpen? thx

4 antwoorden

Gesponsorde links

Shifty - 07/05/2005 16:15
Onbekend		Ik quote mezelf even vanuit een ander topic: Citaat: Probeer ook te letten op eht onnodig aanmaken van variabelen. Ik zie dat je ergens $time = time(); hebt staan. Dit is overbodig en kan dus weg, wat resulteert in een netter en overzichtelijker script. Je zal dit wel met een korrel zout nemen, maar je hebt er echt profijt van. Evenals goede namen verzinnen voor je variabelen. $a zegt voor jou waarschijnlijk genoeg, maar voor een vreemde niet. Wanneer je na een lange tijd weer terugkijkt op het script, zal je het waarschijnlijk ook niet meer snappen, dus zorg ten aller tijde voor de juiste benamingen. In jouw script zou ik $a vervangen door bijvoorbeeld $sBericht. s staat voor string (rij letters en/of letters + cijfers). Haal variabelen buiten quotes! Een goed voorbeeld kan je in deze tutorial onderaan vinden: http://www.site...=6&; id=78#6 Dit geldt ook ongeveer voor jou. Verder maak je onnodig veel variabelen aan en lijn je neit uit. Daarom is het moelijk voor ons om te kijken wat de fout precies is. Wannneer je uitlijnt merk je waarschijnlijk zelf je fout ook op.

Tuinstoel - 07/05/2005 16:15

PHP expert

Waar haal je $van_wie vandan, en $naar_wie? Superglobals gebruiken, want je vraagt ze eerst op, en daaronder defineer je ze.
Je moet gewoon je hele script aanpassen, en een aparte query draaien of deze user wel bestaat, zie het codefragment hieronder.

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

<?php
$sCountSql = "SELECT COUNT(`id`) FROM `bz_gebruikers` WHERE `naam` = '".addslashes($_POST['naar_wie'])."'";
$iCount = mysql_result(mysql_query($sCountSql),0);

if(empty($iCount))
	{ echo 'Deze gebruiker bestaat niet!'; }
else
	{
	// rest van je script,
	// bericht versturen e.d.
	}
?>

<?php
$sCountSql = "SELECT COUNT(`id`) FROM `bz_gebruikers` WHERE `naam` = '".addslashes($_POST['naar_wie'])."'";
$iCount = mysql_result(mysql_query($sCountSql),0);
 
if(empty($iCount))
	{ echo 'Deze gebruiker bestaat niet!'; }
else
	{
	// rest van je script,
	// bericht versturen e.d.
	}
?>

burdy92 - 07/05/2005 16:23
HTML interesse		thx maar sorry:P er staat neit dat de username nit bestaat maar: je gaat toch geen pb naar jezelf versturen?! .... dat staat er!

Tuinstoel - 07/05/2005 16:57
PHP expert		Je script is ook zo barslecht, ik kan geen verschil maken wat bij wat hoort. Je moet netjes leren scripten. Dan kun je de fouten er zelf gemakkelijk uit halen, en dan zul je dit forum niet vaak hoeven te bezoeken. Je moet dus tabs gebruiken, juiste variabele benaming, en ga zo het rijtje maar af. Het is logisch dat het je niet lukt als je zo script, en dit blijft ook maar terug komen terwijl het ze nog steeds niet duidelijk wordt dat ze het verkeerd doen en dus nodig overnieuw php moeten leren.

Gesponsorde links

Dit onderwerp is gesloten.

Actiefste leden van de maand

Actieve forumberichten