"ленивый" __set(), есть идеи?

Question

[ThunderCat]

Собствено внутри класса есть массив, в который часто пишутся / читаются данные в коде, вида:
$this->fields['somefield'] = $someVal;
для чтения я сляпал быстрый геттер:

public function __get($name){
        if($name == 'f'){
            return (object) $this->fields;
        }
        else return false;
    }

и теперь $this->f->somefield красиво отдает из массива значение по ключу.
а с сеттером в стиле $this->f->somefield = $someval; что-то я туплю... Хелп...

ПС: всем спасибо, опять же поймал себя на том что не всегда очевидные мелкие детали меняют картину в целом. Изврат нужен для переопределения ключей от нескольких сторонних апи с большим количеством полей, по этому код выходит скучный, мерзкий, однообразный и простынеподобный. Захотелось слегка упростить ручную работу.

Answer 1

[Александр]

spoiler

<?php

class My {
    private $fields = [
        'hello' => 'world'
    ];
    public function f() {
        return $this;
    }
    public function __get($name) {
        if($name == 'f') {
            return (object) $this->fields;
        }
    }
    public function __set($name, $value) {
        $this->fields[$name] = $value;
    }
    public function printFields() {
        print_r($this->fields);
    }
}

$my = new My();

$my->f()->jetpack = 'azaza';
$my->printFields();

Comments

[ThunderCat]

да, как вариант кстати, очень норм. Именно что-то такое и не пришло в голову, спасибо.

Answer 2

[Immortal_pony]

Сеттер должен быть у того класса, который вы отдаете через "быстро сляпанный геттер". А у вас там stdClass.
Можно сделать, например, так:

<?php
class Fields
{
    protected $parent;
    
    public function __construct(\Data &$parent) 
    {
        $this->parent = $parent;
    }
    
    public function __get($name) 
    {
        return array_key_exists($name, $this->parent->fields) ? $this->parent->fields[$name] : false;
    }

    public function __set($name, $value) {
        $this->parent->fields[$name] = $value;
    }
}

class Data
{
    public $fields = [
        'foo' => "bar"
    ];
    
    public function __get($name){
        if($name == 'f'){
            return new \Fields($this);
        }
        else {
            return false;
        }
    }
}

$data = new Data();
$data->f->testInfo = "test passed";
echo $data->fields['testInfo']; // выведет "test passed"

Comments

[ThunderCat]

мысль ясна, подумаю.

Answer 3

[Станислав Б]

$this->f->somefield красиво отдает из массива значение по ключу.

пожалуйста не делайте так. Это абсолютно необязательное запутывание программы. Плюсов минимум.
Лучше всего - удалить __get и продолжайте использовать $this->fields['somefield'] = $someVal;

UPD
например так, но зачем

class Shit 
{
    private $fields = [];

    public function __get($name){
        if($name !== 'f') {
            return false;
        }

        return new class($this->fields) {
            private $fields;

            public function __construct(&$fields)
            {
                $this->fields = &$fields;
            }

            public function __get($name)
            {
                return $this->fields[$name];
            }

            public function __set($name, $value)
            {
                $this->fields[$name] = $value;
            }

            public function __isset($name)
            {
                return isset($this->fields[$name]);
            }
        };
    }
}

Comments

[ThunderCat]

спасибо, но нет ) запутывание программы меня на данном этапе не интересует, интересует конкретно вопрос как сделать, Как не делать я знаю )

ПС: да, нужен сахар, что с ним делать я уже потом раберусь.

[ThunderCat]

и кстати, я посидел подумал - пока не вижу ни одного аргумента версус...

[Александр Маджугин]

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

[ThunderCat]

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