Forum PHP.pl

Witam,
nie wiem jak zwrócić tablicę obiektów.
mam taką klasę:

[PHP] pobierz, plaintext 
class Sql {
 
	public static function pobierz_uzytkownikow() {
 
		$db = new Database();
		$sql = "SELECT * FROM Persons";
		$db->fetchRows($sql);
 
		$rows = $db->fetchRows($sql);
 
 
		foreach($rows as $row)
		{
			$tab[] = new User($row);
		}
 
		return $row;
	}
}
[PHP] pobierz, plaintext 

Nie wiem jak się zabrać za to:

[PHP] pobierz, plaintext 
foreach($rows as $row)
		{
			$tab[] = new User($row);
		}
 
		return $row;
[PHP] pobierz, plaintext 

Dostaję taki komunikat:
Warning: Invalid argument supplied for foreach()
czyli tablica jest pusta, albo się nie wypełniła.

Do tego jest klasa User, która dziedziczy po klasie abstrakcyjnej Czlowiek:

[PHP] pobierz, plaintext 
abstract class Czlowiek {
 
	protected $waga;
	protected $wzrost;
	protected $wiek;
 
	abstract public function pobierzwage();
	abstract public function pobierzwzrost();
	abstract public function pobierzwiek();
 
}
 
class User extends Czlowiek {
 
	public function __construct() {
 
	}
 
	public function pobierzwage() {
 
	}
 
	public function pobierzwzrost() {
 
	}
 
	public function pobierzwiek() {
 
	}
}
[PHP] pobierz, plaintext 

Dostałem takie zadanie na korepetycjach, możecie mnie nakierować jak się za to zabrać?
Zmieniłem bbcode.

Ten post edytował cykcykacz 12.07.2011, 06:44:48

[PHP] pobierz, plaintext 
        $db->fetchRows($sql);
 
        $rows = $db->fetchRows($sql);
[PHP] pobierz, plaintext 

Coś ci mówi?

Tak BTW, zmień bbcode

Tak wiesz

[PHP] pobierz, plaintext 
 $db->fetchRows($sql);
[PHP] pobierz, plaintext 

Tutaj korzystam z metody którą mam w klasie database

[PHP] pobierz, plaintext 
interface DatabaseConnection {
 
	public function fetchRows($sql);
 
}
 
class Database implements DatabaseConnection {
 
	public $sql;
	public function __construct() {
 
		$this->con = mysql_connect("localhost","root","");
       	if (! is_resource($this->con)){
           	throw new Exception('Could not connect: ' . mysql_error());
        }
 
        $db_selected = mysql_select_db("my_db", $this->con);
        if (!$db_selected) {
            die ('Nie można ustawić foo : ' . mysql_error());
       	}
	}
 
	public function fetchRows($sql) {
		$result = mysql_query($sql);
		if (!$result) {
			echo 'Could not run query: ' . mysql_error();
			exit;
		}
		$row = mysql_fetch_assoc($result);
 
//	foreach ($row as $rows) {
//		echo $rows. "<br />";
//	}
//		    echo $row["personID"];
//    echo $row["FirstName"];
//    echo $row["FirstName"];
 
	}
[PHP] pobierz, plaintext 

Natomiast to:

[PHP] pobierz, plaintext 
$rows = $db->fetchRows($sql);
[PHP] pobierz, plaintext 

Nic mi nie mówi tylko tyle, że przypisuje do zmiennej $rows wynik metody fetchRows. Nie wiem właśnie jak dalej to oprogramować.

Ten post edytował cykcykacz 12.07.2011, 06:45:45

Jaki sens ma pisanie takiej klasy? o_O Nie dość, że jest kompletnie źle napisana, to w dodatku raz używasz wyjątków, raz zabijasz skrypt, a innym razem wypluwasz tylko błąd.



Skoro nic nie zwracasz w metodzie fetchRows, to co Ci ma wypluć? ; d

Zapoznaj się z PDO, bo warto, a to co masz najlepiej usuń.

@cykcykacz
Proszę wykorzystywać poprawny BBCode [php ] <?php echo 'Twój kod'; ?> [/php ] - bez spacji oczywiście.