C++ template error I think?

I'm new to C++ but we're supposed to implement a provided template for a linked list. The error I'm getting says:

Code:

---

..\src\dlist.cc:3: error: expected constructor, destructor, or type conversion before '<' token

---

I get this error for my constructor methods in my .cc file. Without giving too much away, here is what my code looks like:

Code:

---

//Constructor
template <typename T>
Dlist<T>::Dlist()
{
        makeEmpty();
}

//Copy constructor
template <typename T>
Dlist<T>::Dlist(const Dlist &l)
{
        makeEmpty();
        copyAll(l);
}

//Destructor
template <typename T>
Dlist<T>::~Dlist()
{
        makeEmpty();
        removeAll();
}

//Equals operator
template <typename T>
Dlist<T>& Dlist<T>::operator=(const Dlist &l)
{
        if(this!=&l)
                copyAll(l);
        return *this;
}
//more methods

---

I'm certain that the header is correct because it was provided. I think I'm just doing something stupidly wrong.

PS. The other methods all say:

Code:

---

error: expected initializer before '<' token

---

Um. Are you including a file that defines DList? If it's not a known class name, then you can't use it before the :: namespace qualifier.

I'm not sure what you mean. My file dlist.cc doesn't have any includes. It starts with the code I posted. There is no code or anything above what I posted, only some methods afterwards. The only other file is dlist.h which includes dlist.cc

dlist.h:

Code:

---

#ifndef __DLIST_H__
#define __DLIST_H__

class emptyList
{
    // OVERVIEW: an exception class
};

template <typename T>
class Dlist
{
    // OVERVIEW: contains a double-ended list of Objects

 public:

    // Operational methods

    bool isEmpty();
    // EFFECTS: returns true if list is empy, false otherwise

    void insertFront(T *o);
    // MODIFIES this
    // EFFECTS inserts o at the front of the list

    void insertBack(T *o);
    // MODIFIES this
    // EFFECTS inserts o at the back of the list

    T *removeFront();
    // MODIFIES this
    // EFFECTS removes and returns first object from non-empty list
    //        throws an instance of emptyList if empty

    T *removeBack();
    // MODIFIES this
    // EFFECTS removes and returns last object from non-empty list
    //        throws an instance of emptyList if empty

    // Maintenance methods
    Dlist();                                  // ctor
    Dlist(const Dlist &l);                    // copy ctor
    Dlist &operator=(const Dlist &l);          // assignment
    ~Dlist();                                  // dtor

 private:
    // A private type
    struct node
        {
                node  *next;
                node  *prev;
                T      *o;
    };

    node  *first; // The pointer to the 1st node (NULL if none)
    node  *last;  // The pointer to the 2nd node (NULL if none)

    // Utility methods

    void makeEmpty();
    // EFFECT: called by constructors/operator= to establish empty
    // list invariant

    void removeAll();
    // EFFECT: called by destructor/operator= to remove and destroy
    // all list elements

    void copyAll(const Dlist &l);
    // EFFECT: called by copy constructor/operator= to copy elements
    // from a source instance l to this instance
};

/*
Note: this is here *only* because the gnu compiler needs to see the
"templatized" versions of your methods.  This is the *only* instance
in which it is acceptable to #include a cc file.
*/

#include "dlist.cc"

#endif /* __DLIST_H__ */

---

So, what are you trying to compile then? There's nothing in file.h or in file.cc that's compilable, and file.c cannot be used as-is anyway since the definitions in dlist.h must precede it.

So basically I need a file that contains main to get this to work. Does it look right otherwise?

You need to actually write a main, yes. I have no idea whether you're doing anything right. At the very least I'm pretty sure that your operator= needs to take a DList<T> rather than a just plain DList.

Look at the first line in your .cc file. You intend it to be a comment, but have left out one character so it is not.

Sorry, that's not actually in my code, just got messed up in copying over.

I made a simple main class that just returns 0 but it has no effect on the errors. I'm still getting the errors that I mentioned.

Code:

---

#include <iostream>
#include "dlist.h"

using namespace std;

int main()
{
    return 0;
}

---

How did you compile it? If you compile it correctly, as in

Code:

---

$ g++ -Wall jcafaro.cpp

---

then nothing happens (as in no errors). Note that you still must not compile your dlist.cc file, except via dlist.h's include statement; that means it must not appear on your command line at all. If you are using an IDE, then you're going to have to do something fancy to get around the fact that the IDE thinks it knows better when it builds your makefile, by making your own makefile or giving dlist.cc an extension it does not recognize.

Now that you've fixed the defective comment on the first line, your code compiles without problems.

My guess is that the code you're showing us differs from what you're feeding to the compiler in some way.

If all you've done is copy and paste the files into your posts, then the problem is with something that doesn't survive the copy/paste process intact. For example, whitespace, non-graphical characters.

I have seen problems similar to this before, and they were solved by adding a blank line at the end of your header file and at the end of all your source files. This is a problem for some unix compilers (or for files that have been fed repeatedly through programs like dos2unix and unix2dos) as they typically ignore or discard a line that is terminated by an end-of-file marker rather than an end-of-line marker.

But that's just a guess .......

Code:

---

//Constructor
template <typename T>
Dlist<T>::Dlist()
{
        makeEmpty();
}

//Copy constructor
template <typename T>
Dlist<T>::Dlist(const Dlist<T>& l)
{
        makeEmpty();
        copyAll(l);
}

//Destructor
template <typename T>
Dlist<T>::~Dlist()
{
        makeEmpty();
        removeAll();
}

//Equals operator
template <typename T>
Dlist<T>& Dlist<T>::operator=(const Dlist<T>& l)
{
        if(this!=&l)
                copyAll(l);
        return *this;
}

---

I don't really have a compiler to test with at the moment...
But I am pretty sure that you cannot refer to an unspecialized type outside the definition itself.

The parameter list of out-of-line member definitions is in the class scope, so plain DList should work and refer to the current specialization of the template.

I'm using Eclipse and Visual Studio. Both give approximately the same errors. Eclipse shows me what the build command is.

Code:

---

g++ -O0 -g3 -Wall -c -fmessage-length=0 -osrc\dlist.o ..\src\dlist.cc

---

I suppose that's what you said not to do. What am I supposed to get it to build?

I figured it out. In eclipse you can exclude a file from build. I excluded the dlist.cc file from the build and it built the project correctly and even gave the correct output for a simple test program. Thanks for the help guys. How do I know when I can and when I can't include something in the build?

I suppose it's not possible to rename dlist.cc? That would be the easy way, to give it an extenstion that eclipse or VS doesn't recognize so it won't try to compile it. I don't know how to customize make in Eclipse. In Visual Studio (at least VS2005) you can right-click on the file in the Solution Explorer and select "Exclude from Build".