[MySQL][PHP]System tagów - nie dopasowuje tagów do newsa Opcje

[coomler]

Cześć, może temat nazwałem jakoś tak "niewyraźnie", ale nie miałem pomysłu jak go nazwać

Stworzyłem sobie system tagów i teraz mam problem.

To się znajduje w pętli for:

[PHP] pobierz, plaintext 
<?php 
 
				$query = "SELECT tags.tagname FROM tags, newstotags WHERE newstotags.newsid = $id  AND tags.tagid = newstotags.tagid";	
 
				$result = mysql_query($query);
 
				$rows = mysql_num_rows($result);
 
				if($rows >= 1)
				{
						while($row = mysql_fetch_array($result))
						{
								echo '<a href="tag/' . $row['tagname'] . '/">' . $row['tagname'] . '</a>, ';
						}
				}
 
				else
				{
						echo '<span class="no-tags">Nie przypisano żadnych tagów.</span>';
				}
 
?>
[PHP] pobierz, plaintext 

Jak widać na końcu będę mia przecinek. W celu jego usunięcia postanowiłem użyć rtrim więc i mój kod się zmienił:

[PHP] pobierz, plaintext 
<?php 
 
				$query = "SELECT tags.tagname FROM tags, newstotags WHERE newstotags.newsid = $id  AND tags.tagid = newstotags.tagid";		
 
				$result = mysql_query($query);
 
				$rows = mysql_num_rows($result);
 
				if($rows >= 1)
				{
						while($row = mysql_fetch_array($result))
						{
								$tags .= ' <a href="tag/' . $row['tagname'] . '/">' . $row['tagname'] . '</a>,';
						}
 
						echo rtrim($tags, ',');
				}
 
				else
				{
						echo '<span class="no-tags">Nie przypisano żadnych tagów.</span>';
				}
 
?>
[PHP] pobierz, plaintext 

Problem pole na tym, że usuwa przecinek, ale nie tylko... zobaczcie efekt na zrzucie http://img294.imageshack.us/img294/4186/problemm.png Te tagi oddzielone czerwonym nie powinny się tam znajdować. Ten problem występuje tylko, gdy stosuję ten drugi skrypt. Tworzy się tak zwana piramida. Na dole(tych starczych newsach) są wszystkie możliwe tagi.

Problem nie pojawia się również, gdy nie przechodzę pętlą po newsach a wyświetlam tylko jednego nie używając pętli.

Ten post edytował coomler 31.08.2010, 14:21:07

[Blame]

Dodaj sobie po if($rows >=1):

[PHP] pobierz, plaintext 
$tags = '';
[PHP] pobierz, plaintext 

Bo teraz do zmiennej cały czas dodajesz nowe

[Janek85]

Dlaczego cały czas dodajesz do ciągu nowe dane ?

Zamiast:

[PHP] pobierz, plaintext 
$tags .= ' <a href="tag/' . $row['tagname'] . '/">' . $row['tagname'] . '</a>,';
[PHP] pobierz, plaintext 

wpisz ta:

[PHP] pobierz, plaintext 
$tags = ' <a href="tag/' . $row['tagname'] . '/">' . $row['tagname'] . '</a>,';
[PHP] pobierz, plaintext 

Powinno zadziałać.

pozdrawiam

[coomler]

Dodaj sobie po if($rows >=1):

[PHP] pobierz, plaintext 
$tags = '';
[PHP] pobierz, plaintext 

Bo teraz do zmiennej cały czas dodajesz nowe

No to żeś mnie zalatwil Dzięki Ci bardzo.

Zamiast:

[PHP] pobierz, plaintext 
$tags .= ' <a href="tag/' . $row['tagname'] . '/">' . $row['tagname'] . '</a>,';
[PHP] pobierz, plaintext 

wpisz ta:

[PHP] pobierz, plaintext 
$tags = ' <a href="tag/' . $row['tagname'] . '/">' . $row['tagname'] . '</a>,';
[PHP] pobierz, plaintext 

Powinno zadziałać.

pozdrawiam

Zauważ, że dane wyświetlam poza pętlą więc ten sposób nie zadziała. Jeżeli miałbym iść Twoim tropem musiałbym utworzyć tablicę, a dopiero potem wyświetlić ją poza pętlą. Zresztą rozwiązanie podobne do tego, gdzie dodaję tekst do zmiennej, jak się mylę poprawcie.

[Janek85]

Jeśli w pierwszym przykładzie wyświetlasz dane w pętli i wszystko działa, to dlaczego nie chcesz tego zrobić w przykładzie drugim ?

[PHP] pobierz, plaintext 
 
while($row = mysql_fetch_array($result)){
 
								$tags = ' <a href="tag/' . $row['tagname'] . '/">' . $row['tagname'] . '</a>,';
						                echo rtrim($tags, ',');
 
                                                                }
 
 
 
[PHP] pobierz, plaintext 

[coomler]

O ile się nie mylę, bo nie sprawdzałem, to ten kod wyświetli mi

tag1 tag2 tag3

zamiast

tag1, tag2, tag3

[Janek85]

Skoro nie chcesz mieć tylko ostatniego przecinka, to spróbuj po wyświetleniu wyniku poza pętlą (Twój przykład), nadać zmiennej $tags wartość null. Tak żeby do zmiennej dopisywały się tyko dane z aktualnej pętli. Może tak zadziała

pozdrawiam

[coomler]

I właśnie tak działa. Pozdrawiam.