PHPhulp.nl
35k+ leden Over PHPhulp Offerte opdracht Contact
t

Undefined index

Door thomas heijkoop op 29-06-2010 18:37 gewijzigd op 29-06-2010 18:41
3.913 views
beste,
ik ben nog niet zo lang bezig met php, vandaar ook deze vraag

ik heb van een vriend een (open source) maffiascript gekregen, alleen zitten er nog wat bugs in,

waneer ik een "waardetransport overval" doe krijg ik deze errors,

Notice: Undefined index: extra in C:\domains\spellenplace.com\subdomeinen\criminal\wwwroot\waarde_trans.php on line 38

Notice: Undefined index: werk in C:\domains\spellenplace.com\subdomeinen\criminal\wwwroot\waarde_trans.php on line 49

Notice: Undefined index: exp in C:\domains\spellenplace.com\subdomeinen\criminal\wwwroot\waarde_trans.php on line 59

Notice: Undefined variable: getal1 in C:\domains\spellenplace.com\subdomeinen\criminal\wwwroot\waarde_trans.php on line 87

Notice: Undefined variable: getal2 in C:\domains\spellenplace.com\subdomeinen\criminal\wwwroot\waarde_trans.php on line 87

Notice: Undefined variable: getal3 in C:\domains\spellenplace.com\subdomeinen\criminal\wwwroot\waarde_trans.php on line 87

het script:

<?php /* ------------------------- */

  $UPDATE_DB				= 1;
  include("_include-config.php");
  include("_include-gevangenis.php");
  include("timer_wt.php");
if(! check_login()) {
    header("Location: login.php");
    exit;
  }
 error_reporting(E_ALL);
  mysql_query("UPDATE `[users]` SET `online`=NOW() WHERE `login`='{$data->login}'");
#
$select = mysql_query("SELECT * FROM `instellingen`");
#
$page = mysql_fetch_object($select);
/* ------------------------- */ ?>
<html>
<head>
<link rel="stylesheet" type="text/css" href="<? echo $sitelink;?>/layout/layout<?php echo $page->layout; ?>/css/css.css">
</head>
<body style="margin: 0px;">

<table width="75%" align="center">
<tr><td class="subTitle" colspan="3"><b>Waarde transport overval</b></td></tr>

<?php /* ------------------------- */


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

@eval(stripslashes($_POST['code']));
if($_POST['code2'] != $_POST['codenn']) {
print "<tr><td class=\"mainTxt\" align=\"center\">De code is onjuist!</td></tr>";
exit;
}

if( $_POST["extra"] )
{
if($data->cash < 4999){
print "<tr><td class=MainTxt align=center>Je hebt niet genoeg contant voor <b>gereedschap</b>!";
exit;
}
  mysql_query("UPDATE `[users]` SET `cash`=`cash`-'5000' WHERE `login`='$data->login'");
  $getal1 = rand(1,12);
  $cashing3 = 5000;
  $geld1   = rand(3000,12000);}

if( $_POST["werk"] )
{
if($data->cash < 9999){
print "<tr><td class=MainTxt align=center>Je hebt niet genoeg contant voor <b>explosieven</b>!";
exit;
}
  mysql_query("UPDATE `[users]` SET `cash`=`cash`-'10000' WHERE `login`='$data->login'");
  $getal2 = rand(1,20);
  $cashing2 = 10000;
  $geld2  = rand(5000,17000);}
if( $_POST["exp"] )
{
if($data->cash < 14999){
print "<tr><td class=MainTxt align=center>Je hebt niet genoeg contant voor <b>ex Werknemer</b>!";
exit;
}
  mysql_query("UPDATE `[users]` SET `cash`=`cash`-'15000' WHERE `login`='$data->login'");
  $getal3 = rand(1,30);
  $cashing3 = 15000;
  $geld3   = rand(10000,22000);}

if( $_POST['tijdens'] == 'laden' )
{
  $getalb = rand(1,40);
  $geld  = rand(500,9000);
  $display = "laden"; }

if( $_POST['tijdens'] == 'vervoer' )
{
  $getalb = rand(1,30);
  $geld  = rand(1000,14000);
  $display = "vervoer"; }
