Куда лучше поместить метод?

@Passerby · Регистрация: 22.09.2017

Студворк — интернет-сервис помощи студентам

У базового класса есть наследники: классы A, B, C, D. У классов A, B, C есть один и тот же метод, который использует поля базового. Куда его лучше поместить: в базовый класс, тогда этот метод будет и у D, которому он не нужен; или сделать интерфейс для этих трех классов? И еще возможность сделать метод статическим и поместить его куда-нибудь, к примеру в тот же базовый класс и передавать ему в параметры поля.

@nekonihilist · 06.04.2019, 14:01

Я бы сделал интерфейс, если опираться на правила SOLID по крайней мере

@aenye · 06.04.2019, 14:02

Passerby, ИМХО, логичнее описать метод в интерфейсе и реализовать его в тех классах, в которых он нужен.

@OwenGlendower · 06.04.2019, 14:31

Passerby, можно сделать protected метод в базовом. То что он не будет использоваться в классе D не является проблемой.

@kolorotur · 06.04.2019, 14:43

Сообщение от Passerby

базового класса есть наследники: классы A, B, C, D. У классов A, B, C есть один и тот же метод, который использует поля базового. Куда его лучше поместить: в базовый класс, тогда этот метод будет и у D, которому он не нужен;

Ну тогда сделайте промежуточный класс "Базовый с половиной", в котором и будет определен этот метод, который нужен всем, кроме D:

C#
1
2
3
4
5
6
7
8
9
10
11
12
class Base { }
class Derived : Base
{
   protected void Foo()
   {
   }
}
 
class A : Derived { }
class B : Derived { }
class C : Derived { }
class D : Base { }

@Passerby · 06.04.2019, 16:23 **[ТС]**

Думаю сделать, как посоветовал OwenGlendower: лучше одна лишняя ссылка на метод в D, чем лишний интерфейс или лишний промежуточный класс, в которых, даже если они пустые, масса ссылок. Или я в этом рассуждении неправ?

@kolorotur · 06.04.2019, 20:45

Сообщение от Passerby

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

Не улавливаю связи между глубиной наследования и количеством ссылок.

@Kazbek17 · 06.04.2019, 20:51

Вообще интерфейси и нужны для проектирование бизнес логики. Если правильно смотреть с точки зрения, весь алгоритм держится на интерфейсах. А там крути верти как хочешь, и показывай то что тебе нужно.

@ShmelVG · 06.04.2019, 23:52

Passerby, Недостаток наследования – хрупкий базовый класс. Базовый класс может обладать ненужным функционалом, полученным от других его базовых классов, чего можно избежать, применяя интерфейсы.

@Passerby · 07.04.2019, 14:55 **[ТС]**

Сообщение от kolorotur

Не улавливаю связи между глубиной наследования и количеством ссылок.

Я имел в виду, что даже пустой объект класса имеет ссылки (которые не факт, что понадобятся) Equals, GetHeshCode, GetType, ToString. И класс также, как и интерфейс ссылки на статические ReferenceEquals, Equals.
Т.е. в случае класса и интерфейса, в D из первоначального вопроса, получаем не одну ссылку на метод, а ссылку на класс или интерфейс со своими лишними в данном случае ссылками.

@kolorotur · 07.04.2019, 15:15

Сообщение от Passerby

Я имел в виду, что даже пустой объект класса имеет ссылки (которые не факт, что понадобятся) Equals, GetHeshCode, GetType, ToString.

У вас какие-то очень странные представления о расположении объектов в памяти.
Глубина наследования вообще никак не влияет на количество ссылок в конкретном экземпляре.

@Passerby · 07.04.2019, 15:32 **[ТС]**

kolorotur, поясните, как происходит обращение к функции базового класса при наличии промежуточного (как вы посоветовали). Я считал, что происходит лишнее вычисление адреса.

@kolorotur · 07.04.2019, 16:24

Сообщение от Passerby

как происходит обращение к функции базового класса при наличии промежуточного

Зависит от того, как этот метод вызывается и от того, как реализована таблица виртуальных методов в среде выполнения.
Либо производится сразу обращение к нужной реализации, либо в таблице методов указан адрес ближайшей переопределенной версии, либо переопределенная версия ищется вверх по цепочке наследования.

