[PHP] Problem z tablicami Opcje

[Tajgeer]

Witam, posiadam taki kod:

[PHP] pobierz, plaintext 
<?php
public static function ShowTags($separator=",", $tags) 
{    
global $link; 
$tags_array = explode($separator, $tags);
$ile = count($tags_array);
$liczba = 0;
while($liczba <= $ile) {
$name = $tags_array[$liczba];
$link .= "<a href='Tag/$name.html'>$name</a>";
++$liczba;
}
 return $link;
}
?>
[PHP] pobierz, plaintext 

Jest to kod, który ze stringa (w postaci np. (słowo,słowo2)) wyciąga każdy z wyrazów (które są separowane przecinkami), a następnie tworzy link.
Problem w tym, iż otrzymuje błąd: Undefined offset: 2 w linii nr. 9 ($name = $tags_array[$liczba]; ) . Uprzedzam, że ani wzięcie zmiennej w "", ani w taki sposób ($name = $tags_array['.$liczba.']; ) niestety nic nie daje - dalej ten sam błąd.
Na przykład gdy wpiszę liczbe np. 0, to pokazuje mi pierwszy wyraz z tablicy, 1 - drugi itd.
Jak sprawić, aby ten skrypt działał?

Ten post edytował Tajgeer 23.06.2009, 20:14:10

[athei]

Strzelam, że

Kod

while($liczba < $ile)

.
Jeśli używasz funkcji z arg. domyślnym pisz

Kod

ShowTags($tags, $separator=",")

[Pilsener]

Dlaczego po prostu nie dasz foreach?

[PHP] pobierz, plaintext 
<?php
$tags_array = explode($separator, $tags);
foreach($tags_array as $name){
   $link .=  '<a href="Tag/'.$name.'.html">'.$name.'</a>';
}
?>
[PHP] pobierz, plaintext 

- i string dajesz w apostrofach, zwłaszcza, jeżeli używasz kodu HTML.

[Luke_Star]

dla zabezpieczenia polecam dawac

Kod

trim($tags,",")

dzięki temu ominiesz kilka banalnych bledow z pustym elementem.

[alegorn]

tak gwoli wyjasnienia czemu miales errora,
blad lezy w tym ze count zwraca ci liczbe elementow, a nie indeksy,

wiec :

[PHP] pobierz, plaintext 
<?php
$arr = array(1,6,3,4,7);
echo count ($arr);
?>
[PHP] pobierz, plaintext 

wyswietli ci 5, podczas gdy index ostatniej to 4

wiec by twoj kod zadzialal, wystarczy:

[PHP] pobierz, plaintext 
<?php
while($liczba < $ile) {
?>
[PHP] pobierz, plaintext 

i juz.. aczkolwiek lepiej jest uzyc foreach.

tyle ze ja bym jeszcze dopisal

[PHP] pobierz, plaintext 
<?php
$tags_array = explode($separator, $tags);
if(is_array($tags_array  ))
  foreach($tags_array as $name)
    $link .=  '<a href="Tag/'.$name.'.html">'.$name.'</a>';
?>
[PHP] pobierz, plaintext 

kwestia tego ze explode w wyjatkowych warunkach zwraca false, a tego w petli nie chcemy :]

Ten post edytował alegorn 24.06.2009, 09:00:53