Witam

Już kiedyś pisałem że chciałem się wdrożyć w programowanie obiektowe w PHP

Napisałem cosbie taki oto kod składający się z 3 klas:

[PHP] pobierz, plaintext 
<?php
 
class Car{
 protected $mark,$color,$mileage;
 
 protected  function __construct($mark,$color,$milleage){
			 $this->mark=$mark;
			$this->color=$color;
			$this->mileage=$milleage;
		   }
 
 protected	function getMark(){
			 return $this->mark;
			}
 
 protected	function getColor(){
			 return $this->color;
			  }
 
 protected	function getMileage(){
			 return $this->mileage;
			 }
 
 private	function getAllPropertiesOfCar(){
			 return $this->getMark().' '.$this->getColor().' '.$this->getMileage();
			 }
 
 }
 
 
class Person extends Car{
 protected $name,$surname,$age,$profession,$auto;
 
 protected	function setName($name){
			 $this->name=$name;
			 }
 
 protected	function setSurname($surname){
			 $this->surname=$surname;
			  }
 
 protected	function setAgeAndProfession($age,$profession){
			 $this->age=$age; $this->profession=$profession;
			 }
 
 protected	function giveAllPropertiesOfPerson(){
			 print $this->name.' '.$this->surname.' '.$this->age.' '.$this->profession.'<br><br>';
 
			 }
 
 
 public	   function chooseYourCar($mark,$color,$milleage){
			 $this->mark=$mark;
			 $this->color=$color;
			 $this->mileage=$milleage;
			 }
 
 
 
 
  public	 function showNewCar(){
			 print $this->name. ' bought a new car. It`s: '.$this->getMark().' the color is: '.$this->getColor().' and the milleage is not to bad: '.$this->getMileage();
			 }
 
 }
 
 
 
 class Family extends Person{
 private $address,$fortune,$volume;
 
	 function __construct($address,$fortune,$volume){
 
		$this->address=$address;
		 $this->fortune=$fortune;
		 $this->volume=$volume;
	 }
 
 
 public	function addPerson($name,$surname,$age,$profession){
		   $this->setName($name);
		  $this->setSurname($surname);
		  $this->setAgeAndProfession($age,$profession);
		  }
 
 
 public function printFamily(){
		$this->giveAllPropertiesOfPerson();
		}
 
 public function printPropertiesOfFamilyCar(){
		$this->showNewCar();
		}
 
 
 public function hugePersonOfFamily(Person $p){
 print'<br>';
 return $this->name.' is hugging '. $p->name;
 }
 
 };
 
 
 
 
 
 
 
 
 
 // CIAŁO
 
 $family= new Family('London',240000,4);
 $family->addPerson('John','Kowalski',46,'Programmer');
 
 $family_a=new Family('Krakow',120000,2);
 $family_a->addPerson('Izabel','Kowlaska',44,'Designer');
 
 $family->printFamily();
 $family_a->printFamily();
 
 $family->chooseYourCar('volvo','red' ,230000 );
 $family->showNewCar();
 print $family->hugePersonOfFamily($family_a);
 
?>
[PHP] pobierz, plaintext 

Proszę znawców o zweryfikowanie czy dobrze rozumiem pojecie obiektowości. Zastosowałem oddzielne klasy dla obiektów, dodałem dla nich unikalne funkcje gdzie niekiedy inne obiekty korzystają z funkcji innych obiektów.

Proszę o opinie czy moje wypocinki można nazwać choź troszkę OOP

EDIT:

oto wynik zwracany przez powyższy kod

class Person extends Car
A od kiedy to osoba może być samochodem lub przejmować jego cechy? Osoba może mieć samochód.

Dziedziczenie służy do dziedziczenia podobnych cech dla podobnych obiektów. Przykład:
klasa figura
klasa trojkat dziedziczy po figura
klasa kwadrat dziedziczy po figura

klasa samochod
klasa samochodosobowy dziedziczy po samochod
klasa samochodciezarowy dziedziczy po samochod

klasa osoba nie dziedziczy po samochod.
Obiekt osoba może mieć samochód.


Podobnie z rodziną. Rodzina zawiera w sobie osoby. Rodzina nie jest osobą i po niej nie dziedziczy

Dzięki za odpowiedź

1) W takim razie wszystko będzie poprawnie jeśli zrezygnuję z dziedziczenia klasy Car po Family i zmienię odpowiednio wywoływanie metod

