0

Есть следующий код:

template<typename Item>
class Queue{
    enum{QUEUE_SIZE=10};
    static int s_int_queue_elem_counter;
    Item item;
    Queue* next;
  public:
    explicit Queue():item(item){}
    explicit Queue(const Item& _item):item(_item){}
    void addQueue(const Item&);
};

Возникает ошибка

Queue::item has incomplete type.

Почему?

8
  • Видимо в момент инстанцирования Item не является "полностью описанным" типом, компилятор не знает какой именно класс создать. Приведите код класса и код точки инстанцирования.
    – fogbit
    10 июл 2012 в 17:02
  • инстанциируется в main.cpp путем вызова Queue<int> A;
    – PaulD
    10 июл 2012 в 17:19
  • Какой компилятор? GCC (ideone.com/fKRSQ) и VS2008 компилируют без ошибок.
    – fogbit
    10 июл 2012 в 17:23
  • MinGW ver 10
    – PaulD
    10 июл 2012 в 17:24
  • MinGW не компилятор, какой версии там g++.exe ?
    – fogbit
    10 июл 2012 в 17:26

2 ответа 2

1

Так а каким item ты инициализируешь item в дефолтном конструкторе ??? ...:item(item) что это за бред ?! хотябы так :item((Item)0);

    template<typename Item>
    class Queue {
            enum{QUEUE_SIZE=10};
            static int s_int_queue_elem_counter;
            Item item;
            Queue* next;

        public:

            explicit Queue():item((Item)0) {} 
            //Тут скорее всего приведет к ошибке, 
            //если вы в качестве Item укажите класс.
            //Так что лучьше этот конструктор убрать.
            //Ключевое слово explicit тут тоже лишнее.

            explicit Queue(const Item& _item):item(_item) {}
            void addQueue(const Item& var) { item += var; };
    };

    int main()
    {
        Queue<int> A;
        Queue<int> B(1);
        A.addQueue(2);
        return 0;
    }

Ваш код (но рабочий):

    template<typename Item>
    class Queue {
            enum{QUEUE_SIZE=10};
            static int s_int_queue_elem_counter;
            Item item;
            Queue* next;

        public:                 
            explicit Queue(const Item& _item) : item(_item) {}
            void addQueue(const Item& var);
    };
7
  • 1
    (Item)0 Очень плохо. А если в качестве аргумента шаблона будет использоваться какой-нибудь тип вроде класса? Что получится?
    – skegg
    10 июл 2012 в 20:01
  • По этому я и написал "хотябы(для этого случая)", т.е. он явно хотел этот item чем то инициализировать, а чем бы вы его инициализировали ?! т.е. там был еще метод addQueue правда пустой, я добавил что то не очень хорошее "item += var;" при котором инициализировать хоть чем то в любом случае надо! а раз его в любом случае надо инициализировать, то нужно было бы дефолтный конструктор убрать, я его оставил намеренно что бы было видно "Почему возникает ошибка". Естественно тут нужно убрать или это дурацкий конструктор или template... Но да, спасибо!:) Надо было сразу это указать=)
    – lirik90
    10 июл 2012 в 20:21
  • Уж лучше ничем. item() Во всяком случае, это будет работать для простых типов и для тех, у которых определен дефолтный конструктор. Для иных нужно делать специализацию шаблона.
    – skegg
    10 июл 2012 в 20:24
  • Хотя общая идея вашего ответа правильная. item(item) совсем никуда не годится.
    – skegg
    10 июл 2012 в 20:27
  • Ну да, а про инициализацию я же и говорю, что если конструктор оставить (и ничем его не инициализировать) то item += var; будет весьма непредсказуемым, и то, опять же он вообще будет только для тех Item у которых оператор += определен...
    – lirik90
    10 июл 2012 в 20:31
0

Т.к. класс Queue шаблонный, то конструкция вида Queue* next, вообще говоря, ошибочная - не указан шаблонный параметр, и указатель на какой именно тип создавать компилятор сам не решит. Исправьте на Queue<Item>* next.

Ваш ответ

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

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