Правильные геттеры и сеттеры php?

Question

[rak123]

Здравствуйте, делаю рандрмайзер для вывода статей случайным образом. В данном классе частично реализован функционал, но геттером выудить значение после задания его сеттером не получается(то есть гетом можно зацепить дефолтное значение переменной из класса, но вот если его изменить, гет перестает работать), как правильно вообще в php описывать get и set?

class Randomizer {
    public $getValue;
    public $randValue;
    
    public function getRandomContent()
    {
        
        return $this->getValue;
    }

    public function setRandomContent($randValue)
    {
        $randValue = mt_rand(0, 77);
        return $this->getValue = $randValue;
    }

   
}

$randomClass = new Randomizer;

$idContent = $randomClass->getRandomContent();
$idContent = $randomClass->setRandomContent($randValue);

Answer 1

[DevMan]

давайте включим мозг и немного подумаем: в вашем коде сначала идёт гетер и только потом сеттер. в этом вообще нет никакого смысла.
о смысле гетеров/сетеров для публичных полей Danny Chase уже высказался.
ооп - хорошая вещь. но, только, если вы понимаете что и зачем делаете, а не потому, что это стильно/модно/молодежно.

Answer 2

[Danny Chase]

1. зачем вообще нужны геттеры и сеттеры если у вас все свойства класса публичные?) обратитесь к ним как к свойству объекта

$randomClass = new Randomizer;
$randomClass->getValue = 100;

и делайте с ними все что угодно)))

Comments

[Danny Chase]

UPD : зачем для вывода статей писать целый класс) если можно просто сделать ORDER BY RAND() на уровне БД)

[Одиночка Айс]

Danny Chase, ООП головного мозга

[DevMan]

Одиночка Айс, ооп головного мозга у автора вопроса.

[Одиночка Айс]

DevMan, я про него и говорил, не про ответившего)

[DevMan]

Одиночка Айс, сорян, не верно понял.

Answer 3

[Дмитрий]

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