[PHP] problem z bardzo prostą funkcją Opcje

[linx]

Witam,
mam problem ze stworzeniem bardzo prostej funkcji:

[PHP] pobierz, plaintext 
<?php
function miesiac_nazwa($tekst) {
        $tekst = str_replace('1', STYCZEN, $tekst);
        $tekst = str_replace('2', LUTY, $tekst);
        $tekst = str_replace('3', MARZEC, $tekst);
        $tekst = str_replace('4', KWIECIEN, $tekst);
        $tekst = str_replace('5', MAJ, $tekst);
        $tekst = str_replace('6', CZERWIEC, $tekst);
        $tekst = str_replace('7', LIPIEC, $tekst);
        $tekst = str_replace('8', SIERPIEN, $tekst);
        $tekst = str_replace('9', WRZESIEN, $tekst);
        $tekst = str_replace('10', PAZDZIERNIK, $tekst);
        $tekst = str_replace('11', LISTOPAD, $tekst);
        $tekst = str_replace('12', GRUDZIEN, $tekst);
        return $tekst;
}
?>
[PHP] pobierz, plaintext 

funkcja fajnie działa do momentu kiedy wartość są od 1-9, kiedy wartość są od 10-12 to zawsze wyświetla się styczeń.

Prosił bym o jakieś wskazówki.
Za pomoc z góry dziękuje.

[drPayton]

To są stałe czy brakuje apostrofów/cydzysłowów? Jeśli stałe to widocznie nie są zdefiniowane, sama funkcja jest poprawna...

[linx]

Są to wartości pobierane z getdate():

[PHP] pobierz, plaintext 
<?php
 
function miesiac_nazwa($tekst) {
       $tekst = str_replace('1', STYCZEN, $tekst);
       $tekst = str_replace('2', LUTY, $tekst);
       $tekst = str_replace('3', MARZEC, $tekst);
       $tekst = str_replace('4', KWIECIEN, $tekst);
       $tekst = str_replace('5', MAJ, $tekst);
       $tekst = str_replace('6', CZERWIEC, $tekst);
       $tekst = str_replace('7', LIPIEC, $tekst);
       $tekst = str_replace('8', SIERPIEN, $tekst);
       $tekst = str_replace('9', WRZESIEN, $tekst);
       $tekst = str_replace('10', PAZDZIERNIK, $tekst);
       $tekst = str_replace('11', LISTOPAD, $tekst);
       $tekst = str_replace('12', GRUDZIEN, $tekst);
       return $tekst;
}
 
 
$czas = getdate();
$miesiac = $czas['mon'];
 
echo miesiac_nazwa($miesiac);
 
?>
[PHP] pobierz, plaintext 

[bartek12421]

Moim zdaniem nie chodziło tu o stałe

[PHP] pobierz, plaintext 
<?php
function miesiac_nazwa($tekst) {
$liczby = array('1', '2', '3');
$slownie = array('stycznien', 'luty', 'marzec');
$tekst= str_replace($liczby, $slownie, $tekst);
return $text;
?>
[PHP] pobierz, plaintext 

[WojtasSP320]

Jeśli na wejściu podajesz tylko nr miesiąca, to prościej zrobisz to instrukcją switch:

[PHP] pobierz, plaintext 
<?php
function nr2month($nr)
{
switch($nr)
{
case 1: {$text = 'STY'; break;}
case 2: {$text = 'LUT'; break;}
case 3: {$text = 'MAR'; break;}
case 4: {$text = 'KWI'; break;}
case 5: {$text = 'MAJ'; break;}
case 6: {$text = 'CZE'; break;}
case 7: {$text = 'LIP'; break;}
case 8: {$text = 'SIE'; break;}
case 9: {$text = 'WRZ'; break;}
case 10: {$text = 'PAŹ'; break;}
case 11: {$text = 'LIS'; break;}
case 12: {$text = 'GRU'; break;}
}
 
return $text;
}
?>
[PHP] pobierz, plaintext 

Trochę lipa z tym str_replace, bo jak zinterpretuje 11 (jako Listopad, czy StyczeńStyczeń)? Jeśli sie mylę, to proszę mnie poprawić.

Ten post edytował WojtasSP320 5.10.2008, 12:32:09

[Kildyt]

Jeżeli najpierw zmieniasz 1 na styczeń, to nie możesz później zmienić 10 na październik, ponieważ 1 w 10 już nie ma.
Rozwiązanie jest proste. Sprawdź, czy przed cyfrą zawsze jest spacja, wtedy tylko zmieniasz ' 1' na styczeń, albo ' 10' na październik.
Jeżeli nie ma spacji to przed cyframi dodaj jakiś znak, i po. Np. @1@, @10@ i wtedy zmieniasz @1@ na styczeń, a @10@, na październik.

[drPayton]

Jeśli na wejściu podajesz tylko nr miesiąca, to prościej zrobisz to instrukcją switch:
(...)
Trochę lipa z tym str_replace, bo jak zinterpretuje 11 (jako Listopad, czy StyczeńStyczeń)? Jeśli sie mylę, to proszę mnie poprawić.

Racja (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[linx]

Ok.
Dzięki @WojtasSP320 nie pomyślałem o switch`ach.