[PHP] Dlaczego wchodzi w petle if, jesli warunek nie jest spelniony Opcje

[Wolfie]

Witam,


Mam taka funkcje ktora obcina string :

[PHP] pobierz, plaintext 
function truncate($string, $length, $break = "...") {
            $len = strlen($string);
            if ($len > $length) {
                $pos = strpos($string, " ", $length);
                return substr($string, 0, $pos) . $break;
            } else {
            return $string;
            }
        }
[PHP] pobierz, plaintext 

Problem polega na tym ze jesli okresle $length np na 25 to jesli string bedzie krotszy niz $length to zwraca mi tylko '...' zamiast calego stringu, tak jakby za kazdym razem niezalenie od warunku wchodzil w if($len > $length)
Poprostu jesli $len < $length to zamiast zwrocic $string zwraca mi $break czyli '...'

Gdzie tutaj jest blad bo ja nie moge go znalezc...

[Ociu]

A przypadkiem nie chcesz napisać takiej funkcji: wordwrap?

[Wiktor P.]

[PHP] pobierz, plaintext 
function truncate($string, $length, $znak, $break = "...") {
 
 
            $len = strlen($string);
 
 
            if ($len > $length) {
                $pos = stripos($string, $znak, $length);
 
                #echo $pos;
                #echo gettype($pos);
 
                $g1 = substr($string, 0, $pos);
                $g .= $break;
                $g = $g1.$g2;
 
                return $g;
 
            }
 
            else
            {
            return $string;
            }
 
        }
 
 
echo truncate("abcdefgh", 4, "h");
 
[PHP] pobierz, plaintext 

Sorki, że trochę namieszałem w twoim kodzie, ale pewnie dasz sobie radę.
Zmienną $pos otrzymywałeś jako boolean, a nie jako integer. Interpreter więc zgłupiał.
Zmianna $znak może być zamieniona oczywiście na wczesniejszą spację.

Jak bardzo ci przeszkadza, że trchę pozmieniałem to napisz. Pod wieczór to przerobię.

pzdr

[Wolfie]

Wiktor, nie wiem co Ty narobiles ale przeciez to jest to samo co moja funkcja, dzialanie to samo,

zamiast :

[PHP] pobierz, plaintext 
echo truncate("abcdefgh", 4, "h");
[PHP] pobierz, plaintext 

Zrob sobie

[PHP] pobierz, plaintext 
echo truncate("abc", 4, "h");
[PHP] pobierz, plaintext 

I zobacz co sie stanie, wlasnie o to chodzi.....i wcale nie bedzie dzialac poprawnie....

Ten post edytował Wolfie 8.10.2009, 17:54:18

[Wiktor P.]

Chciałem ci pokazać, że jeśli funkcja strpos(), albo lepiej stripos() nie znajdzie w łańcuchu argumentu, który podałeś jako drugi, zwróci false.
Ty przecież potrzebujesz do substr zmienną typu integer.
Co ci przyjdzie z substr(string, int, boolean) ?
Na pewno jakiś błąd.

Zresztą mi ten twój skrypt działa.

[PHP] pobierz, plaintext 
echo truncate1("Jakiś tekst", 2);
echo truncate1("Jakiś tekst", 25);
[PHP] pobierz, plaintext 

Zwracają wyniki:

Jakiś..
Jakiś tekst

Ten post edytował Wiktor P. 8.10.2009, 20:12:44

[Wolfie]

No wlasnie bo on powinien dzialac a mi nie dziala i w tym problem , nie widze bledu a jesli ciag jest krotszy niz $lenght to zwraca mi tylko $break = '...'  - .....ktos wie gdzie moze byc problem ?