Как передать в качестве аргумента функции объект класса внутри его приватного метода?

Question

[karpo518]

Я создал CRUD модель для таблицы БД в виде Active Record. При валидации и сохранении некоторых полей в модели выполняются дополнительные операции. Класс получился достаточно большим. Чтобы сделать архитектуру кода более понятной, я разделил часть полей на группы и вынес их логику в отдельные подмодели, но сами поля оставил в основной модели. Я собирался запускать класс validate_all(), а уже в нём вызывать submodel1->validate(), submodel2->validate().

Но! Чтобы сохранять ошибки валидации в основной модели, а также перезаписывать поля основной модели, подклассам требуется доступ к объекту главной модели. Но я не могу передать в функцию значение $this. Например, я вызываю функцию validate_all() главного класса, а в ней мне нужно вызвать функцию validate_meta дочернего класса. И этот метод должен иметь доступ к экземпляру родительского класса. Как его организовать?

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

$item = new Item();
$item->obj = $item;

Это неудобно, и вообще не уверен, что это сработает

Comments

[karpo518]

VicTHOR, насколько мне известно, в static хранится класс, а не объект. Нужно передавать именно объект

[Максим Федоров]

VicTHOR, хочу отметить, что скорее всего под подклассами имеются в виду не наследники, а зависимости

[Максим Федоров]

почему после вызова submodel1->validate(), submodel2->validate()
нельзя получить ошибки валидации у этих субмоделей? типа submodel1->getValidationErrors()

[Максим Федоров]

VicTHOR, главный и дочерний = объект и зависимость
да и по описанию видно, что речь про ассоциацию

[Максим Федоров]

Должен существовать паттерн, который позволит подкласса получить доступ к классу

Паттерн "Говнокод"

[karpo518]

Максим Федоров, во-первых, чтобы выполнить валидацию субмодели требуются данные из модели. Было бы гораздо удобнее иметь доступ ко всему объекту модели, а не передавать каждое поле сначала в субмодель, а потом в случае изменения передавать обратно. Во-вторых, все поля хранятся в основной модели. Было бы правильнее хранить и ошибки о них в основной модели. А не таскать их туда и обратно из модели в субмодель. Гораздо удобнее у одной модели вызвать get_errors() и получить в шаблоне сразу все ошибки.

Субмодели не имеют собственных полей. Без прямого доступа к объекту родительской модели мне придётся бесконечное количество раз таскать поля из модели в субмодели. И не только поля, но и ошибки валидации. Это дополнительный код. Такие накладные расходы сводят на нет смысл разделения логики на подклассы.

[Максим Федоров]

karpo518, какая-то тогда паутина у вас там... так не сказать, плетите как хотите

Хорошим решением было бы вынести валидацию наружу, раз уж ActiveRecord и всю ответственность НУ ПРЯМО НУЖНО вынести на слой выше...

[Максим Федоров]

VicTHOR, термины верно... и все жу тут нужно понять: наследование там или ассоциация...
ПО терминам -- наследование (тут вы правы), по проблеме -- ассоциация

[Максим Федоров]

VicTHOR,

поля оставил в основном классе. Я собирался запускать класс validate_all(), а уже в нём вызывать submodel1->validate(), submodel2->validate().

на лицо явно есть объект наш и еще два объекта дергаются)

[karpo518]

Максим Федоров, Давайте попробую объяснить по-другому. Потому что как ни крути, я упираюсь в эту проблему. код валидации на несколько десятков строк. Представьте, что я его вынес в класс-валидатор, написанный специально для этой модели. В модели я хочу вызвать Validator::validate($model_object); но не могу. Внутри модели у меня нет $model_object. Как мне передать объект модели в валидатор?

Допустим можно передать сами данные в валидатор в видео массива. Ну а если мне понадобится внутри валидатора прикрутить фильтр, изменяющий данные модели? Как тогда быть? Передавать в валидатор массив полей по ссылке? Мне было бы гораздо удобнее передать в валидатор весь объект модели. Но как это сделать внутри модели?

[Максим Федоров]

karpo518,

В модели я вызову

Когда я говорю наружу -- это наружу, а не во внутрь "в модель"

[Максим Федоров]

karpo518,
крч, не подскажу -- этот ActiveRecord ваш для меня крайне сложный и запутанный... не понимаю, как его представляют простым, когда это невероятная сложность "наружу"

[karpo518]

Максим Федоров, тогда пропадёт вся магия $model->save() с валидацией внутри и раздуется код контроллера

[Максим Федоров]

karpo518,

и раздуется код контроллера

Если раздуется код валидатора (КОТОРЫЙ ВАЛИДИРУЕТ), то код контроллера не раздуется.

Извините, я не совсем в контексте паттерна, потому откланяюсь

[Максим Федоров]

Из удобных способов валидации есть решение в фреймворке Laravel, может быть вам оно будет интересным:
https://laravel.com/docs/7.x/validation

Там валидация в реквесте

[karpo518]

Максим Федоров, мне бы только узнать, как класс в своем приватном нестатическом методе может передать свой экземпляр объекта в другой класс)

[karpo518]

Максим Федоров, у меня самописная ORM. Я вынужден её использовать, потому что пишу модуль для СMS, которая не использует PDO. А все популярные ORM работают именно через PDO. Но дело даже не в этом.

Разберусь с валидатором, потребуется сделать такой же "saver", которому нужно будет брать одни поля, создавать на их основе директории, загружать файлы и в итоге обновить и сохранить основную модель. Этому классу тоже потребуется доступ к объекту основной модели.

Наверно придётся создавать класс обертку для модели, который подружит модель и субмодели между собой. Так как он находится снаружи, у него будет доступ к объектам всех моделей.

[karpo518]

VicTHOR, такой очевидный вариант работает. Странно. Где-то прочитал, что не будет работать, а сам не проверил..

Answer 1

[John Didact]

Мало чего понял... Но, если нужно использовать, так сказать себя в себе, объект класса в методе этого объекта, то $this, думаю, самое то, если я правильно понял.

Comments

[karpo518]

Спасибо. Сам не проверил, полез гуглить и прочитал, что такой вариант не работает. Сейчас проверил. Работает, как нужно