Сообщение от Passerby

Я считал, что происходит лишнее вычисление адреса.

В смысле "лишнее"?

Да, может происходить, если рантайм реализует виртуальный вызов последним из перечисленных выше способов.
Вы это называете ссылкой и вас это беспокоит?

@Passerby · 07.04.2019, 17:40 **[ТС]**

Сообщение от kolorotur

Вы это называете ссылкой и вас это беспокоит?

Не знаю, правильно ли я вас понял. Я имел в виду, что выигрыша нет по следующим причинам:
1) С точки зрения программиста, т.е. с точки зрения читаемости кода введение промежуточного класса облегчает читаемость за счет закрытия в классе в котором не нужен метод ссылки на него, но при этом ухудшает, т.к. вводится новый класс или интерфейс.
2) С точки зрения повышения скорости кода, и уменьшения памяти под программу, то тоже выигрыша нет, а есть незначительный проигрыш, т.к. есть лишний класс и лишние вычисления (хотя, и очень незначительные).
И почему нет такой фички, как закрытие видимости метода или свойства для конкретных наследующих классов: в этом случае количество интерфейсов и классов в коде значительно бы уменьшилось.

@КОП · 07.04.2019, 17:44

Сообщение от Passerby

И почему нет такой фички

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

Сообщение от Passerby

для конкретных наследующих классов

базовый класс не знает ничего о наследниках. это критический удар по читаемости и поддерживаемости

@Passerby · 07.04.2019, 18:00 **[ТС]**

Сообщение от КОП

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

За счет чего понижала бы читаемость?

Сообщение от КОП

базовый класс не знает ничего о наследниках. это критический удар по читаемости и поддерживаемости

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

@КОП · 07.04.2019, 18:25

Сообщение от Passerby

За счет чего понижала бы читаемость?

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

Сообщение от Passerby

В чем удар по читаемости и поддерживаемости?

есть у меня лист цветных фигур. я в цикле пробегаюсь по ним и вывожу только цвет фигуры с соответствующим индексом. петя решил что его треугольник - цветная фигура, но скрыл метод получения цвета.
теперь мой код не компилируется или падает в рантайме. а после исправления КАЖДЫЙ вызов ЛЮБОГО метода находится в try catch, вдруг его в след реализации наследника решат скрыть

@Passerby · 07.04.2019, 18:36 **[ТС]**

Сообщение от КОП

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

Ну так для этого и делаются промежуточные классы и интефейсы. Что читабельней и проще создавать лишний класс или интерфейс (чтобы скрыть базовый метод или поле) или пометить в наследующем классе каким-нибудь словом.

Сообщение от КОП

есть у меня лист цветных фигур. я в цикле пробегаюсь по ним и вывожу только цвет фигуры с соответствующим индексом. петя решил что его треугольник - цветная фигура, но скрыл метод получания цвета.

В чем отличие от интерфейса или промежуточного класса?

Сообщение от КОП

вдруг его в след реализации наследника решат скрыть

И в чем отличие, как его будут скрывать: пометить словом или написав промежуточный класс?

@КОП · 07.04.2019, 18:47

Сообщение от Passerby

Ну так для этого и делаются промежуточные классы и интефейсы.

правильно, реализуйте вариант kolorotur и будет вам счастье. вроде сами понимаете, а делать не хотите.

Сообщение от Passerby

чтобы скрыть базовый метод или поле

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

Сообщение от Passerby

В чем отличие от интерфейса или промежуточного класса?

этоб был пример, почему нельзя вдруг скрыть метод базового класса

Сообщение от Passerby

И в чем отличие, как его будут скрывать: пометить словом или написав промежуточный класс?

это вы же предложили, вот и придумывайте. я лишь привел пример как сломается весь код, если разрешить скрывать методы

В варианте с промежуточным классом петя унаследовался бы от "фигуры", а не "цветной фигуры" и его треугольник никак бы не оказался в моем листе "цветных фигур"

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

@Passerby · 07.04.2019, 19:12 **[ТС]**

