login  Naam:   Wachtwoord: 
Registreer je!
 Forum

Fout bestel script

Offline Alexjeee - 21/10/2014 13:02
Avatar van AlexjeeeLid Heey,

Ik heb een bestel systeem, ik heb alleen een fout erin zitten. Zodra ik door de eerste stap ben ga ik naar stap 2 (Onderataande code). Maar die laat niks zien me scherm blijft leeg en ik krijg ook geen foutmeldingen terug. Kan iemand voor mij even kijken a.u.b?

php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?php
  3. ini_set('display_errors', 1);
  4.  
  5. if(!defined('ALPHA') || ALPHA !== 1) {
  6. 	die('Deze pagina is niet rechtstreeks aan roepbaar!');
  7. }
  8. ?>
  9. <div id="step">
  10. 	<ul class="step">
  11. 		<li class="this-step-one"></li>
  12. 		<li class="step-two-active"></li>
  13. 		<li class="step-three"></li>
  14. 	</ul>
  15. </div>
  16. <div id="show-step">
  17. 	<div class="show-step-top"></div>
  18.  
  19. 	<div class="p">
  20. 	<?php
  21. 	$Asql = "SELECT id FROM packages WHERE type = 0";
  22. 	$Aquery = $con->query($Asql);
  23.  
  24. 	$packs = array();
  25. 	while($Arow = $Aquery->fetch_assoc()) {
  26. 		$packs[] = $Arow['id'];
  27. 	}
  28.  
  29. 	if(!isset($_GET['package']) || !in_array((int) $_GET['package'], $packs)) {
  30. 		$Bsql = "SELECT id, name, details, img FROM packages WHERE type = 0";
  31. 		$Bquery = $con->query($Bsql);
  32. 		if($Bquery->num_rows >= 1) {
  33. 			while($Brow = $Bquery->fetch_assoc()) {
  34. 				?>
  35. 				<div class="p">
  36. 					<a class="link" href="?p=bestel&part=step2&<?=(isset($_GET['domain'])) ? 'domain='.$_GET['domain'].'&' : ''; ?>package=<?=$Brow['id']; ?>">
  37. 						<img src="/images/pakketten/<?=$Brow['img']; ?>" alt="<?=$Brow['name']; ?>" style="float: left; margin-right: 10px;">
  38. 						<p align="left">Klik hier indien u het <?=$Brow['name']; ?> wilt bestellen.</p>
  39. 					</a>
  40. 					<br />
  41. 					<?=$Brow['details']; ?>
  42. 				</div>
  43. 				<br /><br />
  44. 				<?php
  45. 			}
  46. 		}
  47. 	}elseif(isset($_GET['package']) && in_array((int) $_GET['package'], $packs) && !isset($_GET['confirm'])) {
  48. 		?>
  49. 		Indien de onderstaande gegevens correct zijn kunt u dit onderaan de pagina aangeven. U wordt dan doorgestuurd om uw persoonsgegevens in te vullen.<br />
  50. 		<br />
  51. 		<div style="width: 200px; float: left;"><b>Domein:</b></div>
  52. 		<div style="float: left;">
  53. 			<?=(isset($_GET['domain']) && !empty($_GET['domain']) && preg_match('#(.*?).(.*?)#si', $_GET['domain'])) ? $_GET['domain'] : 'geen'; ?>
  54. 		</div>
  55. 		<div style="clear: left;"></div>
  56. 		<br />
  57.  
  58. 		<div style="width: 200px; float: left;"><b>Pakket:</b></div>
  59. 		<div style="width: 400px; float: left;">
  60. 			<?php
  61. 			$Bsql = "SELECT name, domains, disk_quota, datalim, ftpaccs, price FROM packages WHERE id = ".(int) $_GET['package']." LIMIT 1";
  62. 			$Bquery = $con->query($Bsql);
  63. 			if($Bquery->num_rows == 1) {
  64. 				$Brow = $Bquery->fetch_assoc();
  65. 				echo $Brow['name'].'<br />';
  66. 				echo $Brow['domains'].' '.(((count($Brow['domains'])) == 1) ?  'Domeinnaam' : 'Domeinnamen').'<br />';
  67. 				echo $Brow['disk_quota'].' Ruimte<br />';
  68. 				echo $Brow['datalim'].' Dataverkeer<br />';
  69. 				echo $Brow['ftpaccs'].' FTP Account'.((count($Brow['ftpaccs']) == 1) ? '' : 's').'<br />';
  70. 				echo '&euro; '.$Brow['price'].' Per maand<br />';
  71. 				?>
  72. 				</div>
  73. 				<div style="clear: left;"></div>
  74. 				<br /><br />
  75. 				<a href="?p=bestel&part=step2&<?=(isset($_GET['domain']) && !empty($_GET['domain']) && preg_match('#(.*?).(.*?)#si', $_GET['domain'])) 
  76. 							? 'domain='.$_GET['domain'].'&' 
  77. 							: ''; ?>package=<?=$_GET['package']; ?>&confirm=1" class="a_p">
  78. 					<small><img src="images/bestel/partakkoord.png"></small>
  79. 				</a><br />
  80.  
  81. 				<a href="?p=bestel" class="a_p"><small><img src="images/bestel/partweiger.png"></small></a>
  82. 				<?php
  83. 			}else{
  84. 				echo 'Helaas werd het door u geselecteerde pakket niet gevonden, gelieve een ander pakket te kiezen<br />';
  85. 				echo '<a href="?p=bestel&part=step2&'.((isset($_GET['domain']) && !empty($_GET['domain']) && preg_match('#(.*?).(.*?)#si', $_GET['domain'])) 
  86. 												? '?domain='.$_GET['domain'] 
  87. 												: '').'" class="a_p">Ga terug</a>';
  88. 			}
  89. 			?>
  90.  
  91. 		<?php
  92. 	}elseif(isset($_GET['package']) && in_array((int) $_GET['package'], $packs) && isset($_GET['confirm'])) {
  93. 		if(!defined('DELTA')) {
  94. 			define('DELTA', 1);
  95. 		}
  96.  
  97. 		if($_GET['confirm'] == 1) {
  98. 			include 'includes/nl/bestel/particulier.php';
  99. 		}elseif($_GET['confirm'] == 2) {
  100. 			include 'includes/nl/bestel/particulier.php';
  101. 		}else{
  102. 			include 'includes/nl/bestel/particulier.php';
  103. 		}
  104. 	}
  105. 	?>
  106. 	</div>
  107.  
  108. 	<div class="show-step-footer"></div>
  109.  
  110. 	<!-- //end step -->
  111. 	<div style="clear: both;"></div>
  112. </div>