2) czy poza tym błędem jest jeszcze coś nie tak

3) I odpowiedź na pytanie czy to już OOP czy nie

ad1) mowilem o dwoch zlych dziedziczeniach

ad2) Jak poprawisz tamto to pogadamy

ad3) Przy bledach co zrobiles to jeszcze to nie jest OOP

Nie wiem czy dobrze zrozumiałem

[PHP] pobierz, plaintext 
<?php
class Family{
 private $address,$fortune,$volume;
 private $person; // <-- TERAZ MA W SOBIE OSOBY 
?>
[PHP] pobierz, plaintext 

private $person; // <-- TERAZ MA W SOBIE OSOBY
Nie, teraz ma w sobie jedną osobe.

Czyli rozumiem że trzeba tu zrobić kolekcję tych osób w postaci tablicy

Czy o to chodziło

[PHP] pobierz, plaintext 
<?php
class Family{
 private $address,$fortune,$volume;
 private $person,$name; // <-- TERAZ MA W SOBIE OSOBY 
 
 
	 function __construct($address,$fortune,$volume){
 
		$this->address=$address;
		 $this->fortune=$fortune;
		 $this->volume=$volume;
	 }
 
 public function addPerson(){
 
	   $this->person=array(
	   'name'=>$name);
 
 }
?>
[PHP] pobierz, plaintext 

pomóżcie bo naprawdę chce się nauczyć OOP

[PHP] pobierz, plaintext 
<?php
class Family{
 private $address,$fortune,$volume;
 private $persons,$name; 
 
 
	 function __construct($address,$fortune,$volume){
 
		$this->address=$address;
		 $this->fortune=$fortune;
		 $this->volume=$volume;
$this->persons = array();
	 }
 
 public function addPerson($person){
 
	   $this->persons[] = $person;
 
 }
?>
[PHP] pobierz, plaintext 

edit:
PRzeciez tym kodem:

[PHP] pobierz, plaintext 
<?php
public function addPerson(){
 
	   $this->person=array(
	   'name'=>$name);
 
 }
?>
[PHP] pobierz, plaintext 

Za kazdym wywolaniem addPerson nadpisywalbys zmienna $person. Gdzie tu tablica osob? Pomijam już fakt, że $name to z kosmosu bierzesz.
Moze zamiast za OOP to weź się najpierw za podstawy php?

Ziomus, zacznij od zaimplementowania tego:

[PHP] pobierz, plaintext 
<?php
$family = new Family(  'Longon', 240000 );
$john = new Person( 'John', 'Kowalski', 46 );
$john->setOccupation( 'Programmer' ); // bo moze byc przeciez bezrobotny, nie?
$bob = new Person( 'Bob', 'Kowalski', 10 );
$family->addPerson( $john );
$family->addPerson( $bob );
 
$family_a = new Family('Krakow',120000,2); 
$iza = new Person( 'Izabela', 'Kowlaska', 44 );
$iza->setOccupation( 'Designer' );
 
 
$family->printFamily(); // jak chcesz mega wypasione OOP to "printera" mozesz do innej klasy przeniesc, ale tutaj powinna ta metodka wystarczyc
// ma wypisac wsyzstkich czlonkow rodziny
$family_a->printFamily();
 
$family->addCar( new Car( 'Volvo', 'red', 230000 ) );
$family->printLatestCarInfo(); // moze korzystac wewnetrznie z $car->printInfo(); zalezy od potrzeb
 
// print $family->hugePersonOfFamily($family_a); - bez sensu - rodzina ma przytulac rodzine, a tak na prawde przytula 1 osobe?? WTF
 
$john->giveHugTo( $iza ); // wystarczy?
?>
[PHP] pobierz, plaintext 

3 klasy, 0 dziedziczenia

Witam ponownie

@dr_bonzo

napisałem to co poleciłeś poszło całkiem dobrze już chyba pomalutku zaczynam rozumieć OOP.

[PHP] pobierz, plaintext 
<?php
 
class Family{
private $city,$fortune,$volume;
public $person;
public $car;
 
