Можете привести пример, когда использовать private, а когда protected?

Question

[Роман Сарваров]

Я вот до конца не могу понять преимущества использования модификатора доступа private вместо protected.
Ведь один из принципов SOLID гласит, что чтобы поменять что-то в классе, не нужно изменять его код - нужно делать это через наследование. Можно ли в таком случае использовать всегда protected?

Можете привести пример, когда лучше использовать private у функций и свойств класса, а когда лучше использовать protected? Можно на любом языке.

Comments

[kafkiansky]

protected - если у класса будут наследники и они будут пользоваться свойствами родителя, private во всех остальных случаях, если у вас не php 7.4.

не нужно изменять его код - нужно делать это через наследование

Сами придумали?

[Роман Сарваров]

kafkiansky, разницу между private и protected в техническом плане я понимаю.
А в чем особенность php 7.4? Не нашел что-то я с этим связанное в ченджлоге.

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

Т.е. по сути функции private мешают наследедоваться.

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

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

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

Answer 1

[Дмитрий]

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

Answer 2

[Северное Сияние]

Мамкины ответчики, написавшие про то, что везде должен быть private, умных книжек не читали, кроме сухой теории и что-то своё, судя по всему, не разрабатывали.

Класс почти ВСЕГДА должен иметь protected свойства. Ибо они ничем не отличаются от private, но позволяют использовать свойства в классах-наследниках, которые очень часто возникают. И что бы не переписывать код, лучше всегда использовать protected.

Ситуация, когда действительно нужен private-метод или свойство - очень редкое явление.

Comments

[Роман Сарваров]

Большинство, как я понимаю, ставит private по умолчанию, но ваш ответ мне всеравно кажется более грамотным и логичным в ООП.

[Северное Сияние]

Роман Сарваров, это не только моё мнение, это мнение выразил однажды преподаватель из какого-то минского университета, который приехал к нам в компанию в 2013 году, когда команда переучивалась с PHP на Java.

Можете привести пример, когда лучше использовать private у функций и свойств класса, а когда лучше использовать protected?

В данном примере я использовал статические методы и свойства, но не суть.
Хранилище счётчиков $data доступно только через методы абстрактного класса-родителя и никак иначе к этому свойству ты не подлезешь.

abstract class Car
{
    private static $data = [];

    protected static function setCounter()
    {
        if (!isset(self::$data[static::class])) {
            self::$data[static::class] = 0;
        }

        self::$data[static::class]++;
    }

    public static function getCounter()
    {
        return self::$data[static::class] ?: 0;
    }
}

class VAZ extends Car
{
    public function __construct()
    {
        parent::setCounter();
    }
}

class BMV extends Car
{
    public function __construct()
    {
        parent::setCounter();
    }
}

new VAZ(); new VAZ(); new VAZ(); new VAZ(); new VAZ();
new BMV();

echo "Класс VAZ вызывался " . (VAZ::getCounter()) . ", BMV вызывался " . (BMV::getCounter());

[Дмитрий]

То что не нужно в наследниках, оно и не нужно, если нужно свойство пиши геттер, лишнее в объекте лучше не носить
Пример на счётчиках вообще странный

[Северное Сияние]

Дмитрий,
1. Ты не можешь твердо знать, особенно на этапе разработки, что нужно будет наследнику, а что - нет.
2. Бессмысленное сокрытие свойств класса потянет за собой кучу геттеров и усложнит код.
3. PRIVATE, по сути, это способ защиты от выстрела в ногу, когда это приведёт к фатальным последствиям. Нужно скрывать действительно то, что жизненно необходимо скрывать.