Wrong declaration, Right result ?

I'm reading a beginners guide to C++, and it say's,
that this code should give the result: -5536

But when I compile it using Visual C++ 6.0,
the result is: 60000
which sure is correct, but when "result" is
declared as an "int", the result should be an
overflow right ?

Code:

---

#include <iostream.h>

void main(void)
{
        int result;

        result = 200 * 300;

        cout << "200 * 300 = " << result << endl;
}

---

ints are usually 32 bits, which means they usually have a range from -(2^31)+1 to 2^31-1

Check to make sure you got the type from your book right. Also, it looks like that book might be outdated since it uses <iostream.h>, which is deprecated. You should use <iostream> and put the following line after your includes:

Code:

---

using namespace std;

---

the maximum value that can be stored in an int can be found in the file limits.h. If you attempt to store a larger value than can be stored, the result is undefined behavior, which means you can not count on any value. Most compilers I have worked with just wrap the number, but there is no guarentee that all compilers will do that.

On your platform a short is probably only 16 bits, so try it with a short instead to see what the book was trying to show you. Or get a newer, more standards compliant book.

If I write:

Code:

---

#include <iostream>
instead of <iostream.h>

---

I'll get this:

Code:

---

Compiling...
regn_overflow.cpp
c:\temporary files\regn_overflow\regn_overflow.cpp(9) : error C2065: 'cout' : undeclared identifier
c:\temporary files\regn_overflow\regn_overflow.cpp(9) : error C2297: '<<' : illegal, right operand has type 'char [13]'
c:\temporary files\regn_overflow\regn_overflow.cpp(9) : error C2065: 'endl' : undeclared identifier
Error executing cl.exe.

regn_overflow.exe - 3 error(s), 0 warning(s)

---

I have iostream in the
C:\Program Files\Microsoft Visual Studio\VC98\Include directory (837 bytes).

iostream.h is 2.397 bytes

But iostream hasn't got any extention...


In my book it says that an int has a range from:
-32.768 to 32.767
so when I get 60.000 it should get an overflow accordingly.

Your book is outdated or for a different operating system.

Since Win95 and 32Bit, and int is 32 bits long. The limits your book quotes are from 16 bit times. If you want to know the size of variable types in bits, use

Code:

---

cout << ( sizeof( type ) * 8 ) << endl;

---

To bring your book up to date and let this example work out the way the author intended, use short instead of int as type.

Read Jawibs whole text:

Quote:

---

You should use <iostream> and put the following line after your includes:

Code:
using namespace std;

---

I still wonder why I get this "undeclared identifier"
when I write:
#include <iostream> :confused:

because you also must identify the namespace in which the cout function resides -- std. you can do that a couple ways

Code:

---

#include <iostream>

int main()
{
  std::cout << "Hello World" << std::endl;

  return 0;
}

---

or

Code:

---

#include <iostream>
using namespace std;

int main()
{
  cout << "Hello World" << endl;

  return 0;
}

---

or

Code:

---

#include <iostream>
using std::cout;
using std::endl;

int main()
{
  cout << "Hello World" << endl;

  return 0;
}

---

Great, thanks !

I'll use:

Code:

---

#include <iostream>

using namespace std;

and then:

cout << "Hello World" << endl;

---

So now I guess that iostream without
the ".h" should work in vc6 ;)


I'll try this when I get home from work ;)

Code:

---

#include <iostream>

using std::cout;

void main(void)
{
        int result;

        result = 200 * 300;

        cout << "200 * 300 = " << result << endl;
}

---

If I use iostream.h it works, but when I use this code
with iostream, I'll get the following error message:

[QUOTE]Compiling...
regn_overflow.cpp
c:\temporary files\regn_overflow\regn_overflow.cpp(11) : error C2065: 'endl' : undeclared identifier
Error executing cl.exe.

Of course I can remove 'endl', then it works...
hmm...

regn_overflow.exe - 1 error(s), 0 warning(s) :confused:

Code:

---

#include <iostream>

using std::cout;

void main(void)
{
        int result;

        result = 200 * 300;

        cout << "200 * 300 = " << result << endl;
}

---

If I use iostream.h it works, but when I use this code
with iostream, I'll get the following error message:

Quote:

---

Compiling...
regn_overflow.cpp
c:\temporary files\regn_overflow\regn_overflow.cpp(11) : error C2065: 'endl' : undeclared identifier
Error executing cl.exe.

---

Of course I can remove 'endl', then it works...
hmm...

regn_overflow.exe - 1 error(s), 0 warning(s) :confused:

Code:

---

#include <iostream>

using std::cout;

void main(void)
{
        int result;

        result = 200 * 300;

        cout << "200 * 300 = " << result << endl;
}

---

If I use iostream.h it works, but when I use this code
with iostream, I'll get the following error message:

Quote:

---

Compiling...
regn_overflow.cpp
c:\temporary files\regn_overflow\regn_overflow.cpp(11) : error C2065: 'endl' : undeclared identifier
Error executing cl.exe.

regn_overflow.exe - 1 error(s), 0 warning(s)

---

Of course I can remove 'endl', then it works...
hmm...

When someone replies to your question, it is in your best interest to read the entire post.

First: don't include iostream.h, it's not standard. Include iostream. Forget iostream.h exists.

Second: As a result of the first item, you have to define what namespace you'll be using for what statements, and where you'll be using it. There are a number of ways to do this. Ancient Dragon laid it out all very nicely for you.

Look at the options. Look at your code. How do the they differ? For the option you seem to have chosen:

Code:

---

#include <iostream>

using std::cout;
using std::endl; <<-- You need to include as well to do it this way.

void main(void)
{
        int result;

        result = 200 * 300;

        cout << "200 * 300 = " << result << endl;
}

---

Speaking of standard, I'd stop using void main(void) if you want to get out of this thread alive...that really ticks people off around here. ;) Use int main() instead.

The best way would just be to put:

Code:

---

using namespace std;

---

Then you havent got to put using std::cout and using std::endl

Hi Decrypt !

Quote:

---

When someone replies to your question, it is in your best interest to read the entire post.

---

You're quite right.

regards,

The SharK