иногда private пишут только потому что так захотелось
16.05.2020, 10:05
Каждый класс создаётся с конкретной целью. Один или несколько публичных методов исполняют контракт класса. Остальные методы вспомогательные, существуют только для обеспечения выполнения основных методов и они не должны быть публичными, поскольку засоряют собой пространство имён класса, чем усложняют его использование к примеру другими людьми .
Не вижу проблемы поменять контракт класса, если он в привате, как оказалось, делает нечто, что может быть полезно извне. Или вообще вынести к херам этот функционал в отдельный класс.
Для относительно неспецифического, то есть, не привязанного к конкретной задаче, класса - вполне частый кейс.
Для относительно неспецифического, то есть, не привязанного к конкретной задаче, класса - вполне частый кейс.
Вынести да, изменить контракт класса который кем то используется - так себе идея, в лучшем случае тесты упадут, в худшем баг в продакшн. Изменение на public усложнит код создавая лишние зависимости. Если бизнес логика изменится, что происходит довольно таки часто, придется проверять все связанные ветки выполнения. На больших проектах single responsibility - хлеб и соль.
Если это, например, хитрая коллекция/холдер/монада, а пртватный метод позволяет узнать некоторое, по мнению автора класса коллекции, "ненужное" свойство внутреннего состояния этой коллекции, то тут только открытие приватного метода, ну или новый метод, вызывающий тот приватный. И ни на какие тесты и прочее это не повлияет, если этот метод не вносит изменений(а вообще, copy on write рулит).
Чтобы написать коммент, необходимо залогиниться
