Error C2228, though left of .x has a class

Question

I am trying to call class function, while Visual Studio throws above error at me. I have been calling another function in constructor and it works there.

What is worth mentioning, is that GameLoop is static. I have a strange feeling that this might be the cause. If it is, how do I make it work?

GameApp::GameApp()
{
    winApi.CreateBasicWindow("---===| Test |===---", 1024, 768, WS_OVERLAPPEDWINDOW);

    bool err = d3d.BasicInit(winApi.GetWindowHandle(),
                             winApi.GetInstance(),
                             1024,
                             768,
                             60,
                             1,
                             true);

    if(!err)
        MessageBox(0, "Could not initialize DirectX 10.", "Error!", MB_OK | MB_ICONERROR);

    winApi.RunMessageLoop(GameLoop);
}

void GameApp::GameLoop()
{
    D3DXCOLOR color(1.0f, 0.0f, 0.0f, 1.0f);
    d3d.Redraw(color); // Error here
}

---- edit

Error message

Error   1   error C2228: left of '.Redraw' must have class/struct/union

You should include the full text of the error message in you question — Captain Obvlious, May 29 '13 at 20:26
is d3d a nonstatic member variable? if so, whose d3d do you expect GameLoop to redraw? — Kate Gregory, May 29 '13 at 20:27

score 2 · Accepted Answer · answered May 29 '13 at 20:27

2

If GameLoop is static, that means it can only access static fields. d3d is probably not a static field.

You probably made GameLoop static to keep a single instance of GameApp available throughout the code. To do that properly, read up on the singleton pattern, for it probably is what you are looking for.

answered May 29 '13 at 20:27

Kornel Kisielewicz

55,802
15
111
149

Or "Singleton anti-pattern" if you prefer. – chris May 29 '13 at 20:36
@chris one has to learn why Singletons are awesome before learning why the actually suck :P – Kornel Kisielewicz May 29 '13 at 20:44

Error C2228, though left of .x has a class

1 Answers1