login  Naam:   Wachtwoord: 
Registreer je!
 Forum

ik kan niet uitloggen??

Offline Simeken - 06/05/2005 17:10
Avatar van SimekenHTML beginner ik heb loguit.php aangemaakt maar als ik naar die pagina ga ik toch naar ingelogd.php...

login.php
php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?
  2.  
  4. 	require("config.php");
  5. 	$ipadres=$_SERVER['REMOTE_ADDR'];
  6. 	//query opbouwen
  7. 	$query="SELECT * FROM ipban WHERE ipadres='" .$ipadres ."'";
  8. 	$result=mysql_query($query) or die ("FOUT IN DE QUERY: " . mysql_error());
  9.  
  10. 	//als het ipadres van de bezoeker is gevonden in de ipbanlijst dan
  11. 	//doorsturen naar een zelf te kiezen pagina
  12. 	$aantal = mysql_num_rows($result);
  13. 	if ($aantal >= 1)
  14. 	{
  15.    		header("Location: banned.php");
  16.    		die ();
  17. 	}
  18.  
  19.  
  20. if($submit)
  21. {
  22.  
  23. 	$query = mysql_query("SELECT * FROM members WHERE gebruikersnaam = '$gebruikersnaam' AND wachtwoord = '$wachtwoord' AND geactiveerd = '1'");
  24. 	$result = mysql_num_rows($query);
  25.  
  26. 	if(!$result)
  27. 		{
  28. 			$a = "Gebruikersnaam en/of Wachtwoord is/zijn incorrect. Of uw account is nog niet geactiveerd...<br><br><a href=\"?gebruikersnaam=$gebruikersnaam\">Ga terug</a>";
  29. 		}
  30.  
  31. 	else
  32. 		{
  33. 			while($object = mysql_fetch_assoc($query))
  34. 			{
  35. 				$memberid = $object['memberid'];
  36. 				$logintimes = $object['logintimes'];
  37. 			}
  38.  
  39. 		$ipadres=$_SERVER['REMOTE_ADDR'];
  40. 		$time=time();
  41. 		mysql_query("UPDATE members SET ipadres='$ipadres' WHERE memberid = '$memberid'") or die (mysql_error());
  42.  
  43. 		if($logintimes == 0)
  44. 			{
  45. 				$logintimes2 = $logintimes+1;
  46. 				mysql_query("UPDATE members SET logintimes='$logintimes2', laatstonline=NOW(), sessiebegin='$time', cashgeld='3000', bankgeld='0' WHERE memberid = '$memberid'") or die (mysql_error());
  47. 			}
  48.  
  49. 		elseif($logintimes > 0)
  50. 			{
  51. 				$logintimes2 = $logintimes+1;
  52. 				mysql_query("UPDATE members SET logintimes='$logintimes2', laatstonline=NOW(), sessiebegin='$time'  WHERE memberid = '$memberid'") or die (mysql_error());
  53. 			}
  54.  
  55. 		else
  56. 			{
  57.  
  58. 			}
  59.  
  60. 	$login = 1;
  61. 	session_register("login");
  62. 	session_register("memberid");
  63. 	}
  64. }
  65.  
  66. if($login == 0)
  67. {
  68. ?>
  69.  
  70.  
  71. <form method="POST" action="">
  72. 	<input type="hidden" name="submit" value="1">
  73. 	Gebruikersnaam:<br>
  74. 	<input type="text" name="gebruikersnaam" size="20" value=""><br>
  75. 	Wachtwoord:<br>
  76. 	<input type="password" name="wachtwoord" size="20" value=""><br><br>
  77. 	<input type="submit" name="submit" value="Login"> 
  78. 	<input type="reset" value="Registreren"	onclick="document.location.href=('aanmelden.php')">
  79. </form>
  80. <?
  81. }
  82.  
  83. elseif($login == 1)
  84. {
  85. header("Location: ingelogd.php");
  86. }
  87.  
  88.  
  89. // Verwijder deze comment en de onderstaande regel als je een eigen stylesheet gebruikt...
  90. echo "<link rel=\"stylesheet\" content=\"text/css\" href=\"style.css\">";
  91. ?>


loguit.php
php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?php
  3. $_SESSION = array();
  5. $login = 0; 
  6. if(isset($_COOKIE['gebruikersnaam'])) {
  7.   		setcookie("gebruikersnaam", "", time(), "/");
  8. }
  9. if(isset($_COOKIE['wachtwoord'])) {
  10.   		setcookie("gebruikersnaam", "", time(), "/");
  11. }
  12. header("Location: index2.php");
  13.  
  14. ?>
