9

У меня есть класс машина и класс электромашина. Дело в том, что класс электромашина наследует от класса машина поле Fuel и мне ее нужно удалить или переименовать просто потому-что у электромашины нет топлива а есть заряд.

class Car
{
    public int MaxSpeed = 180;
    public int Range = 100;
    public int CurrentSpeed = 100;
    public int Fuel = 100;
    public void Drive()
    {
        if (Fuel <= 0)
            throw new Exception("Not enough fuel");
        else
        {
            this.Fuel--;
            Range -= Range / CurrentSpeed;
        }
    }
}
class ElectroCar : Car
{
    int Charge;
    public ElectroCar()
    {
       Charge = base.Fuel;
    }
}
2
  • А почему бы не сделать поле которое лишнее просто приватным, не уверен про C# но в С++ такое легко делается, тогда у класса потомка это поле не будет доступным.
    – Arty
    21 окт 2017 в 16:31
  • 1
    @ArtyOneSoul: Сделать-то такое можно. Вопрос не в этом. Вопрос в смысле наследования: производный класс должен быть разновидностью базового. А электрическая машина не является разновидностью машины с топливом.
    – VladD
    21 окт 2017 в 19:21

2 ответа 2

15

Никак.

Если у класса-предка есть поля, не имеющие смысла в классе-потомке, то вы неправильно выстроили иерархию наследования.

Есть разные пути исправить ситуацию. Например: выделите общее у Car и ElectroCar в базовый класс Vehicle, Car переименуйте в FuelCar, и свойства, относящиеся к топливу, пускай содержит только FuelCar.

10
  • тоже раньше об этом думал, думал есть альтернатива, спасибо!
    – user245150
    21 окт 2017 в 15:08
  • @Duoxx: Пожалуйста!
    – VladD
    21 окт 2017 в 15:14
  • @Duoxx, поправка: не CarBase, а Vehicle. Ведь выделенные общие черты можно наследовать не только для различных видов автомобилей, но и для троллейбуса, к примеру. 21 окт 2017 в 16:39
  • @Duoxx поддерживаю Arhad, правилнее Vehicle, тогда и упряжку с осликом тоже можно наследовать.
    – 0xdb
    21 окт 2017 в 17:03
  • А почему не сделать Fuel свойством - Fuel {private get; private set;} ? 21 окт 2017 в 17:07
1

Я предлагаю добавить перечисление Source

enum Sources
{
    Gas,
    Diesel,
    Petrol,
    Kerosene,
    Electricity   
}

Поле fuel заменить на powerSource типа Sources и добавить поле числовое powerIndex которое характеризует числовой показатель расхода энергии

1
  • 2
    Идея-то хорошая. Но вопрос, в каких единицах powerIndex? А если у нас гибридная машина, нужно два powerIndex'а? Не, если уж переходить к композиции, сделайте PowerSource полноценным интерфейсом, и пусть каждая имплементация работает по-своему.
    – VladD
    21 окт 2017 в 19:25

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.