Логика в геттере возможна ли?

Question

[Мега Синяк]

Есть геттер, дающий бонус 20% к хп, если страна - Беларусь, применяется так:

// геттер сущности Test для примера
$this->hp = 100; // берётся из БД

public function getHp(): int
{
    return $this->country_code == 'by' ? $this->hp * 1.2 : $this->hp;
}

// где-то в дебрях для примера
public function test(Test $test)
{
    $test->hp = $test->getHp() - 20;
    $test->save(); // пишем 80, если Беларусь - 100
}

Просто пишут везде "гуру Ларавеля", что не должно быть логики в гетерах-сеттерах.

Могу ли я делать такой геттер ну вот в данном случае?
Если нет, то почему?

Comments

[Дмитрий]

Можете

[Мега Синяк]

Дмитрий, а что не в ответ-то, что за скромность? :)

[Мега Синяк]

Дмитрий, ээээээ неееееет,

$this->hp = 100; // берётся из БД

public function getHp(): int
{
    return $this->country_code == 'by' ? $this->hp * 1.2 : $this->hp;
}

public function test(Test $test)
{
    $test->hp = $test->getHp() - 20;
    $test->save(); // пишем 80, если Беларусь - 100
}

// ПРОДОЛЖАЕМ
$this->hp; // 80, если Беларусь - 100

public function getHp(): int
{
    return $this->country_code == 'by' ? $this->hp * 1.2 : $this->hp;
}

public function test(Test $test)
{
    $test->hp = $test->getHp() - 20;
    $test->save(); // пишем 60, если Беларусь - 100
}

Такое вот крутое обломилово))) Беларусь всегда живой будет)))

Answer 1

[Евгений]

Просто это не getter, а decorator.

Comments

[Мега Синяк]

Буду теперь везде всех гуру отшивать этим словом)

[Мега Синяк]

а вот и засада)))

$this->hp = 100; // берётся из БД

public function getHp(): int
{
    return $this->country_code == 'by' ? $this->hp * 1.2 : $this->hp;
}

public function test(Test $test)
{
    $test->hp = $test->getHp() - 20;
    $test->save(); // пишем 80, если Беларусь - 100
}

// ПРОДОЛЖАЕМ
$this->hp; // 80, если Беларусь - 100

public function getHp(): int
{
    return $this->country_code == 'by' ? $this->hp * 1.2 : $this->hp;
}

public function test(Test $test)
{
    $test->hp = $test->getHp() - 20;
    $test->save(); // пишем 60, если Беларусь - 100
}

[Boris Köln]

Мега Синяк, говнокод это

[Евгений]

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

[Дмитрий]

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

[Мега Синяк]

Дмитрий, какая разница я напрямую свойство заполню или оно будет приватным и я его сеттером заполню)))

взяв начальное значение геттером

[Дмитрий]

Мега Синяк, ну так вы на прямую без обработки не получите

[Мега Синяк]

Boris Korobkov,

говнокод это

нет, это - декоратор

[Мега Синяк]

Дмитрий, я уже сообразил что к чему, спс

Answer 2

[toxa82]

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

Answer 3

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

Нет

Геттеры могут быть в транспортных классах и иногда в инфраструктурных/библиотечных.

Иначе это нарушение инкапсуляции: открывают доступ к тому, что закрыли. Как и сеттеры — дают доступ к тому, что закрыто, но на запись.
Крч чит, хотя соответствуют определению инкапсуляции и дату-хиддингу

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