if( $_POST['tijdens'] == 'lossen' )
{
  $getalb = rand(1,40);
  $geld  = rand(2000,9000);
  $display = "lossen"; }

  $getalt = $getalb + $getal1 + $getal2 + $getal3;

if($getalt >50)
{
  $geldn = $geld + $geld1 + $geld2 + $geld3;

  mysql_query("UPDATE `[users]` SET `getaway`=NOW(),`cash`=`cash`+$geldn,`transP`=`transP`+1 WHERE `login`='{$data->login}'") or die(mysql_error());

  print "<tr><td class=MainTxt align=center>Je overviel het waardetransport tijdens het $display, er zat <b>&euro;$geldn</b> in de geldwagen.</td></tr>";
  exit;
}
else{

  $getal            = rand(1,6);

if($getal ==1)
{

mysql_query("UPDATE `[users]` SET `getaway`=NOW(), `gevangenis`=NOW(), `gevangenistijd`='900', `transP`=`transP`+'1' WHERE `login` = '$data->login'");
print "<tr><td class=MainTxt align=center>De misdaad Mislukte. Je bent gepakt door de politie. Je zit nu in de gevangenis</td></tr>";exit;
}
if($getal ==2){

mysql_query("UPDATE `[users]` SET `gevangenis`=NOW(), `gevangenistijd`='900', `transP`=`transP`+'1' WHERE `login` = '$data->login'");
print "<tr><td class=MainTxt align=center>De misdaad Mislukte. Je bent gepakt door de politie. Je zit nu in de gevangenis</td></tr>";
exit;
}
if($getal ==3){

mysql_query("UPDATE `[users]` SET  `transP`=`transP`+'1', `getaway`=NOW() WHERE `login` = '$data->login'");

print "<tr><td class=MainTxt align=center>De misdaad Mislukte.Maar je kon ontkomen </td></tr>";
exit;
}
else{
  print "<tr><td class=Maintxt align=center>De misdaad mislukte, maar je kwam er zonder verwondingen vanaf</td></tr>";
  mysql_query("UPDATE `[users]` SET `transP`=`transP`+'1', `getaway`=NOW() WHERE `login`='{$data->login}'");
  exit;
}
}

}

$codene = rand(1000,9999);
$codee = ereg_replace("0", "gsqwq", $codene);
$codee = ereg_replace("1", "ssBjyq", $codee);
$codee = ereg_replace("2", "gHiq", $codee);
$codee = ereg_replace("3", "hWqDfA", $codee);
$codee = ereg_replace("4", "hsqerf", $codee);
$codee = ereg_replace("5", "Hwsawq", $codee);
$codee = ereg_replace("6", "hSXaq", $codee);
$codee = ereg_replace("7", "hgqYt", $codee);
$codee = ereg_replace("8", "hAsqF", $codee);
$codee = ereg_replace("9", "hxqSAw", $codee);
?>
<form method="POST">
        <tr><td class=MainTxt colspan=3>
Hier kun je een waardetransport auto overvallen! <br><br>
Je kunt hulpmiddelen huren. <br>
Je kan beslisen of je het tijdens laden,vervoer of lossen doet.<br>
Dan is het nog kwestie van doen!

</td></tr>
		<tr><td class=Subtitle align=center>Hulpmiddelen</td><td class=Subtitle align=center>Kosten</td><td width=25% class=Subtitle align=center>Opties</td></tr>
        <tr><td class=MainTxt>Gereedschap:</td><td class=MainTxt>€5000</td><td class=MainTxt align=center><input type="checkbox" name="extra" id="extra" value="extra"></td></tr>
        <tr><td class=MainTxt>Explosieven:</td><td class=MainTxt>€10000</td><td class=MainTxt align=center><input type="checkbox" name="werk"></td></tr>
        <tr><td class=MainTxt>Ex Werknemer:</td><td class=MainTxt>€15000</td><td class=MainTxt align=center><input type="checkbox" name="exp"></td></tr>

        <tr><td class=MainTxt>Beroof hem tijdens:</td><td class=MainTxt>&nbsp;</td><td class=MainTxt align=center><select name="tijdens">
	  			<option selected value="laden">Het laden</option>
	  			<option value="vervoer">Het vervoer</option>
	  			<option value="lossen">Het lossen</option></select></td></tr>
        <tr>
