Wątek: klasa a klasa która lepsza

Mam dwa kody i się zastanawiam która lepsza:

Kod php:

  <?php
  class player{ 
  var $info; 
  function getBasic($login){ 
  $mysql = mysql_query("select * from `user` where `login` = '$login'");
  $this->info = mysql_fetch_array($mysql); 
  } 
  } 
  $player = new player; 
  $player->getBasic($login); 
  ?>

i drugi

Kod php:

<?
class player{
    public $dane;
    function pobranie_danych($nazwa){
        $this->dane = mysql_fetch_array(mysql_query("select plec, klasa, energia, czasen from `konta` where `nazwa` = '$nazwa' limit 1"));
    } 
} 
$gracz = new player;
$gracz->pobranie_danych($nazwa); 
echo $gracz->dane['energia'];
?>

Który z nich jest praktyczniejszy??
można to jeszcze w konstruktorze zrobic ale pomijam fakt

Obie sa na tym samym etapie. Przydatnosc moglyby pokazac gdyby posiadaly rozne zastosowanie. Jednak to co one wykonuja to jedno ZBEDNE zapytanioe ktore lepiej umiescic w kodzie niz w klasie.

A ja bym proponował coś takiego:

Kod php:

<?php
class gracz {
    function GetInfo($info, $id = false) {
        if(isset($id)) $PlayerID = $id; else $PlayerID = $_SESSION['PlayerID'];
        if(is_array($info)) {
            $i = 1;
            $text = "";
            foreach($info as $key => $value) {
                if($i == 1) $text .= "`".$value."`"; else $text .= ", `".$value."`";
                $i++;
            }
        } else if($info == "all") {
            $text = "*";
        } else {
            $text = "`".$info."`";
        }
        $q = mysql_query("SELECT ".$text." FROM `users` WHERE `id`='".$PlayerID."'") or die(mysql_error());
        $r = mysql_fetch_assoc($q) or die(mysql_error());
        if(is_array($info) || $info == "all") return $r; else return $r[$info];
    }
}
/*
* $klasa = new gracz();
* $kolumny = $klasa->GetInfo(array("kolumna_a", "kolumna_b"), 12); // Pobieramy informacje (kolumny "kolumna_a" i "kolumna_b") o graczu posiadającym ID = 12
* echo "kolumna_a: ".$kolumny['kolumna_a']."<br />kolumna_b: ".$kolumny['kolumna_b'];
*/
?>

Pozwala pobrać jedną kolumnę (jeśli jako $info damy stringa z nazwą kolumny), kilka kolumn (jeśli jako $info damy tablicę (klucze obojętne, liczą się tylko wartości)) lub wszystkie kolumny (jeśli jako $info damy stringa o treści "all").

PS. Czy można by ją (moja funkcję) ocenić?

No fajna funkcja. Do tworzenia klas taka sie przyda. sama w sobie jest srednioprzydatna bo ma wady i zalety:
zalety: zautomatyzowany proces
wady: kiepsko sie jej uzywa indywidualnie. Lepiej stosowac w klasie gdzie nie zagladamy do kodu.

Mam jeszcze rade: dodaj do niej opcje "debug" ktora wyswietli wszystkie tresci takie jak zapytanie, zwrot wynikow i komunikat bledow. Cos takiego moze fajnie usprawnic prace bo wtedy mozemy bardzo latwo debugowac swoj skrypt i wykluczyc pomylke przy tworzeniu zapytania.

Zamieszczone przez karer

wady: kiepsko sie jej uzywa indywidualnie. Lepiej stosowac w klasie gdzie nie zagladamy do kodu.

Tego kawałka nie rozumiem. ^^ Przecież jest w klasie. (class gracz)

Tak tak ale chodzi mi o to ze nie ma wiekszego sensu wywolywac jej recznie bo jest malo przejrzysta dla czlowieka. Lepiej zrobic ja raz i niech skrypt z automaty ja zawsze wywoluje gdy jest potrzebna.

No to już chyba wiem o co ci chodzi. ^^