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

[mrWodoo]

nie ma zmiennej $mysql zdeklarowanej w obrębie klasy, więc to pewnie jest zmienna zewn. czyli musisz zrobić

[PHP] pobierz, plaintext 
global $mysql;
[PHP] pobierz, plaintext 

Zadajesz nieprecyzjne pytania.
Jeśli faktycznie pytasz o

jak zrobić aby wyświetlały się wszystkie dostępne newsy w bazie Mysql.

to polecam poczytać więcej artykułów

[LeNy]

Dzięki wielkie, mrWodoo zapomniałem o global

Odnalazłem pewien poradnik, trochę mnie naprowadził.

[PHP] pobierz, plaintext 
public function __construct()
{
	global $mysql;
	$news_query = $mysql->query("SELECT * FROM `News`");
	$returned = array();
	while ($row = $news_query->fetch_assoc())
	{
		$mx = array();
		foreach ($row as $key => $val)
		{
			$mx[$key] = $val;
		}
		$returned[] = $mx;
	}
	return $returned;
}
[PHP] pobierz, plaintext 

Dlaczego podczas użycia funkcji foreach nic mi się nie wyświetla:

[PHP] pobierz, plaintext 
$news = new printNews;
foreach($news as $var)
{
	echo $var['UID'];
}
[PHP] pobierz, plaintext 

[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

[solr]

Kluczami w $var są nazwy pól z tabeli News w bazie danych.

[LeNy]

Mortus, dzięki wielgachne. Dzięki Tobie zrozumiałem o co chodzi dokładnie z tymi class'ami, jak je budować (IMG:style_emoticons/default/smile.gif)
Za dużo kombinowałem, jak zwykle (IMG:style_emoticons/default/wstydnis.gif)

Jeszcze raz dzięki wielkie (IMG:style_emoticons/default/specool.gif)

Ten post edytował LeNy 24.03.2012, 22:38:11