		function __construct($city,$fortune,$volume){
		$this->city=$city;
		$this->fortune=$fortune;
		$this->volume=$volume;
		$this->person=array();
		}
 
public	function addPerson($person){
		$this->person[]=$person;
		}
 
public	function addCar($car){
		$this->person[]=$car;
		}
 
public function printFamily(){
	/**   foreach($this->person as $key => $value)
			  print $this->person[$key] -> name;
			  print $this->person[$key] -> surname;
			  print $this->person[$key] -> age;
			  print $this->person[$key] -> occupation;
			  print '<br>';
			  print_r($this->person);
	   }  NIESTETY NIE DZIAŁA POPRAWNIE :(*/
			for($int = 0; $int < count($this->person); $int++){
			print $this->person[$int]->name .'<br>';
			print $this->person[$int]->surname .'<br>' ;
			print $this->person[$int]->age .'<br>';
			print $this->person[$int]->occupation .'<br><br>';
			}
	   }
};
 
class Person{
public $name,$surname,$age;
public $occupation;
 
		function __construct($name,$surname,$age){
		$this->name=$name;
		$this->surname=$surname;
		$this->age=$age;
		}
 
public	function setOccupation($occupation){
		$this->occupation=$occupation;
		   }
 
public function huge(Person $p){
		print $this->name.' is hugging '.$p->name;
		}
};
 
class Car{
private $mark,$color,$mileage;
 
public  function __construct($mark,$color,$milleage){
		 $this->mark=$mark;
		$this->color=$color;
		$this->mileage=$milleage;
		}
 
public function printLatestCarInfo(){
	  print $this->mark.' '.$this->color.' '.$this->mileage;
}
};
 
// CIAŁO
$family= new Family('London','240000',2);
$john= new Person('John','Kowalski',46);
$john->setOccupation('Programmer');
$bob= new Person('Bob','Kowalski',10);
$family->addPerson($john);
$family->addPerson($bob);
 
$family_a= new Family('Krakow','120000',1);
$iza= new Person('Izabela','Nowak',44);
$iza->setOccupation('Designer');
$family_a->addPerson($iza);
 
$family->printFamily();
print'Wydruk drugiej rodzinki';
$family_a->printFamily();
 
$family->addCar($car=new Car('Volvo','red',230000));
print'Wydruk samochodu rodzinki<br>';
$car->printLatestCarInfo();
print'<br>Ściskają się<br>';
$john->huge($iza);
?>
[PHP] pobierz, plaintext 

WYNIK



Może jakiś kolejny przykład


@ NOSPOR




uznam to za złośliwość z Twojej strony ( wczoraj było dosyć późno ... nie każdy jest robokopem a każdy potrzenuje wypoczynku stąd być może moje błędy )

@kosmos: OOP to jest to co ja napisalem, reszta to implementacja
I proboj pisac zaczynajac wlasnie od tej strony, a nie od razu od implementacji [nie mowie ze zawsze, ale czasem jest to lepsza droga,a juz na pewno do poznania oop].

edit: no i skoro ci kod dziala - to jest ok

@dr_bonzo
Dzięki wielkie

w niedługim czasie postaram się napisać coś większego i swojego ( bardziej użytecznego ). Może wtedy wkleje tu kilka wierszy kodu do przeanalizowana

Pozdrawiam

Pokazujesz kod OOP i prosisz o ocene i wskazanie bledow. Dostales to odemnie. Dostales tez sugestie bys jednak zaczal od nauki podstaw bo z tym tez u Ciebie krucho. Jak chcesz to traktuj to sobie jako zlosliwosc. Wybacz, ale ja nie wiem czy ty gadasz ze mna na forum po nieprzespanej nocy, czy po trzech nieprzespanych nocach. Oddajesz cos pod ocene, znaczy ze chcesz oceny. Koniec kropka. Jak jestes zmeczony to sie przespij i napisz kod dnia nastepnego. Ludzie jak cos analizują Twojego to poswiecają na to swoj czas, wiec oddawaj cos co piszesz będąc w pelni swiadomym. Szanuj czas innych, ktorych prosisz o pomoc.

Widze ze dales pomogl dla dr_bonzo. Ok. Ale ja Ci nie pomogłem? Pokazałem ci na samym początku jakie robisz podstawowe błedy. dr_bonzo niejako to podsumował nie dodając nic więcej. Odnosze wrażenie, ze skoro napisalem bys sie za podstawy wziął to mnie w ten sposob karzesz :/ Dziwne postepowanie.
Nie zeby mi zalezalo na pomogl od Ciebie. Chce ci jedynie zwrocic uwage bys na przyszlosc troche inaczej postepowal.

edit: dziękuję za Pomogł