В чем загвоздка данного вопроса и как «идеально» решается такое?

Question

[Антон Середний]

В задании нужнор написать класс Cat, который наследуется от класcа Animal. Класс Animal имеет метод getName (name можно передать в конструктор).

Класс Cat имеет метод meow (возвращает строку «Cat {catname} is sayig meow».

Пример использования:

$cat = new Cat ('garfield');
$cat->getName () === 'garfield' // true;
$cat->meow () === 'Cat garfield is saying meow' // true;

Вот моя реализиция... Что не так с ней? Почему проверяющий "завернул" ее?

<?php
abstract class Animal {
	public $name;
	public function __construct($cat_name) {
		$this->name = $cat_name;
	}
}

class Cat extends Animal {
	public function meow() {
		echo "<br>Cat {$this->name} is saying meow ";
	}
	public function getName() {
		return $this->name;
	}
}

$cat = new Cat('garfield');
echo $cat->getName();
$cat->meow();

Comments

[Антон Середний]

дополнил вопрос.

[Sergej]

Антон Середний, Все так, но в чем вопрос?

[Антон Середний]

Doc, дополнил вопрос. Дело в том, что моя релаизация не понравилась проверяющему. Что не так с ней?

[Sergej]

Антон Середний, Смотря что требовалось.
Объект задачу выполняет.
Другое дело, что типы принимаемы и возвращаемых данных можно указать, добавить сеттер и геттер для имени, сделать имя не публичным.... и еще с десяток улучшений.

[Sergej]

Как то так вероятно. Естественно объекты разносятся по разным файлам и должны иметь неймспейсы.

/**
 * Class AbstractAnimal
 */
abstract class AbstractAnimal
{

    /**
     * @var string
     */
    private $name;

    /**
     * AbstractAnimal constructor.
     * @param string $cat_name
     */
    public function __construct(string $cat_name)
    {
        $this->name = $cat_name;
    }

    /**
     * @return string
     */
    public function getName(): string
    {
        return $this->name;
    }

}

/**
 * Interface CatInterface
 */
interface CatInterface
{
    /**
     * @return string
     */
    public function meow();
}

/**
 * Class Cat
 */
class Cat extends AbstractAnimal implements CatInterface
{
    /**
     * @return string
     */
    public function meow(): string
    {
        return  'Cat ' . $this->getName() . ' is saying meow ';
    }
}

$cat = new Cat('garfield');
echo $cat->meow();
echo '<hr>'. $cat->getName();

Answer 1

[Антон Антон]

Не совпадение с заданием в пункте

Класс Animal имеет метод getName

Также метод meow() должен возвращать строку, а не выводить ее. Ну и <br> там быть не должно

Comments

[SharuPoNemnogu]

а еще метод meow() должен возвращать строку, а не выводить ее, судя по

$cat->meow () === 'Cat garfield is saying meow' // true;

[Антон Антон]

SharuPoNemnogu, согласен, дополнил ответ

Answer 2

[nozzy]

Имеется getter:

public function getName() {
    return $this->name;
  }

Я бы сделал $name => protected, вместо public.

Answer 3

[Вячеслав]

Потому что " Класс Animal имеет метод getName"

Answer 4

[Александр Аксентьев]

Никакой загвоздки нет.

Comments

[Антон Середний]

Тоже не вижу, но как-то слишком просто, чтобы быть правду. Может, параноя уже

[Владислав Гайдук]

Класс Animal имеет метод getName

У тебя этот метод в классе cat