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

Transferlijst script

SomaQan - 25/03/2007 16:57

PHP beginner

naar aanleiding van mij vorige advies topic heb ik wat geprobeert en ik kwam op dit uit

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

<?php
session_start();

include('config.php');

if (isset($_POST['bumpers']))

$sql = mysql_query("SELECT * FROM lid WHERE gebruikernaam ='".$_GET['gebruikernaam']."' LIMIT 1") or die(mysql_error());

while($show = mysql_fetch_object($sql)) {
$geld = htmlspecialchars($show->geld); 
$som = $geld;

if($som['num'] > 209) 
echo 'Je hebt geen genoeg geld om bumpers tekopen !'
}
else{
echo'Je hebt 1 bumper gekocht';
mysql_query("UPDATE lid SET bumpers = bumpers + 1") or die(mysql_error());
}

?>

<?php
session_start();
 
include('config.php');
 
if (isset($_POST['bumpers']))
 
$sql = mysql_query("SELECT * FROM lid WHERE gebruikernaam ='".$_GET['gebruikernaam']."' LIMIT 1") or die(mysql_error());
 
while($show = mysql_fetch_object($sql)) {
$geld = htmlspecialchars($show->geld); 
$som = $geld;
 
if($som['num'] > 209) 
echo 'Je hebt geen genoeg geld om bumpers tekopen !'
}
else{
echo'Je hebt 1 bumper gekocht';
mysql_query("UPDATE lid SET bumpers = bumpers + 1") or die(mysql_error());
}
 
?>

Maar ik kwam jammer genoeg op een error uit die ik niet weet te oplossen

Wie kan mij helpen

Alvast heel erg bedankt !

12 antwoorden

Gesponsorde links

Koen - 25/03/2007 17:04
PHP expert		php code - Bekijk de code zonder highlighting - Klap code in echo 'Je hebt geen genoeg geld om bumpers tekopen !'; echo 'Je hebt geen genoeg geld om bumpers tekopen !'; ?

SomaQan - 25/03/2007 17:07
PHP beginner		Hier heb ik niks aan wat bedoel je met dat vragen teken

JBke - 25/03/2007 17:09
PHP gevorderde		Je bent je ; aan het einde vergeten das al

Koen - 25/03/2007 17:13
PHP expert		Ja, had ik er misschien moeten bijzetten

SomaQan - 25/03/2007 17:16
PHP beginner		@sliphead toch bedankt Maar ik heb nog een probleem als ik op het pagina van het script kom dan laat hij meteen dat ELSE zien in plaats van dat IF eerst te laten zien want hij moet eerst checken of gene genoeg geld bij zich heeft

Koen - 25/03/2007 17:27

PHP expert

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

<?php
session_start();

include('config.php');

if (isset($_POST['bumpers']))
{

$sql = mysql_query("SELECT * FROM lid WHERE gebruikernaam ='".$_GET['gebruikernaam']."' LIMIT 1") or die(mysql_error());

while($show = mysql_fetch_object($sql)) {
$geld = htmlspecialchars($show->geld); 
$som = $geld;

if($som['num'] < 209)
{
echo 'Je hebt geen genoeg geld om bumpers tekopen !';
} else {
echo'Je hebt 1 bumper gekocht';
mysql_query("UPDATE lid SET bumpers = bumpers + 1") or die(mysql_error());
}
}
}
?>

<?php
session_start();
 
include('config.php');
 
if (isset($_POST['bumpers']))
{
 
$sql = mysql_query("SELECT * FROM lid WHERE gebruikernaam ='".$_GET['gebruikernaam']."' LIMIT 1") or die(mysql_error());
 
while($show = mysql_fetch_object($sql)) {
$geld = htmlspecialchars($show->geld); 
$som = $geld;
 
if($som['num'] < 209)
{
echo 'Je hebt geen genoeg geld om bumpers tekopen !';
} else {
echo'Je hebt 1 bumper gekocht';
mysql_query("UPDATE lid SET bumpers = bumpers + 1") or die(mysql_error());
}
}
}
?>

zo dan?

SomaQan - 25/03/2007 17:41

PHP beginner

Bedankt @sliphead je hebt het probleem halve opgelost

Alleen voegt hij nu geen bumpers toe en krijg geen foutmeldingen + ook geen echo meldingen

Mij script ziet er nu zo uit

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

<?php
session_start();

include('config.php');

if( $_SERVER['REQUEST_METHOD'] == 'POST')
{

$bumpers = addslashes($_POST['bumpers']);

$sql = mysql_query("SELECT * FROM lid WHERE gebruikernaam ='".$_GET['gebruikernaam']."' LIMIT 1") or die(mysql_error());

while($show = mysql_fetch_object($sql)) {
$geld = htmlspecialchars($show->geld); 
$som = $geld;



if($som['num'] < 209)
{
echo 'Je hebt geen genoeg geld om bumpers tekopen !';
} 

else {
echo'Je hebt 1 bumper gekocht';
mysql_query("UPDATE lid SET bumpers = bumpers + 1") or die(mysql_error());
}

}
}
?>
<form action="<? echo $PHP_SELF; ?>" method="POST">
<b>Bumpers:</b><input type="radio" name="bumpers"><br />
<br /><input type="submit" name="submit" value="Items Kopen">
</form>

