C / C++ / MFC

9-Sep-06 15:51

Hello everyone!
(Note: I first posted this question here)
But I'll repost it, hopefully more clearly... Smile | :)

So let's say I have an error-free Player class... It's located on the files Player.h and Player.cpp. But there's a problem, the files don't include correctly... I've tried putting the class along with another class in Game.h/cpp, and that compiled, but not otherwise... Here's the code at the top of my files:

[Main.cpp]<br />
#include "Includes.h"<br />
<br />
[Includes.h]<br />
#pragma once<br />
<br />
// C++<br />
#include <iostream><br />
#include <string><br />
#include <stdlib.h><br />
#include <time.h><br />
#include <vector><br />
using namespace std;<br />
<br />
// SDL<br />
#include "SDL.h"<br />
#include "SDL_image.h"<br />
<br />
// Game<br />
#include "Defines.h"<br />
#include "Structures.h"<br />
#include "HelperFuncs.h"<br />
#include "Vehicle.h"<br />
#include "Player.h"<br />
#include "Game.h"<br />
<br />
[Game.h]<br />
#pragma once<br />
<br />
#include "Includes.h"<br />
<br />
[Game.cpp]<br />
#include "Game.h"<br />
<br />
[Player.h]<br />
#pragma once<br />
<br />
#include "Includes.h"<br />
<br />
[Player.cpp]<br />
#include "Player.h"

I tried everything on the previous thread, nothing worked... The error in VC++ 2005 is:

1>c:\incoming\war game\Game.h(10) : error C2146: syntax error : missing ';' before identifier 'mainPlayer'<br />
1>c:\incoming\war game\Game.h(10) : error C4430: missing type specifier - int assumed. Note: C++ does not support default-int<br />
1>c:\incoming\war game\Game.h(10) : error C4430: missing type specifier - int assumed. Note: C++ does not support default-int

