Thread: need help with handelling multiple source files

1. Don't put definitions in a header, #include it in multiple files, and then expect the linker not to complain.
2. Don't #include source files.
3. Don't post C++ code on the C board.

Still not posting all relevant info.

My main cpp file, is called "main.cpp"

I tryed to include this in the main.cpp file:

Code:

#include "color.h"

color.h only says this in the entire file for testing reasons:

Code:

void blue_grey(void);

BUT colorIt.cpp has the actual function, and includes the color.h:

Code:

#include "color.h"

#define _WIN32_WINNT WINVER

#ifdef WIN32
#endif

void blue_grey()
{
	#ifdef WIN32
		HANDLE hstdo;
		hstdo = GetStdHandle(STD_OUTPUT_HANDLE);
		SetConsoleTextAttribute(hstdo, FOREGROUND_RED|FOREGROUND_GREEN|FOREGROUND_BLUE|BACKGROUND_INTENSITY|BACKGROUND_BLUE);
	#endif  
}

Now I know somethings wrong, but I basically just want to split up my huge file into other files, and have it all compile and work together.

So basically, my main.cpp is supposed to gather all other files, and compile it as if it was really in one file. I guess that works?


EDIT:

OK IT NOW COMPILES! But it does NOT show any of my colors!

Where does the WIN32 macro get defined? Or should it be _WIN32?

Originally Posted by DarkMortar

Code:

void blue_grey()
{
#ifdef WIN32
   HANDLE hstdo;
   hstdo = GetStdHandle(STD_OUTPUT_HANDLE);
   SetConsoleTextAttribute(hstdo, FOREGROUND_RED|FOREGROUND_GREEN|FOREGROUND_BLUE|BACKGROUND_INTENSITY|BACKGROUND_BLUE);
#endif  
}

OK IT NOW COMPILES! But it does NOT show any of my colors!

Perhaps add an #error statement in an #else to verify that the code you are trying to conditionally compile is really there.

Code:

void blue_grey()
{
#ifdef WIN32
   HANDLE hstdo;
   hstdo = GetStdHandle(STD_OUTPUT_HANDLE);
   SetConsoleTextAttribute(hstdo, FOREGROUND_RED|FOREGROUND_GREEN|FOREGROUND_BLUE|BACKGROUND_INTENSITY|BACKGROUND_BLUE);
#else
#error !WIN32
#endif  
}

Just so you're clear, this is a header file (hpp)

Code:

#include "color.h" 
// Don't do this either. You do not have to include your custom made header file when you
// define it here:
#ifndef _WIN32_WINNT WINVER
#define _WIN32_WINNT WINVER
#ifdef WIN32
void blue_grey(void);
#endif

This is an implementation file (cpp)

Code:

#include "color.h"
void blue_grey(void)
{
		HANDLE hstdo;
		hstdo = GetStdHandle(STD_OUTPUT_HANDLE);
		SetConsoleTextAttribute(hstdo, FOREGROUND_RED|FOREGROUND_GREEN|FOREGROUND_BLUE|BA  CKGROUND_INTENSITY|BACKGROUND_BLUE);
}

Only include color.h where it needs to be used now.

but this is everything in my color.h file:

Code:

// INTERFACE FUNCTIONS:
// font colors>>

void tred();
void tgreen();
void tblue();
void tmagenta();
void tteal();
void ttan();
void tdefault();
void tred_i();
void tgreen_i();
void tblue_i();
void tmagenta_i();
void tteal_i();
void ttan_i();
void tdefault_i();

// background colors:

void bred();
void bgreen();
void bblue();
void bmagenta();
void bteal();
void btan();
void bdefault();
void bred_i();
void bgreen_i();
void bblue_i();
void bmagenta_i();
void bteal_i();
void btan_i();
void bdefault_i();
void face_intro();
void face_darktech();
void face_menu_titlebar();
void cosmos_logo();
void dark_logo();
void light_side();
void red_i();
void green_i();
void blue_white();
void teal_white();
void blue_grey();

EDIT: I beleive the problem is that the program is looking at my color functions as all void, which it is no doing anything because there is nothing to do in color.h. But how do I make it so color.h can do what colorIt.cpp does with the defs.

Moved to C++ From C

>>well i defined something in colorIt.cpp, that is about WIN32.

"something" isn't good enough. You need to define WIN32 if you want that code between #ifdef WIN32 and #endif to run. Ideally, you should use a preprocessor option passed to the compiler, but if you just put #define WIN32 at the top of your files (or in files included) it should work.