За последние 24 часа нас посетили 51137 программистов и 3179 роботов. Сейчас ищут 1127 программистов ...

абстрактные классы и наследование

Тема в разделе "PHP для новичков", создана пользователем Dimon2x, 9 апр 2017.

Метки:
Страница 2 из 2
  1. mkramer

    mkramer Суперстар
    Команда форума Модератор

    С нами с:
    20 июн 2012
    Сообщения:
    8.600
    Симпатии:
    1.764
    Вес не включил в базовый, потому что он только в одном месте фигурирует. В принципе, можно и включить. Название сделал приватным, поскольку по-хорошему все поля должны быть или приватными, или защищёнными, но не публичными. Одну нельзя - на уровне базового класса мы знаем только, что может быть скидка, а как она вычисляется - определяется производными классами. У тебя куча функций непонятного назначения, и самое главное - клиентский код точно знает, с каким классом он работает - это неинтересно. Так можно было и вообще не городить всё это наследование. На самом деле, тот код, который я показал, тоже далеко не идеальное решение задачи, но тебе пока показывать самое правильное решение рано, ты бы совсем ничего не понял. Прочитай книжку Зандстры.
     
    #26 mkramer, 12 апр 2017
  2. Dimon2x

    Dimon2x Старожил

    С нами с:
    26 фев 2012
    Сообщения:
    2.211
    Симпатии:
    186
    я 100 страниц прочитал
     
    #27 Dimon2x, 12 апр 2017
  3. mahmuzar

    mahmuzar Старожил

    С нами с:
    6 апр 2012
    Сообщения:
    4.631
    Симпатии:
    425
    Адрес:
    РД, г. Махачкала.
    @Dimon2x, твой код стал лучше, чем был, но есть моменты которые надо улучщить.

    PHP:
    1. header('Content-Type: text/html; charset:utf-8');
    3.  
    4. abstract class Product {
    5.  
    6.     private $title;
    7.     private $price;
    8.     private $weight;
    9.     private $discount = 10;
    10.  
    11.     public function getDiscount() {
    12.         return $this->discount;
    13.     }
    14.  
    15.     public function getPrice() {
    16.         return $this->price - (($this->price * $this->getDiscount()) / 100);
    17.     }
    18.  
    19.     function getPriceWithoutDiscount() {
    20.         return $this->price;
    21.     }
    22.  
    23.     function setTitle($title) {
    24.         $this->title = $title;
    25.     }
    26.  
    27.     function setPrice($price) {
    28.         $this->price = $price;
    29.     }
    30.  
    31.     function setWeight($weight) {
    32.         $this->weight = $weight;
    33.     }
    34.  
    35.     function getWeight() {
    36.         return $this->weight;
    37.     }
    38.  
    39.     public function __construct($title, $price, $weight) {
    40.         $this->setTitle($title);
    41.         $this->setWeight($weight);
    42.         $this->setPrice($price);
    43.     }
    44.  
    45. }
    46.  
    47. class Planshet extends Product {
    48.    
    49. }
    50.  
    51. class Potato extends Product {
    52.  
    53.     function getPrice() {
    54.         if ($this->getWeight() > 10000) {
    55.             return parent::getPrice();
    56.         } else {
    57.             return $this->getPriceWithoutDiscount();
    58.         }
    59.     }
    60.  
    61. }
    62.  
    63. class Jacket extends Product {
    64.    
    65. }
    66.  
    67. $planshet = new Planshet('Планшет', 2000, 400);
    68. $potato = new Potato('Картошка', 700, 12000);
    69. $jacket = new Jacket('Куртка', 700, 2000);
    70.  
    71. var_dump($potato->getPrice());
    По условию задачи, у всех продуктов скидка 10%, поэтому в родительском классе определяем этот геттер getPrice с ценой где скидка уже учтена.(такое у тебя условие)
    Для продуктов где скидка может быть другая при определенных условиях переопределяешь getPrice как я уже сделал. а если этот порог не превзойден, вызывается метод определенный в родительском классе, для получения цены без скидки.

    Выще в коде, почти всегда я работаю со свойсвами через геттеры и сеттеры. Это поможет тебе в будущем, вдруг ты решить фильтровать какие либо значения, или же решишь проделать над ними кое-какие операции. Да и для доступа из дочерних классов они нужны.

    И свойства лучше всегда херачить в столбик, это удобно+ ты можешь документировать каждое свойство как полагается.
     
    #28 mahmuzar, 12 апр 2017
    Dimon2x нравится это.
  4. mkramer

    mkramer Суперстар
    Команда форума Модератор

    С нами с:
    20 июн 2012
    Сообщения:
    8.600
    Симпатии:
    1.764
    Ну значит ещё дочитаешь, и начнёшь понимать, о чём я
     
    #29 mkramer, 12 апр 2017
  5. Dimon2x

    Dimon2x Старожил

    С нами с:
    26 фев 2012
    Сообщения:
    2.211
    Симпатии:
    186
    А тут же такие методы не нужны

    PHP:
    1. function setTitle($title) {
    2.         $this->title = $title;
    3.     }
    4.     function setPrice($price) {
    5.         $this->price = $price;
    6.     }
    7.     function setWeight($weight) {
    8.         $this->weight = $weight;
    9.     }
    В конструкторе же можно так прописать

    PHP:
    1. $this->title = $title;
    2. $this->price = $price;
    3. $this->weight = $weight;
     
    #30 Dimon2x, 12 апр 2017
  6. mahmuzar

    mahmuzar Старожил

    С нами с:
    6 апр 2012
    Сообщения:
    4.631
    Симпатии:
    425
    Адрес:
    РД, г. Махачкала.
    Можно. Но представь, что ты захочешь проделать какие либо операции с входными данными? Будешь это все городить в контсрукторе?
    Всегда работай с свойсвами через сеттеры и геттеры.
     
    #31 mahmuzar, 12 апр 2017
  7. Dimon2x

    Dimon2x Старожил

    С нами с:
    26 фев 2012
    Сообщения:
    2.211
    Симпатии:
    186
    Я ещё сделал доставку на все продукты - 250 рублей, но если на продукт была скидка - 300 рублей

    PHP:
    1. <?php
    2. header('Content-Type: text/html; charset:utf-8');
    4. abstract class Product{
    5.     private $title;
    6.     private $price;
    7.     private $weight;
    8.     private $discount = 10;
    9.     static $delivery = 250;
    10.    
    11.     public function incDelivery(){
    12.         return self::$delivery+50;
    13.     }
    14.    
    15.     public function getDiscount(){
    16.         return $this->discount;
    17.     }
    18.    
    19.     public function getPrice(){
    20.         return round($this->price - ($this->price * $this->getDiscount()/100));
    21.     }
    22.    
    23.     function getPriceWithoutDiscount() {
    24.         return $this->price;
    25.     }
    26.    
    27.     public function getTitle(){
    28.         return $this->title;
    29.     }
    30.    
    31.     public function getWeight(){
    32.         return $this->weight;
    33.     }
    34.    
    35.     public function dostavka(){
    36.         if($this->getPriceWithoutDiscount() > $this->getPrice()){
    37.             return $this->incDelivery();
    38.         }
    39.        
    40.         else{
    41.             return self::$delivery;
    42.         }
    43.     }
    44.     public function __construct($title, $price, $weight){
    45.         $this->title = $title;
    46.         $this->price = $price;
    47.         $this->weight = $weight;
    48.     }
    49. }
    50.  
    51. class Planshet extends Product{}
    52.  
    53. class Veggies extends Product{
    54.     function getPrice(){
    55.         if($this->getWeight() > 10000){
    56.             return parent::getPrice();
    57.         }
    58.         else{
    59.             return $this->getPriceWithoutDiscount();
    60.         }
    61.     }
    62. }
    63.  
    64. class Clothing extends Product{}
    65.  
    66. $planshet = new Planshet('Asus', 2000, 400);
    67. $veggies = new Veggies('Морковка', 700, 12000);
    68. $veggies2 = new Veggies('Лук', 700, 3000);
    69. $clothing = new Clothing('Куртка', 700, 2000);
    70.  
    71. ?>
    72. <!DOCTYPE html>
    73. <html>
    74.     <head>
    75.         <meta charset="utf-8">
    76.     </head>
    77.     <body>
    78.         <?php
    79.        
    80.             echo $planshet->getTitle() . ' цена без скидки ' . $planshet->getPriceWithoutDiscount() . ', со скидкой ' . $planshet->getPrice() . ', доставка ' . $planshet->dostavka();
    81.             echo '<br>';
    82.             echo $veggies->getTitle() . ' цена без скидки ' . $veggies->getPriceWithoutDiscount() . ', со скидкой ' . $veggies->getPrice() . ', доставка ' . $veggies->dostavka();
    83.             echo '<br>';
    84.             echo $veggies2->getTitle() . ' цена без скидки ' . $veggies2->getPriceWithoutDiscount() . ', со скидкой ' . $veggies2->getPrice() . ', доставка ' . $veggies2->dostavka();
    85.             echo '<br>';
    86.             echo $clothing->getTitle(). ' цена ' . $clothing->getPrice() . ', доставка ' .  $clothing::$delivery;
    87.         ?>
    88.     </body>
    89. </html>
     
    #32 Dimon2x, 12 апр 2017
  8. mahmuzar

    mahmuzar Старожил

    С нами с:
    6 апр 2012
    Сообщения:
    4.631
    Симпатии:
    425
    Адрес:
    РД, г. Махачкала.
    Не вижу где ты определяешь составила ли скидка 300р

    PHP:
    1. <?php
    2.  
    3. header('Content-Type: text/html; charset:utf-8');
    5.  
    6. abstract class Product {
    7.  
    8.     private $title;
    9.     private $price;
    10.     private $weight;
    11.     private $discount = 10;
    12.  
    13.     public function getDiscount() {
    14.         return $this->discount;
    15.     }
    16.  
    17.     public function getPrice() {
    18.         return round($this->price - ($this->price * $this->getDiscount() / 100));
    19.     }
    20.  
    21.     function getPriceWithoutDiscount() {
    22.         return $this->price;
    23.     }
    24.  
    25.     public function getTitle() {
    26.         return $this->title;
    27.     }
    28.  
    29.     public function getWeight() {
    30.         return $this->weight;
    31.     }
    32.  
    33.     public function costDelivery() {
    34.  
    35.         $discount = $this->getPriceWithoutDiscount() - $this->getPrice();
    36.         switch ($discount) {
    37.             case ($discount > 400):
    38.                 return 200;
    39.             case ($discount > 300):
    40.                 return 250;
    41.             default :
    42.                 return 300;
    43.         }
    44.     }
    45.  
    46.     public function __construct($title, $price, $weight) {
    47.         $this->title = $title;
    48.         $this->price = $price;
    49.         $this->weight = $weight;
    50.     }
    51.  
    52. }
    53.  
    54. class Planshet extends Product {
    55.    
    56. }
    57.  
    58. class Veggies extends Product {
    59.  
    60.     function getPrice() {
    61.         if ($this->getWeight() > 10000) {
    62.             return parent::getPrice();
    63.         } else {
    64.             return $this->getPriceWithoutDiscount();
    65.         }
    66.     }
    67.  
    68. }
    69.  
    70. class Clothing extends Product {
    71.    
    72. }
    73.  
    74. $planshet = new Planshet('Asus', 2000, 400);
    75. $veggies = new Veggies('Морковка', 10000, 12000);
    76. $veggies2 = new Veggies('Лук', 700, 3000);
    77. $clothing = new Clothing('Куртка', 700, 2000);
    78.  
    79. var_dump($veggies->costDelivery());

    Теперь самое время подумать о рефактоинге. :D
    Советую перечитать/прочитать о полиморфизм и композиции.
     
    #33 mahmuzar, 12 апр 2017
  9. Dimon2x

    Dimon2x Старожил

    С нами с:
    26 фев 2012
    Сообщения:
    2.211
    Симпатии:
    186
    Теперь я правильно сделал абстрактный класс? Подскажите, что можно добавить?

    PHP:
    1. <?php
    2. header('Content-Type: text/html; charset=utf8');
    3.  
    4. abstract class Animal{
    5.     public $name;
    6.    
    7.     public function __construct($name){
    8.         $this->name = $name;
    9.     }
    10.    
    11.     public function eat(){
    12.         echo 'ням-ням';
    13.     }
    14.    
    15.     abstract public function voice();
    16. }
    17.  
    18. class Cat extends Animal{
    19.     public function voice(){
    20.         return  'Мяу-мяу';
    21.     }
    22. }
    23.  
    24. class Dog extends Animal{
    25.     public function voice(){
    26.         return 'Гав-гав';
    27.     }
    28. }
    29.  
    30. $cat = new Cat('Мурка');
    31. $dog = new Dog('Мухтар');
    32.  
    33. echo $cat->name . ' говорит ' . $cat->voice();
    34. echo '<br>';
    35. echo $dog->name . ' говорит ' . $dog->voice();
     
    #34 Dimon2x, 18 апр 2017
  10. mkramer

    mkramer Суперстар
    Команда форума Модератор

    С нами с:
    20 июн 2012
    Сообщения:
    8.600
    Симпатии:
    1.764
    Ну класс сделал, но так его использовать Ну скучно же. Этот код будет работать и без абстрактных классов. Ну хотя бы в массив их объедини и перебором. Или сделай функцию, которая принимает Animal, а передай её $dog и $cat, просто, чтоб почувствовать, зачем это всё нужно - все эти абстрактные классы и прочие приблуды
     
    #35 mkramer, 18 апр 2017
    Dimon2x нравится это.
  11. Dimon2x

    Dimon2x Старожил

    С нами с:
    26 фев 2012
    Сообщения:
    2.211
    Симпатии:
    186
    @mkramer так?

    PHP:
    1. <?php
    2. header('Content-Type: text/html; charset=utf8');
    3.  
    4. abstract class Animal{
    5.     public $name;
    6.  
    7.     public function __construct($name){
    8.         $this->name = $name;
    9.     }
    10.  
    11.     public function eat(){
    12.         echo 'ням-ням';
    13.     }
    14.  
    15.     abstract public function voice();
    16. }
    17.  
    18. class Cat extends Animal{
    19.     public function voice(){
    20.         return  'Мяу-мяу';
    21.     }
    22. }
    23.  
    24. class Dog extends Animal{
    25.     public function voice(){
    26.         return 'Гав-гав';
    27.     }
    28. }
    29.  
    30. $cat = new Cat('Мурка');
    31. $dog = new Dog('Мухтар');
    32.  
    33. function createAnimal($animal, $name){
    34.     return new $animal($name);
    35. }
    36.  
    37. $cat = createAnimal('Cat', 'Мурка');
    38. echo $cat->name . ' говорит ' . $cat->voice();
     
    #36 Dimon2x, 19 апр 2017
  12. igordata

    igordata Суперстар
    Команда форума Модератор

    С нами с:
    18 мар 2010
    Сообщения:
    32.408
    Симпатии:
    1.768
    та ни
    PHP:
    1. $animals = [];
    2. $animals[] = new Cat('Мурка');
    3. $animals[] = new Dog('Мухтар');
    4. function doSomeWork(Animal $creature) {
    5.   echo $creature->name . ' does ' . $creature->eat();
    6.   echo $creature->name . ' says ' . $creature->voice();
    7. }
    8.  
    9. foreach ($animals as $animal) {
    10.   doSomeWork($animal);
    11. }
     
    #37 igordata, 19 апр 2017
    mkramer и Dimon2x нравится это.
(Вы должны войти или зарегистрироваться, чтобы разместить сообщение.)
Страница 2 из 2