[PHP]Usunięcie niechcianego przecinka Opcje

[Greg23]

Mam skrypt, który wyświetla mi na stronie tagi:

[PHP] pobierz, plaintext 
if (!empty($rekordow2[8])){
echo '<p>Tagi: ';
$tags24=explode(",", $rekordow2[8]);
 
foreach($tags24 as $f ){
echo "<a  href='' title='".$f."'><strong>$f</strong></a>, ";
 
}
echo '</p>';
}
[PHP] pobierz, plaintext 

W wyniku dostaję coś takiego:

[HTML] pobierz, plaintext 
Tagi: ala, ma, kota,
[HTML] pobierz, plaintext 

Jak pozbyć się tego ostatniego, niepotrzebnego przecinka ?

[!*!]

[PHP] pobierz, plaintext 
trim($tag, ',')
[PHP] pobierz, plaintext 

Ten post edytował !*! 12.07.2012, 12:41:32

[matty7]

[PHP] pobierz, plaintext 
$x = 0;
foreach($tags24 as $f ){
echo "<a  href='' title='".$f."'><strong>$f</strong></a>";
if($x++ == count($rekordow2[8])) echo ", "; else echo " ";
}
[PHP] pobierz, plaintext 

[Greg23]

Matty, Twój pomysł nie do końca działa, co prawda nie mam na końcu przecinka, ale na początku też
Coś takiego dostaje:

[HTML] pobierz, plaintext 
ala ma, kota
[HTML] pobierz, plaintext 

a gdy zmienie $x=0 na $x=1 to dostaje:

[HTML] pobierz, plaintext 
ala, ma kota
[HTML] pobierz, plaintext 

Próbowałem zmienić:

[HTML] pobierz, plaintext 
if($x++ == count($rekordow2[8])) echo ", "; else echo " ";
[HTML] pobierz, plaintext 

na

[HTML] pobierz, plaintext 
if($x++ < count($rekordow2[8])) echo ", "; else echo " ";
[HTML] pobierz, plaintext 

ale dalej lipa :/


Ten post edytował Greg23 12.07.2012, 13:07:31

[piotrex41]

Powinno zadziałać:

[PHP] pobierz, plaintext 
if (!empty($rekordow2[8]))
{
    $tags24=explode(',', $rekordow2[8]);
    foreach($tags24 as $f)
    {
        $s .= '<a  href='' title='".$f."'><strong>$f</strong></a>, ';
    }
    $s = substr($s, 0, -1);
    echo '<p>Tagi: '.$s.' </p>';
}
[PHP] pobierz, plaintext 

Ten post edytował piotrex41 12.07.2012, 13:13:54

[Greg23]

Działa świetnie
Bardzo dziękuję za pomoc!

[irmidjusz]

Nie wiem, co to za dziwactwa ludziska wyprawiają z tym doklejaniem łańcuchów i usuwaniem ostatniego przecinka (czy jakiegokolwiek innego znaku).
A można prościej, tak:

[PHP] pobierz, plaintext 
if (!empty($rekordow2[8]))
{
    $tagi = array();
    foreach(explode(',', $rekordow2[8]) as $f)
    {
        //tutaj odpowiednio sformatowane elementy, co tam chcemy...
        $tagi[] = '<a  href="" title="'.$f.'"><strong>'.$f.'</strong></a>';
    }
    //a to ładnie skleja je do stringa z dowolnym łańcuchem łączącym pomiędzy elementami, niczego zbędnego na końcu ani na początku nie ma...
    $s = implode(',', $tagi);
    echo '<p>Tagi: '.$s.' </p>';
}
[PHP] pobierz, plaintext 

[SmokAnalog]

Nie wiem, co to za dziwactwa ludziska wyprawiają z tym doklejaniem łańcuchów i usuwaniem ostatniego przecinka (czy jakiegokolwiek innego znaku).

Zgadzam się! Niestety PHP w całej tej swojej "łatwości" w wielu osobach zakorzenia programistyczną bylejakość, złe nawyki, klejenie potworków. Chociaż Twój sposób, irmidjusz, jest niewiele lepszy Zamiast tych zmiennych/tablic-potworków, wystarczy sprawdzać czy osiągnięto już koniec (lub przerobić na sprawdzanie czy początek):

[PHP] pobierz, plaintext 
<?php if($f !== end($tags24)): ?>, <?php endif; ?>
[PHP] pobierz, plaintext 

[markonix]

SmokAnalog co jeżeli wartość się powtarza?

ala, ma, kota, i, nic, nie, ma

[irmidjusz]

złapał markonix smoka za ogon

