0

Есть класс:

template <typename T>
class cFunctionVector {
public:
    cFunctionVector()
    {
        functionsVector = new vector<T>();
    }

    cFunctionVector(vector<T> _functionsVector)
    {
        functionsVector = _functionsVector;
    }

    void addFunction(T _function)
    {
        functionsVector.resize(functionsVector.size()+1);
        functionsVector.push_back(_function);
    }

    int getAmount()
    {
        return functionsVector.size();
    }

    T getFunctionValue(int _number)
    {
        vector<T>::iterator it = functionsVector.begin(); //проблемная строка
        return *(it + _number);
    }
private:
    vector<T> functionsVector;
};

Компилятор ругается на отмеченную строку, вывод ошибки:

need 'typename' before 'std::vector<T>::iterator' because 'std::vector<T>' is a dependent scope

метод писался для получения i-го элемента поля functionsVector, и ругается, я так понимаю, компилятор на то, что для итератора не задан явно тип (int, float и т.д.), а стоит некий класс T. Но в том то и дело, что вектор в данном классе может быть любого типа. Как можно это исправить? Или я вообще неверно пытаюсь вернуть i-й элемент?

2
  • 1
    А зачем resize() перед push_back() ? Да и в getFunctionValue можно просто return functionsVector[_number];
    – avp
    30 ноя 2012 в 13:23
  • @MorkOFF48, а Вы не пробовали подключить <functional> и использовать класс std::function? Почему-то мне кажется, что Вы уже много дней изобретаете велосипед. Наверняка наличие такого функционала заставило бы Вас задуматься о решении самой задачи, а не о том, какими инструментами ее решать.
    – mega
    30 ноя 2012 в 19:20

3 ответа 3

5

А почему бы и вправду не написать

typename vector<T>::iterator it = ...

? Вот обсуждение, почему это требуется стандартом: Why do I need to use typedef typename in g++ but not VS.

8
  • сделать можно, но в данном случае есть значительно проще и красивее решения.
    – KoVadim
    30 ноя 2012 в 13:40
  • а кстати по поводу итератора, он не будет работать быстрее, чем at в данном случае (как к примеру в случае с указателями на массивы)?
    – MorkOFF
    30 ноя 2012 в 13:47
  • хотите скорость - используйте operator[]. Но он, как и Ваш код имеет один недостаток - не проверяет выход за пределы. А вот at(index) - проверяет. думаю at может оказаться медленее Вашего кода, но не думайте сейчас об этом. Всегда можно заменить на []. А вот код с итератором в данном случае - жуть, которую не нужно писать.
    – KoVadim
    30 ноя 2012 в 13:52
  • @KoVadim: я хотел показать топикстартеру, как решить проблему локально. По поводу глобальной оптимизации, я бы просто заменил cFunctionVector<T> на vector<T> при текущем состоянии кода. Но возможно код упрощён, чтобы не нагружать нас несущественными деталями, поэтому я не рискнул оптимизировать глобально.
    – VladD
    30 ноя 2012 в 14:18
  • 1
    зачем нужен класс, мне понятно. Это же классическая методика при программировании - создание специализированных классов для решения проблемы (то есть задачи). В будущем класс получит новые, специализированные методы. А пока он такой. В противном случае придется делать либо отдельные функции, либо размазывать методы по другим классам. А это уже плохая идея. Плюс, у такого подхода есть хороший плюс при отладке. Ставим бряку внутри класса и отлавливаем все вызовы. А иначе придется искать по всему коду и смотреть во все методы.
    – KoVadim
    30 ноя 2012 в 14:54
4

Начнем с первой проблемы. Переменная functionsVector определяется как вектор типа T

vector<T> functionsVector;

Но в конструкторе почему-то присутствует такая строка

   functionsVector = new vector<T>();

Оператор new возвращает указатель на созданный объект. Думаю, ошибочность этой строки сами увидите.

Как сделать грамотно

cFunctionVector() : functionsVector () {}

По поводу typename в функции T getFunctionValue(int _number), то он должен быть обязательно. Но использование итератора в данном контексте - маразм (об этом уже писали).

Также очень плохо написан другой конструктор. Надо бы так

cFunctionVector(const vector<T> & _functionsVector): 
      functionsVector (_functionsVector)   {}
9
  • @mikillskegg, вот чего я не понимаю, так это зачем вообще нужен класс cFunctionVector? Неужели нельзя обойтись просто вектором для адресов функций?
    – avp
    30 ноя 2012 в 14:48
  • В случае нормальной рабочей программы это было бы абсолютным быдлокодом. Просто сделать typedef. Но в качестве упражнения - почему бы и нет. Цель моей речи было указать на принципиальные ошибки, чтобы ТС чему-то на будущее научился.
    – skegg
    30 ноя 2012 в 14:56
  • Для упражнения, конечно. Тренировка на простой, но более-менее осмысленной задачке.
    – avp
    30 ноя 2012 в 14:58
  • 1
    @mikillskegg: не вызванный деструктор = undefined behaviour, разве нет? stackoverflow.com/a/2034936/276994 update: вот тут говорится, что вызов не того конструктора таки undefined behaviour со ссылкой на релевантную часть стандарта.
    – VladD
    30 ноя 2012 в 16:17
  • 1
    ОК, спасибо, убедили. Но если не использовать позднего связывания, то проблем не будет и в случае с наследованием. В таком случае ТС прав и надо использовать делегирование и has-a.
    – skegg
    30 ноя 2012 в 16:44
3

А зачем тулить итераторы, где они не нужны? Почему просто не вызвать метод at у вектора и не переживать.

T getFunctionValue(int _number)
{
    return functionsVector.at(number);
}
8
  • теперь ругается на это functionsVector = new vector<T>(); no match for 'operator=' in '((cFunctionVector<cFunction*>*)this)->cFunctionVector<cFunction*>::functionsVector = (operator new(12u), (<statement>, ((std::vector<cFunction*, std::allocator<cFunction*> >*)<anonymous>)))'
    – MorkOFF
    30 ноя 2012 в 13:13
  • а, не, понял ошибку, это же не указатель у меня. Тогда теперь другой вопрос, судя по всему элементы не добавляются, ибо при вызове getFunctionValue() ничего не возвращается, хотя было добавлено 2 элемента, да и плюс при этом getAmount() возвращает 4
    – MorkOFF
    30 ноя 2012 в 13:18
  • я бы тоже ругался бы. зачем вызывать new для поля, которое не является указателем? В данном случае, вначале вызывается конструктор, а потом оператор присваивания. А его нет... Если смотреть в код, то строка functionsVector = new vector<T>(); не нужна.
    – KoVadim
    30 ноя 2012 в 13:18
  • 3
    >> и плюс при этом getAmount() возвращает 4 логично. Вы же вначале неявно добавляете один элемент, потом ещё один явно. functionsVector.resize(functionsVector.size()+1); не нужен, push_back сам его сделает. не путаете с reserve ? А ничего не возвращается потому что запрашиваете 0 элемент. А он "пустой".
    – KoVadim
    30 ноя 2012 в 13:22
  • о, спасибо большое, да, перепутал с reserve, и кстати, если не делать reserve, то все равно при добавлении на каждый добавленный элемент размер вектора дважды инкрементируется, при использовании reverse все работает адекватно
    – MorkOFF
    30 ноя 2012 в 13:34

Ваш ответ

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

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