<?php
session_start();
 
include('config.php');
 
if( $_SERVER['REQUEST_METHOD'] == 'POST')
{
 
$bumpers = addslashes($_POST['bumpers']);
 
$sql = mysql_query("SELECT * FROM lid WHERE gebruikernaam ='".$_GET['gebruikernaam']."' LIMIT 1") or die(mysql_error());
 
while($show = mysql_fetch_object($sql)) {
$geld = htmlspecialchars($show->geld); 
$som = $geld;
 
 
 
if($som['num'] < 209)
{
echo 'Je hebt geen genoeg geld om bumpers tekopen !';
} 
 
else {
echo'Je hebt 1 bumper gekocht';
mysql_query("UPDATE lid SET bumpers = bumpers + 1") or die(mysql_error());
}
 
}
}
?>
<form action="<? echo $PHP_SELF; ?>" method="POST">
<b>Bumpers:</b><input type="radio" name="bumpers"><br />
<br /><input type="submit" name="submit" value="Items Kopen">
</form>

Kan iemand mij helpen ?

Koen - 25/03/2007 17:56 (laatste wijziging 25/03/2007 18:00)

PHP expert

en als je het zo doet?:

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

<?php
session_start();

include('config.php');

if( $_SERVER['REQUEST_METHOD'] == 'POST')
{

$bumpers = addslashes($_POST['bumpers']);

$sql = mysql_query("SELECT * FROM lid WHERE gebruikernaam ='".$_GET['gebruikernaam']."' LIMIT 1") or die(mysql_error());

while($show = mysql_fetch_object($sql)) {
$geld = htmlspecialchars($show->geld); 
$som = $geld;



if($som['num'] < 209)
{
echo 'Je hebt geen genoeg geld om bumpers tekopen !';
} 

else {
echo 'Je hebt 1 bumper gekocht';
$bumber = htmlspecialchars($show->bumbers);
mysql_query("UPDATE lid SET bumpers = '".$bumber++."' WHERE gebruikernaam ='".$_GET['gebruikernaam']."'") or die(mysql_error());
}

}
}
?>
<form action="<? echo $PHP_SELF; ?>" method="POST">
<b>Bumpers:</b><input type="radio" name="bumpers"><br />
<br /><input type="submit" name="submit" value="Items Kopen">
</form>

<?php
session_start();
 
include('config.php');
 
if( $_SERVER['REQUEST_METHOD'] == 'POST')
{
 
$bumpers = addslashes($_POST['bumpers']);
 
$sql = mysql_query("SELECT * FROM lid WHERE gebruikernaam ='".$_GET['gebruikernaam']."' LIMIT 1") or die(mysql_error());
 
while($show = mysql_fetch_object($sql)) {
$geld = htmlspecialchars($show->geld); 
$som = $geld;
 
 
 
if($som['num'] < 209)
{
echo 'Je hebt geen genoeg geld om bumpers tekopen !';
} 
 
else {
echo 'Je hebt 1 bumper gekocht';
$bumber = htmlspecialchars($show->bumbers);
mysql_query("UPDATE lid SET bumpers = '".$bumber++."' WHERE gebruikernaam ='".$_GET['gebruikernaam']."'") or die(mysql_error());
}
 
}
}
?>
<form action="<? echo $PHP_SELF; ?>" method="POST">
<b>Bumpers:</b><input type="radio" name="bumpers"><br />
<br /><input type="submit" name="submit" value="Items Kopen">
</form>

timo - 25/03/2007 18:08
PHP ver gevorderde		"Je hebt geen genoeg geld om bumpers te kopen" staat niet echt prof in een game ofzo (lijkt dat het dat is) als het zulk crappy nl is..

Koen - 25/03/2007 18:17
PHP expert		Tekst heeft geen belang nu, het is het script dat telt.

SomaQan - 25/03/2007 18:54
PHP beginner		@timo tekst wordt wel later verbetert het gaat nu om het script

Dark_Paul - 26/03/2007 12:37 (laatste wijziging 26/03/2007 12:48)

PHP ver gevorderde

Spring in met tabs, dan zie je meteen waar je accolades bent vergeten (zoals net).
Vertel ons ook even wat de error is die je nu krijgt, dat werkt gemakkelijker.

Edit:

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

<?php
session_start();

include('config.php');

if (isset($_POST['bumpers'])) {

	$sql = mysql_query("SELECT * FROM lid WHERE gebruikernaam ='".$_GET['gebruikernaam']."' LIMIT 1") or die(mysql_error());

	while($show = mysql_fetch_object($sql)) {
		$geld = htmlspecialchars($show->geld); 
		$som = $geld;

		if($som['num'] > 209) {
			echo 'Je hebt geen genoeg geld om bumpers tekopen !'
		}
		else{
			echo 'Je hebt 1 bumper gekocht';
			mysql_query("UPDATE lid SET bumpers = bumpers + 1") or die(mysql_error());
		}
	}
}

