[MySQL][PHP]Klasa OOP Opcje

[neuroine]

Stworzyłem taką prostą klasę, prosiłbym o informację czy jest napisana zgodnie z OOP.

class.articles.php

[PHP] pobierz, plaintext 
class Articles {
 
		private $db_host = 'localhost';
		private $db_name = 'database';
		private $username = 'root';
		private $password = 'root';
		private $section;
		private $mysqli;
		private $result;
 
 
		public function __construct() {
 
				$this->mysqli = @new mysqli($this->db_host,$this->username,$this->password,$this->db_name);
				$this->mysqli->query('SET NAMES utf8');
 
						if ($link->connect_error) 
						{
							echo("Connect failed: %s\n". $mysqli->connect_error);
							exit();
						}
 
		}
 
		public function display () {
 
				$this->result  = $this->mysqli->query("SELECT id, title, date, content, autor  FROM articles ORDER BY date DESC");
				while ($data = $this->result->fetch_object()) 
				{
					echo('<h1>'.$data->title.'</h1><p class="right">'.$data->date.'</p>');
					echo('<p class="justify">'.$data->content.'</p>');
					echo('<p>Add by: <b>'.$data->autor.'</b></p>');
					echo('<hr />');
				}
				$this->result -> close();
				$this->mysqli -> close();
		}
}
[PHP] pobierz, plaintext 

index.php

[PHP] pobierz, plaintext 
require_once ('class.articles.php');
 
 
$testing = new articles();
$testing->display();
[PHP] pobierz, plaintext 

[zend]

Nie. Ani troche nie jest zgodna z podejściem obiektowym. to tylko kod opakowany w klase. W klasach się nic nie wyświetla. A jeśli będziesz chciał stworzyć inną klasę np komentarzy to też tam nawiążesz połączenie? Klasa powinna przyjmować połączenie do bazy jako parametr w konstruktorze, dodaj jeszcze kilka metod, dodawanie newsa, usuwanie newsa(ów), edycja i daj jeszcze raz do oceny

Ten post edytował zend 8.06.2010, 12:44:51

[vokiel]

Mam chwilę czasu, to taki mały wzór (oczywiście można to zrobić na wiele różnych sposobów, napisać interfejs dla samych artykułów, a później odpowiednie klasy do wyświetlania w wybranych formatach (na stronie, RSS, mailing etc)

[PHP] pobierz, plaintext 
class Articles {
    private $_db;
 
    public function __construct(database $db){
        $this->_db = $db;
    }
 
    public function displayArticles($limit=3){
        $articles = $this->getArticles($limit);
        /* sprawdzenie poprawności etc */
        $ToReturn = '';
        foreach ($articles as $article){
            $ToReturn .= '<h1>'.$article['title'].'</h1>'; /* i reszta */
        }
        return $ToReturn;
    }
 
    private function getArticles($limit){
        /* połączenie z bazą, zapytanie, zwrócenie wyników w formie tabeli */
    }
 
}
[PHP] pobierz, plaintext 

Użycie:

[PHP] pobierz, plaintext 
$art = new Articles($db);
echo $art->displayArticles();
[PHP] pobierz, plaintext 

[neuroine]

Jak mogę najkorzystniej połączyć się z bazą danych, spotkałem się z kilkoma podejściami

1. Dwie klasy mysql.php i articles.php jedna jest rozszerzeniem drugiej .
2. Wzorzec Singleton [nie wiem jak powinien wyglądać na moim przykładzie po dodaniu kilku klas]
3. Wzorzec Fabryka.

Walidacja powinna być wykonywana w klasie Articles, czy lepiej napisać uniwersalną umieszczoną w oddzielnym pliku?

[PHP] pobierz, plaintext 
class Articles {
 
	private $db_host = 'localhost';
	private $db_name = 'db';
	private $username = 'root';
	private $password = 'root';
 
 
		public function __construct($section, $title, $content, $autor) {
 
		$this->mysqli = new mysqli($this->db_host,$this->username,$this->password,$this->db_name);
			$this->mysqli->query('SET NAMES utf8');
 
					if ($link->connect_error) 
					{
						echo("Connect failed: %s\n". $mysqli->connect_error);
						exit();
					}
 
					$this->content = addslashes($content);
					$this->date = date('Y-m-d H:i:s');
					$this->autor = $autor;
					$this->section = $section;
					$this->title = $title;			
		}
 
		public function displayArticles() {
 
			  $result = $this->mysqli->query("SELECT id, title, date, content, autor  FROM $this->section ORDER BY date DESC");
 
			  while ($row = $result->fetch_object()) 
			  {
				  echo('<h1>'.$row->title.'</h1><p class="right">'.$row->date.'</p>');
				  echo('<p class="justify">'.$row->content.'</p>');
				  echo('<p>Add by: <b>'.$row->autor.'</b></p><p id="'.$row->article_id.'" class="comment" >Comments: ['); 
				  echo(']</p><p class="right">( Read more )</p>');
				  echo('<hr />');
			  } 
 
 
			  $result->close();		
		}
 
		public function checkArticle() {
 
			if (empty($this->title) || empty($this->content) || empty($this->autor) || empty($this->section)) {
				echo ('You did not provide all the necessary information.');
 
			}		
		}
 
		public function addArticles() { 
 
			$result = $this->mysqli->prepare("INSERT INTO $this->section (`title`, `content`, `date`, `autor`) VALUES (?, ?, ?, ?)");
			$result->bind_param("ssss", $this->title, $this->content, $this->date, $this->autor);
			$result->execute();
			$result->close();
		}
 
		public function __destruct() { 
 
$this->mysqli->close();
}
 
}
[PHP] pobierz, plaintext 

Ten post edytował neuroine 6.07.2010, 16:13:16

[zend]

1) Kiepskie podejście, lepiej stworzyć klasę podelu, a połączenie przekazywać jako parametr
2) Czego nie wiesz?
3) Jeśli korzystasz z podpowiadania składni, to nie polecam
4) Co osoba, to innej podejście, ja bym napisał osobną klasę, nie będziesz przez to musiał powielać tego samego kodu przy innych walidacjach

[neuroine]

1. Dobrze, mam klasę modelu articles.php, rozumiem że połączenie przekazuje w kontrolerze includując tam klasę z mysql.php
Wiadomo, że klasę mysql.php będę chciał nie raz użyć, wiec jak powinienem to rozwiązać. Sytuacja podobna gdy stworzę klasę walidacji.

2. Nie bardzo wiem od czego mam zacząć.

3. Nie korzystam z podpowiadania składni.

4. Walidacja na pewno przyda się nie raz.