Forum PHP.pl

Witam,
właśnie zacząłem próbować pisać cos obiektowo w php...

na razie wyszło mi coś takiego:


klasa do łączenia z bazą danych

[PHP] pobierz, plaintext 
<?php
error_reporting( E_ALL ); 
 
class polacz_sql {
 
  private $sql;
 
  public function __construct() {
 
	  $this -> db_host = '127.0.0.1'; // Adres serwera
	  $this -> db_name = 'oop_cms'; // Nazwa bazy danych
	  $this -> db_user = 'root'; // Nazwa uzytkownika
	  $this -> db_password = 'root'; // Haslo
 
	  $this -> sql = mysql_connect($this -> db_host, $this -> db_user, $this -> db_password);
	  mysql_select_db($this -> db_name);
 
	  if (!is_resource($this -> sql)) {
		die('Brak polaczenia z baza danych.' . mysql_error());
	  }
 
  }
 
  public function __destruct() {
 
	  if(!is_resource($this -> sql)) {
		mysql_close($this -> sql);
	  }
 
  }
 
}
?>
[PHP] pobierz, plaintext 

klasa do zarządzania podstronami:

[PHP] pobierz, plaintext 
<?php
error_reporting( E_ALL ); 
 
require_once 'class.polaczenie_sql.php';
 
class podstrona extends polacz_sql {
 
  public function pobierz_podstrone($id) {
 
	  $this -> res = mysql_query("SELECT * FROM podstrony WHERE id = '$id' LIMIT 1");
	  return $this -> res;
 
  }
 
  public function edytuj_podstrone($id, $menu, $tresc) {
 
	  mysql_query("UPDATE `oop_cms`.`podstrony` SET `menu` = '$menu', `tresc` = '$tresc' WHERE `id` = $id LIMIT 1");
 
  }
 
  public function lista_podstron() {
 
	  $this -> res = mysql_query("SELECT id, menu FROM podstrony");
	  return $this -> res;
 
  }
 
  public function usun_podstrone($id) {
 
	  mysql_query("DELETE FROM `podstrony` WHERE `id` = $id LIMIT 1");
 
  }
 
