login  Naam:   Wachtwoord: 
Registreer je!
 Forum

rare bug...

Offline Simon - 08/09/2005 20:31
Avatar van SimonPHP expert ik heb een zeer rare bug
ik werk met een forum waarbij je moet aanmelden bij bepaalde clubs voordat je kan posten
php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?
  2. $gAbo2 = mysql_query("SELECT * FROM abonnementen WHERE userid='".$user."'") or die(mysql_error());
  3. 					while($Abo = mysql_fetch_assoc($gAbo2)) { 						
  4. 						$gTav2 = mysql_query("SELECT * FROM tavernes WHERE t_id != '".$Abo['clubid']."' ORDER BY t_naam ASC") or die(mysql_error());
  5. 					}
  6. 					while($tav = mysql_fetch_assoc($gTav2)) {		
  7. ?>

een stukje waar het fout loopt is hier:
Bij geabonneerde tavernes loopt alles mooi en zoals ik het wil,
maar als je niet geabonneerd bent treed er een rare fout op, het volgende:
er zijn 16 clubs, ik ben bij 15 geabonneerd...
maar ipv van dat er 1 club staat bij niet geabonneerd staan er 15, de verklaring is dat er slechts 1 taverne eruit wordt gehaald hier:
php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?
  2. 						$gTav2 = mysql_query("SELECT * FROM tavernes WHERE t_id != '".$Abo['clubid']."' ORDER BY t_naam ASC") or die(mysql_error());
  3. ?>

de andere 15 worden er niet uigefilterd, hoe komt dit?:(

10 antwoorden

Gesponsorde links
Offline Steven - 08/09/2005 20:36 (laatste wijziging 08/09/2005 20:37)
Avatar van Steven HTML interesse Heel makkelijk, je verandert: niet geabonerd in geaboneerde clubs

dus ipv:

15 niet geabonneerde clubs,
krijg je dit:
15 geaboneerde clubs ;)

[edit]
Ik weet niet hoe die bug komt, maar dit lijkt mij makkelijker ;)
Offline Simon - 08/09/2005 20:47 (laatste wijziging 10/09/2005 10:03)
Avatar van Simon PHP expert dat is dus echt onzin 
ik zou graag mijn volledig script hier posten, maar is 200 regels, mag dat?
als het niet mag dan neem je mijn post maar weg:
php code - Bekijk de code zonder highlighting - Klap code in 
  1. ik heb de foute code gepost :s


ik probeer het nog beter uit te leggen:
bijv:
er zijn 10 clubs.
ik ben geabonneerd bij 9.
er staat dus bij geabonneerde clubs (9) en de juiste clubs,
maar bij nietgeaboneerd staat (1) maar er komen 9 clubs te staan, wat dus niet klopt.
hoe dit komt, als je kijkt naar het stukje in mijn eerste post zie je:

WHERE t_id != ...

dus: er wordt door deze query maar 1 club eruit gehaald, namelijk de eerste club in het rijtje.
nochtans staat deze query in een while loop, maar toch schijnt deze niet te werken.
hoe komt dit? hoe kan ik het verhelpen?

niemand?
Offline Ontani - 09/09/2005 17:56
Avatar van Ontani Gouden medailleGouden medailleGouden medailleGouden medaille

-1		 WHERE t_id <> ...
Offline Simon - 09/09/2005 18:02 (laatste wijziging 09/09/2005 19:35)
Avatar van Simon PHP expert neej, werkt niet

nog iemand een idee?

echt niemand? dat kan toch nie?
Offline Rens - 09/09/2005 20:37
Avatar van Rens Gouden medaille

Crew algemeen		 Over dit stukje:

