login  Naam:   Wachtwoord: 
Registreer je!
 Forum

Fout bij controleren

Offline DeepMafia - 22/04/2005 19:36 (laatste wijziging 22/04/2005 21:51)
Avatar van DeepMafiaNieuw lid Ik heb een script geschreven en nu geeft hij een fout aan. Dit komt doordat ik hem laat controleren of hij wel genoeg energie heeft maar er komt een rare fout. Weet iemand hem?

php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?php $title = "Misdaden"; include("header.php"); ?>


php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?php
  2. if ($action != crime) {
  3. 	print "<tr>";
  4. 	print "<td>";
  5. 	print "<form>";
  6. 	print "De misdaden werken nog niet goed. En als ik merk dat je in de min staat zit je voor een week in de gevangenis!<p>Selecteer welke misdaad je wilt doen:<br>";
  7. 	print "<br>";
  8. 	print "<table>";
  9. 	print "<tr>";
  10. 	print "<td width=200><b>Misdaad</font></b></td>";
  11. 	print "<td width=60><b>Kans</font></b></td>";
  12. 	print "<td width=10><b>Energie</font></b></td>";
  13. 	print "<td width=50><b>Opties</font></b></td>";
  14. 	print "</tr>";
  15. 	print "<tr>";
  16. 	print "<td width=200>Steel snoep van een kind.</td>";
  17. 	print "<td width=60>0%</td>";
  18. 	print "<td width=10>1</td>";
  19. 	print "<td width=50><INPUT TYPE=radio NAME=get_money value=1></td>";
  20. 	print "</tr>";
  21. 	print "<tr>";
  22. 	print "<td width=200>Beroof iemand op straat.</td>";
  23. 	print "<td width=60>0%</td>";
  24. 	print "<td width=10>1</td>";
  25. 	print "<td width=50><INPUT TYPE=radio NAME=get_money value=2></td>";
  26. 	print "</tr>";
  27. 	print "<tr>";
  28. 	print "<td width=200>Beroof een supermarkt.</td>";
  29. 	print "<td width=60>0%</td>";
  30. 	print "<td width=10>1</td>";
  31. 	print "<td width=50><INPUT TYPE=radio NAME=get_money value=3></td>";
  32. 	print "</tr>";
  33. 	print "<tr>";
  34. 	print "<td width=200>Beroof iemand bij een pinautomaat.</td>";
  35. 	print "<td width=60>0%</td>";
  36. 	print "<td width=10>1</td>";
  37. 	print "<td width=50><INPUT TYPE=radio NAME=get_money value=4></td>";
  38. 	print "</tr>";
  39. 	print "<tr>";
  40. 	print "<td width=200>Gijzel een belangrijk zakenman.</td>";
  41. 	print "<td width=60>0%</td>";
  42. 	print "<td width=10>1</td>";
  43. 	print "<td width=50><INPUT TYPE=radio NAME=get_money value=5></td>";
  44. 	print "</tr>";
  45. 	print "</table>";
  46. 	print "<br>";
  47. 	print "<INPUT TYPE=submit value=Start!><P>";
  48. } else {
  49. 	if ($stat[energy] <= .3) {
  50. 		print "Je hebt niet genoeg energie om de te stelen.";
  51. 	} else {
  52. 		$get_money = get_money;
  53. 		}
  54. 		if ($get_money == 1) { 
  55. 			$gain = rand(1,10);
  56. 		print "Het is gelukt!Je hebt het snoep verkocht en het leverde je $gain euro op!";
  57. 		mysql_query("update players set energy=energy-1 where id=$stat[id]");
  58. 		mysql_query("update paying set energy=energy-1 where id=$stat[id]");
  59. 		mysql_query("update players set credits=credits+$gain where id=$stat[id]");
  60. 		mysql_query("update paying set credits=credits+$gain where id=$stat[id]");
  61. 		}
  62. 		if ($get_money == 2) {
  63. 			$gain = rand(50,100);
  64. 		print "Het is gelukt! Het leverde je $gain euro op!";
  65. 		mysql_query("update players set energy=energy-1 where id=$stat[id]");
  66. 		mysql_query("update paying set energy=energy-1 where id=$stat[id]");
  67. 		mysql_query("update players set credits=credits+$gain where id=$stat[id]");
  68. 		mysql_query("update paying set credits=credits+$gain where id=$stat[id]");
  69. 		}
  70. 		if ($get_money == 3) {
  71. 			$gain = rand(100,250);
  72. 		print "Het is gelukt! Het leverde je $gain euro op!";
  73. 		mysql_query("update players set energy=energy-1 where id=$stat[id]");
  74. 		mysql_query("update paying set energy=energy-1 where id=$stat[id]");
  75. 		mysql_query("update players set credits=credits+$gain where id=$stat[id]");
  76. 		mysql_query("update paying set credits=credits+$gain where id=$stat[id]");
  77. 		}
  78. 		if ($get_money == 4) {
  79. 			$gain = rand(150,350);
  80. 		print "Het is gelukt! Het leverde je $gain euro op!";
  81. 		mysql_query("update players set energy=energy-1 where id=$stat[id]");
  82. 		mysql_query("update paying set energy=energy-1 where id=$stat[id]");
  83. 		mysql_query("update players set credits=credits+$gain where id=$stat[id]");
  84. 		mysql_query("update paying set credits=credits+$gain where id=$stat[id]");
  85.  
  86. 		}
  87. 		if ($get_money == 5) {
  88. 			$gain = rand(450,800);
  89. 		print "Het is gelukt! Het leverde je $gain euro op!";
  90. 		mysql_query("update players set energy=energy-1 where id=$stat[id]");
  91. 		mysql_query("update paying set energy=energy-1 where id=$stat[id]");
  92. 		mysql_query("update players set credits=credits+$gain where id=$stat[id]");
  93. 		mysql_query("update paying set credits=credits+$gain where id=$stat[id]");
  94.  
  95. 			}
  96. 	print "</form>";
  97. 	print "</td>";
  98.  
  99.  
  100.  include("footer.php"); ?>

13 antwoorden

Gesponsorde links
Offline Rens - 22/04/2005 19:42
Avatar van Rens Gouden medaille

Crew algemeen		 Misschien de fout ook even geven?
Is wat makkelijker zoeken voor ons...
Offline DeepMafia - 22/04/2005 19:42 (laatste wijziging 22/04/2005 19:44)
Avatar van DeepMafia Nieuw lid
php code - Bekijk de code zonder highlighting - Klap code in 
  1. if ($stat[energy] <= .3) {
  2.         print "Je hebt niet genoeg energie om de te stelen.";
Offline nemesiskoen - 22/04/2005 19:44 (laatste wijziging 22/04/2005 19:45)
Avatar van nemesiskoen Gouden medaille

PHP expert
php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?php 
  2.         $get_money = get_money;
  3. ?>


php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?php
  2.  
  3.         $get_money = htmlspecialchars( addslashes( $_GET['money'] ) );
  4.  
  5. ?>


maar ik raad je wel aan met post te gebruiken en je form een naam en action te geven


edit: waar is dat puntje voor nodig?
Offline DeepMafia - 22/04/2005 19:47
Avatar van DeepMafia Nieuw lid Betekend 0.3

In andere scripts werkte het wel dus waarom hier niet?
Offline Rens - 22/04/2005 19:50 (laatste wijziging 22/04/2005 19:50)
Avatar van Rens Gouden medaille

Crew algemeen
Citaat:
Misschien de fout ook even geven?
Is wat makkelijker zoeken voor ons...


Als je geen error krijgt, zet dan eens ERROR_REPORTING(E_ALL); bovenaan je script...
Offline DeepMafia - 22/04/2005 19:53
Avatar van DeepMafia Nieuw lid In dat stukje dat jij had verbeterd heb ik geprobeert en hij gaf dezelfde fout aan. Op dat moment dat ik het stukje weg laat dat hij controleert doet hij het. Zonder geeft hij aan dat de fout in de laatste regel zit met openen van een navigatie bar.

Iemand anders een idee?
Offline Rens - 22/04/2005 19:56 (laatste wijziging 22/04/2005 19:56)
Avatar van Rens Gouden medaille

Crew algemeen
php code - Bekijk de code zonder highlighting - Klap code in 
  1. if ($stat[energy] <= .3) {
  2.         print "Je hebt niet genoeg energie om de te stelen.";
  3.     } else {
  4.         $get_money = get_money;
  5.         }


$stat[energy] ==> $stat['energy']
get_money ==> waar komt dit vandaan?

edit; Kopieer eens gewoon alle fouten die je krijgt, want aan omschrijvingen van fouten hebben we niet veel:s
Offline DeepMafia - 22/04/2005 19:56
Avatar van DeepMafia Nieuw lid Ik heb het er in gezet en eens gekeken.

Hij zegt nu dat de fouten zitten in:

if ($get_money == 1) {

if ($get_money == 2) {

if ($get_money == 3) {

if ($get_money == 4) {

if ($get_money == 5) {

Nog zo iets. Ik snap dit neit en daarom vraag ik om hulp. Ziet iemand hier een fout in? Ik niet dit is gewoon een "alsje".
Offline DeepMafia - 22/04/2005 20:02 (laatste wijziging 22/04/2005 20:04)
Avatar van DeepMafia Nieuw lid [img]http://deepmafia.funpic.org/naamloos.bmp[/img]

edit: klik maar even op de link dan zie je de fouten.
edit: Even gedult hij geeft het verkeerde plaatje aan.

edit: http://img228.echo.cx/my.php?image=naamloos4dg.png

dit is de goede
Offline Rens - 22/04/2005 20:15 (laatste wijziging 22/04/2005 20:16)
Avatar van Rens Gouden medaille

Crew algemeen		 Ik kon het niet aanzien (je code) dus heb ik het even aangepast.

php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?PHP
  3. if ($_SERVER['REQUEST_METHOD'] != "POST")
  4. {
  5. ?>
  6. <tr>
  7. <td>
  8. <form method="post" action="<?=$_SERVER['PHP_SELF'];?>">
  9. De misdaden werken nog niet goed. En als ik merk dat je in de min staat zit je voor een week in de gevangenis!<p>Selecteer welke misdaad je wilt doen:<br>
  10. <br>
  11. <table>
  12. <tr>
  13. <td width=200><b>Misdaad</font></b></td>
  14. <td width=60><b>Kans</font></b></td>
  15. <td width=10><b>Energie</font></b></td>
  16. <td width=50><b>Opties</font></b></td>
  17. </tr>
  18. <tr>
  19. <td width=200>Steel snoep van een kind.</td>
  20. <td width=60>0%</td>
  21. <td width=10>1</td>
  22. <td width=50><INPUT TYPE=radio NAME=get_money value=1></td>
  23. </tr>
  24. <tr>
  25. <td width=200>Beroof iemand op straat.</td>
  26. <td width=60>0%</td>
  27. <td width=10>1</td>
  28. <td width=50><INPUT TYPE=radio NAME=get_money value=2></td>
  29. </tr>
  30. <tr>
  31. <td width=200>Beroof een supermarkt.</td>
  32. <td width=60>0%</td>
  33. <td width=10>1</td>
  34. <td width=50><INPUT TYPE=radio NAME=get_money value=3></td>
  35. </tr>
  36. <tr>
  37. <td width=200>Beroof iemand bij een pinautomaat.</td>
  38. <td width=60>0%</td>
  39. <td width=10>1</td>
  40. <td width=50><INPUT TYPE=radio NAME=get_money value=4></td>
  41. </tr>
  42. <tr>
  43. <td width=200>Gijzel een belangrijk zakenman.</td>
  44. <td width=60>0%</td>
  45. <td width=10>1</td>
  46. <td width=50><INPUT TYPE=radio NAME=get_money value=5></td>
  47. </tr>
  48. </table>
  49. <br>
  50. <INPUT TYPE=submit value=Start!><P>
  51. </form>
  52. </td>
  53. <?PHP
  54. } else
  55. {
  56. 	if ($stat['energy'] <= .3) 
  57. 	{
  58. 		echo "Je hebt niet genoeg energie om te stelen.";
  59.     } else
  60. 	{
  61. 		if(IsSet($_POST['get_money']))
  62. 		{
  63. 			if ($get_money == 1)
  64. 			{
  65. 				$gain = rand(1,10);
  66. 				echo "Het is gelukt!Je hebt het snoep verkocht en het leverde je ".$gain." euro op!";
  67. 			} else if ($get_money == 2)
  68. 			{
  69. 				$gain = rand(50,100);
  70. 				echo "Het is gelukt! Het leverde je ".$gain." euro op!";
  71. 			} else if ($get_money == 3)
  72. 			{
  73. 				$gain = rand(100,250);
  74. 				echo "Het is gelukt! Het leverde je ".$gain." euro op!";
  75. 			} else if ($get_money == 4) {
  76. 				$gain = rand(150,350);
  77. 				echo "Het is gelukt! Het leverde je ".$gain." euro op!";
  78.  
  79. 			} else if ($get_money == 5)
  80. 			{
  81. 				$gain = rand(450,800);
  82. 				echo "Het is gelukt! Het leverde je ".$gain." euro op!";
  83. 			}
  84. 			if(!mysql_query("update players set energy=energy-1 where id=".$stat['id']))
  85. 			{
  87. 			}
  88. 			if(!mysql_query("update paying set energy=energy-1 where id=".$stat['id']))
  89. 			{
  91. 			}
  92. 			if(!mysql_query("update players set credits=credits+".$gain." where id=".$stat['id']))
  93. 			{
  95. 			}
  96. 			if(!mysql_query("update paying set credits=credits+".$gain." where id=".$stat['id']))
  97. 			{
  99. 			}
  100. 		} else
  101. 		{
  102. 			echo "Niet opgegeven wat je wilt beroven!<BR>";
  103. 		}
  104. 	}
  105. }
  106.  
  107. include("footer.php");
  108. ?>


Probeer het zo nog eens.
Offline xSc - 22/04/2005 21:58 (laatste wijziging 22/04/2005 22:01)
Avatar van xSc Onbekend Hmm,

36 ifjes?

php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?php
  2.  
  3.    if(isSet($_POST['get_money']))
  4.    {
  5.       switch($_POST['get_money'])
  6.       {
  7.          case 1:
  8.             echo 'Het is gelukt! Je hebt het snoep verkocht en het leverde je ' . rand(1, 10) . ' euro op!';
  9.             break;
  10.         case 2:
  11.            echo 'Het is gelukt! Het leverde je ' . rand(50, 100) . ' euro op!';
  12.            break;
  13.       }
  14.    }
  15. ?>


Je kunt uiteraard nog meer case toevoegen. Als else geldt default. Laatstgenoemde wordt echter niet afgesloten met een break.

Verder: gebruik consequent superglobals waar dat verwacht wordt. Als je eerst $_POST['get_money'] gebruikt, moet je later niet zomaar $get_money gebruiken.

Maak niet onnodig variabelen aan. Dingen die je 1 keer gebruikt, hoef je in principe niet in een variabele te stoppen.

Als je een random inhoud maakt, die ook in je database geplaatst wordt, dan heb je wel een variabele nodig natuurlijk, omdat je anders verschillende waarden krijgt.
Offline delta_004 - 23/04/2005 07:57 (laatste wijziging 23/04/2005 10:45)
Avatar van delta_004 Onbekend ub bij if ($get_money == 2) {
moet worden if ($get_money == '2') {
Als het geen variable is dan zet je die erom

Edit door Henri
Bovenstaande reactie is pure onzin. Nummers zet je niet tussen quotes. De rest wel. Voortaan beter lezen!
Offline Rens - 23/04/2005 10:40
Avatar van Rens Gouden medaille

Crew algemeen		 Als het een getal is zet je die er NIET om;-)
Gesponsorde links
Dit onderwerp is gesloten.
Actieve forumberichten
© 2002-2024 Sitemasters.be - Regels - Laadtijd: 0.303s