(dit is overgenomen van een membersysteem)
ingelogd.php
php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?
  3.  
  4. require("config.php");
  5.  
  6. // Verwijder deze comment en de onderstaande regel als je een eigen stylesheet gebruikt...
  7. echo "<link rel=\"stylesheet\" content=\"text/css\" href=\"style.css\">";
  8.  
  9. if(!$login){
  10. ?>
  11.  
  12. Niet ingelogd<br><br>
  13. > <a href="login.php">Log in</a><br>
  14. > <a href="aanmelden.php">Meld aan</a>
  15.  
  16. <?
  17. }
  18. if($login AND session_is_registered('memberid')){
  19. $query = mysql_query("SELECT * FROM members WHERE memberid = '$memberid'");
  20.  
  21. while( $object = mysql_fetch_assoc($query))
  22. {
  23.     $gebruikersnaam = $object['gebruikersnaam'];
  24.     $priveberichten = $object['pm'];
  25. }
  26.  
  27. $query = mysql_query("SELECT * FROM priveberichten WHERE naar = '$memberid'");
  28. $result = mysql_num_rows($query);
  29. ?>
  30.  
  31. Ingelogd als :
  32. <?php
  33. echo "<b>".$gebruikersnaam."</b>";
  34.  
  35. $query = mysql_query("SELECT * FROM members WHERE memberid = '$memberid'");
  36. while( $object = mysql_fetch_assoc($query))
  37. {
  38. 	if ( $object['rang'] == 1 )
  39. 	{
  40. 	      echo "(Speler)";
  41. 	}
  42. 	elseif ( $object['rang'] ==2 )  
  43. 	{
  44. 	      echo "(Forum-admin)";
  45. 	}
  46. 	elseif ( $object['rang'] ==3 )
  47. 	{
  48. 	       echo "(Forum-admin en moderator)";
  49. 	}
  50. 	elseif ( $object['rang'] ==4 )
  51. 	{
  52. 	       echo "(Admin)";
  53. 	}
  54. }	
  55. ?>
  56. <br><br>
  57.  
  58. <SCRIPT TYPE="text/javascript" LANGUAGE="JavaScript">
  59. <!--
  60.  
  61. function FrameUpdate() {
  62. self.location.href = "pm.php";
  63. parent.frame1.location.href = "menu_links.php";
  64. }
  65.  
  66. //-->
  67. </SCRIPT>
  68.  
  69. > <a href="login.php" onclick="FrameUpdate(); return false;">Ga naar het spel</a><br>
  70. <?
  71. }
  72. ?>


index2.php
php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?
  3.  
  4. require("config.php");
  5.  
  6. // Verwijder deze comment en de onderstaande regel als je een eigen stylesheet gebruikt...
  7. echo "<link rel=\"stylesheet\" content=\"text/css\" href=\"style.css\">";
  8.  
  9. if($login AND session_is_registered('memberid'))
  10. {
  11. ?>
  12. <html>
  13. <title>Homepage MKIQ</title>
  14. <frameset cols="30%,*" frameborder="0">
  15. <frame src="menu_links.php" name="frame1" noresize>
  16. 	<frameset rows="17%,83%">	 
  17. 	<frame src="logo.html" name="frame2" scrolling="no" noresize>	
  18. 	<frame src="pm.php" name="frame3" scrolling="yes" noresize>	
  19. 	</frameset>
  20. </frameset>
  21. </html>
  22. <? 
  23. } //sluiten sessie
  24.  
  25. elseif(!$login AND !session_is_registered('memberid'))
  26. {
  27. ?>
  28.  
  29. <html>
  30. <title>Homepage MKIQ</title>
  31. <frameset cols="30%,*" frameborder="0">
  32. <frame src="homepage_menu_links.html" name="frame1" noresize>
  33. 	<frameset rows="17%,83%">	 
  34. 	<frame src="logo.html" name="frame2" scrolling="no" noresize>	
  35. 	<frame src="home.html" name="frame3" scrolling="yes" noresize>	
  36. 	</frameset>
  37. </frameset><noframes></noframes>
  38. </html>
  39.  
  40. <?
  41. }
  42. ?>


Weet iemand hoe het komt dat ik toch niet uitgelogd geraak.
Dan bij voorbaat

(ik weet dat het een lang script is :$ )

6 antwoorden

