Witam,

prosiłbym o wypowiedź osób, które mają dość duże doświadczenie w programowanie aplikacji webowych. Otóż mam paginację wraz z sortowaniem, wartości dotyczące sortowania wypełnia się poprzez formularz na stronie (GET). Wszystkie te flagi muszę później załadować do metody, która pobiera listę elementów i mam to tak zakodowane:

[PHP] pobierz, plaintext 
        // Sorting rules
            if(isset($this->get['sort_date']) && $this->get['sort_date'] == "asc")  $options['order_by_date_asc'] = TRUE;
            else if(isset($this->get['sort_date']) && $this->get['sort_date'] == "desc")  $options['order_by_date_desc'] = TRUE;
            else if(isset($this->get['sort_title']) && $this->get['sort_title'] == "desc")  $options['order_by_title_desc'] = TRUE;
            else if(isset($this->get['sort_title']) && $this->get['sort_title'] == "asc")  $options['order_by_title_asc'] = TRUE;
[PHP] pobierz, plaintext 

Dodatkowo mam jeszcze kilka innych elementów srotujących więc trochę if-ów się tworzy pytanie czy to jedyny bezpieczne optymalne rozwiązanie

Przekombinowałeś moim zdaniem. 10 pól w bazie i 2 możliwe kierunki to "tylko" 20 możliwych kombinacji jako pola tablicy, a na dodatek wzajemnie się wykluczających Znaczne prościej byłoby:

[PHP] pobierz, plaintext 
$this->sort = array('pole' => 'kierunek', 'pole2' => 'kierunek2');
[PHP] pobierz, plaintext 

Czemu tak? Posłużę się najprostszym możliwym klejeniem zapytania (query builder byłby lepszy, ale nie znam implementacji):

[PHP] pobierz, plaintext 
$order = 'ORDER BY';
if(count($this->sort) > 0) {
    foreach($this->sort AS $column => $direction) {
        $order .= ' '.$column.' '.$direction.',';
    }
    $order = substr($order, 0, -1);
}
[PHP] pobierz, plaintext 

Oczywiście to tylko idea. Osobiście zabezpieczałbym to przed głupotami z kierunkiem czy sprawdzeniem istnienia kolumn i takie tam "drobiazgi"

Zauważ... Jedna tablica z kluczem jako nazwa pola i wartością jako kierunek, a dodatkowo kolejność w tablicy odgrywa rolę gdy mamy order by z użyciem wielu pól. Przy odrobinie pomyślunku można nawet to przerobić tak by generował także:

[SQL] pobierz, plaintext 
ORDER BY count(FIELD) DESC, id ASC
[SQL] pobierz, plaintext 

I nie jest to trudne Odrobinka chęci i przemyślenia sprawy

A jak zabezpieczać? Przykładowo zastosowałbym mapowanie. Skoro ze skryptu otrzymuję konkretne wartości to mam też zapewne tablicę, gdzie mogę tę wartość zamienić na odpowiednią nazwę kolumny. Sprawdzam to co otrzymuję od usera pod kątem zgodności z tablicą mapującą. Jeśli mam coś niezgodnego z nią - wywalam. To samo gdy otrzymam ze skryptu coś, co nie mapuje się na ASC lub DESC jako kierunek sortowania. Takie coś też wylatuje. Ostatecznie coś mi tam zostaje (lub nie) i to wykonuję.

Budowanie zapytań odpada bo używam Active Record w Codeigniterze.

[PHP] pobierz, plaintext 
 if(isset($this->get['sort_date']) && in_array($this->get['sort_date'], $this->sorting_template['sort_date'])) $options['sort_date'] =  $this->get['sort_date'];
                if(isset($this->get['sort_title']) && in_array($this->get['sort_title'], $this->sorting_template['sort_date'])) $options['sort_title'] =  $this->get['sort_title'];
                $options['positions'] =  array_filter(elements(array_keys($this->get), $this->sorting_template['positions'] ));
                $options['options'] =  array_filter(elements(array_keys($this->get), $this->sorting_template['options'] ));
                $options['employments'] =  array_filter(elements(array_keys($this->get), $this->sorting_template['employments'] ));
                $options['works'] =  array_filter(elements(array_keys($this->get), $this->sorting_template['works'] ));      
[PHP] pobierz, plaintext 

Zamknąłem się w takim zapytaniu.

Myślisz, że Active Record to naprawdę problem?

[PHP] pobierz, plaintext 
foreach($this->sort AS $column => $direction) {
    $this->db->order_by($column, $direction);
}
[PHP] pobierz, plaintext 

Nie bez powodu napisałem, że query buildery byłby lepszy