Можно ли вот так разделять класс?

Question

[Max Ba]

Мужики, строго не критикуйте) Вопрос исключительно по грамматике.
Есть класс, в нем два метода. И я вдруг решил разделить методы по разным файлам. Не спрашивайте "зачем мне это". Меня интересует реализация!!! Верно ли я сделал в целом?

class A{
  public $val = 10;
  public function method1(){
    $m2 = new B();
    return $m2->method2();
  }
}

class B extends A{
  public function method2(){
    return $this->val + 1;
  }
}

$a = new A();
echo $a->method1();

Comments

[Mikhail Osher]

нет

[Max Ba]

Mikhail Osher, почему?)

[Владислав Лысков]

ты через неделю сам в этом разберёшься?

[Max Ba]

Владислав Лысков, сложно сказать. на практике такое не применял. Ну а вообще, разве нельзя никак разделить класс на разные файлы? Когда все в одном это жесткач.

[Александр Новиков]

Макс Васильев, разделять класс на 2 по отдельным файлам можно и нужно (декомпозиция), но с умом и в определенных случаях
PSR-4 и composer autoload так и работают, 1 класс = 1 файл, 2 класса в одном файле нельзя

Ответы выше скорее к тому, что в примере классы не в файлах, а в одном блоке кода - это раз

И их содержимое - это мешанина из нечитаемых названий классов, имен переменных и имен методов с перекрестными вызовами
+ имеется инстанцирование и дальнейший вызов метода наследника из родителя, хотя по идее родителю знать о наследниках нет никакого смысла

[Max Ba]

Ну а если я решил сделать 1 класс в двух файлах, теоретически это ведь как-то можно сделать? Ну не так ведь делать же)

class Class1{
  public $val = 10;
  public function methodOne(){
    require_once 'Class1One.php';
  }

public function methodTwo(){
    require_once 'Class1Two.php';
  }
}

Answer 1

[hack504]

Если нужно работать в рамках одного экземпляра класса то можно попробовать так

class A{
  public $val = 10;
  public function method1(){
    $m2 = new B($this);
    return $m2->method2();
  }
}

class B {
  public function __construct($obj) {$this->obj = $obj;}
  public function method2(){
    return $this->obj->val + 1;
  }
}

$a = new A();
echo $a->method1();

Правда все методы и свойства класса должны быть только публичными, но судя по всему это не критично.

Comments

[Max Ba]

Поэтому я и сделал extends, чтоб не передавать сам объект во второй.
Понял, спасибо. Рассматривал мельком и такой подход, кстати весьма грамотнее моего. Сейчас конкретно это видно.

[hack504]

Для extends тогда не нужно в рамках одного метода подгружать класс. Получается обычное наследование.

class B {
  public function method2(){
    return $this->val + 1;
  }
}
class A extends B {
  public $val = 10;
  public function method1(){
    return $this->method2();
  }
}

$a = new A();
echo $a->method1();

Answer 2

[ipokos]

Работать это будет. Если опишите в чем цель, возможно будет более адекватное решение. (например трейты)
Но если вы "разделяете" класс, т.к. в нем много методов и вы хотите их структурировать, это значит изначально что-то пошло не так.

Comments

[Max Ba]

Я просто хочу, чтобы определенный функционал (какой-нибудь большой метод сложного расчета, ну например) находился не внутри основного класса, а подключался. При этом подключаемому методу нужно знать все про родителя, поэтому он и наследник.

[ipokos]

Пример с трейтом

class A{
// объявление трейта
    use Count;

    public $val = 10;

    public function method1(){
// вызов метода трейта
        $this->calculation();

        echo $this->val;
    }
}

trait Count {
    public function calculation() {
// доступ из трейта к свойству класса в котором он объявлен
        $this->val++;
    }
}

$a = new A();
echo $a->method1();

[Max Ba]

ipokos, изучу, спасибо

Answer 3

[Павло Пономаренко]

Впринципе, можно. Но то, что класс А знает о своем наследнике, классе В - пахнет оооочень стрёмно.