confusion about header and implementation files

[Roman Töngi]

I don't yet understand how to divide a C++ project
into header and implementation files. The following won't
compile:

// Apple.cpp
#include <iostream>
#include "random.h"
using namespace std;
int main()
{
cout << random();
return 0;
}

// random.h
double random();

// random.cpp
#include <cstdlib>
#include <ctime>
// is it necessary to include random.h?
#include "random.h"
double random()
{
double rnd;
std::srand(static_cast<unsigned>(std::time(NULL))) ;
rnd = static_cast<double>(std::rand()) / RAND_MAX;
return rnd;
}

There is an error message something like this: searching error for directive
for the precompiled header file.

[Larry I Smith]

Roman Töngi wrote:

I don't yet understand how to divide a C++ project
into header and implementation files. The following won't
compile:

// Apple.cpp
#include <iostream>
#include "random.h"
using namespace std;
int main()
{
cout << random();
return 0;
}

// random.h
double random();

// random.cpp
#include <cstdlib>
#include <ctime>
// is it necessary to include random.h?
#include "random.h"
double random()
{
double rnd;
std::srand(static_cast<unsigned>(std::time(NULL))) ;
rnd = static_cast<double>(std::rand()) / RAND_MAX;
return rnd;
}

There is an error message something like this: searching error for directive
for the precompiled header file.

The <cstdlib> (aka <stdlib.h>) already has a function
named random(). Your compiler is getting confused.
Rename your function (myRandom() perhaps) -or- put
your random() in a seperate namespace.

Larry

--
Anti-spam address, change each 'X' to '.' to reply directly.

[Jay Nabonne]

On Fri, 06 May 2005 22:42:12 +0200, Roman Töngi wrote:

There is an error message something like this: searching error for directive
for the precompiled header file.

Turn off precompiled headers in Visual Studio. Or include "stdafx.h" as
the first thing in each .cpp file.

- Jay

[Roman Töngi]

Do I separate the header and implementation files right
in this example? Is it necessary to include random.h in random.cpp?

Thanks

[Jay Nabonne]

On Fri, 06 May 2005 23:53:03 +0200, Roman Töngi wrote:

Do I separate the header and implementation files right
in this example? Is it necessary to include random.h in random.cpp?

Thanks

You have the right idea. The only change I would do is make it:

extern double random();

in random.h. Doesn't matter much, but it shows the intent more.

And while not technically necessary to include random.h in random.cpp,
I think it's better; you can catch some kinds of prototype mismatches if
the cpp file sees the prototype for the function (e.g. if you
accidentally change the return type in one place but not the other).

- Jay

[Jack Klein]

On Fri, 06 May 2005 21:06:47 GMT, Larry I Smith
<la***********@verizon.net> wrote in comp.lang.c++:

Roman Töngi wrote:

I don't yet understand how to divide a C++ project
into header and implementation files. The following won't
compile:

// Apple.cpp
#include <iostream>
#include "random.h"
using namespace std;
int main()
{
cout << random();
return 0;
}

// random.h
double random();

// random.cpp
#include <cstdlib>
#include <ctime>
// is it necessary to include random.h?
#include "random.h"
double random()
{
double rnd;
std::srand(static_cast<unsigned>(std::time(NULL))) ;
rnd = static_cast<double>(std::rand()) / RAND_MAX;
return rnd;
}

There is an error message something like this: searching error for directive
for the precompiled header file.

The <cstdlib> (aka <stdlib.h>) already has a function
named random(). Your compiler is getting confused.
Rename your function (myRandom() perhaps) -or- put
your random() in a seperate namespace.

Larry

No, it does not. Not if it is a standard conforming C++ compiler.
There is no function named "random" in the standard C++ (or C)
library, and that name is in the application's namespace so it is
invalid for an implementation-specific extension.

