Warning: Cannot use a scalar value as an array in /home/admin/public_html/forum/include/fm.class.php on line 757

Warning: Invalid argument supplied for foreach() in /home/admin/public_html/forum/include/fm.class.php on line 770
Форумы портала PHP.SU :: Версия для печати :: Изменение видимости при наследовании
Форумы портала PHP.SU » » Объектно-ориентированное программирование » Изменение видимости при наследовании

Страниц (1): [1]
 

1. ArsWolfBeast - 19 Февраля, 2016 - 16:55:30 - перейти к сообщению
Столкнулся тут с такой проблемой. Пытаюсь сделать в классе-наследнике более строгие настройки видимости.
PHP:
скопировать код в буфер обмена
  1.  
  2. class ClassOne {
  3.         public $i;
  4. }
  5.  
  6. class ClassTwo extends ClassOne{
  7.         protected $i;
  8. }
  9.  

Скрипт полностью вешается, скрип при этом полностью вешается. Не выдает вообще ничего, ни сообщений об ошибках, ни нужного текста. А вот назначить более мягкие правила видимости позволяет. Вопрос - это оно действительно невозможно назначить более строгие правила потомкам, или у меня какой-то локальный глюк?

Версия пхп 5.4.45
2. DelphinPRO - 19 Февраля, 2016 - 17:10:23 - перейти к сообщению
ArsWolfBeast пишет:
Не выдает вообще ничего

Нужно включить вывод ошибок. и вы увидите заветный fatal error
CODE (htmlphp):
скопировать код в буфер обмена
  1. Fatal error: Access level to a::b() must be public
3. ArsWolfBeast - 19 Февраля, 2016 - 22:20:50 - перейти к сообщению
Благодарю.

Вроде стоит и в php.ini, и в .htaccess и в настройках скрипта стоит показывать все типы ошибок.

Значит так и должно быть. Хотя, имхо, логичнее было бы наоборот. Будем придумывать костыль.
4. Мелкий - 20 Февраля, 2016 - 09:36:01 - перейти к сообщению
ArsWolfBeast пишет:
Значит так и должно быть. Хотя, имхо, логичнее было бы наоборот.

Как раз не логичнее.
Все наследники могут вести себя различным образом, но контракт базового класса соблюдать обязаны. Если в базовом классе был публичный метод - этот метод обязан быть во всех наследниках. Если было публичное свойство - оно обязано быть в наследнике и быть тоже публичным.
Иначе теряем все прелести наследования. Вместо лаконичного
PHP:
скопировать код в буфер обмена
  1. function somefunction(baseclass $obj)
  2. {
  3.     echo $obj->foo;
  4. }

Ведь мы знаем, что у базового класса foo - публичен и можем спокойно передавать любой подкласс.
Получится простыня условий: для объекта такого класса надо делать X, для такого - вообще невозможно, для третьего - через метод Y

И вот это - немаловажная причина почему публичные свойства - зло и должны быть геттеры и сеттеры. Их запросто в подклассе можно переопределить, будет изменение поведения, но сохранён контракт.
5. ArsWolfBeast - 20 Февраля, 2016 - 13:57:01 - перейти к сообщению
Мелкий пишет:
И вот это - немаловажная причина почему публичные свойства - зло и должны быть геттеры и сеттеры. Их запросто в подклассе можно переопределить, будет изменение поведения, но сохранён контракт.

Ну переменные у меня не публичные (кроме редких исключений). Столкнулся я с этим, когда хотел поменять видимость методов. Пример привел с переменными просто для сокращения кода.
6. MiksIr - 20 Февраля, 2016 - 16:09:20 - перейти к сообщению
Почитайте про LSB (Принцип подстановки Барбары Лисков)

 

Powered by ExBB FM 1.0 RC1