Poczuc OOP - zakladanie bazy firm Opcje

[wojckr]

Witam,
postaram się pisać zwięźle i na temat.
Dotychczas projektowalem proceduralnie. Zapoznalem sie z odstawiami i filozofia OOP. Jednak jeszcze tego nie "czuje".
Chcę stworzyć bazę firm przy użyciu OOP. Nie wiem czy dobrze mysle, ale obiektem bylaby firma, ktora mialaby swoja nazwe, adres,telefon itd.(własciwosci), a firme ta moznaby bylo dodawac, usuwac, edytowac i wyswietlac (metody).
Sprobowalem projektowac klase, ale zauwazylem, ze wrzucam funkcje do klasy i w zasadzie proceduralnie zrobilbym to chyba szybciej...

Czy mozecie podpowiedziec jako doswiadczeni w OOP, jak zabralibyscie sie do tworzenia takiej bazy firm z uzyciem OOP i czy do tego wystarczy jedna klasa czy wiecej niz jedna?

Dzieki wielkie z gory za wskazowki (mam wrazenie, ze potrzebuje kilku wskazowek, a potem pojdzie z gorki).

[kicaj]

Wklej tutaj to co dotychczas zrobiles, podpowiemy co zmienic, poprawic, tak by bylo to OOP:)

[dr_bonzo]

1. skoro sie uczysz czegos nowego to bedziesz to robil N razy wolniej niz znana tobie metoda
2.uzyj jakiego frameworka, pozostanie ci tylko zamodelowanie dziedziny (danych o firmach) + kontrolery/html
3. opisz wiecej co chcesz zrobic, bo baza firm w najprostszej postaci to jedna klasa : Firma, a mozesz miec np. branże itd.

[wojckr]

Dzięki za pierwsze wskazowki :-)

Za frameworki zabiore sie innym razem, na razie chce pojąć OOP, a N razy wolniejsze robienie to fakt, ale może zaprocentuje?

Wklejam _wstępny_szkielet_ klasy:

[PHP] pobierz, plaintext 
<?
 
include '../53/mysqlconn.inc.php';
 
class firma {
	public $id;
	public $nazwa;
	public $adres;
	public $telefon;
 
	function __construct() {
		$sql=mysql_query("SELECT * FROM firmy LIMIT 1 ;");
		if (mysql_num_rows($sql)>0) {
			$row=mysql_fetch_assoc($sql);
			$this->id = $row['id'];
			$this->nazwa = $row['nazwa'];
			$this->adres = $row['adres'];
			$this->telefon = $row['telefon'];
		} else {
			echo 'Brak firm.';
		}
	}
 
	function dodajFirme() {
		//pomijam walidację (a moze powinno się już tearaz ją uwzględnić?)
		if (mysql_query("INSERT INTO firmy SET nazwa='".$this->nazwa."', adres='".$this->adres."', telefon='".$this->telefon."'  ;")) {
			echo 'Dodano firme';
		} else {
			echo 'Nie dodano firmy';
		}
	}
 
	function updateFirmy() {
		if (mysql_query("UPDATE firmy SET nazwa='".$this->nazwa."', adres='".$this->adres."', telefon='".$this->telefon."' WHERE id=".$this->id."  ;")) {
			echo 'Uaktualniono firmę.';
		} else {
			echo 'Nie uaktualniono firmy.';
		}
	}
 
	function usunFirme() {
		if (mysql_query("DELETE FROM firmy WHERE id=".$this->id."  ;")) {
			echo 'Firma usunieta.';
		} else {
			echo 'Firma nie usunieta.';
		}
	}
 
	function pokazFirme() {
		echo '<LI>Nazwa: '.$this->nazwa;
		echo '<LI>Adres: '.$this->adres;
		echo '<LI>Telefon: '.$this->telefon;
	}
 
	function formFirma() {
		echo '<form action="" method="post">';
		echo '<input type="hidden" name="id" value="'.$this->id.'"><BR>';
		echo 'Nazwa: <input type="text" name="nazwa" value="'.$this->nazwa.'"><BR>';
		echo 'Adres: <input type="text" name="adres" value="'.$this->adres.'"><BR>';
		echo 'Telefon: <input type="text" name="telefon" value="'.$this->telefon.'"><BR>';
		echo '<input type="submit" value="dodaj" name="formFirma">';
		echo '<input type="submit" value="usun" name="formFirmaUsun">';
		echo '</form>';
	}
}
 
$firmaA = new firma;
 
if (isset($_POST['formFirma']) and $_POST['id']=='') {
	$firmaA->nazwa=$_POST['nazwa'];
	$firmaA->adres=$_POST['adres'];
	$firmaA->telefon=$_POST['telefon'];
	$firmaA->dodajFirme();
} elseif (isset($_POST['formFirma']) and $_POST['id']!='') {
	$fimraA->id=$_POST['id'];
	$firmaA->nazwa=$_POST['nazwa'];
	$firmaA->adres=$_POST['adres'];
	$firmaA->telefon=$_POST['telefon'];
	$firmaA->updateFirmy();
} elseif (isset($_POST['formFirmaUsun']) and $_POST['id']!='') {
	$fimraA->id=$_POST['id'];
	$firmaA->usunFirme();
} else {
	$firmaA->pokazFirme();
	$firmaA->formFirma();
}
 
 
?>
[PHP] pobierz, plaintext 