Gesponsorde links
Offline Legolas - 06/05/2005 17:16
Avatar van Legolas Onbekend
php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?php
  3. $_SESSION = array();
  5. unset($login);
  6. if(isset($_COOKIE['gebruikersnaam'])) {
  7.           setcookie("gebruikersnaam", "", time()-3600, "/");
  8. }
  9. if(isset($_COOKIE['wachtwoord'])) {
  10.           setcookie("gebruikersnaam", "", time()-3600, "/");
  11. }
  12. echo "Succesvol uitgelogd";
  13. ?>

Is volgens mij beter, mss werkt et
Offline Gerard - 07/05/2005 00:47
Avatar van Gerard Ouwe rakker
php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?php
  2. $query="SELECT * FROM ipban WHERE ipadres='" .$ipadres ."'";
  3.     $result=mysql_query($query) or die ("FOUT IN DE QUERY: " . mysql_error());
  4.  
  5.     //als het ipadres van de bezoeker is gevonden in de ipbanlijst dan
  6.     //doorsturen naar een zelf te kiezen pagina
  7.     $aantal = mysql_num_rows($result);
  8.     if ($aantal >= 1)
  9.     {
  10.            header("Location: banned.php");
  11.            die ();
  12.     }
  13. ?>


php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?php
  2.  
  3.     if (@mysql_result(mysql_query("SELECT COUNT(`ipadres`) FROM `ipban` WHERE `ipadres` = '".$ipadres."'"), 0) >= 1) {
  4.  
  5.         header("Location: banned.php");
  6.         exit();
  7.  
  8.     }
  9.  
  10. ?>


Gewoon suggestie 
Offline Legolas - 07/05/2005 08:21 (laatste wijziging 07/05/2005 08:31)
Avatar van Legolas Onbekend @Proximus:
dat maakt toch niks uit, het bannen werkt dacht ik goed.
ff moment volgens mij is er iets fout aant cookie gebruik...
edit: je sette de cookie op time(), en dat is dus niet in het verleden... dus blijft die cookie bestaan:!:
zoals mijn voorbeeld moet het werken, dus time()-3600 bijvoorbeeld.^^
Offline Thomas - 07/05/2005 13:09 (laatste wijziging 07/05/2005 13:10)
Avatar van Thomas Moderator - je (re)set (bij uitloggen) 2x hetzelfde cookie:
setcookie("gebruikersnaam", "", time()-3600, "/");

- session_register() is verouderd
- je maakt geen gebruik van superglobals:
if($submit) ...
if(!$login) { ... (gevaarlijk !!! denk aan je_pagina.php?login=1 !!!)
unset($login); ? unset($_SESSION['login']) misschien? maar je sessie al weggegooid / vernietigd / whatever

Dus tijd om met een flinke bezem je scripts eens door te werken.

Tis een wonder dat je in de eerste plaats in kunt loggen .
Offline Gerard - 07/05/2005 15:56
Avatar van Gerard Ouwe rakker
Citaat:
@Proximus:
dat maakt toch niks uit, het bannen werkt dacht ik goed.


Performance zal met mijn query iets omhoog gaan. Niet veel maar alle kleine beetjes helpen. Vooral aangezien dit, volgens mij dan, weer een opzet is voor een text-based online game waar 100den mensen tegelijk kunnen zitten te spelen.
Offline Shifty - 07/05/2005 16:04
Avatar van Shifty Onbekend Het kan inderdaad beter zijn om eerst wat tutorials door te lezen, want je maakt nog gebruik van verouderde functies (zoals murfy al aangaf). Je moet proberen het nieuwste van het nieuwste te gebruiken, want dat is (vaak) het veiligste).

Probeer ook te letten op eht onnodig aanmaken van variabelen. Ik zie dat je ergens $time = time(); hebt staan. Dit is overbodig en kan dus weg, wat resulteert in een netter en overzichtelijker script. Je zal dit wel met een korrel zout nemen, maar je hebt er echt profijt van. Evenals goede namen verzinnen voor je variabelen. $a zegt voor jou waarschijnlijk genoeg, maar voor een vreemde niet. Wanneer je na een lange tijd weer terugkijkt op het script, zal je het waarschijnlijk ook niet meer snappen, dus zorg ten aller tijde voor de juiste benamingen.

In jouw script zou ik $a vervangen door bijvoorbeeld $sBericht. s staat voor string (rij letters en/of letters + cijfers). Haal variabelen buiten quotes! Een goed voorbeeld kan je in deze tutorial onderaan vinden: http://www.site...mp;id=78#6
Gesponsorde links
Dit onderwerp is gesloten.
Actieve forumberichten
© 2002-2024 Sitemasters.be - Regels - Laadtijd: 0.255s