?>

<?php
session_start();
 
include('config.php');
 
if (isset($_POST['bumpers'])) {
 
	$sql = mysql_query("SELECT * FROM lid WHERE gebruikernaam ='".$_GET['gebruikernaam']."' LIMIT 1") or die(mysql_error());
 
	while($show = mysql_fetch_object($sql)) {
		$geld = htmlspecialchars($show->geld); 
		$som = $geld;
 
		if($som['num'] > 209) {
			echo 'Je hebt geen genoeg geld om bumpers tekopen !'
		}
		else{
			echo 'Je hebt 1 bumper gekocht';
			mysql_query("UPDATE lid SET bumpers = bumpers + 1") or die(mysql_error());
		}
	}
}
 
?>

Qua haakjes zou dit moeten werken, maar dit stukje kan ik niet helemaal begrijpen:

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

		$geld = htmlspecialchars($show->geld); 
		$som = $geld;

		if($som['num'] > 209) {

$geld = htmlspecialchars($show->geld); 
		$som = $geld;
 
		if($som['num'] > 209) {

Je zegt $som = $geld, vervolgens ga je verder alsof $som een array is, terwijl $geld al geen array meer is.
Volgens mij bedoel je met dat stukje dit:

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

if (htmlspecialchars($show->geld) > 209) {
//
}

if (htmlspecialchars($show->geld) > 209) {
//
}

Verder haal je onnodige dingen op uit je database, wat voor extra serverload zorgt.
Dus

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

$sql = mysql_query("SELECT * FROM lid WHERE gebruikernaam ='".$_GET['gebruikernaam']."' LIMIT 1")

$sql = mysql_query("SELECT * FROM lid WHERE gebruikernaam ='".$_GET['gebruikernaam']."' LIMIT 1")

kan je beter vervangen door:

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

$sql = mysql_query("SELECT geld FROM lid WHERE gebruikernaam ='".$_GET['gebruikernaam']."' LIMIT 1")

$sql = mysql_query("SELECT geld FROM lid WHERE gebruikernaam ='".$_GET['gebruikernaam']."' LIMIT 1")

Verder is je While ook overbodig, die kan je dus ook weghalen.

Edit 2:
Ik heb een aantal dingen gewijzigd in je script.

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

<?php
session_start();

include('config.php');

if (isset($_POST['bumpers'])) {
	$sql = mysql_query("SELECT geld FROM lid WHERE gebruikersnaam='".$_GET['gebruikersnaam']."' LIMIT 1");
	
	$show = mysql_fetch_object($sql);
	
	if (htmlspecialchars($show->geld) > 209) {
		echo 'Je hebt niet genoeg geld om bumpers te kopen!';
	}
	else {
		echo 'Je hebt 1 bumper gekocht';
		mysql_query("UPDATE lid SET bumpers = bumpers + 1 WHERE gebruikersnaam='".$_GET['gebruikersnaam']."' ");
	}
}
?>

<?php
session_start();
 
include('config.php');
 
if (isset($_POST['bumpers'])) {
	$sql = mysql_query("SELECT geld FROM lid WHERE gebruikersnaam='".$_GET['gebruikersnaam']."' LIMIT 1");
 
	$show = mysql_fetch_object($sql);
 
	if (htmlspecialchars($show->geld) > 209) {
		echo 'Je hebt niet genoeg geld om bumpers te kopen!';
	}
	else {
		echo 'Je hebt 1 bumper gekocht';
		mysql_query("UPDATE lid SET bumpers = bumpers + 1 WHERE gebruikersnaam='".$_GET['gebruikersnaam']."' ");
	}
}
?>

Een aantal opmerkinge/veranderingen:
1) Je had bij je update geen WHERE ingevoerd, dus bij iedereen in de tabel 'lid' zou dan 1 bij de bumpers worden opgeteld;
2) Je queries zijn niet veilig, omdat je gebruik maakt van een $_GET-waarde. Het lijkt erop dat je een ledensysteem gebruikt, dan houd je mensen dus ook ingelogd. Dan zou je dus het ID van die persoon in een sessie op kunnen slaan. Zo kan je ook voorkomen dat mensen voor elkaar bumpers kunnen kopen door simpelweg de naam in de url te wijzigen, maar je voorkomt ook SQL-injections via de url.
Als het de bedoeling is dat mensen voor elkaar bumpers kunnen kopen kan dat ook via $_GET, maar is het beter om dan in de url het ID van het lid mee te geven. Getallen zijn gemakkelijk te controleren of het wel echt getallen zijn..

Gesponsorde links

Dit onderwerp is gesloten.

Actiefste leden van de maand

Actieve forumberichten