  public function dodaj_podstrone($nazwa) {
 
	  mysql_query("
		INSERT INTO `oop_cms`.`podstrony` (
		`id` ,
		`menu` ,
		`tresc`
		)
		VALUES (
		NULL , '$nazwa', ''
		)
	  ");
 
  }
 
}
?>
[PHP] pobierz, plaintext 

oraz zastosowanie:

[PHP] pobierz, plaintext 
<?php
  error_reporting( E_ALL ); 
  include("class.podstrona.php");
  $objPodstrona = new podstrona();
?>
 
<h2>Edycja podstron</h2>
 
<?php
 
if(isset($_GET['zmien'])) {
  $objPodstrona -> edytuj_podstrone($_GET['zmien'], $_POST['menu'], $_POST['tresc']);
}
 
if(isset($_GET['usun'])) {
  $objPodstrona -> usun_podstrone($_GET['usun']);
}
 
if(isset($_GET['dodaj'])) {
  $objPodstrona -> dodaj_podstrone($_POST['menu']);
}
 
if(isset($_GET['edycja'])) {
 
  $res = $objPodstrona -> pobierz_podstrone($_GET['edycja']);
  while ($rekord = mysql_fetch_array($res)) {
	echo '
		  Edycja podstrony:
		  <form method="post" action="?zmien='.$rekord['id'].'">
			<input type="text" name="menu" value="'.$rekord['menu'].'" /><br />
			<textarea name="tresc">'.$rekord['tresc'].'</textarea><br />
			<input type="submit" value="Edytuj" />
		  </form>
	';
  }
 
}
 
?>
 
<hr />
Lista podstron:<br /><br />
 
<?php
 
$res = $objPodstrona -> lista_podstron();
 
while ($rekord = mysql_fetch_array($res)) {
  echo $rekord['menu'].' <a href="?edycja='.$rekord['id'].'">edycja</a> | <a href="?usun='.$rekord['id'].'">usun</a> <br />';
}
 
?>
 
<br /><hr /><br />
 
Dodaj podstrone:
 
<form method="post" action="?dodaj">
  <input type="text" name="menu" /><br />
  <input type="submit" value="Dodaj" />
</form>
[PHP] pobierz, plaintext 

nie pisałem narazie nic wiecej, sprawdzania poprawnosci wpisanych danych itp.
czytałem kilka tematów na tym forum, starałem się nie popełniać błędów innych... wątpie żeby mi się to udało.. ale

mam kilka pytań (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)
1. co jest źle? co napisalibyście inaczej?
2. jak to poprawnie napisać stosując szablony MVC?
z tego co wiem to strona widoku może mieć PHP ale tylko podstawowa skladnia typu for, if, while...
tylko jak to rozbić na kontroler i model? czy przy tak prostych klasach jest to możliwe?
3. czy dobrze wykombinowałem z łączenie z bazą danych? czy może ta klasa być dziedziczona przez inne?
4. czy na stronie widoku mogę używać normalnych zmiennych typu $jakas_zmienna ? (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

zaraz pewnie ktoś napisze ze to co zrobiłem to jeszcze nie prawdziwe OOP, ale to dopiero mój początek z OOP więc proszę o wyrozumiałość oraz wskazówki co mógłbym jeszcze tutaj zastosować (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Ten post edytował terabit 9.07.2008, 18:27:43

Pierwsze co się rzuca to brak oddzielenia kodu od treści.

[PHP] pobierz, plaintext 
<?php
public function edytuj_podstrone($id, $menu, $tresc) {
 
	  mysql_query("UPDATE `oop_cms`.`podstrony` SET `menu` = '$menu', `tresc` = '$tresc' WHERE `id` = $id LIMIT 1");
 
  }
?>
[PHP] pobierz, plaintext 

Powinieneś zastosować returny abyś wiedział co dana funkcja zwraca.

[PHP] pobierz, plaintext 
<?php
$this -> db_host = '127.0.0.1'; // Adres serwera
	  $this -> db_name = 'oop_cms'; // Nazwa bazy danych
	  $this -> db_user = 'root'; // Nazwa uzytkownika
	  $this -> db_password = 'root'; // Haslo
?>
[PHP] pobierz, plaintext 

Takie coś jest niezbyt ciekawe. Ja robię to tak przed wywołaniem głównego systemu systemu:

[PHP] pobierz, plaintext 
<?php
$db = new db;
	$db->setParam('user', $ini_array['mysql']['user']);
	$db->setParam('pass', $ini_array['mysql']['pass']);
	$db->setParam('name', $ini_array['mysql']['name']);
	$db->setParam('host', $ini_array['mysql']['host']);
	$db->connect();
?>
[PHP] pobierz, plaintext 

Nie stosuj die a return
Filtruj zmienne

ok ale jak mam np. cos takiego:

[PHP] pobierz, plaintext 
<?php
if(isset($_GET['usun'])) {
  $objPodstrona -> usun_podstrone($_GET['usun']);
}
?>
[PHP] pobierz, plaintext 

to co mam przez zmienna wrzucić echo żeby wyświetliło to co zwraca return?

Masz to źle zrobione powinno być tak:

[PHP] pobierz, plaintext 
<?php
class jakas {
 
function cos() {
 
$this->view->panel_admin($this->zapytanie_do_bazy_z_returnem($_POST['cos']));
}
 
}
 
class view {
 
function panel_admina($result) {
 
echo $result;
//Lub jakiś system szablonów
 
}
 
}
?>
[PHP] pobierz, plaintext 

Musisz przekazać dane z modelu do widoku który je wyświetli

sorry wlamywacz ale nie kapuje :/
widok w osobnej klasie?

Jasne że tak!
M - model - pobiera dane
C - controler - router itd.
V - view - pokazuje dane

ok,
tylko nie wiem jak to wszystko laczyc...

mam model - czyli zarzadzanie wszystkimi danymi w bazie (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) serce aplikacji

pozniej widok - pobiera/wysyla wszystko co mu potrzebne od/do kontrolera...

kontoroler - zarzadza danymi miedzy modelem a widokiem...


teraz tak mam widok,
chce np. usunac wpis z bazy
czyli mam jakas funkcje z kontrolera...
przekazuje mu te dane...
on je obrabia, sprawdza czy wszystko z nimi ok, jesli tak to przekazuje do modelu ktory juz sie nimi odpowiednio zajmuje? (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)


hmm czy dobrze to rozumiem ?

tylko jak te dane przykazywać miedzy kontrolerem a modelem? :/

Ten post edytował terabit 9.07.2008, 20:25:10

Nie będę Ci pisał bo na forum naprawdę wiele przykładów jest.

A ja powiem nie stosuje return ani die() w tym przypadku tylko Exception, bo przecież nie udane połączenie właśnie jest wyjątkiem, w ten sposób masz zatrzymanie skryptu jeśli ktoś nie obsłuży, ale jak ktoś obsłuży to ma dane co się stało.
Dlaczego nie die(), bo przerywa działanie skryptu i nic nie można z tym zrobić, a jednak często chcemy np. wyświetlić odpowiedni komunikat, czy coś innego zrobić.
Dlaczego nie return, bo PHP pozwala zignorować to, więc ktoś może nawet nie wiedzieć, że wystąpił błąd a to się okaże dopiero dalej.

Dalej, kontrolerów nie dziedziczysz po klasie połączenia, bo to by przypominało jakbyś ty był potomkiem ubrania, trochę głupie co nie? Przekazujesz do kontrolera instancje obiektu bazy.


Moim zdaniem zamiast brać się za pisanie własnego MVC, mógł byś chociażby nauczyć się Symfony, przejść manual krok po kroku, nawet nie zaglądając jak to działa, byś zrozumiał jak to powinno działać.
Co do pisania w tak ostatnio popularnym skrócie OOP, to masz jeszcze braki w podstawach, więc powinieneś je poprawić, tzn. kiedy się powinno dziedziczyć a kiedy agregować obiekty, dzielenia na odpowiednią funkcjonalność (wywołanie w kontrolerze mysql_query() jest bez sensu, bo po co Ci wtedy klasa bazy danych?, do tego nie wiesz do którego połączenia się odwołujesz)

Mógłbyś to rozwinąć ? Oczywiście sam stosuje:

[PHP] pobierz, plaintext 
<?
function __autoload($class) {
	require_once './class/' . $class . '.class.php';
}
 
try {
 
	$ini_array = parse_ini_file("config.ini", true);
 
	$db = new db;
	$db->setParam('user', $ini_array['mysql']['user']);
	$db->setParam('pass', $ini_array['mysql']['pass']);
	$db->setParam('name', $ini_array['mysql']['name']);
	$db->setParam('host', $ini_array['mysql']['host']);
	$db->connect();
 
	$action = new action;
	$action->setParam('defClass', $ini_array['action']['defClass']);
	$action->setParam('startClass', $ini_array['action']['startClass']);
	$action->setParam('explode', $ini_array['action']['explode']);
	$action->setParam('exceptions', $ini_array['action']['exceptions']);
	$action->action();
 
} catch (Exception $e) {
	echo 'Wystąpił błąd systemu xxxxx.pl: <b>',  $e->getMessage(), "</b>\n";
}
?>
[PHP] pobierz, plaintext 

Ten post edytował wlamywacz 10.07.2008, 11:17:34

Już pominę fakt return w __construct(), ogólnie masz kod:

[PHP] pobierz, plaintext 
<?php
class Something {
 
	/**
	 * If returns -1 means error, other case returns null
	 *
	 * @return integer
	 */
	public function doIt(){
		return -1; 
	}
 
}
$s = new Something();
$s->doIt();
//And go ahead
?>
[PHP] pobierz, plaintext 

Czyli ktoś nie sprawdza czy wystąpił błąd, i nie wie że wystąpił. Takie przypadki można często spotkać jak ludzie korzystają z wbudowanych funkcji, gdzie często informacja o błędzie jest właśnie w tym co zwraca. Najczęściej nie ma problemów, więc nawet nie wiedzą, że powinni to sprawdzać, a jak wystąpią to nie wiadomo co się dzieje. "Przecież wszystko działa"

EDIT poprawa konwencji, nie wiem czemu napisałem nazwę klasy z małej litery.

Ten post edytował Sedziwoj 10.07.2008, 11:31:57

Zawszę sprawdzam co zwraca funkcja więc nie widziałem tego problemu (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Wiesz, można być optymistą i się na tym przejechać. Po co masz ryzykować, że zapomnisz, tak masz pewność, że tego nie zrobisz, a jak coś będzie nie tak, to się dowiesz co.
Gdyby wszystko było tak pięknie jak piszesz, nie tworzyło by się testów jednostkowych, a się robi, bo człowiek nie jest omylny.
Powinno się robić to tak dobrze, jak się potrafi, a w przyszłości zwróci się to. (pamiętaj że Ty pracujesz sam, ale jak jest zespół, to wygląda inaczej, trzeba dbać, aby zawsze było dobrze)