2 antwoorden

Gesponsorde links
Offline Jointjeff - 21/10/2014 15:17
Avatar van Jointjeff HTML interesse Moet er geen $_GET['confirm'] gedefinieerd worden in de url die je aanklikt?

Verder is het vrij lastig te zien door alle if-statements die gemaakt worden (beetje lastig overzicht te behouden).
Offline Thomas - 21/10/2014 17:45
Avatar van Thomas Moderator Ik zou serieus overwegen om je code anders in te delen, want hier is geen touw aan vast te knopen.

- splits je stappen op, niet in één groot if-elseif-elseif-elseif-elseif-...-else blok
- als je bepaalde informatie over verschillende pagina's (stappen) wilt bijhouden, sla deze dan op in $_SESSION
- gebruik commentaar in je code die op zijn minst omschrijft wat iets zou moeten doen, dit is ook handig voor andere mensen die code moeten lezen of aanpassen en het dient tevens als documentatie
- ampersands in URLs dienen als &amp; geschreven te worden, niet als &
- let heel goed op wanneer je gegevens rechtstreeks verwerkt in (een) SQL(-string) en zorg dat deze DATA niet gebruikt kan worden als SQL

Daarnaast, je hebt nu vijf topics openstaan over vrij uiteenlopende onderwerpen. Ik verzoek je vriendelijk doch dringend eerst bestaande topics af te ronden voordat je nieuwe topics start.

Ook lijkt het mij in je eigen belang dat je jezelf (nog beter) PHP aanleert, temeer omdat je daar je hele webtoko op lijkt te draaien. Dit forum is echter niet jouw persoonlijke supportlijn. Vragen stellen is vrij (mits met mate, en na blijk van eigen inspanning), maar je zult zelf toch echt wat meer uit moeten gaan zoeken.
Gesponsorde links
Je moet ingelogd zijn om een reactie te kunnen posten.
Actieve forumberichten
© 2002-2024 Sitemasters.be - Regels - Laadtijd: 0.216s