php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?PHP
  2. $gAbo2 = mysql_query("SELECT * FROM abonnementen WHERE userid='".$user."'") or die(mysql_error());
  3.                     while($Abo = mysql_fetch_assoc($gAbo2)) {                         
  4.                         $gTav2 = mysql_query("SELECT * FROM tavernes WHERE t_id != '".$Abo['clubid']."' ORDER BY t_naam ASC") or die(mysql_error());
  5.                     }
  6.                     while($tav = mysql_fetch_assoc($gTav2)) {        
  7. ?>


Verander eens in dit:

php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?PHP
  2. $gAbo2 = mysql_query("SELECT * FROM abonnementen WHERE userid='".$user."'") or die(mysql_error());
  3. while($Abo = mysql_fetch_assoc($gAbo2))
  4. {                         
  5. 	$gTav2 = mysql_query("SELECT * FROM tavernes WHERE t_id != '".$Abo['clubid']."' ORDER BY t_naam ASC") or die(mysql_error());
  6. 	while($tav = mysql_fetch_assoc($gTav2))
  7. 	{
  8. 		// gegevens weergeven
  9. 	}
  10. }       
  11. ?>


(de verandering van inspringen e.d. is niet nodig, heb ik alleen ff gedaan zodat het voor mij iets makkelijker leesbaar is, en ik je dus makkelijker kan helpen...)
Offline Simon - 09/09/2005 21:43
Avatar van Simon PHP expert dan is het een echte catastrofe, dan krijg ik bij niet geabonneerd een lijst van meer dan 200 clubs, steeds met dezelfde naam 
Offline Rens - 09/09/2005 22:16 (laatste wijziging 09/09/2005 22:25)
Avatar van Rens Gouden medaille

Crew algemeen		 Owjah, klopt, foutje...
Ik zie het nu staan.

In de 2e query staat dit: !=
php code - Bekijk de code zonder highlighting - Klap code in 
  1. $gTav2 = mysql_query("SELECT * FROM tavernes WHERE t_id != '".$Abo['clubid']."' ORDER BY t_naam ASC") or die(mysql_error());


Die moet je ff veranderen in: =
php code - Bekijk de code zonder highlighting - Klap code in 
  1. $gTav2 = mysql_query("SELECT * FROM tavernes WHERE t_id = '".$Abo['clubid']."' ORDER BY t_naam ASC") or die(mysql_error());


edit; Nog iets, je zou misschien ook het aantal tavernes kunnen tellen, en dan:

php code - Bekijk de code zonder highlighting - Klap code in 
  1. $iAantalNietGeaboneerd = $iAantalTavernes-$iAantalGeaboneerd;
Offline Simon - 10/09/2005 10:05
Avatar van Simon PHP expert het spijt me maar ik was zo dom om de foute code te posten 
dit is de juiste, nu zal je ook zien hoe het in elkaar zit Rensjuh
php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?php
  3. require("config.php"); // connectie met database maken en database selecteren
  4. require("functies.php");
  6. ?>
  7. <html>
  8. <head>
  9.   <link rel="stylesheet" href="styletav.css" type="text/css">
  10.   <title>Alle tavernes</title>
  11. </head>
  12. <body bgcolor="#000000">
  13. <?php
  14.  
  15. if(!isset($_SESSION['suser']))
  16. { //dus als er niemand is ingelogd
  17. 	?>
  18. 	<div class="okbox">
  19. 	 	 <div class="oktitle">FOUT</div>
  20. 	  	<div class="noktext">U dient ingelogd te zijn om dit te bekijken.</div>
  21. 	</div>
  22. 	<?php
  23. } 
  24. else 
  25. {
  26.  
  27. $userid = mysql_query("SELECT * FROM users WHERE username = '".$_SESSION['suser']."'") or die(mysql_error());
  28. $id = mysql_fetch_assoc($userid);
  29. $user = $id['id']; 
  30. ?>
  31. <table width="70%" border="0" class="sh" cellpadding="0" cellspacing="0">
  32. 	<tr bgcolor="#333399">
  33. 		<th width="35%" nowrap><b>&nbsp;Naam taverne</b></th>
  34. 		<th width="17%" nowrap><b>&nbsp;Oprichter</b></th>
  35. 		<th width="4%" nowrap><b><center>#L</center></b></th>
  36. 		<th width="4%" nowrap><b>&nbsp;Posts</b></th>
  37. 		<th width="10%" nowrap><b>&nbsp;</b></th>
  38. 	</tr>
  39. 	<?php
  40. 	$gAbo = mysql_query("SELECT * FROM abonnementen WHERE userid='".$user."'") or die(mysql_error());
  41. 	$result = mysql_num_rows($gAbo);
  42. 	if($result >= 1) {
  43. 		echo "&nbsp;Geabonneerde tavernes (".$result.")<br><br>"; 
  44. 		while($abo = mysql_fetch_assoc($gAbo)) {
  45.  
  46. 			$gTav = mysql_query("SELECT * FROM tavernes WHERE t_id = '".$abo['clubid']."' ORDER BY t_naam ASC") or die(mysql_error());
  47.  
  48. 				while($tav = mysql_fetch_assoc($gTav)) {
  49. 				?>
  50. 				<tr bgcolor="#444444">
  51. 					<td width="35%"><b>&nbsp;<a href="tavernes.php?tavid=<?= $tav['t_id']; ?>" target="_blank"><font color="#FFFF00"><?= $tav['t_naam']; ?></font></a></b></td>
  52. 					<td width="17%">&nbsp;<?= afkorten($tav['t_eigenaar'], 15); ?></td>
  53. 					<td width="4%"><center>
  54. 					<?php
  55. 					$gLid = mysql_query("SELECT * FROM abonnementen WHERE clubid='".$tav['t_id']."'") or die(mysql_error());
  57. 					?>
  58. 					</center></td>
  59. 					<td width="4%">&nbsp;
  60. 					<?php
  61. 					$gPost = mysql_query("SELECT * FROM t_topics WHERE taverne_id='".$tav['t_id']."'") or die(mysql_error());
  62. 					while($topic = mysql_fetch_assoc($gPost)) {
  63. 						$gPost2 = mysql_query("SELECT * FROM t_replys WHERE topic_id = '".$topic['id']."'") or die(mysql_error());
  64. 					}
  65. 					$gMax = mysql_query("SELECT * FROM tavernes WHERE t_id = '".$tav['t_id']."'") or die(mysql_error());
  66. 					$tav = mysql_fetch_assoc($gMax);
  67. 					echo mysql_num_rows($gPost) + mysql_num_rows($gPost2) ."/".$tav['t_maxposts'];
  68. 					?>
  69. 					</td>
  70. 					<td width="10%">&nbsp;<a href="opzeggen.php?tavid=<?= $tav['t_id']; ?>"><u>opzeggen</u></a></td>
  71. 				</tr>
  72. 				<?
  73. 				}
  74. 		}
  75. 	}
  76. 	else {
  77. 		echo "&nbsp;Geabonneerde tavernes (0)<br><br>";
  78. 			?>
  79. 			<tr bgcolor="#444444">
  80. 				<td colspan="5">&nbsp;Geen abonnementen</td>
  81. 			</tr>
  82. 			<?php
  83. 	}
  84. 	?>
  85. </table>
  86. <br>
  87. <table width="70%" class="sh" border="0" cellpadding="0" cellspacing="0">
  88. 	<tr bgcolor="#333399">
  89. 		<th width="35%" nowrap><b>&nbsp;Naam taverne</b></th>
  90. 		<th width="17%" nowrap><b>&nbsp;Oprichter</b></th>
  91. 		<th width="4%" nowrap><b><center>#L</center></b></th>
  92. 		<th width="4%" nowrap><b>&nbsp;Posts</b></th>
  93. 		<th width="10%" nowrap><b>&nbsp;</b></th>
  94. 	</tr>
  95. 	<?php
  96. 	$gAbo = mysql_query("SELECT * FROM abonnementen WHERE userid='".$user."'") or die(mysql_error());	
  97. 	$gTavs = mysql_query("SELECT * FROM tavernes  ORDER BY t_naam ASC") or die(mysql_error());
  98. 	$allAbo = mysql_num_rows($gAbo);
  99. 	$alltavs = mysql_num_rows($gTavs);
  100.  
  101. 	$result = $alltavs - $allAbo;
  102.  
  103. 	if($result >= 1) {		
  104. 			echo "&nbsp;Niet-geabonneerde tavernes (".$result.")<br><br>"; 
  105. 			$gAbo = mysql_query("SELECT * FROM abonnementen WHERE userid='".$user."'") or die(mysql_error());
  106. 			$result3 = mysql_num_rows($gAbo);	
  107. 				if($result3 == 0) {
  108. 					$gTav1 = mysql_query("SELECT * FROM tavernes ORDER BY t_naam ASC") or die(mysql_error());
  109. 					while($tav = mysql_fetch_assoc($gTav1)) {
  110. 					?>					
  111. 					<tr bgcolor="#444444">
  112. 						<td width="35%"><b>&nbsp;<font color="#FFFF00"><?= $tav['t_naam']; ?></font></b></td>
  113. 						<td width="17%">&nbsp;<?= afkorten($tav['t_eigenaar'], 15); ?></td>
  114. 						<td width="4%"><center>
  115. 						<?php 
  116. 						$gLid = mysql_query("SELECT * FROM abonnementen WHERE clubid='".$tav['t_id']."'") or die(mysql_error());
  118. 						?>
  119. 						</center></td>
  120. 						<td width="4%">&nbsp;
  121. 						<?php
  122. 						$gPost = mysql_query("SELECT * FROM t_topics WHERE taverne_id='".$tav['t_id']."'") or die(mysql_error());
  123. 						while($topic = mysql_fetch_assoc($gPost)) {
  124. 							$blaat = $topic['id'];
  125. 						}
  126. 						$gPost2 = mysql_query("SELECT * FROM t_replys WHERE topic_id = '".$blaat."'") or die(mysql_error());
  127. 						$gMax = mysql_query("SELECT * FROM tavernes WHERE t_id = '".$tav['t_id']."'") or die(mysql_error());
  128. 						$tav = mysql_fetch_assoc($gMax);
  129. 						echo mysql_num_rows($gPost) + mysql_num_rows($gPost2) ."/".$tav['t_maxposts'];
  130. 						?>
  131. 						</td>
  132. 						<td width="10%">&nbsp;<a href="abonneren.php?tavid=<?= $tav['t_id']; ?>"><u>abonneren</u></a></td>
  133. 					</tr>
  134. 					<?php	
  135. 					}			
  136. 				}
  137. 				else {
  138.  
  139. 					$gAbo2 = mysql_query("SELECT * FROM abonnementen WHERE userid='".$user."'") or die(mysql_error());
  140. 					while($Abo = mysql_fetch_assoc($gAbo2))
  141. 					{                         
  142.     					$gTav2 = mysql_query("SELECT * FROM tavernes WHERE t_id != '".$Abo['clubid']."' ORDER BY t_naam ASC") or die(mysql_error());
  143.     				}
  144. 						while($tav = mysql_fetch_assoc($gTav2))
  145.    						{					
  146. 						?>
  147. 						<tr bgcolor="#444444">
  148. 							<td width="35%"><b>&nbsp;<font color="#FFFF00"><?= $tav['t_naam']; ?></font></b></td>
  149. 							<td width="17%">&nbsp;<?= afkorten($tav['t_eigenaar'], 15); ?></td>
  150. 							<td width="4%"><center>
  151. 							<?php
  152. 							$gLid = mysql_query("SELECT * FROM abonnementen WHERE clubid='".$tav['t_id']."'") or die(mysql_error());
  154. 							?>
  155. 							</center></td>
  156. 							<td width="4%">&nbsp;
  157. 							<?php
  158. 							$gPost = mysql_query("SELECT * FROM t_topics WHERE taverne_id='".$tav['t_id']."'") or die(mysql_error());
  159. 							while($topic = mysql_fetch_assoc($gPost)) {
  160. 								$blaat = $topic['id'];
  161. 							}
  162. 							$gPost2 = mysql_query("SELECT * FROM t_replys WHERE topic_id = '".$blaat."'") or die(mysql_error());
  163. 							$gMax = mysql_query("SELECT * FROM tavernes WHERE t_id = '".$tav['t_id']."'") or die(mysql_error());
  164. 							$tav = mysql_fetch_assoc($gMax);
  165. 							echo mysql_num_rows($gPost) + mysql_num_rows($gPost2) ."/".$tav['t_maxposts'];
  166. 							?>
  167. 							</td>
  168. 							<td width="10%">&nbsp;<a href="abonneren.php?tavid=<?= $tav['t_id']; ?>"><u>abonneren</u></a></td>
  169. 						</tr>
  170. 						<?php					
  171. 						}
  172. 				}			
  173. 	}
  174. 	else {
  175. 		echo "&nbsp;Niet-geabonneerde tavernes (".$result.")<br><br>"; 
  176. 		?>
  177. 		<tr bgcolor="#444444">
  178. 			<td colspan="5">&nbsp;U bent reeds op alle tavernes geabonneerd.</td>			
  179. 		</tr>
  180. 		<?php
  181. 	}
  182. 	?>
  183. </table>
  184. <br><br>
  185. <?php
  186. }
  187. ?>
  188. </body>
  189. </html>

dit is beter, nu zie je bij niet geabonneerd dezeflde query als bovenaan, en daar loopt het mis
Offline ikkedikke - 10/09/2005 11:06
Avatar van ikkedikke PHP expert je overschrijft iedere keer in die eerste regel het id, in de 2e query zal die dus alleen nog het id van de laatste vorige query onthouden hebben...
Offline Simon - 10/09/2005 13:40 (laatste wijziging 10/09/2005 15:23)
Avatar van Simon PHP expert en hoe valt dit op te lossen?
iemand?
Gesponsorde links
Dit onderwerp is gesloten.
Actieve forumberichten
© 2002-2025 Sitemasters.be - Regels - Laadtijd: 0.269s