Cześć.

Ostatnio czytam o klasach/funkcjach.
Chciałem stworzyć klasę, która by zwracała wyniki na podstawie danego ID.
Stworzyłem coś takiego:

[PHP] pobierz, plaintext 
function user($i) {
global $baza;  
$zapytanie = $baza->query("SELECT login, username, avatar, email, gwiazdki FROM users WHERE id = ".$i);
$gracz = $zapytanie->fetch();
    $out['id'] = $i;
    $out['login'] = $gracz['login'];
    $out['username'] = $gracz['username'];
    $out['email'] = $gracz['email'];
    $out['gwiazdki'] = $gracz['gwiazdki'];
 
    if(empty($gracz['avatar'])){
    $out['avatar'] = 'avatars/noimage.png';   
    }else{
    $out['avatar'] = ''.$gracz['avatar'].'';   
    }
 
    return $out;
} 
[PHP] pobierz, plaintext 

niby działa ale chyba kiedyś widziałem gdzieś inne rozwiązanie.
Co sądzicie zostawić tak czy polecacie mi inny sposób?
Dziękuję za pomoc.

Mówisz o klasie, a to zwykła funkcja

Ogólnie takie pobieranie po ID jest popularnym rozwiązaniem w bibliotekach będących warstwą abstrakcji dla bazy danych, ale w Twoim kodzie jest kilka problemów. Przede wszystkim zobacz sobie prepared statements, żeby Twój kod nie był podatny na SQL Injection. Nie powinieneś tak wstawiać $id z parametru do zapytania. Po drugie, polecam unikać global, bo to się prosi o problemy. Zamiast tego możesz np. przekazywać instancję bazy jako parametr albo mieć singleton (poczytaj o tym). A w idealnym przypadku możesz użyć bardziej zaawansowanych technik wstrzykiwania zależności (Dependency Injection).

Możesz też uprościć kod z $out, bo klucze się pokrywają. Zamiast linii 5-17 wystarczy jedna:

[PHP] pobierz, plaintext 
return ['avatar' => $user['avatar'] ?: 'avatars/noimage.png'] + $user;
[PHP] pobierz, plaintext 

Przy okazji dodaj w zapytaniu wybór id, czyli: "SELECT id, login, username, avatar, email, gwiazdki", wtedy zmienna $gracz będzie zawierać wszystkie pola z pierwotnego kodu.

P.S. Zamiast $user powinno być $gracz (w kodzie SmokAnalog).

Też zauważyłem to $gracz po fakcie, ale sugerowałbym jednak nazwę $user, bo po pierwsze nazwa funkcji nie wskazuje, że chodzi o jakichś graczy, a po drugie mieszanie polskiego z angielskim w kodzie jest ble. To uwagi do kolegi diamentowego króla, bo drugi kolega zapewne to już wie.