php - forkert opsætning - Udvikleren.dk - Programmering, webdesign og grafik

Tags: php

Bruger #14510 @ 21.08.10 22:47

hejsa, jeg har kodet et system som nogle information ind og så ser om det kan lade sig gøre at fortage en handling,
det er til et php spil,
hvor man skal bruge 2000 gold og 300 food for at bygge en arbejder, men selvom man kun har 100 gold kan man stadig bygge det...
kan du finde fejlen

Kode

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<title>Untitled Document</title>

</head>

<meta http-equiv="REFRESH" content="5;url=/GAMEDYNAMITE/ARMY.php">
<body>
<?php 
session_start(); 
?>
<?
if(isset($_SESSION['myusername'], $_SESSION['mypassword'])) {
include 'http://www.jkay.dk/GAMEDYNAMITE/Train.php';
$host="localhost"; // Host name 
$username="jkay_dk"; // Mysql username 
$password="((((/"; // Mysql password 
$db_name="jkay_dk"; // Database name 
$tbl_name="users"; // Table name

// Connect to server and select databse.
mysql_connect($host, $username, $password)or die("cannot connect"); 
mysql_select_db($db_name)or die("cannot select DB");
$arbejder = $_POST['arbejder'];
//pris for en arbejder er 300 food og 2000 gold
$brugerr = $_SESSION['myusername'];
$gold = mysql_query("SELECT gold FROM users WHERE username = '$brugerr'") or die(mysql_error());
$gold = mysql_fetch_assoc($gold);
$food = mysql_query("SELECT food FROM users WHERE username = '$brugerr'") or die(mysql_error());
$food = mysql_fetch_assoc($food);
$wood = mysql_query("SELECT wood FROM users WHERE username = '$brugerr'") or die(mysql_error());
$wood = mysql_fetch_assoc($wood);
$guldpris = $arbejder*2000;
$foodpris = $arbejder*300;
if($foodpris<$food){
	echo " du har ikke nok Food til at træne " . $arbejder . "arbejdere";
} elseif($guldpris<$gold) {
		echo " du har ikke nok Gold til at træne " . $arbejder . "arbejdere";
	}else{ 
	mysql_query("UPDATE users SET gold = gold-$guldpris WHERE username='$brugerr'");
	mysql_query("UPDATE users SET food = food-$foodpris WHERE username='$brugerr'");
	mysql_query("UPDATE users SET arbejder = arbejder+$arbejder WHERE username='$brugerr'");
	echo " du trænede " . $arbejder . "arbejdere";
	}

}
?>
</body>
</html>

3 svar postet i denne tråd vises herunder
1 indlæg har modtaget i alt 1 karma

Sorter efter stemmer Sorter efter dato

Bruger #10216 @ 21.08.10 23:39

4.283

Dine if-sætninger er vent forkert.
goldpris skal være større-end $gold for at du får en fejl.

Jeg har ryddet lidt op i din kode og vendt det hele lidt på hovedet for at du får en mere korrekt PHP/HTML-kode.

PHP kode

 <?php
 /*
  * Man starter ALTID med session_start(). session er header baseret og skriver du til klienten før du sender en header (det svarer til at maven kommer før hovedet), så går det galt. Prøv selv at forestille dig hvordan det vil se ud.
  */  
 session_start(); 
 
 // Altid simplificer de opgaver der ligger i en if-sætning
 if(!isset($_SESSION['myusername'], $_SESSION['mypassword'])) {
     header('location: login.php');
 }
 
 // ved ikke hvad dette er - men det giver ikke mening at bruge URLer i include.
 include $_SERVER['DOCUMENT_ROOT'].'/GAMEDYNAMITE/Train.php';
 
 // Da du vist ikke kender til objekt-orieteret programmering er dette okay, men man kan lægge DB-connection i en global include-fil.
 $host="localhost"; // Host name 
 $username="jkay_dk"; // Mysql username 
 $password="((((/"; // Mysql password 
 $db_name="jkay_dk"; // Database name 
 $tbl_name="users"; // Table name
 
 // Connect to server and select databse.
 mysql_connect($host, $username, $password)or die("cannot connect"); 
 mysql_select_db($db_name)or die("cannot select DB");
     
 $arbejder = $_POST['arbejder'];
 
 //pris for en arbejder er 300 food og 2000 gold
 $brugerr = $_SESSION['myusername'];
 
 // Man kan sagtens hente flere felter i samme query.
 $query = mysql_query("SELECT gold, food, wood FROM users WHERE username = '$brugerr'") or die(mysql_error());
 $result = mysql_fetch_assoc($query);
 $gold = $result['gold'];
 $food = $result['food'];
 $wood = $result['wood'];
 
 $guldpris = $arbejder*2000;
 $foodpris = $arbejder*300;
 
 if($food > $foodpris && $gold > $goldpris) {
     // man kan sagtens opdatere flere felter i en query.    
     mysql_query("UPDATE users SET gold = gold-$guldpris, food = food-$foodpris, arbejder = arbejder+$arbejder WHERE username='$brugerr'");
     $msg = " du trænede " . $arbejder . "arbejdere";
 } else {
     if($food < $foodpris){
         $msg = " du har ikke nok Food til at træne " . $arbejder . "arbejdere";
     } else if($gold < $goldpris) {
         $msg = " du har ikke nok Gold til at træne " . $arbejder . "arbejdere";
     }else{ 
         $msg = " du kan ikke træne ". $arbejder . "arbejdere";
     }
 }
 
 ?>
 <!--  HTML SKAL ALTID VÆRE SIDST, DET ER OUTPUT TIL KLIENTEN OG ALLE ANDRE SERVER-BASERET HANDLINGER SKAL VÆRE UDFØRT FØRST. -->
 <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
 <html xmlns="http://www.w3.org/1999/xhtml">
     <head>
         <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
         <meta http-equiv="REFRESH" content="5;url=/GAMEDYNAMITE/ARMY.php">
         <title>Untitled Document</title>
     </head>
     <body>
         <?php if($msg): ?>
             <div style="border: 1px solid #a00; background: #faa; color: #000; padding: 10px;"><?php echo $msg; ?></div>
         <?php endif; ?>
     </body>
 </html>

Bruger #14510 @ 22.08.10 12:07

mange tak for koden michael

den virker super

men lige en ting som jeg ikke kan få styr på, for hvis jeg har minus(i gold) kan jeg lave så mange arbejdere som jeg vil?
hvordan sætter jeg en begrænsing på dette?

Bruger #10216 @ 22.08.10 14:45

4.283

Sørg for at vende dine betingelser rigtigt. Se nedestående eksempler, se efter "krokodillenæbbet".

Hvis $guld= 100 og $guldpris=50:
$guld > $guldpris : true;

Hvis $guld = 40 og $guldpris = 50
$guld > $guldpris : false;

Hvis $guld= 100 og $guldpris=50:
$guld < $guldpris : false;

Hvis $guld = 40 og $guldpris = 50
$guld < $guldpris : true;

forkert opsætning

Karma barometer (30 dage)

Modtaget

Givet

Favorit hos
Forum tråde
Artikler