Co do tego co chcę zrobić, to moje założenia:
1. Firmy podzielone na branze: branzaGlowna1 skladajaca sie z podBranzy1, podBranzy2 itd. (na razie dwa poziomy, ale potem bycmoze podbranze beda sie skaldac z podpodbranz), branzaGlowna2, 3, 4 itd.
2. Na stronie wyswietlają się branże główne i podbranże. Po kliknieciu na najniższy poziom (teraz podBranza, np. na podBranza3 branzyGlownej7) pokazuje sie lista firm z danej podbranzy.
3. Po kliknieciu na firme, pokazują się szczegolowe dane o firmie.
4. Teraz mozna edytować dane o tej firmie lub usunąć.
5. Po edycji/usunięciu w panelu administracyjnym zatwierdzam lub nie, ew. coś jeszcze poprawiam, jesli byla edycja. ALBO Jesli byla zalogowana uprawniona osoba, to wprowadzone zmiany od razu sa widoczne na stronie.

6. Jest też możliwość dodania firmy z dowolnego miejsca strony (czy uzytkownik jest na str glownej, czy w jakiejs branzy to moze z tego poziomu dodawac firme).


Oto co chcę zrobić. Zagadką dla mnie jest jakie korzyści osiągnę robiąc taką bazę firm w OOP ?

[Hacker]

Dobrze by było jakby konstruktor pobierał id (__construct($id = 0)) i jak jest 0 to nie pobiera nic...
Początek klasy wygląda tak, jakbyś chciał dla każdej firmy tworzyć nowy obiekt (dobre podejście), ale potem... zrób zamiast dodajFirme i updateFirmy metodę save.
Pole $id powinno być private lub protected jak będziesz dziedziczył po tej klasie.
Echo w klasie nie wygląda zbyt ładnie... Może smarty? I pamiętaj przy tym, żeby nie klasa assignowała...
Patrząc na (X)HTML to najpierw piszesz <LI>, a potem <form>. Pisz wszystko małymi literami jak nowe standardy wymagają!

Ten post edytował Hacker 7.05.2007, 08:09:39

[Sedziwoj]

Pisz wszystko małymi literami jak nowe standardy wymagają!

Nowe standardy wymają małym, jak również zamykanie każdego taga, jak również umieszczenie w jakiejś konstrukcji a nie bezpośrednio po <form>... i pewnie parę innych rzeczy.

Co do użycia Smarty, to raczej MVC niż OOP, ale też bym sugerował zastosowanie MVC, bo kod staje się o wiele czytelniejszy.

Co do konstruktora, to jeśli nie zawsze mają być pobierane dane z bazy, to niech to nie będzie robione w konstruktorze.
Do tego nie tylko właściwości/atrybuty mają 'uprawnienia dostępu', ale również metody.
Nigdzie nie sprawdzasz, czy też nie inicjujesz połączenia z bazą. (i znów można by się czepić brak MVC...) ale również dane pobierane od użytkownika powinny być sprawdzane.

EDIT: lit.

Ten post edytował Sedziwoj 7.05.2007, 08:20:39

[Hacker]

@Sedziwoj nie czepiałem się już tak dokładnie (X)HTML-a, tylko zauważyłem to, że wojckr nie trzyma się konwencji raz pisze dużymi raz małymi...
Jakby nie miały być pobierane dane w konstruktorze to __get, __set lub/i gettery i settery (żeby wszyscy łącznie z mike_mechem byli zadowoleni )


---

~mike_mech

[Cysiaczek]

Zaprezentowałeś klasyczny przykład ujęcia kodu proceduralnego w klasie. Programowanie obiektowe to nie tylko klasy z kodem, ale przede wszystkim relacje pomiędzy obiektami. Obiekt firma może składać się z kilku obiektów, zwłaszcza a Twoim przykładzie, w którym wydzielasz funkcje zapisujące i odtwarzające dane z bazy.
Każda klasa powinna mieć ściśle wydzielone zadanie. Twoja tego nie ma, bo:
1. Obsługuje bazę danych
2. Wypisuje informacje dla użytkownika
3. Przechowuje informacje (o firmie)
4. Sama siebie kontroluje.

Widzę tu co najmniej 3 obiekty.

Również logika nieco szwankuje, bo jeśli za nią pójść, to obiekt Człowiek powinien mieć takie metody:

[PHP] pobierz, plaintext 
<?php
$człowiek= new Czlowiek('cysiaczek');
$czlowiek->dodajCzlowiek('wojckr');
$czlowiek->wypiszCzlowiek();// a może wypiszCzlowieki() ?
?>
[PHP] pobierz, plaintext 

Krótko mówiąc - nie ma to ani rąk, ani nóg.
Tu pomoże dobra książka na temat projektowania, do czego Cię zachęcam.

Małe propozycje:
1. Patrz na obiekty również pod kątem ich użytkowników (kodu, który je użytkuje)
2. Metody powinny zwracać wartości do użytkownika

Pozdrawiam. : )

P.S
Na pocieszenie napiszę, że jedna z moich klas o nazwie News dziedziczyła po obiekcie Pager, który odpowiadał za łamanie stron wyników