Accessing a function in "private:"

[eman2013]

I have a problem accessing a function in my "private:", named void display();

class Board
{
private:
    Piece list[64];
    int x1, y1;
    int status, redpiece, greenpiece, winner;
    int inRange(int x, int y);
    int pieceAtLoc(int x, int y);
    int validPlacement(int x, int y);
    void flip(int x, int y);
    void display();
    void move();
public:
    Board();
    void execute();
};
Board::Board()
{
    list[0].setposition(4,4,0);
    list[1].setposition(5,4,1);
    list[2].setposition(4,5,0);
    list[3].setposition(5,5,1);
}
void Board::execute()
{
    Board::display()
    {
        cout<<"hello"<<endl;
    }
}
int main()
{
    Board b;
    b.execute();
}

Can someone tell me how to access display correctly?...Thank You!

[S G H]

void Board::execute()
{
    this->display();
    // or
    display(); // "this->" is implicit on this line.
}

void Board::display()
{
    std::cout<<"hello"<<std::endl;
}

[long double main]

You need to move the definition of Board::display() out of your Board::execute() function:

void Board::display()
{
    cout << "hello" << endl;
}
void Board::execute()
// ... 

Then to call display(), you just call it like any other function:

void Board::execute()
{
    display();
}

[eman2013]

thanks long double main!..

[eman2013]

I tried doing it but it gives me a error for "operator<<"

class Board
{
private:
    Piece list[64];
    int x1, y1;
    int status, redpiece, greenpiece, winner;
    int inRange(int x, int y);
    int pieceAtLoc(int x, int y);
    int validPlacement(int x, int y);
    void flip(int x, int y);
    void display();
    void move();
public:
    Board();
    void execute();
};
Board::Board()
{
    list[0].setposition(4,4,0);
    list[1].setposition(5,4,1);
    list[2].setposition(4,5,0);
    list[3].setposition(5,5,1);
}
void Board::display()
{
    for(int i=0; i<64; i++)
    {
        if((i==8)||(i==16)||(i==24)||(i==32)||(i==40)||(i==48)||(i==56)||(i==64))
        {
            cout<<endl;
            cout<<list[i];
        }
        else
        {
            cout<<list[i];
        }

    }
}
void Board::execute()
{
    display();
}
int main()
{
    Board b;
    b.execute();
}

[long double main]

In order to do cout << list[i];, you'll need to define operator<< for Piece.

If you don't want to/don't feel like doing that, then you'll have to display the members of Piece somehow (I don't know what your Piece class looks like, so I don't know what cout << somePiece; should look like).

[eman2013]

class Piece
{
private:
    int x,y;
    char color;
public:
    Piece()
    {
        x=-1;
        y=-1;
        color=2;
    }
    void setcolor(char c)
    {
        color=c;
    }
    void setposition(int x_pos,int y_pos, char c)
    {
        x=x_pos;
        y=y_pos;
        color=c;
    }
    void set(int x_pos, int y_pos)
    {
        x=x_pos;
        y=y_pos;
    }
    int getc()
    {
        return color;
    }
    int getx()
    {
        return x;
    }
    int gety()
    {
        return y;
    }
    void display();
};

[long double main]

What if, instead of cout << list[i];, you just did list[i].display();?

[eman2013]

It brings an error that Piece::display() is undefined. But if you can give me a few minutes I finish some code on Piece::display()...hopefully it works

[eman2013]

I need to figure out how to make display as if it was on a board but here is the raw code:

void Piece::display()
{
    int a=getc();
    if(a==0)
    {
        cout<<"O ";
    }
    else if(a==1)
    {
        cout<<"X ";
    }
    else
    {
        cout<<"  ";
    }
}

[S G H]

Don't call that function "getc".
getc is an actual function existing in the C standard. Call it getcolor or similar, it's hard to understand it on-the-fly for me.
Besides, it still works.

As to make a grid, you should edit Board::display, but as I'm on mobile it's hard to say more right now.

[eman2013]

Thank you! For the heads and if you could get back to me on the grid thing that would be great!

[S G H]

I have to think it's a 8x8 board?

In this case:

void Board::display()
{
    const int board_width = 8;
    const int board_height = 8;
    const int board_size = board_width*board_height;
    for(int i=0; i<board_size; i++)
    {
        int x = i%board_width; // result is in range 0-7. 0 being the leftmost, 7 the rightmost.
        int y = (i-x)/board_width; // result is also in range 0-7. 0 being the top, 7 the bottom.
        if(i && (!x) )
        {
            cout<<endl; // If "i" is not 0 and the X position is 0, go down one line
// if the Y position equals to the last Y position, don't write a horizontal separator
            if(y != (board_height-1))
            {
                for(int x = 0; x < board_width; ++x)
                    cout<<"---";
// They are three lines, because two spaces are used in Piece::display, and one for vertical separators
            }
        }
        list[i].display();
// If the X position equals to the last X position, don't write a vertical separator
        if(x != (board_width-1))
            cout<<'|';
    }
}

Didn't test but should work.
To change the board's size you can just change board_width and board_height.