Whats wrong with my code? ::sigh::

I am not good with c++ but i did take one class on it and that was enough to see it wasnt for me. I made this project for my work it worked fine in codewarrior but i couldnt compile itno an executable because it was student version. So i came to this forum which I had recieved help when was in my class still to find a free compiler then to download Dev C++. When I pasted the exact same code that worked fine in codewarrior into Dev C++ it wouldnt compile :confused:

Any help would be greatly appreciated,

Charlie



#include <iostream.h>
#include <iomanip.h>


int main ()

{

int rmeters;
float lspeed;
float rtime;
float htime;







cout << endl;
cout << endl;
cout << " Run Time Calculator \n";

cout << endl;
cout << endl;
cout << endl;
cout << endl;
cout << " Enter the number of meters \n";

cin >> rmeters ;
cout << endl;
cout << " What is the estimated average line speed? \n";
cin >> lspeed;

rtime=rmeters/lspeed;
htime=(rmeters/lspeed)/60;
cout << endl;
cout << endl;
cout << " You have aproximately " << rtime <<" minutes \n";
cout << " or " << htime << "hours of runtime";

cout << endl;
cout << endl;
cout << endl;

Worked for me.

You must include the "return 0;" and the closing "}".

Also,
Instead of this:

Code:

---

#include <iostream.h>
#include <iomanip.h>

---

Use

Code:

---

#include <iostream>
#include <iomanip>

using namespace std;

---

And use code tags please

Thanks for the help it compiled but now it doesnt show the meters remaining it just closes after I insterd the meters and line speed.

Any ideas?

http://faq.cprogramming.com/cgi-bin/...&id=1043284385
http://faq.cprogramming.com/cgi-bin/...&id=1043284385

the last line of ur code before the return 0, add a cin.get();

Quote:

---

Originally Posted by jlf029

the last line of ur code before the return 0, add a cin.get();

---

I made a little program today at school, showing a friend of mine a little more advanced programming, He's on the "Hello World" first program thingy, and I tried using the cin.get(); , but it didn't work. Why not? I did it on the last line before the return 0;

Perhaps neither of you have read the FAQ

Quote:

---

Originally Posted by Salem

Perhaps neither of you have read the FAQ

---

I have, but I'm not familliar with the other things. only cin.get();

You probably have an return character in the buffer (from when you previously hit enter on your cin), so when it goes to cin.get() it has an enter in it.


Use:

Code:

---

cin.ignore();
cin.get();

---

Or just do cin.ignore() after every cin input line.

It'd also be a good idea to use "cout << endl << endl" instead of using a new cout for each endl. Or do as I do, and if you have more than one endl right next to eachother, drop both of them and use "count << "\n\n" ". Same thing as "cout << endl << endl;" except you can get rid of that second <<.

Here's your code, a bit cleaned up.

Code:

---

#include <iostream>
#include <iomanip>

using namespace std;


int main ()
{

int rmeters;
double lspeed, rtime, htime; //Doubles are better than floats

cout << "\n\nRun Time Calculator";
cout << "\n\nEnter the distance in meters: ";
cin >> rmeters ;
cin.ignore();
cout << "\nWhat is the estimated speed? ";
cin >> lspeed;
cin.ignore();

rtime = rmeters / lspeed;
htime = ( rmeters / lspeed ) / 60;

cout << "\n\nYou have aproximately " << rtime <<" minutes ";
cout << " or " << htime << " hours of runtime.";

//Those last few cout << endl; s were useless.
cin.get();
return 0;
}

---

>Same thing as "cout << endl << endl;" except you can get rid of that second <<.
Not quite. std::endl actually flushes the stream too, while '\n' doesn't. To get the similar functionality you'd have to do this:

Code:

---

cout << "\n\n" << flush;

---

Which doesn't get rid of the second operator. ;)