Witam.

Mam pytanie projektowe.
Tworzę system generujący umowy.
Posiadam w nim klasę Worker zawierającą informacje o pracowniku. Z tym, że do jednej umowy potrzebuję 10 różnych informacji o pracowniku, a do innej umowy potrzebuję jedynie 4 właściwości(takich przypadków może być więcej - do rachunku potrzebuję jedynie imię i nazwisko).
Stąd moje pytanie:
Powinienem utworzyć klasę Worker zawierającą 4 właściwości, a potem rozszerzać ją do WorkerExtended zawierającej 10 właściwości?
Może powinienem utworzyć abstrakcyjną klasę Worker oraz dwie dziedziczące po niej?

Będę bardzo wdzięczny za podpowiedź.
Dzięki.

jesli jedna umowa wynika z drugiej (jest jej rozszerzoną wersją) to powinienes zrobic jedną z 4 ma wlasciwosciami a pozniej z niej odziedziczyc i w dziedziczacej dodac pozostale szesc.

jezeli sa to rozne umowy to raczej klase abstrakcyjną i w nia wrzucic te 4 pola.

ewentualnie niech kazda umowa posiada w sobie liste swoich pól, i tworz je poprzez factory method, zamiast robic osobną klase dla kazdej umowy.

class umowa{
private $fields;
public function addFields($field)
.....
itd

}

Orzeszekk dziękuję za Twoją odpowiedź.

Troszkę chyba chaotycznie napisałem bo troszkę o co innego mi chodziło.

Uzupełniając moje pytanie - co w przypadku, gdy:

[PHP] pobierz, plaintext 
class WorkerSmall {
 
   private $_name;
   private $_surname;
 
}
 
class WorkerNormal {
 
   private $_name;
   private $_surname;
   private $_city;
   private $_address;
 
}
 
class WorkerBig {
 
   private $_name;
   private $_surname;
   private $_city;
   private $_address;
   private $_idNumber;
   private $_peselNumber;
 
}
[PHP] pobierz, plaintext 

I teraz tak:
- do umowy A potrzebuję tylko znikome dane z WorkerSmall,
- do umowy B potrzebuję więcej danych czyli WorkerNormal,
- do umowy C potrzebuję całej wiedzy o pracowniku czyli WorkerBig.

Powinienem zrobić łańcuszek dziedziczeń ?

Dzięki jeszcze raz. Pozdrawiam.

3 poziomy dziedziczenia to jeszcze nie jest za dużo, martwić należy się przy 5 i więcej. Osobiście jednak zastosowałbym tu wzorzec strategia, oddzielne klasy Workerów wymieniane według potrzeb.

strategia będzie duzo lepsza jezeli logika roznych formularzy bedzie się różniła, to zapobiegnie burdlowi w kodziem, wiec to faktycznie moze byc lepsze rozwiazanie.

Tym bardziej ze w PHP refaktoryzacja nalezy do srednio przyjemnych przyjemnosci a nie wiadomo jak mocno sie portal kiedys rozrosnie. lepiej zrobic od razu porządnie.

Jak zrobic strategie pewnie wiesz wiec nie bedziemy pisać.

@tolomei to co rozpisałeś to jakaś kicha, bo jeżeli masz te dane w bazie w tabeli worker to np.

Worker
id_worker
imie,
nazwisko
adres,
miasto,
kod_pocztowy,
nip,
pesel

i w 2 tabeli naprzykład masz dane o zatrudnieniu to klasa worker powinna zwracać wszystkie dane z zapytania i jeżeli będziszz miał

wokers_companies
worker_id,
companies_id

to wtedy rozszerz klasę worker_companies o klasę worker

Bo zaraz będziesz dopisywał następne klasy workerEmail, workerPhone i takie inne h*j wie co.

Też mi się wydaje lipa

skowron-line racja!
Bo ja planowałem trzymać obiekt Worker w jednym z atrybutów klasy Umowa, ale każda umowa wymaga innych danych o pracowniku. Dane o pracowniku bezpośrednio w obiekcie umowy to najlepsze wyjście.

Dziękuję!

Hmmm a dla mnie to co napisałeś skowron-line jest lekko mówiąc.. nie zrozumiałe. Jeżeli będę chciał pobrać markę samochodu to po co mi wiedzieć jaki ten samochód ma silnik, przebieg czy kolor? Druga sprawa to dziedziczenie, które zaproponowałeś - co gdy z worker_companies będę potrzebował danych firmy a nie pracownika? pisać drugą klasę worker_companies, która będzie rozszerzała companies? Trochę absurdalne

Wg mnie to klasa umowy powinna decydować, które dane powinny być dostarczone:

[PHP] pobierz, plaintext 
<?php
 
abstract class Umowy {
 
	protected $_worker;
 
	protected $_company;
 
	public function addWorker($worker) {
 
		$this->_worker = $worker;
	}
 
	public function addCompany($company) {
 
		$this->_company = $company;
	}
 
	public function setFields($fields) {
 
		$this->_worker -> setFields($fields);
	}
}
 
class UmowaA extends Umowy {
 
	public function initWorker() {
 
		$this->setFileds(array('id', 'imie', 'nazwisko'));
	}
}
 
class UmowaB extends Umowy {
 
	public function initWorker() {
 
		$this->setFileds(array('id', 'adres'));
	}
 
	public function initCompany() {
 
		....
	}
}
 
/**
 * Umowa A
 */
$w = new Worker(67);
 
$u = new UmowaA();
$u -> addWorker($w);
$u -> initWorker();
 
 
/**
 * Umowa B
 */
$w = new Worker(1267);
$c = new Company($w -> getWorkerId());
 
$uB = new UmowaB();
$uB -> addWorker($w);
$uB -> addCompany($c);
$uB -> initWorker();
$uB -> initCompany();
[PHP] pobierz, plaintext 

$this->_worker -> setFields($fields); dodaje do zapytania odpowiednie pola

Odpada problem z różną ilością danych i dziedziczeniem