login  Naam:   Wachtwoord: 
Registreer je!
 Forum

lijn 286

Offline stijnhau - 08/05/2005 09:30 (laatste wijziging 08/05/2005 09:50)
Avatar van stijnhauOnbekend er zit een fouy in lin 286 maar weet ni wat.
lijn 5 is 286
php code - Bekijk de code zonder highlighting - Klap code in 
  1. else if($data) {
  2.     if($data->activated == 0)
  3.       print "  <tr><td class=\"mainTxt\">Je account is nog niet geactiveerd</td></tr>\n";
  4.     else{
  5.       print "  <tr><td class=\"subTitle\"><b>Login</b></td></tr>\n  <tr><td class=\"mainTxt\">Je bent nu ingelogd.<a href="http://www.members.lycos.nl/webscripters/home2.php">klik hier om verder te gaan</a></td></tr>\n";
  6.   }
  7.   else {
  8.  
  9.     print "  <tr><td class=\"subTitle\"><b>Login</b></td></tr>\n";
  10.     if(isset($_POST['naam'],$_POST['wachtwoord']))
  11.       print "  <tr><td class=\"mainTxt\">Verkeerde login of wachtwoord</td></tr>\n";
  12.  
  13.     print <<<ENDHTML


Admin edit:
Lees de regels, codefragmenten, geen hele scripts. Als het bij lijn 286 is, post alleen dan dat fragment.

14 antwoorden

Gesponsorde links
Offline Martijn1989 - 08/05/2005 09:34
Avatar van Martijn1989 PHP ver gevorderde lijn 268 :

php code - Bekijk de code zonder highlighting - Klap code in 
  1. print "  <tr><td class=\"subTitle\"><b>Login</b></td></tr>\n  <tr><td class=\"mainTxt\">Je bent nu ingelogd.<a href="www.members.lycos.nl/webscripters/home2.php"> klik hier om verder te gaan</a></td></tr>\n";


Probeer dit eens 
Offline stijnhau - 08/05/2005 09:39
Avatar van stijnhau Onbekend nee het werkt ni hoor.
Offline gameplazaa - 08/05/2005 09:46 (laatste wijziging 08/05/2005 09:49)
Avatar van gameplazaa Lid
php code - Bekijk de code zonder highlighting - Klap code in 
  1. print "  <tr><td class=\"subTitle\"><b>Login</b></td></tr>\n  <tr><td class=\"mainTxt\">Je bent nu ingelogd.<a href=\"http://www.members.lycos.nl/webscripters/home2.php\"> klik hier om verder te gaan</a></td></tr>\n";


backslashes bij de a tag vergeten..
Offline stijnhau - 08/05/2005 09:49 (laatste wijziging 08/05/2005 09:51)
Avatar van stijnhau Onbekend is nu in 288 hier dus lijn 7
Offline gameplazaa - 08/05/2005 10:23 (laatste wijziging 08/05/2005 11:04)
Avatar van gameplazaa Lid Je moest de else nog afsluiten..

php code - Bekijk de code zonder highlighting - Klap code in 
  1. else if($data) {
  2.     if($data->activated == 0)
  3.       print "  <tr><td class=\"mainTxt\">Je account is nog niet geactiveerd</td></tr>\n";
  4.     else{
  5.       print "  <tr><td class=\"subTitle\"><b>Login</b></td></tr>\n  <tr><td class=\"mainTxt\">Je bent nu ingelogd.<a href="http://www.members.lycos.nl/webscripters/home2.php">klik hier om verder te gaan</a></td></tr>\n";
  6.     }
  7.   }
  8.   else {


@ Simeken: Tis van een criminals.. geen wonder dus dat het niet netjes is ;)
Offline Simeken - 08/05/2005 10:39 (laatste wijziging 08/05/2005 11:08)
Avatar van Simeken HTML beginner
php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?
  2. elseif($data)
  3. {
  4.     if($data->activated == 0)
  5.     {
  6.       print "  <tr><td class=\"mainTxt\">Je account is nog niet geactiveerd</td></tr>\n";
  7.     }
  8.     else
  9.     {
  10.       print "  <tr>
  11. 		<td class=\"subTitle\"><b>Login</b>
  12. 		</td>
  13. 	       </tr>
  14. 	       \n
  15. 	       <tr>
  16. 		<td class=\"mainTxt\">Je bent nu ingelogd.
  17. 			<a href=\"http://www.members.lycos.nl/webscripters/home2.php\">klik hier om verder te gaan</a>
  18. 	        </td>
  19. 	       </tr>\n";
  20.     }
  21.  
  22. else
  23. {
  24.  
  25.     print "  <tr>
  26. 		<td class=\"subTitle\"><b>Login</b>
  27. 		</td>
  28. 	     </tr>\n";
  29.  
  30.     		if(isset($_POST['naam'],$_POST['wachtwoord']))
  31. 		{
  32.       		print "  <tr><td class=\"mainTxt\">Verkeerde login of wachtwoord</td></tr>\n";
  33. 		}
  34. }
  35.     print <<<ENDHTML
  36. ?>

mss kan je ook iets netter scripten
EDIT:
waarom plaats jij \n na een </tr>?? als je een nieuw <tr> opent gaat hij toch auromatisch naar een nieuwe lijn...
EDIT2:
idd, gameplazaa, je hebt gelijk, hoe kan je daar nou aan uit, het staat allemaal op elkaar, geen wonder dat je de fout niet ziet
Offline Scott - 08/05/2005 11:08
Avatar van Scott Gouden medaille

PHP ver gevorderde		 Nee hoor... Als je alle HTML-code zou echoën zonder gebruik te maken van nieuwe regels zou er in de bron 1 regel komen met alle html-code
Offline Shifty - 08/05/2005 11:45 (laatste wijziging 08/05/2005 11:48)
Avatar van Shifty Onbekend Waarom trekt niemand hier zich iets van het netjes scripten aan? Je moet proberen html zo weinig mogelijk te echoën. Hierdoorkrijg je zulke foutmeldingen waar je aanhalingstekens vergeet te escapen. Dit gebeurt bij jou ook (het laatste stukje in de regel, de link).

Verder vind ik het gebruik van \n ook vreemd. Je gaat het nu allemaal een beetje door elkaar gooien. Die kan je dus gewoon weglaten. Misschien moet je eerst even kijken hoe tabellen nu precies werken, want volgens mij hem je dat ook nog niet eens onder de knie.

Criminalsscripts zijn inderdaad niet zo goed gescript. Zelf zou ik een eigen maken ipv. het bestaande script te nemen, want die zit nog vol met oude functies.

En waarom doe je
php code - Bekijk de code zonder highlighting - Klap code in 
  1. if($date)

Gebruik dan de functie isset(). Daarmee kijk je of een variabele geset is. Wanneer deze niet geset is of 0 bevat, dan retourneert hij false, anders true. Dit is veel veiliger. Verder zou ik het gebruik van mysql_fetch_object afraden, maar mysql_fetch_assoc gebruiken. Dit werkt sneller omdat er gebruik is gemaakt van array's. Je kunt hier ook veel meer mee dan wanneer er een object wordt geretourneerd.
Offline Legolas - 08/05/2005 12:03 (laatste wijziging 08/05/2005 12:04)
Avatar van Legolas Onbekend @Shifty:
dat over mysql_fetch_object, kan het best worden gedaan met $row = $query->fetchRow(); dat is nog sneller dan assoc hoor.
Code:
php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?
  2. else if(isset($data)) {
  3.     if($data->activated == 0)
  4. ?>
  5. <tr><td class="mainTxt">Je account is nog niet geactiveerd</td></tr>
  6. <?
  7.     else{
  8. ?>
  9. <tr><td class="subTitle"><b>Login</b></td></tr><tr><td class="mainTxt">Je bent nu ingelogd.<a href="http://www.members.lycos.nl/webscripters/home2.php">klik hier om verder te gaan</a></td></tr>
  10. <?
  11.   }
  12.   else {
  13.  
  14. ?>
  15. <tr><td class="subTitle"><b>Login</b></td></tr>
  16. <?
  17.     if(isset($_POST['naam'],$_POST['wachtwoord']))
  18. ?>
  19. <tr><td class="mainTxt">Verkeerde login of wachtwoord</td></tr><?
  20.  
  21.     print <<<ENDHTML
  22. ?>

Is beter. en volgens mij heb ik de fout gevonden:
bij de link, <a href="http://www.members.lycos.nl/webscripters/home2.php">, had je geen escapers gebruikt. en in de php gezet, dus gaat ie dit als een var zien, wat het niet is 
mss hiermee opgelost
Offline timmie_loots - 08/05/2005 13:13
Avatar van timmie_loots PHP gevorderde
Citaat:
Verder vind ik het gebruik van n ook vreemd. Je gaat het nu allemaal een beetje door elkaar gooien. Die kan je dus gewoon weglaten.


Nee hoor, is heel handig voor de leesbaarheid van de HTML.

Verder ben ik het ook niet met je eens dat echoën niet netjes is. Als jij het goed doet kan het ook netjes zijn. Het is net hoe je het zelf doet.
Offline nemesiskoen - 08/05/2005 13:20
Avatar van nemesiskoen Gouden medaille

PHP expert		 voor die escapers is er de heredoc-syntax uitgevonden... hebje die escapers ni nodig
Offline Simeken - 08/05/2005 13:31 (laatste wijziging 08/05/2005 13:33)
Avatar van Simeken HTML beginner
php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?
  2. elseif($data)
  3. {
  4. 	if($data->activated == 0)
  5. 	{
  6.        	?>
  7. 		<tr>
  8. 		<td class="mainTxt">
  9. 			Je account is nog niet geactiveerd
  10. 		</td>
  11. 		</tr>
  12. 		<br>
  13. 	<?
  14.     	}
  15. 	else
  16. 	{
  17. 	?>
  18.       		<tr>
  19. 		<td class="subTitle">
  20. 			<b>Login</b>
  21. 		</td>
  22. 		</tr>
  23.  
  24. 		<br> 
  25.  
  26. 		<tr>
  27. 		<td class="mainTxt">
  28. 			Je bent nu ingelogd.
  29. 			<a href="http://www.members.lycos.nl/webscripters/home2.php">
  30. 				klik hier om verder te gaan
  31. 			</a>
  32. 		</td>
  33. 		</tr>
  34.  
  35. 		<br>
  36. 	<?
  37.   	}
  38.  
  39. else 
  40. {
  41. ?>
  42.  
  43. 	<tr>
  44. 	<td class="subTitle">
  45. 		<b>Login</b>
  46. 	</td>
  47. 	</tr>
  48.  
  49. 	<br>
  50.  
  51. <?
  52.  
  53. 	if(isset($_POST['naam'],$_POST['wachtwoord']))
  54. 	{
  55. 	?>
  56.         <tr>
  57. 	<td class="mainTxt">
  58. 		Verkeerde login of wachtwoord
  59. 	</td>
  60. 	</tr>
  61.  
  62. 	<br>
  63. 	<?
  64. 	}
  65.  
  66. print <<<ENDHTML
  67.  
  68. ?>

Zo is het denk ik beter;-)
Offline stijnhau - 08/05/2005 14:37
Avatar van stijnhau Onbekend bdankt legolas werkt nu wel enkel een foutje noch hij geeft tijd waar je naam moet in vullen.
Offline stijnhau - 08/05/2005 15:08
Avatar van stijnhau Onbekend is opgelost bedankt hoor mag gesloten worden.
Gesponsorde links
Dit onderwerp is gesloten.
Actieve forumberichten
© 2002-2024 Sitemasters.be - Regels - Laadtijd: 0.522s