Taking arrays of pointers in function prototypes in header files: function declared void

Question

I'm a bit new to programming and ran into these errors tracing to the function prototype in the function header file and I think it may have something to do with the array of pointers it's getting.

error variable or field 'clean_up' declared void

error 'Button' was not declared in this scope

error 'buttons' was not declared in this scope

error expected primary variable before ']' token

    //Function.h

    #include "SDL/SDL.h"
    #include "SDL/SDL_image.h"
    #include <string>
    #include "functions.h"
    #include "globals.h"
    #include "Button.h"
    #include <fstream>

    void clean_up( Button *buttons[] ); // errors here

    //Function.cpp
  void clean_up( Button *buttons[] )

    {
        SDL_FreeSurface( background );
        SDL_FreeSurface( X );
        SDL_FreeSurface( O );

        for( int t = 0; t < TOTAL_BUTTONS; t++ )
        {
            delete buttons[ t ];
        }

        SDL_Quit();
    }
    //Button.h
    class Button
    {
    private:

    SDL_Rect box;
    SDL_Surface *sprite;

    public:


        bool in_use;
        bool xoro;
        int p_id;


Button( int x, int y, int id );
~Button();

void handle_events();

void show();

};

//Button.cpp

Button::Button( int x, int y, int id )
{
    box.x = x;
    box.y = y;
    box.w = 120;
    box.h = 120;
    in_use = false;
    p_id = id;
}
Button::~Button()
{
    SDL_FreeSurface( sprite );
}

I wasn't really sure where to look for a solution so any help is appreciated. Thanks.

try to move the Button class definition to the "Button.h" header — Cheers and hth. - Alf
– Cheers and hth. - Alf, Commented Dec 18, 2012 at 4:02

Some programmer dude · Accepted Answer · 2012-12-18 07:04:44Z

1

Try adding forward declaration of class Button in the .h file.

Function.h

#include "SDL/SDL.h"
#include "SDL/SDL_image.h"
#include <string>
#include "functions.h"
#include "globals.h"
#include "Button.h"
#include <fstream>

class Button;
void clean_up( Button *buttons[] );

edited Dec 18, 2012 at 7:04

Some programmer dude

411k36 gold badges420 silver badges655 bronze badges

answered Dec 18, 2012 at 4:22

Manoj R

3,2671 gold badge24 silver badges37 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Yakk - Adam Nevraumont · Accepted Answer · 2012-12-18 04:26:51Z

0

Function.h includes too many header files. It only depends on Button.

So only include Button.h. or even better, just forward declare class Button;

This will remove the problem in function.h, but it probably the will pop up elsewhere.

As a psychic guess, you are missing a ; or a } somewhere.

answered Dec 18, 2012 at 4:26

Yakk - Adam Nevraumont

280k31 gold badges367 silver badges585 bronze badges

Comments

interjay · Accepted Answer · 2012-12-20 13:20:57Z

0

I guess the class should be declared before the function declaration/definition.

edited Dec 20, 2012 at 13:20

interjay

111k22 gold badges278 silver badges264 bronze badges

answered Dec 18, 2012 at 4:25

user1908151

Collectives™ on Stack Overflow

Taking arrays of pointers in function prototypes in header files: function declared void

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related