Redefinition of class

When I compile code from a program I'm making it give me these two errors

Code:

|4|error: redefinition of 'class Bitmap'
|5|error: previous definition of 'class Bitmap'

Here is the Bitmap class

Code:

class Bitmap
{
    protected:
      //Memeber variables
      HBITMAP m_hBitmap;
      int m_iWidth, m_iHeight;

      //Helper Methods
      void Free();

   public:
     //Constructor(s)/Destructor
     Bitmap();
     Bitmap(HDC hDC, LPSTR szFileName);
     Bitmap(HDC hDC, UINT uiResID, HINSTANCE hInstance);
     Bitmap(HDC hDC, int iWidth, int iHeight, COLORREF crColor = RGB(0,0,0));
     virtual ~Bitmap();

     //General Methods
     BOOL Create(HDC hDC, LPSTR szFileName);
     BOOL Create(HDC hDC, UINT uiResID, HINSTANCE hInstance);
     BOOL Create(HDC hDC, int iWidth, int iHeight, COLORREF crColor);
     void Draw(HDC hDC, int x, int y, BOOL bTrans = FALSE, COLORREF crTransColor = RGB(255, 0, 255));
     int GetWidth(){return m_iWidth; };
     int GetHeight(){return m_iHeight; };
};

Well, that seems valid to me. I suspect that your problem is elsewhere. What file and line did you get the error in? Please post that file instead.

I suspect the bitmap class is in a header? If so, try adding include guards.

bijan311: Header guards, if you don't know, are preprocessor-fu for "don't double declare my stuff"

foo.h

Code:

#ifndef FOO_H
#define FOO_H

// Stuff

#endif

Know that FOO_H might not be unique enough, especially if it's a short name. It is better, then, to add something that is rather unique, namely time and date. That almost guarantees uniqueness. I make it a habit when I create include guards.

Originally Posted by Elysia

Know that FOO_H might not be unique enough, especially if it's a short name. It is better, then, to add something that is rather unique, namely time and date. That almost guarantees uniqueness. I make it a habit when I create include guards.

Lol, you really guard the hell out of your headers!