Сообщение от КОП

насколько я знаю невозможно скрыть публичный

Не только публичный, но и протектед.

Сообщение от КОП

я лишь привел пример как сломается весь код, если разрешить скрывать методы

Мне ваш пример непонятен. Если вы сделали класс в котором объявлены методы, поля, то в чем разница как вы решили скрыть что-либо: тем кто будут использовать этот класс без разницы как вы скрыли из-за чего и сломался их код: из-за введения промежуточного класса или просто применили новое ключевое слово. Я знаю, что такого слова нет: это видно из вопроса.

Новые блоги и статьи Все статьи Все блоги /
Символьное дифференцирование igorrr37 13.02.2026 / * Логарифм записывается как: (x-2)log(x^2+2) - означает логарифм (x^2+2) по основанию (x-2). Унарный минус обозначается как ! */ #include <iostream> #include <stack> #include <cctype>. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .	И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»
SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image 8Observer8 10.02.2026 Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .	Установка Qt-версии Lazarus IDE в Debian Trixie Xfce volvo 10.02.2026 В общем, достали меня глюки IDE Лазаруса, собранной с использованием набора виджетов Gtk2 (конкретно: если набирать текст в редакторе и вызвать подсказку через Ctrl+Space, то после закрытия окошка. . .

@Passerby -7 / 3 / 1 Регистрация: 22.09.2017 Сообщений: 242

	Куда лучше поместить метод? 06.04.2019, 13:56. Показов 3127. Ответов 67 Метки нет (Все метки) У базового класса есть наследники: классы A, B, C, D. У классов A, B, C есть один и тот же метод, который использует поля базового. Куда его лучше поместить: в базовый класс, тогда этот метод будет и у D, которому он не нужен; или сделать интерфейс для этих трех классов? И еще возможность сделать метод статическим и поместить его куда-нибудь, к примеру в тот же базовый класс и передавать ему в параметры поля. 0

@nekonihilist 38 / 27 / 13 Регистрация: 06.11.2018 Сообщений: 141
	06.04.2019, 14:01
	Я бы сделал интерфейс, если опираться на правила SOLID по крайней мере 1

@aenye 304 / 186 / 45 Регистрация: 05.07.2018 Сообщений: 580
	06.04.2019, 14:02
	Passerby, ИМХО, логичнее описать метод в интерфейсе и реализовать его в тех классах, в которых он нужен. 1

@OwenGlendower Администратор 18285 / 14208 / 5368 Регистрация: 17.03.2014 Сообщений: 28,888 Записей в блоге: 1
	06.04.2019, 14:31
	Passerby, можно сделать protected метод в базовом. То что он не будет использоваться в классе D не является проблемой. 1

@Passerby -7 / 3 / 1 Регистрация: 22.09.2017 Сообщений: 242
	06.04.2019, 16:23 [ТС]
	Думаю сделать, как посоветовал OwenGlendower: лучше одна лишняя ссылка на метод в D, чем лишний интерфейс или лишний промежуточный класс, в которых, даже если они пустые, масса ссылок. Или я в этом рассуждении неправ? 0

@Kazbek17 1484 / 939 / 454 Регистрация: 06.02.2012 Сообщений: 2,868
	06.04.2019, 20:51
	Вообще интерфейси и нужны для проектирование бизнес логики. Если правильно смотреть с точки зрения, весь алгоритм держится на интерфейсах. А там крути верти как хочешь, и показывай то что тебе нужно. 1

@ShmelVG 171 / 106 / 51 Регистрация: 27.11.2015 Сообщений: 1,269
	06.04.2019, 23:52
	Passerby, Недостаток наследования – хрупкий базовый класс. Базовый класс может обладать ненужным функционалом, полученным от других его базовых классов, чего можно избежать, применяя интерфейсы. 1

@Passerby -7 / 3 / 1 Регистрация: 22.09.2017 Сообщений: 242
	07.04.2019, 15:32 [ТС]
	kolorotur, поясните, как происходит обращение к функции базового класса при наличии промежуточного (как вы посоветовали). Я считал, что происходит лишнее вычисление адреса. 0