(Note: The 10th line doesn't matter, as I said the thing works if I merge Game and Player...)

If you can help me, I'll give you a cookie! Laugh | :laugh:

Thanks!

PS: At first, I thought it was a problem with SDL not being compatible with VC++ 2005 because SDL download page says VC 5/6/7, not 8, but it doesn't compile under Code::Blocks (MinGW) either, it gives a similar error...

Windows Calculator told me I will die at 28. Frown | :(

Christian Graus9-Sep-06 16:24

Christian Graus

9-Sep-06 16:24

I assume that mainPlayer is an instance of player ? You're not using precompiled headers ? I'd put all these includes in the stdafx, if you were using them.

Does it have to be an instance of Player ? Could it be a pointer ? class Player; at the top of game.h, and the include of player.h in the CPP would then work fine.

Christian Graus - Microsoft MVP - C++
Metal Musings - Rex and my new metal blog

Lord Kixdemp9-Sep-06 16:41

9-Sep-06 16:41

You mean something like...

<br />
[Game.h]<br />
<br />
#pragma once<br />
<br />
class Player;<br />
#include "Player.h"<br />
<br />
#include "Includes.h"<br />
<br />
class Game<br />
{<br />
	// Main player<br />
	Player mainPlayer;<br />
<br />
	// All the vehicles available for the game<br />
	VehicleType AllVehicles[1];

That gives me:
c:\incoming\war game\sources\Game.h(13) : error C2079: 'Game::mainPlayer' uses undefined class 'Player'

Smells like I'll lose a cookie soon... Frown | :(

Thanks!

PS: Christian Graus, do you live here at CP? I don't recall ever having a thread without you in it... Laugh | :laugh:

Windows Calculator told me I will die at 28. Frown | :(

John M. Drescher9-Sep-06 16:58

John M. Drescher

9-Sep-06 16:58

In game.h get rid of

class Player;

because you are not using Player as a pointer and you

#include "Player.h"

in the next line.

John

Lord Kixdemp9-Sep-06 17:10

9-Sep-06 17:10

Choose[^] any kind you want! Thanks! Wink | ;)

Windows Calculator told me I will die at 28. Frown | :(

Christian Graus9-Sep-06 17:27

Christian Graus

9-Sep-06 17:27

I'm a bit slow to reply today, I am sick. Has your problem been solved ? I thought so, from reading the replies, but if not, then yell.

Lord Kixdemp wrote:
PS: Christian Graus, do you live here at CP?

Pretty close. Smile | :)

One other possible problem I thought of, does Player have a default constructor ( one that takes no arguments and is public ) ?

Christian Graus - Microsoft MVP - C++
Metal Musings - Rex and my new metal blog

Michael Dunn9-Sep-06 17:18

Michael Dunn

9-Sep-06 17:18

From that description, you need to clean up your include situation. For starters, your "includes.h" (analagous to stdafx.h in wizard-generated code) should not be included in other header files, only cpp files. The way it is now, if you're not using precompiled headers, you're seeing much longer compile times than necessary.

--Mike--
Visual C++ MVP Cool | :cool:

LINKS~! Ericahist | PimpFish | CP SearchBar v3.0 | C++ Forum FAQ

Lord Kixdemp9-Sep-06 18:41

9-Sep-06 18:41

Yeah, problem solved! Wink | ;)

And yes, it did have a default constructor...

And Michael Dunn, is there any way I can change the default stdafx.h file to something else? I'm using various compilers/platforms for this program, and I don't want to have one for every one of them... Poke tongue | ;-P

Thanks!

PS: I learned how to use VC++'s debugger - now I see how important those are...
PS: Hope you get better, Christian Graus! Wink | ;)

Windows Calculator told me I will die at 28. Frown | :(

Michael Dunn9-Sep-06 18:54

Michael Dunn

9-Sep-06 18:54

Each compiler has predefined symbols specific to it, so you can have conditionally-compiled code for different compilers. For example, VC has _MSC_VER which also tells you the compiler version.

--Mike--
Visual C++ MVP Cool | :cool:

LINKS~! Ericahist | PimpFish | CP SearchBar v3.0 | C++ Forum FAQ

Lord Kixdemp11-Sep-06 9:33

copy data from two buffer into one

11-Sep-06 9:33

Yeah, so I'll finish my game in VC++ and then I'll worry about the other compilers, since VC++ is the easiest to use IDE... Laugh | :laugh:

Thanks!

PS: Precompiled headers do go fast... OMG | :OMG:

Windows Calculator told me I will die at 28. Frown | :(

venadder9-Sep-06 12:49

venadder

9-Sep-06 12:49

Hi,
I am curious abt shortcuts to copying data from two buffers into one.
let's say I have an integer and a buffer of LPVOID type. Now i want to copy these two, int first and then all contents of LPVOId buffer( size known ) into the buffer supplied by the user(LPVOID type). memcpy or memmove will do only wholesale copy, no way to specify from what byte to start. is there any way other then byte by byte copy??
how do u do this kinda thing anyways?

any pointers will help, thnx

will something like this work
mydata is allocated and is of LPVOID type
myDataSize is UINT and have correct size( hopefully ) of myData
void copy( LPVOID buff, size )
{
memmove_s( buffer, size, (LPVOID)&someint, sizeof(int) );
buff += sizeof( int );
memmove_s( buffer, ( size - sizeof( int ) ), myData, myDataSize );
//return the input pointer to original value
buff -= sizeof( int );
}

//this could be totally laughable thing, but am jsut new to this kinda thing
//please excuse me if this code seems ridiculous

thnx for help.

Re: copy data from two buffer into one

venadder9-Sep-06 13:06

venadder

9-Sep-06 13:06

yupp it was ridiculous. VOID does not have size so the compiler does not allow pointer aithemetic on it.

Any suggestions please!!!!

Re: copy data from two buffer into one

Justin Tay9-Sep-06 15:23

Justin Tay

9-Sep-06 15:23

venadder wrote:
//return the input pointer to original value
buff -= sizeof( int );

You're not returning anything. A pointer is not a reference. A pointer could be used as a reference, in which case you'd need a double pointer to modify buff so the calling function can see a change. Anyway what you are attempting to do is pointless as you are trying to preseve the original value, but the original value passed in by the calling function can't be changed in your function, you are only operating on a stack copy of the pointer value.

memcpy(buffer, &someint, sizeof(int));
memcpy(((LPBYTE) buffer) + sizeof(int), myData, myDataSize);

Re: CTyptedPtrList error !

tanvon malik9-Sep-06 16:39

tanvon malik

9-Sep-06 16:39

the problem is here
CTyptedPtrList m_ElementList;

instead use this
CTypedPtrList<CObList, CMyObject*> m_ElementList;

you are missing its parameters

Tanvon
the brain behind ...

Met problems with my first win32 program

bloodwinner9-Sep-06 11:04

bloodwinner

9-Sep-06 11:04

code:

// Windows Programming Tutorial Series

#include "stdafx.h"
#include <windows.h>

int WINAPI
WinMain(HINSTANCE hInst,
HINSTANCE hPrevInstance,
LPSTR lpCmdLine,
int nCmdShow)
{
MessageBox (NULL, "Hello World! This is my first WIN32 program",
"Lesson 1", MB_OK);
return 0;
}

2 errors:
Linking...
LIBCD.lib(crt0.obj) : error LNK2001: unresolved external symbol _main
Debug/a.exe : fatal error LNK1120: 1 unresolved externals
Error executing link.exe.

a.exe - 2 error(s), 0 warning(s)

Who can make some suggestions please?
I guess it may be I did not include the lib well.

Rick York9-Sep-06 11:51

Rick York

9-Sep-06 11:51

The linker thinks this is a console program whose entry point is the main function. You need to tell it that it is a windows program. In the compiler options, preprocess definitions section you will probably see a macro "_CONSOLE" defined. You need to remove that and add in the macro _WINDOWS there.

bloodwinner9-Sep-06 12:33

bloodwinner

9-Sep-06 12:33

when I include this C++ source file into a "Win32 Application" project, it works;

when include it into a "Win32 Console Application" project, it fails;

my question is how to change the settings of the latter to make it work, without creating a new project of Wind32 Application.

I tried include the macro _WINDOWS in the setting->c/c++->preprocess definitions , but it did not work.

Rick York9-Sep-06 12:41

Rick York

9-Sep-06 12:41

There is obviously more to it than just the macro and I'm not certain what they are exactly.

What I would do is rename the folder your project is in to Lesson1_Old or something like that, create a new project, and then copy the old code files that you changed into the new project's directory (do NOT copy any project files - only .cpp, h, or .rc files) That should be very easy and take only a minute or two.

Sorry I couldn't be more helpful.

BTW - it could be informative to compare the new project files generated with the old ones that do not work.

TylerD759-Sep-06 13:15

TylerD75

9-Sep-06 13:15

I'm just a beginner my self, and in the process of writing my first application.
So this might be a shot in the dark, so to speak...

But are you trying to display a msgbox in a console application?
I was under the impression that that would be more or less impossible (unless you
program the actual code for displaying such a thing).
If you intend to write something in a console application, shouldn't you just print it
to the console?
Console::WriteLine(L"Hello World!");

If you want to use a msgbox, I think you either have to make a Win32 Application or a MFC application?
This is because in an empty Console Application you don't have window handling etc...

Someone please correct me if I'm wrong?

TylerD75