login  Naam:   Wachtwoord: 
Registreer je!
 Forum

[request] bugcheck

Offline merlin_nl - 03/07/2005 00:00
Avatar van merlin_nlPHP interesse Ik zit met het probleem dat volgens het volgende script altijd $error 1 is..
Echter de checks kloppen allemaal volgens mij, kan iemand mij hier een goede hint geven wat er fout gaat ?

php code - Bekijk de code zonder highlighting - Klap code in 
  1. <link href="../css/lanparty.css" rel="stylesheet" type="text/css">
  2. <TABLE cellpadding=0 cellspacing=10 width="100%" height="500">
  3. <TR>
  4. <TD valign=top>
  5.  
  6. <?PHP
  7. // controle of lid is ingelogd of niet
  8. if ($_COOKIE[email] != "" AND $_COOKIE[memberid] != "" AND $_COOKIE[hash] != "") {
  9. 	// zo ja, dan error
  10. 	?>
  11. 	<table cellpadding=5 cellspacing=0 class="grayBox" >
  12. 	<tr>
  13. 	<td width="50%" valign="top"><h2><img src="/images/bruger.gif" alt="" width="16" height="16" border=0> Fout</h2></td>
  14. 	</tr>
  15. 	<tr>
  16. 	<td width="50%" valign="top" >
  17. 	Je bent al ingelogd, hierdoor kun je je niet inschrijven.<br>
  18. 	<br>
  19. 	<br>
  20. 	Met vriendelijke groet,<br>
  21. 	<br>
  22. 	De LelyLAN Crew
  23. 	</td>
  24. 	</tr>
  25. 	</table>
  26. 	<?PHP
  27. } else {
  28. 	// zo nee, dan verder met controle of formulier is ingevuld.
  29. 	if($_POST['Aanmelden']) {
  30. 		// zo ja, dan verwerken
  31. 		// zet eerst alle errors op 0, zodat de gebruiker netjes met schone lijst begint.
  32. 		$error = "0";
  33. 		$naam = htmlentities($_POST['naam']);
  34. 		$nickname = htmlentities($_POST['gebruikersnaam']);
  35. 		$geslacht = $_POST['geslacht'];
  36. 		$woonplaats = htmlentities($_POST['woonplaats']);
  37. 		$provincie = $_POST['provincie'];
  38. 		$wachtwoord1 = $_POST['wachtwoord1'];
  39. 		$wachtwoord2 = $_POST['wachtwoord2'];
  40. 		$email1 = $_POST['email1'];
  41. 		$email2 = $_POST['email2'];
  42. 		$geb1 = $_POST['dag'];
  43. 		$geb2 = $_POST['maand'];
  44. 		$geb3 = $_POST['jaar'];
  45. 		$geboortedatum = "".$geb1."-".$geb2."-".$geb3."";
  46. 		$nu = date("d-m-Y");
  47. 		$errortext = "";
  48.  
  49. 		if($naam == ""){
  50. 			$errortext .= "Je moet je naam invullen!<br>";
  51. 			$error = "1";
  52. 		}
  53.  
  54. 		if(!preg_match('#^[a-zA-Z0-9-_ ./(/)]{3,100}$#', $naam)){
  55. 			$errortext .= "Je moet een correcte snaam invullen!<br>";
  56. 			$error = "1";
  57. 		}
  58.  
  59. 		if($gebruikersnaam == ""){
  60. 			$errortext .= "Je moet je gebruikersnaam invullen!<br>";
  61. 			$error = "1";
  62. 		}
  63.  
  64. 		if (trim($gebruikersnaam)=='admin' || trim($gebruikersnaam)=='crew' || trim($gebruikersnaam)=='webmaster' || trim($gebruikersnaam)=='lelylan'){
  65. 			$errortext .= "Deze gebruikersnaam is beschermd!<br>";
  66. 			$error = "1";
  67. 		}
  68.  
  69.         $AanmeldingUsernameQuery = mysql_query ("SELECT * FROM gebruikers WHERE nickname='$gebruikersnaam'") or die(mysql_error());
  70.         $AanmeldingUsernameExist = mysql_num_rows($AanmeldingUsernameQuery);                                                            // Returns 0 if not yet existing
  71.  
  72.         if ($AanmeldingUsernameExist > 0) {
  73.                 $errortext .= "Deze gebruikersnaam is al in gebruik!<br>";
  74.                 $error = "1";
  75.         }
  76.  
  77. 		if($geb1 == "" || $geb2 == "" || $geb3 == ""){
  78. 			$errortext .= " Je moet je geboortedatum invullen!<br>";
  79. 			$error = "1";
  80. 		}
  81.  
  82. 		if($woonplaats == ""){
  83. 			$errortext .= " Je moet je woonplaats invullen!<br>";
  84. 			$error = "1";
  85. 		}
  86.  
  87. 		if($provincie == ""){
  88. 			$errortext .= " Je moet je provincie invullen!<br>";
  89. 			$error = "1";
  90. 		}
  91.  
  92. 		if($geslacht == ""){
  93. 			$errortext .= " Je moet je geslacht kiezen!<br>";
  94. 			$error = "1";
  95. 		}
  96.  
  97. 		if($email1 == ""){
  98. 			$errortext .= " Je moet de eerste keer je email adres invullen<br>";
  99. 			$error = "1";
  100. 		}
  101.  
  102. 		if($email2 == ""){
  103. 			$errortext .= " Je moet ook de tweede keer je email adres invullen<br>";
  104. 			$error = "1";
  105. 		}
  106.  
  107. 		if($email1 != $email2){
  108. 			$errortext .= " Je moet wel beide keren je email adres invullen<br>";
  109. 			$error = "1";
  110. 		}
  111.  
  112. 		if(!control_email("$email1")){
  113. 			$errortext .= " Je moet wel een echt emailadres invullen.. bullshit als vraag maar doen we hier niet aan.<br>";
  114. 			$error = "1";
  115. 		}
  116.  
  117.         $email1 = strtolower($email1);
  118.         $EmailQuery = mysql_query("SELECT * FROM gebruikers WHERE email='$email1'") or die(mysql_error());
  119.         $EmailExist = mysql_num_rows($EmailQuery);                                                                              // Returns 0 if not yet existing
  120.  
  121.         if ($EmailExist>0) {
  122.                 $errortext .= " Dit emailadres is al in gebruik!<br>";
  123.                 $error = "1";
  124.         }
  125.  
  126. 		if($wachtwoord1 == ""){
  127. 			$errortext .= " Je moet je wachwoord (1) invullen!<br>";
  128. 			$error = "1";
  129. 		}
  130.  
  131. 		if($wachtwoord2 == ""){
  132. 			$errortext .= " Je moet ook je wachtwoord (2) invullen!<br>";
  133. 			$error = "1";
  134. 		}
  135.  
  136. 		if($wachtwoord1 != $wachtwoord2){
  137. 			$errortext .= " Je wachtwoorden zijn NIET gelijk!<br>";
  138. 			$error = "1";
  139. 		}
  140.  
  141.  
  142.  
  143. 		if($error = '1'){
  144. 			?>
  145. 			<table cellpadding=5 cellspacing=0 class="grayBox" >
  146. 			<tr>
  147. 			<td width="50%" valign="top"><h2><img src="/images/bruger.gif" alt="" width="16" height="16" border=0> Fout</h2></td>
  148. 			</tr>
  149. 			<tr>
  150. 			<td width="50%" valign="top" >
  151. 			De volgende fouten zijn opgetreden:<br>
  152. 			<br>
  153. 			<?PHP echo "$errortext"; ?>
  154. 			<br>
  155. 			<br>
  156. 			Met vriendelijke groet,<br>
  157. 			<br>
  158. 			De LelyLAN Crew
  159. 			</td>
  160. 			</tr>
  161. 			</table>
  162. 			<?PHP
  163. 		} else {
  164. 			$wachtwoordcoded = md5($wachtwoord1);
  165.             $email = $email1;
  166.  
  167. 			mysql_query("INSERT INTO gebruikers (id, naam, wachtwoord, nickname, clan, status, quote, email, actief, act_code, regdate, paydate, woonplaats, provincie, geboortedatum, geslacht) VALUES ('', '" . $naam . "', '" . $wachtwoordcoded . "', '" . $gebruikersnaam . "', '', 'site', '', '" . $email . "', '', '', '" .$nu. "',  '', '". $woonplaats ."', '". $provincie ."', '". $geboortedatum ."', '". $geslacht ."')") OR die(mysql_error());
  168.  
  169. mail("$email", "LelyLAN aanmelding op de website", "Beste $naam,
  170.  
  171. Je hebt je aangemeld op www.lelylan.nl.
  172.  
  173. Je gebruikersnaam is : $gebruikersnaam
  174. Je emailadres is : $email
  175. Je wachtwoord is : $wachtwoord1
  176.  
  177. Met vriendelijke groet,
  178. De LelyLAN Crew
  179. ", "From: info@lelylan.nl");
  180.  
  181. 		?>
  182. 			<table cellpadding=5 cellspacing=0 class="grayBox" >
  183. 			<tr>
  184. 			<td width="50%" valign="top"><h2><img src="/images/bruger.gif" alt="" width="16" height="16" border=0> Gelukt</h2></td>
  185. 			</tr>
  186. 			<tr>
  187. 			<td width="50%" valign="top" >
  188. 			De aanmelding is voltooid.<br>
  189. 			<br>
  190. 			Je kan nu inloggen op de site om je aan te melden voor de LAN.
  191. 			<br>
  192. 			<br>
  193. 			Met vriendelijke groet,<br>
  194. 			<br>
  195. 			De LelyLAN Crew
  196. 			</td>
  197. 			</tr>
  198. 			</table>
  199. 		<?PHP
  200. 		}
  201. 	} else {
  202. 		// zo nee dan formulier
  203. 		?>
  204. 		<table cellpadding=5 cellspacing=0 class="grayBox" >
  205. 		<tr>
  206. 		<td width="50%" valign="top"><h2><img src="/images/bruger.gif" alt="" width="16" height="16" border=0> Aanmelden</h2></td>
  207. 		</tr>
  208. 		<tr>
  209. 		<td width="50%" valign="top" >
  210. 		<!-- START FORMULIER -->
  211. 		<form action="" method="post" enctype="multipart/form-data" name="aanmelden">
  212.         <table width="450" border="0" align="center" cellpadding="3" cellspacing="0" class="kader">
  213. 	    <tr class="rij1">
  214. 	    <td width="125">Naam:</td>
  215. 	    <td> <input name="naam" type="text" id="naam" onFocus="this.style.backgroundColor='#EFEFEF'" onBlur="this.style.backgroundColor='#F7F7F7'" size="40"></td>
  216. 	    </tr>
  217. 	    <tr class="rij2">
  218. 	    <td width="125">Gebruikersnaam:</td>
  219.         <td><input name="gebruikersnaam" type="text" id="gebruikersnaam" onFocus="this.style.backgroundColor='#EFEFEF'" onBlur="this.style.backgroundColor='#F7F7F7'" size="40"></td>
  220. 	    </tr>
  221. 	    <tr class="rij1">
  222. 		<td>&nbsp;</td>
  223. 		<td>&nbsp;</td>
  224. 		</tr>
  225. 		<tr class="rij1">
  226. 		<td width="125">Geslacht:</td>
  227. 		<td>
  228.         <select name="geslacht" id="geslacht" onfocus="this.style.backgroundColor='#EFEFEF'" onblur="this.style.backgroundColor='#F7F7F7'">
  229.     	<option value="Man">Man</option>
  230.     	<option value="Vrouw">Vrouw</option>
  231.     	</select>
  232. 		</td>
  233.     	</tr>
  234.     	<tr class="rij2">
  235.     	<td>Geboortedatum:</td>
  236.     	<td>
  237.     	<SELECT name="dag" id="dag" onfocus="this.style.backgroundColor='#EFEFEF'" onblur="this.style.backgroundColor='#F7F7F7'">
  238.     	<OPTION value=01>01</OPTION>
  239. 		<OPTION value=02>02</OPTION>
  240. 		<OPTION value=03>03</OPTION>
  241. 		<OPTION value=04>04</OPTION>
  242. 		<OPTION value=05>05</OPTION>
  243. 		<OPTION value=06>06</OPTION>
  244. 		<OPTION value=07>07</OPTION>
  245. 		<OPTION value=08>08</OPTION>
  246.         <OPTION value=09>09</OPTION>
  247.         <OPTION value=10>10</OPTION>
  248.         <OPTION value=11>11</OPTION>
  249.         <OPTION value=12>12</OPTION>
  250.         <OPTION value=13>13</OPTION>
  251.         <OPTION value=14>14</OPTION>
  252.         <OPTION value=15>15</OPTION>
  253.         <OPTION value=16>16</OPTION>
  254.         <OPTION value=17>17</OPTION>
  255.         <OPTION value=18>18</OPTION>
  256.         <OPTION value=19>19</OPTION>
  257.         <OPTION value=20>20</OPTION>
  258.         <OPTION value=21>21</OPTION>
  259.         <OPTION value=22>22</OPTION>
  260.         <OPTION value=23>23</OPTION>
  261.         <OPTION value=24>24</OPTION>
  262.         <OPTION value=25>25</OPTION>
  263.         <OPTION value=26>26</OPTION>
  264.         <OPTION value=27>27</OPTION>
  265.         <OPTION value=28>28</OPTION>
  266.         <OPTION value=29>29</OPTION>
  267.         <OPTION value=30>30</OPTION>
  268.         <OPTION value=31>31</OPTION>
  269.     	</SELECT>
  270.     	<SELECT name="maand" id="maand" onfocus="this.style.backgroundColor='#EFEFEF'" onblur="this.style.backgroundColor='#F7F7F7'">
  271.         <OPTION value=01>Januari</OPTION>
  272.         <OPTION value=02>Februari</OPTION>
  273.         <OPTION value=03>Maart</OPTION>
  274.         <OPTION value=04>April</OPTION>
  275.         <OPTION value=05>Mei</OPTION>
  276.         <OPTION value=06>Juni</OPTION>
  277.         <OPTION value=07>Juli</OPTION>
  278.         <OPTION value=08>Augustus</OPTION>
  279.         <OPTION value=09>September</OPTION>
  280.         <OPTION value=10>October</OPTION>
  281.         <OPTION value=11>November</OPTION>
  282.         <OPTION value=12>December</OPTION>
  283.     	</SELECT>
  284.         <SELECT name="jaar" id="jaar"onfocus="this.style.backgroundColor='#EFEFEF'" onblur="this.style.backgroundColor='#F7F7F7'">
  285.         <OPTION value=1950>1950</OPTION>
  286.         <OPTION value=1951>1951</OPTION>
  287.         <OPTION value=1952>1952</OPTION>
  288.         <OPTION value=1953>1953</OPTION>
  289.         <OPTION value=1954>1954</OPTION>
  290.         <OPTION value=1955>1955</OPTION>
  291.         <OPTION value=1956>1956</OPTION>
  292.         <OPTION value=1957>1957</OPTION>
  293.         <OPTION value=1958>1958</OPTION>
  294.         <OPTION value=1959>1959</OPTION>
  295.         <OPTION value=1960>1960</OPTION>
  296.         <OPTION value=1961>1961</OPTION>
  297.         <OPTION value=1962>1962</OPTION>
  298.         <OPTION value=1963>1963</OPTION>
  299.         <OPTION value=1964>1964</OPTION>
  300.         <OPTION value=1965>1965</OPTION>
  301.         <OPTION value=1966>1966</OPTION>
  302.         <OPTION value=1967>1967</OPTION>
  303.         <OPTION value=1968>1968</OPTION>
  304.         <OPTION value=1969>1969</OPTION>
  305.         <OPTION value=1970>1970</OPTION>
  306.         <OPTION value=1971>1971</OPTION>
  307.         <OPTION value=1972>1972</OPTION>
  308.         <OPTION value=1973>1973</OPTION>
  309.         <OPTION value=1974>1974</OPTION>
  310.         <OPTION value=1975>1975</OPTION>
  311.         <OPTION value=1976>1976</OPTION>
  312.         <OPTION value=1977>1977</OPTION>
  313.         <OPTION value=1978>1978</OPTION>
  314.         <OPTION value=1979>1979</OPTION>
  315.         <OPTION value=1980>1980</OPTION>
  316.         <OPTION value=1981>1981</OPTION>
  317.         <OPTION value=1982>1982</OPTION>
  318.         <OPTION value=1983>1983</OPTION>
  319.         <OPTION value=1984>1984</OPTION>
  320.         <OPTION value=1985>1985</OPTION>
  321.         <OPTION value=1986>1986</OPTION>
  322.         <OPTION value=1987>1987</OPTION>
  323.         <OPTION value=1988>1988</OPTION>
  324.         <OPTION value=1989>1989</OPTION>
  325.         <OPTION value=1990>1990</OPTION>
  326.         <OPTION value=1991>1991</OPTION>
  327.         <OPTION value=1992>1992</OPTION>
  328.         <OPTION value=1993>1993</OPTION>
  329.     	</SELECT>
  330.     	</td>
  331.     	</tr>
  332.     	<tr class="rij2">
  333.     	<td>Woonplaats:</td>
  334.     	<td><input name="woonplaats" type="text" id="woonplaats" onFocus="this.style.backgroundColor='#EFEFEF'" onBlur="this.style.backgroundColor='#F7F7F7'" size="40"></td>
  335.     	</tr>
  336.     	<tr class="rij1">
  337.     	<td>Provincie:</td>
  338.     	<td>
  339.     	<select name="provincie" id="provincie" onFocus="this.style.backgroundColor='#EFEFEF'" onBlur="this.style.backgroundColor='#F7F7F7'">
  340.     	<option value="Flevoland">Flevoland</option>
  341.         <option value="Utrecht">Utrecht</option>
  342.         <option value="NoordHolland">NoordHolland</option>
  343.         <option value="ZuidHolland">ZuidHolland</option>
  344.         <option value="Friesland">Friesland</option>
  345.         <option value="Groningen">Groningen</option>
  346.         <option value="Drenthe">Drenthe</option>
  347.         <option value="Overijsel">Overijsel</option>
  348.         <option value="Gelderland">Gelderland</option>
  349.         <option value="NoordBrabant">NoordBrabant</option>
  350.         <option value="Zeeland">Zeeland</option>
  351.         <option value="Limburg">Limburg</option>
  352.          <option value="Anders">Anders</option>
  353.     	</SELECT>
  354.         </td>
  355.         </tr>
  356.     	<tr class="rij1">
  357.       	<td>&nbsp;</td>
  358.       	<td>&nbsp;</td>
  359.     	</tr>
  360.     	<tr class="rij1">
  361.     	<td width="125">E-mail adres:</td>
  362.     	<td><input name="email1" type="text" id="email1" onFocus="this.style.backgroundColor='#EFEFEF'" onBlur="this.style.backgroundColor='#F7F7F7'" size="40"></td>
  363.     	</tr>
  364.     	<tr class="rij2">
  365.     	<td width="125">E-mail adres: (2)</td>
  366.     	<td><input name="email2" type="text" id="email2" onFocus="this.style.backgroundColor='#EFEFEF'" onBlur="this.style.backgroundColor='#F7F7F7'" size="40"></td>
  367.     	</tr>
  368.     	<tr class="rij1">
  369.     	<td width="125">Wachtwoord:</td>
  370.     	<td><input name="wachtwoord1" type="text" id="wachtwoord1" onFocus="this.style.backgroundColor='#EFEFEF'" onBlur="this.style.backgroundColor='#F7F7F7'" size="40"></td>
  371.         </tr>
  372.         <tr class="rij2">
  373.         <td width="125">Wachtwoord: (2)</td>
  374.         <td><input name="wachtwoord2" type="text" id="wachtwoord2" onFocus="this.style.backgroundColor='#EFEFEF'" onBlur="this.style.backgroundColor='#F7F7F7'" size="40"></td>
  375.         </tr>
  376.         </table>
  377.         <br>
  378.         <table width="450" border="0" align="center" cellpadding="3" cellspacing="0" class="kader">
  379.         <tr class="rij1">
  380.         <td width="30"><input type="checkbox" name="regels" value="ja"></td>
  381.         <td>Ja, ik ken en ga akkoord met de <a href="?page=info/regels">regels </a></td>
  382.         </tr>
  383.         <tr class="rij2">
  384.         <td colspan="2"><div align="center">
  385.         <input type="submit" name="Aanmelden" value="Aanmelden">
  386.         <input type="reset" name="reset" value="Opnieuw beginnen">
  387.         </div></td>
  388.         </tr>
  389.         </table>
  390.         </form>
  391.  
  392.  
  393. 		<!-- EINDE FORMULIER -->
  394. 		</td>
  395. 		</tr>
  396. 		</table>
  397. 		<?PHP
  398. 	}	
  399. }	
  400. ?>
  401. </TD>
  402. </TR>
  403. </TABLE>

2 antwoorden

Gesponsorde links
Offline MothZone - 03/07/2005 00:19 (laatste wijziging 03/07/2005 00:36)
Avatar van MothZone PHP ver gevorderde :p:p normaal 

php code - Bekijk de code zonder highlighting - Klap code in 
  1. if($error = '1'){

moet dit zijn:
php code - Bekijk de code zonder highlighting - Klap code in 
  1. if($error == '1'){

(lijn 143 hier)

trouwens, een paar dingen kunnen mss ook errors veroorzaken, maar die niet getoond worden wegens de instellingen vd host.

dit doe je vb vaak:
php code - Bekijk de code zonder highlighting - Klap code in 
  1. if(!control_email("$email1")){


dus een var tussen "" zetten
conclusie:
Slecht: echo("$var is een var");
Goed: echo($var." is een var");

en dit bij alle functies.

ook in het begin doe je dit:
php code - Bekijk de code zonder highlighting - Klap code in 
  1. if ($_COOKIE[email] != "" AND ...

Hierbij gebruik je geen '', dus dat word hier $_COOKIE['email']

en jajah, zag nog iets 
php code - Bekijk de code zonder highlighting - Klap code in 
  1. if($_POST['Aanmelden']) {

kan je beter vervangen door
php code - Bekijk de code zonder highlighting - Klap code in 
  1. if(isset($_POST['Aanmelden'])) {


wat ik ook zag:
php code - Bekijk de code zonder highlighting - Klap code in 
  1. $geboortedatum = "".$geb1."-".$geb2."-".$geb3."";

waarom doe je dat nu?
dit geeft hetzelfde resultaat, is korter, en correcter:
php code - Bekijk de code zonder highlighting - Klap code in 
  1. $geboortedatum = $geb1."-".$geb2."-".$geb3;


en dan dit ook nog
php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?PHP echo "$errortext"; ?>

is sneller zo:
php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?=$errortext?>


en wees gerust, het is niet alleen in de scripting dat fouten staan 
php code - Bekijk de code zonder highlighting - Klap code in 
  1. $errortext .= " Je moet je wachwoord (1) invullen!<br>";

Zeker dat het niet een wachtwoord is?

voor de rest zag ik (toen ik het vluchtig bekeek, dus niet zo grondig) geen fouten^^ 

trouwens, wees gerust, heb al scripts gezien waar ik nòg meer in verbeterde 
Offline merlin_nl - 03/07/2005 00:52
Avatar van merlin_nl PHP interesse hahahahaha txt 
ik moest SNEL wat maken voor een komende lan..
tja als ze nou eerder hadden gezegd dat het zo kortdag was he 
Gesponsorde links
Dit onderwerp is gesloten.
Actieve forumberichten
© 2002-2025 Sitemasters.be - Regels - Laadtijd: 0.304s