Thread: C++ template error I think?

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.

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 .......

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".