0

Хочу добавить string мой метод find_next. Пытаюсь наследовать:

class advstring:public string{
public:
string find_next(string str);
  static int i;
  advstring.find // выдаёт ошибку (advstring не содержит find)
};

Как отнаследовать так, чтобы в advstring были все методы string?

1
  • Для "отнаследовать так, чтобы в advstring были все методы string" ничего особенного делать не надо - все само собой получится. Но у вас написана синтаксическая каша. Сначала исправляйте ее, а уж потом будете беспокоиться о наследовании. 21 сен 2016 в 20:16

2 ответа 2

4

1) код, который Вы привели - неверный. Неверный синтаксически. то есть, метод find_next должен выглядеть как минимум где то так

string find_next(string str);
  static int i;
  advstring.find(/*здесь параметры*/);
  return /*что то возвратить*/;
};

2) static int i; - это поле будет общее для всех экземпляров данного класса. Поэтому поиск может давать самые интересные результаты.

3) наследоваться от std::string не стоит. У него деструктор невиртуальный. А это грозит утечками памяти.

В данном случае лучше не наследоваться, а делегировать. То есть, создать класс, у которого есть приватное поле типа std::string и выносим отдельно все нужные методы.

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

1
  • "А это грозит утечками памяти." Это грозит неопределенным поведением. Зачем снова упоминать этот расхожий миф про некие "утечки памяти" - совершенно не ясно. 21 сен 2016 в 20:11
2

От себя добавлю уже сказанное - класс std::string изначально не разрабатывался, как базовый. В нем нет виртуальных функций, как и нет виртуального деструктора. С точки зрения законов ООП, наследоваться от него запрещено, но при большом желании можно - компилятор ничего не скажет вам, но обязательно будут проблемы.

Вариант расширить его функционал - создать класс с композицией (включающий член данных) std::string

class advstring {
public:
  std::string find_next(const std::string& str);
private:
  std::string m_str;
}

Проблема такой реализации в том, что хочешь не хочешь в классе advstring придется продублировать в секции public весь открытый функционал std::string и прокинуть мосты через член класса m_str

Проще релизовать свободную функцию если в ее теле не требуется обращения к закрытым членам базового класса std::string

std::string find_next(const std::string& str);
4
  • "...но обязательно будут проблемы". Проблемы тут могут быть только в одном случае - при попытке полиморфного динамического удаления, т.е. delete объекта-наследника через указатель типа std::string *. Если не делается попыток полиморфного delete, то и никаких проблем не будет. Поэтому ни о каком "обязательно" речи быть не может. 21 сен 2016 в 20:13
  • Полностью согласен. Это я, конечно, перегнул палку на счет обязательно будут.. 21 сен 2016 в 20:32
  • Сегодня не делают попыток, а завтра кто то может и сделать.
    – KoVadim
    21 сен 2016 в 20:39
  • @KoVadim: Завтра кто-то может и на ноль разделить. Но это не повод запрещать операцию деления или даже избегать ее. 21 сен 2016 в 21: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.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.