[SmokAnalog]

Racja, wtedy pozjada część przecinków

Ale i tak jest lepszy sposób:

[PHP] pobierz, plaintext 
$separator = '';
foreach($tablica as $element) {
   echo $separator.'<a href="">'.$element.'</a>';
   $separator = ', ';
}
[PHP] pobierz, plaintext 

[irmidjusz]

No, co kto lubi, ja preferuję:

[PHP] pobierz, plaintext 
$temp = array();
foreach($tablica as $element){
    $temp[] = '<a href=""'.$element.'</a>';
}
echo implode(', ', $temp);
[PHP] pobierz, plaintext 

Rzecz gustu

[markonix]

Wydaje mi się, że sposób z tymczasową tablicą i funkcją implode jest ździebko bardziej pamięciożerny więc osobiście wybrałbym pomysł poprzednika.
Wygodniejszy byłby także gdy stosujemy system szablonów oparty o surowe PHP. Wtedy wygląda to znacznie prościej dla ewentualnego kodera HTML.
Wygląda także znacznie lepiej bo gdyby struktura HTMLowa znajdująca się w pętli byłaby dość pokaźna (tak, że wygodniej zamiast echo byłoby lepiej umieścić poza znacznikami php) to wrzucanie tego do tablicy wyglądałoby niefortunnie.

[SmokAnalog]

Rzecz gustu

Nie tylko gustu Wrzucanie fragmentów kodu HTML do zmiennych jest przeważnie złym pomysłem. Jeśli można coś wypluć od razu z echo, to najlepiej tak zrobić Tym bardziej, że jak tych tagów będzie więcej niż kilka, to zbieranie ich kodu w tablicy, a następnie łączenie z implode wcale nie jest błyskawiczną operacją.

[irmidjusz]

Przekonaliście mnie, rzeczywiście ten zapis SmokaAnaloga wygląda prościej, czytelniej
Chociaż osobiście nie lubię pisać tego typu konstrukcji, gdzie wykonuje się coś niepotrzebnie mnóstwo razy (jak to przypisanie separatora w pętli), to jednak w typowych zastosowaniach nie ma to żadnego znaczenia i dla uproszczenia kodu jest OK. Działa dobrze i jest łatwe w czytaniu i to mnie przekonało.

Natomiast jeśli chodzi o czas wykonania czy zużycie pamięci:

1.
Jeśli echo ma wypluć niewielkie ilości niedługich łańcuchów, to czas wykonania czy pamięciożerność w ogóle się nie liczy Dla tak małych liczb, są to rzeczy zupełnie bez znaczenia i pomijalne i nie ma nawet się co zastanawiać nad tym czy coś się wykona szybciej albo zajmie mniej RAMu, bo są to nieistotne różnice. Gdybanie o tym, co jest szybsze od czego nie jest profilowaniem i często ma się nijak do rzeczywistości. W ogóle rozmyślanie o tym i używanie jako argumentów, że jakieś rozwiązanie jest lepsze od innego, jest w takim przypadku (małych ilości przetwarzanych elementów) bez sensu.

Co to są "niewielkie ilości"? Zależy Ale w praktyce, ile takich elementów wyświetlisz? 100? 500? Więcej się zwykle nie daje, bo to utrudnia przeglądanie strony - do tego jest paginacja albo filtrowanie. Nawet dla 500 kilkuset znakowych łańcuchów nie ma praktycznego znaczenia, który sposób zastosujesz. No chyba, że piszesz w PHP aplikację na najnowszy zegarek Bonda z wbudowanym mikroserwerem www z 1 MB RAMu dla PHPa, wówczas to co innego... ;P

A teraz zagadka: przy jakiej ilości elementów w tablicy będzie różnica wykonania rzędu 100 ms? Zależy od serwera ale mogę strzelać, że będą to ilości 100000+ (o ile oczywiście nie sprawdzamy na zegarku Bonda z prockiem 400 MHz... ) Poza tym, takie rzeczy to się sprawdza w realnym środowisku wykonawczym podczas profilowania i optymalizuje tylko wtedy, gdy w ogóle ma jakiekolwiek istotne znaczenie.

2.
Jeśli echo nie jest buforowane, to im więcej operacji echo, tym dłużej to trwa i prosty test pokazuje, że dla dużych ilości echo, lepiej jest zgromadzić stringi w tablicy i wypluć jednym echo implode, bo trwa to sumarycznie krócej. Oczywiście, można też gromadzić w tymczasowym stringu i wypluć go na koniec w całości. A najszybciej (i zużywa najmniej RAMu) jest stosować echo przy włączonym buforowaniu.