Jestem poczatkujacy w dziedzinie OOP. Pisze a raczej probuje pisac aplikacje baze danych kin, seansow, filmow. Napisalem 2 klasy: 1.laczenia z baza SQL
2.kina (jak narazie tylko pokazuje nazwy kin)

Chcialbym zapytac czy to przypomina obiektowke? I jak powinno sie konstuowac klasy typu kino,film w takim projekcie.

class.connect.php

[PHP] pobierz, plaintext 
<?php
 class DB {
 private $link;
 public $result;
 public $row;
 
 public function __construct(){
    $this-> link = mysql_connect(localhost,root,krasnal);
    mysql_select_db(kino);
    
    if(!is_resource($this -> link)){
            die('Brak połączenia z bazą!');
         }
 }
 public function setCinema(){
 $this-> result=mysql_query(&#092;"SELECT name FROM cinema ORDER BY id DESC\")or die('Wystapil blad');
 }
 public function getCinema(){
 while ($this-> row = mysql_fetch_assoc( $this->result )){
 print $this->row['name'].&#092;" \";
     }
 }
 
}
$baza = new DB;
?>
[PHP] pobierz, plaintext 

class.cinema.php

[PHP] pobierz, plaintext 
<?php
 class cinema {
  
  public function __construct(){
    require_once(&#092;"class.connect.php\");
    DB::setCinema();
    DB::getCinema();
 }
 
 }
 $kino = new cinema;
?>
[PHP] pobierz, plaintext 

Prosze o wszelkie uwagi.

Twój kod nie ma najmniejszego sensu. Nie rozdzielasz zadań jakie mają mieć klasy DB i cinema. Db - ma za zadanie łączenia się z bazą danych, ułatwienie w pobieraniu rekordów, a nie już pobieranie danych. Pobieranie danych należy do klas dziedziczących po Db, np. cinema.

pozdrawiam.

A moge prosic o szablon jak powino to wygladac (jak porozdzielac metody klas).Chcialem rowniez dodac opcje 'dodawania,usuwanie i modyfikowania danych kina'. Czy powinienem w tym celu zrobic klase cinema_manager w ktorej beda metody dodawanie kina, usuwanie kina a w klasie cinema tylko wyswietlanie?



Dopiero ucze sie OOP. Czytam tutoriale i probuje sie w tym wszystkim polapac.

@Noctrum
Jak się uczysz, to powinieneś czytać, już na tym podforum było wiele pytań o takie rzeczy, poczytaj co tam proponują jakie uwagi były, a wtedy jak czegoś nadal nie będziesz rozumiał pisz.
Ale w tym kodzie jak napisał Ociu nie ma krzty podejścia obiektowego. Dlaczego? Powiedz w jednym dwóch zdaniach co robi dany obiekt? Może to głupio brzmi ale tak jest, każdy obiekt ma robić tylko konkretną czynność, jak robi wszystko to znaczy że to nie jest podejście obiektowe. (mimo że są obiekty)
Co robi baza? Można wybrać coś, wstawić, aktualizować i usunąć, to są główne aspekty. Więc obiekt bazy to powinien mieć, nic więcej, bez konkretnych zapytań, bo to nie jego robota. Chcesz obsługiwać kina, seanse, klientów itd. więc nie możesz zaszywać rzeczy związanych z nimi w klasie DB.

(Są zresztą ORM, co by robiły co chcesz zrobić, ale jak chcesz tylko pisać aby się nauczyć to nie ma sensu korzystać, możesz najwyżej zobaczyć jak tam jest to zrobione, ale na tym poziomie chyba byś się pogubił)

Witam, też raczkuję w OOP ale myślę że lepszym rozwiązaniem jest:

[PHP] pobierz, plaintext 
<?php
class DB {
 private $link;
 public $result;
 public $row;
 
 public function __construct(){
	$this-> link = mysql_connect(localhost,root,krasnal);
	mysql_select_db(kino);
 
	if(!is_resource($this -> link)){
			die('Brak połączenia z bazą!');
		 }
 }
 
 public function query($ARG_SQL)
 {
   $this->result = mysql_query($ARG_SQL);
 }
 
 public function fetch_array()
 {
  $this->row = @mysql_fetch_array($this->result);
   if($this->row)
	 return $this->row;
   else
	throw new Exception ('Nie można odczytać elementów z bazy danych i zapisać do tablicy.');
 }
 
 class Cinema extends DB
 {
 
  public function setCinema()
  {
  $this->query("SELECT name FROM cinema ORDER BY id DESC");
   }
 
   public function getCinema()
   {
	 while ($this-> result = mysql_fetch_assoc( $this->result )){
	 print $this->row['name']." ";
   }
 }
?>
[PHP] pobierz, plaintext 

Nie wiem czy dobrze, nie testowałem
Ale przynajmniej z dziedziczeniem jest chyba jakiś sens...

Dziękuję, Babcia@Stefa

Też nie ma sensu i też nie jest to OOP. Powinna być osobna klasa odpowiedzialna za bazę danych (czyli w niej metody np. "select", "query", ta klasa łączy się z bazą danych). Jest kilka możliwości rozwiązania problemu, np ActiveRecord, DAO. Przykładowo (pseudokod):

[PHP] pobierz, plaintext 
<?php
class Database{
	//uchwyt bazy danych
	protected $hConn;
	public function __construct(){
		//polaczenie z bd
	}
 
	public function query($sql){
		return $query($sql, $this->hConn);
	}
 
	//jakieś inne metody
}
 
interface DAO{
	public function getById($id);
	public function delete();
	public function getAll();
	public function save(RowObject $o);
}
 
 
abstract class SqlDao implements DAO{
	protected $className;
	protected $tableName;
	protected $db;
 
	public function __construct($className, $tableName, Database $db){
		$this->className = (string) $className;
		$this->tableName = (string) $tableName;
		$this->db = $db;
	}
 
	public function getById($id){
		$table = $this->tableName;
		$class = $this->className;
		$sql = "SELECT * FROM $table WHERE id='$id';";
		$res = $this->db->query($sql);
		$record = mysql_fetch_assoc($res);
		return new $class($record);
	}
 
	public function delete(RowObject $o){
		//implementacja
	}
 
	public function getAll(){
		//implementacja
	}
 
	public function save(RowObject $o){
		//implementacja
	}
}
 
interface RowObject{
	public function getId();
}
 
abstract class AbstractRowObject implements RowObject{
	protected $fields = array();
	protected $data = array();
 
	public function __construct(array $fields, array $data = array()){
		$this->fields = $fields;
		$this->data = $data;
	}
 
	public function __get($name){
		if(array_value_exists($name, $this->fields)){
			return $this->data[$name];
		}
	}
 
	public function __set($name, $value){
		if(array_value_exists($name, $this->fields)){
			$this->data[$name] = $value;
		}
	}
 
	public function getId(){
		return $this->data['id'];
	}
}
 
class CinemaDAO extends SqlDao{
 
	public function __construct(Database $sql){
		parent::__construct('cinema', 'Cinema', $sql);
	}
 
}
 
class Cinema extends AbstractRowObject{
	public function __construct($data = array()){
		parent::__construct(array('id', 'name', 'location'), $data);
	}
}
?>
[PHP] pobierz, plaintext 

zastosowanie:

[PHP] pobierz, plaintext 
<?php
$dao = new CinemaDao($db);
$cinema = $dao->getById($id);
 
//lub
$c = new Cinema();
$c->name = "nazwa";
$c->location = "lokacja";
$dao->save($c);
?>
[PHP] pobierz, plaintext 

Na forum jest dosyć dużo rozwiązań tego "problemu", więc szukaj...

Kilka, kilka dni temu, za paroma topikami...
Temat: PHPskrypt w OOP
kilko
Pozdrawiam.

Przy okazji - proszę poprawić temat na sensowny