Optymalizacja kodu, instrukcja switch Opcje

[alkamid]

Cześć!

Napisałem pewien prosty skrypt php i chcę go teraz zoptymalizować. Mam jedną zagwozdkę, na którą nie mam pomysłu. Oto ona:

W skrypcie ważną rolę odgrywa zmienna miesiąc wybrana przez użytkownika - chodzi o to, że użytkownik ma do wyboru kilka miesięcy, przekazuje wybór za pomocą $_GET, a następnie wyświetlane są dane z wybranego miesiąca.
Jest jednak mały problem: jeśli istnieją dane z miesiąca poprzedniego, to ostatni ich wiersz musi się pojawić w bieżącym zestawieniu. Zmienna $_GET['mth'] (czyli ta przechowująca aktualnie wybrany miesiąc) ma format MM, a zapytanie sql które stosuję do wyciągnięcia wierszy z danego miesiąca:

[SQL] pobierz, plaintext 
SELECT DATA, wartosc FROM fw WHERE DATA LIKE "%-'.$_GET['mth'].'-%" ORDER BY DATA ASC;
[SQL] pobierz, plaintext 

Jeśli nie widać tego od razu, to tłumaczę: dla marca zapytanie to wyszukuje wyrazów z "-03-", dla kwietnia z "-04" etc.
Żeby pobrać ostatni wiersz z poprzedniego miesiąca stosuję:

[SQL] pobierz, plaintext 
SELECT wartosc FROM fw WHERE DATA LIKE "%-'.$prev.'-%" ORDER BY DATA DESC LIMIT 1 ;
[SQL] pobierz, plaintext 

Niestety, zmienną $prev otrzymuję w nieelegancki sposób, a mianowicie instrukcją switch:

[PHP] pobierz, plaintext 
<?php
switch($_GET['mth']) { // ustawienie poprzedniego miesiąca... jakby to zrobić krócej?
        case '02':
            $prev='01';
            break;
        case '03':
            $prev='02';
            break;
        case '04':
            $prev='03';
            break;
        case '05':
            $prev='04';
            break;
        case '06':
            $prev='05';
            break;
        case '07':
            $prev='06';
            break;
        case '08':
            $prev='07';
            break;
        case '09':
            $prev='08';
            break;
        case '10':
            $prev='09';
            break;
        case '11':
            $prev='10';
            break;
        case '12':
            $prev='11';
            break;
        }
?>
[PHP] pobierz, plaintext 

Nie wiem jednak jak zrobić to inaczej, bo zmienna $_GET['mth'] nie jest przecież liczbą, więc nie mogę po prostu odjąć od niej jedności.
Bardzo proszę o wskazówki - być może już sam pomysł na pobieranie wierszy z danego miesiąca jest zbyt skomplikowane, bo można to zrobić za pomocą narzędzi ukrytych w MySQL, kto wie?

[alkamid]

A pozatym jesli trzymasz daty w bazie jako DATE to mozesz uzyc mysqlowej funkcji MONTH() i sobie porownujesz normalnie z 1, 4, 12, 11

Świetne, właśnie o to mi chodziło. Wielkie dzięki!

To jeszcze jedno: do wyświetlenia listy dostępnych miesięcy używam takiej konstrukcji:

[PHP] pobierz, plaintext 
<?php
$months_exist = 'SELECT data FROM fw;'; // sprawdza jakie miesiące istnieją w bazie
 
    if($zap0 = $mysqli->query($months_exist)) {
        
        $months_num_rows = $zap0->num_rows;
        
        for($i=0; $i<$months_num_rows; $i++) {
            $months_element = $zap0->fetch_assoc();
            for($j=1; $j<13; $j++) { // jeśli znajdzie datę która zawiera w sobie j (1-12), to przypisze elementowi tablicy $miesiace $j (do wydrukowania listy)
                if (($j<10) && (preg_match("/-0".$j."-/", $months_element['data'])))
                    $miesiace[$j] = "0".$j;
                elseif (($j>9) && (preg_match("/-".$j."-/", $months_element['data'])))
                    $miesiace[$j] = $j;
                
            }
        }
    }
?>
[PHP] pobierz, plaintext 

Co też jest trochę głupie, bo muszę robić dwa przypadki dla j od 1 do 9 i od 10 do 12. Skoro już się uporałem z tamtym to tu też mogłyby być liczby w zwykłym formacie: 1,2,3...
Wyobrażam sobie to tak:

[PHP] pobierz, plaintext 
<?php
if($j == month($months_element['data']))
$miesiace[$j] = $j;
?>
[PHP] pobierz, plaintext 

Tylko w php nie ma funkcji tak samo działającej jak ta w MySQL, tzn. pewnie jest ale ja się nie mogę jej doszukać.

Ten post edytował alkamid 28.03.2009, 14:03:25