--
Jack Klein
Home: http://JK-Technology.Com
FAQs for
comp.lang.c http://www.eskimo.com/~scs/C-faq/top.html
comp.lang.c++ http://www.parashift.com/c++-faq-lite/
alt.comp.lang.learn.c-c++
http://www.contrib.andrew.cmu.edu/~a...FAQ-acllc.html

[Larry I Smith]

Jack Klein wrote:

On Fri, 06 May 2005 21:06:47 GMT, Larry I Smith
<la***********@verizon.net> wrote in comp.lang.c++:

Roman Töngi wrote:

I don't yet understand how to divide a C++ project
into header and implementation files. The following won't
compile:

// Apple.cpp
#include <iostream>
#include "random.h"
using namespace std;
int main()
{
cout << random();
return 0;
}

// random.h
double random();

// random.cpp
#include <cstdlib>
#include <ctime>
// is it necessary to include random.h?
#include "random.h"
double random()
{
double rnd;
std::srand(static_cast<unsigned>(std::time(NULL))) ;
rnd = static_cast<double>(std::rand()) / RAND_MAX;
return rnd;
}

There is an error message something like this: searching error for directive
for the precompiled header file.

The <cstdlib> (aka <stdlib.h>) already has a function
named random(). Your compiler is getting confused.
Rename your function (myRandom() perhaps) -or- put
your random() in a seperate namespace.

Larry

No, it does not. Not if it is a standard conforming C++ compiler.
There is no function named "random" in the standard C++ (or C)
library, and that name is in the application's namespace so it is
invalid for an implementation-specific extension.

From 'man -S 3 random':

<quote>

NAME
random, srandom, initstate, setstate - random number generator

SYNOPSIS
#include <stdlib.h>

long int random(void);

[Victor Bazarov]

Larry I Smith wrote:

Jack Klein wrote:

On Fri, 06 May 2005 21:06:47 GMT, Larry I Smith
<la***********@verizon.net> wrote in comp.lang.c++:

[...]
The <cstdlib> (aka <stdlib.h>) already has a function
named random(). Your compiler is getting confused.
Rename your function (myRandom() perhaps) -or- put
your random() in a seperate namespace.

Larry

No, it does not. Not if it is a standard conforming C++ compiler.
There is no function named "random" in the standard C++ (or C)
library, and that name is in the application's namespace so it is
invalid for an implementation-specific extension.

[..]
Conforming, or not, there may be a conflicting 'random()'
in his <stdlib.h>.

May be or may NOT be. Tha's the whole point Jack is making: you
said "The <cstdlib> .. already has a function named random". The
reality is that it does on *your* platform. How you arrive to the
conclusion that it causes conflict on the OP's platform is beyond
us. You must know something we don't.

V

[Larry I Smith]

Victor Bazarov wrote:

Larry I Smith wrote:

Jack Klein wrote:

On Fri, 06 May 2005 21:06:47 GMT, Larry I Smith
<la***********@verizon.net> wrote in comp.lang.c++:

>[...]
The <cstdlib> (aka <stdlib.h>) already has a function
named random(). Your compiler is getting confused.
Rename your function (myRandom() perhaps) -or- put
your random() in a seperate namespace.

Larry
No, it does not. Not if it is a standard conforming C++ compiler.
There is no function named "random" in the standard C++ (or C)
library, and that name is in the application's namespace so it is
invalid for an implementation-specific extension.

[..]
Conforming, or not, there may be a conflicting 'random()'
in his <stdlib.h>.

May be or may NOT be. Tha's the whole point Jack is making: you
said "The <cstdlib> .. already has a function named random". The
reality is that it does on *your* platform. How you arrive to the
conclusion that it causes conflict on the OP's platform is beyond
us. You must know something we don't.

V

Yeah, I tend to forget that GCC on unix/linux is not the only
package folks use. MS Windows confuses me (:

Sorry.

Regards,
Larry

--
Anti-spam address, change each 'X' to '.' to reply directly.