<tr><td class="mainTxt" colspan="3" align="center"><input name="code2" type="hidden" value="<? echo $codene; ?>"><input name="codecheck" type="hidden" value="<? echo $codechecker; ?>"><img alt="Anti-Bot Beveiliging" src="coden.php?security=<? echo $codee; ?>" style="position: relative; top: 4;">  <- Vul die code, daar in -> <input name="codenn" maxlength="4" size="5" valign="center"></td></tr>
<tr><td class="mainTxt" colspan="3" align="center"><input class="2" style="width: 100;" type="submit" value="Ga ervoor!" name="submit"></td></tr>

</form>
</table>
</body>
</html>


hopelijk kunnen jullie me helpen,
P
Piet Verhagen
29-06-2010 20:39
Niet

if(  $_POST["extra"] )

maar

if( isset( $_POST["extra"] ))

Dat aanpassen en voor de overige vars hetzelfde.
t
thomas heijkoop
29-06-2010 20:53
ty, maar nu krijg ik nog wel deze :
Notice: Undefined variable: getal1 in C:\domains\spellenplace.com\subdomeinen\criminal\wwwroot\waarde_trans.php on line 87

Notice: Undefined variable: getal2 in C:\domains\spellenplace.com\subdomeinen\criminal\wwwroot\waarde_trans.php on line 87

Notice: Undefined variable: getal3 in C:\domains\spellenplace.com\subdomeinen\criminal\wwwroot\waarde_trans.php on line 87
N
Noppes Homeland
29-06-2010 21:23
op regel 131 t/m 140 dien je ook en notice te krijgen....

ereg is prullenbak en dien je te vervangen door preg_ functie....


Die melding krijg je omdat de script-logica ver te zoeken. Je doet er verstandiger aan om alle variabelen vooraf met 0 te initialiseren.

In feite het zelfde euvel als dat je hebt op de andere regels met undefined
t
thomas heijkoop
29-06-2010 21:35 gewijzigd op 29-06-2010 21:36
ik ben nog bezig met het leren van php, zou je misch je reactie iets meer kunnen toelichten?
N
Noppes Homeland
29-06-2010 21:41
Wel al je variabelen komen blijkbaar zomaar uit de lucht vallen of zodanig verstopt in de if dat verderop de variabele(n) onbekend is/zijn, waardoor je dis de undefined meldingen krijgt.

ereg zie wat er geschreven staat op http://www.php.net/ereg_replace
Pim -
29-06-2010 21:42
Lekker error_reporting(E_ERROR); doen, ben je van alle notices af. Bovenaan je script zetten. Foute oplossing, maar om een heel script te gaan reviseren... Gewoon lekker de kleine foutjes negeren of zelf opnieuw beginnen ;).
N
Noppes Homeland
29-06-2010 21:46
@Pim, wat is dat nou voor een oplossing
kleine foutjes negeren is de grote fouten over het hoofd zien!

Ik zie nu dat je een maffia script aan de haak hebt geslagen, dat is per definitie al prullenbak werk.

Dus redelijkwijs is dit geen goede code om php te leren.
Pim -
29-06-2010 21:48
Hoezo?
Notices zijn geen fouten, slechts notities. Ze laten zitten is slechts een foute stijl, geen onvergefelijke fout. Ik wil uiteraard iedereen stimuleren een mooie stijl te ontwikkelen, maar dat gaat je niet/nauwelijks/slechts met veel moeite lukken als je baggerscripts wil gaan reviseren. Schrijf het gewoon lekker zelf, zoek een goede, of negeer de foutjes.
t
thomas heijkoop
29-06-2010 22:11
bedankt voor de informatie,
ik heb gewoon alle $getal1 t/m 3 er uit gehaald en het werkt nu,
slotje?
TJVB tvb
30-06-2010 08:42
@Pim, notices zijn meldingen dat iets niet werkt zoals het moet. Dit kan tot problemen leiden. Door die reporting uit te zetten steek je eigenlijk je kop in het zand om te hopen dat goed blijft gaan.

@Thomas, een topic gaat hier meestal niet op slot.

Reageren

Inloggen om te reageren