Klasa newsów Opcje

[LeNy]

Witam. Tworzę właśnie klasę do newsów, lecz nie wiem jak zrobić aby wyświetlały się wszystkie dostępne newsy w bazie Mysql.
Czytałem dużo poradników, lecz nie znalazłem interesującej mnie odpowiedzi.
Aktualnie mój kod wygląda tak:

[PHP] pobierz, plaintext 
class printNews
{
	public function __construct()
	{
		$news_query = $mysql->query("SELECT * FROM `News`");
		return $news_query->fetch_assoc();
	}
}
 
$newsy = new printNews;
var_dump($newsy);
[PHP] pobierz, plaintext 

Proszę o pomoc (IMG:style_emoticons/default/sadsmiley02.gif)

Ten post edytował LeNy 24.03.2012, 21:28:21

[mortus]

Zapomnij o słowie global. W OOP takie sytuacje rozwiązuje się za pomocą np. DI (Dependency Injection, wstrzykiwania zależności). W tym przypadku obiekt $mysql odpowiedzialny za obsługę połączenia z bazą danych powinieneś przekazać jako argument konstruktora. Skoro klasa ma reprezentować nowinki, to niech się tak właśnie nazywa, nazwa printNews nadaje się bardziej dla odpowiedniej metody, która te news-y wyświetla. Swoją drogą tym już klasa News nie powinna się zajmować. Przykład:

[PHP] pobierz, plaintext 
class News {
  private $_db = null;
  public function __construct($db) {
    $this->_db = $db;
  }
  public function fetchAll() {
    $results = $this->_db->query("SELECT * FROM `News`");
    while($row = $results->fetch_assoc()) {
      $news[] = $row;
    }
    return $news;
  }
}
 
$news = new News($mysql);
foreach($news->fetchAll() as $data) {
  echo $data['UID'];
  // ...
}
[PHP] pobierz, plaintext 

Chyba trochę przedobrzyłeś.

Ten post edytował mortus 24.03.2012, 22:32:41