Forum PHP.pl

Tak się zastanawiam, mam coś takiego

[PHP] pobierz, plaintext 
function get_users($sort = null, $order = null)
{
    $query = $this->qb->select(
        array(
            'user.name',
            'user.surname',
            'user_group.name as group_name',
            'user_role.name as role_name',
            ... itd
        )
    )
    ->from('user', 'user_group', 'user_role')
    ->where(
        array(
            'user.id_user_grou' => 'user_group.id_user_group',
            'user.id_user_role' => 'user_role.id_user_role'
        )
    )
    if(is_null($sort) == false)
    {
        $query = $query->where($sort);
    }
 
    if(is_null($order) == false)
    {
        $query = $query->orderby($order);
    }
    else
    {
        $query = $query->orderby(
            array(
                'user.id_group' => 'desc'
            )
        );
    }
}
[PHP] pobierz, plaintext 

i tak się zastanawiam jak wy to rozwiązaliście, czy w przypadku kiedy chcę tylko imiona i nazwiska userów to czy wybierać wszystkie kolumny które sobie zadeklarowałem, czy dodać zmienną w której będę przekazywał nazwy pól które mnie interesują

Ten post edytował skowron-line 31.10.2010, 13:29:59

no jasne, że wybierać konkretne pola, po co wszystko wyciągać, kiedy niepotrzebne
tak btw, ciekawy zapis

[PHP] pobierz, plaintext 
if(is_null($sort) == false)
{
$query = $query->where($sort);
}
[PHP] pobierz, plaintext 

nie wystarczy tak?

[PHP] pobierz, plaintext 
if($sort != null)
// lub
if($sort !== null)
// lub
if($sort)
[PHP] pobierz, plaintext 

Ten post edytował my salsa 31.10.2010, 17:05:34

To prawda, nie mogę się z tym nie zgodzić, ale czy jeżeli zaczniemy podawać listę wszystkiego to czy taki zapis nie traci sensu ?

to jest 4 możliwość i niestety ona mi się wryła w pamięć.

Jak to przyjęte w wielu frameworkach, trzymaj listę wszystkich pól i w przypadku kiedy nie podasz konkretnych, tylko dasz nulla, niech korzysta z tej listy i wybiera wszystko, albo *
Co do tej 4 możliwości...
Nie ma chyba najmniejszego sensu, korzystać z funkcji, kiedy nie jest to potrzebne, a wypada się wyzbywać złych nawyków

Ten post edytował my salsa 31.10.2010, 18:33:44