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

[dr_bonzo]

Masakra, to zes wymyslil switcha.

Odejmij po prostu 1 i dopisz ew. zero z przodu.

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

[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

[dr_bonzo]

explode" title="Zobacz w manualu PHP" target="_manual ?
albo bardziej poprawnie, ale raczej wolno: strtotime() + date('m')

[alkamid]

Dzięki raz jeszcze!