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?

[Fifi209]

Kod

$_GET['mth']

$mth = (int) $_GET['mth'];

Zostanie przekonwertowany na int ;d

@edit

Radzę wyłączyć dyrektywę register globals w apache ;d (chyba że korzystasz z hostingu to pewnie wyłączyli)

Kod

if (($mth > 1) && ($mth < 13)) {
$prev = $mth-1;
}

Ten post edytował fifi209 28.03.2009, 10:06:39

[alkamid]

Kod

$_GET['mth']

$mth = (int) $_GET['mth'];

Zostanie przekonwertowany na int ;d

Ale wtedy będzie to "1", "2"..., zamiast "01", "02" - a to robi różnicę w zapytaniu do bazy danych.
Mógłbym zrobić dwa różne zapytania: jeśli $month<10 to niech szuka %-0'.$month.'-%, a jeśli >=10 to po prostu %-".$month.", bez dopisanego 0. Na razie ten pomysł wygrywa (jest trochę mniej kodu niż switch, ale mimo wszystko wolałbym w jakiś sposób być zdolnym do zmieniania "02" na "01").

Edit: Musiałem dopisać dwa zapytania:

[PHP] pobierz, plaintext 
<?php
if(($month>=1) && ($month<10)) {
        $list = 'SELECT data, wartosc FROM fw 
        WHERE data LIKE "%-0'.$month.'-%" order by data asc;'; 
    }
    else {
         $list = 'SELECT data, wartosc FROM fw 
        WHERE data LIKE "%-'.$month.'-%" order by data asc;';
    }
?>
[PHP] pobierz, plaintext 

I to samo ze zmienną $prev. Nie wiem więc czy switch nie był jednak lepszy.

Ten post edytował alkamid 28.03.2009, 13:16:35