Thread: Whats wrong with my program?

I'm not really making a real program. I'm just playing around with the new functions I recently learned.

#include <iostream.h>
#include <conio.h>
#include <conio.c>

typedef unsigned short int USHORT;
void after();

void after()
{
USHORT Width = 5;
USHORT Length;
Length = 10;
USHORT Area = Width * Length;
cout << "\nWidth:" << Width << "\n";
cout << "Length: " << Length << endl;
cout << "Area: " << Area << endl;
}

int main()
{
int coul;
cout << "Press 1 and/or 2 to continue.......\n";
cin >> coul;
if (coul==1 || coul==2) {
void after(); }
getch();
return 0;
}

in main() you have the line:

void after(); }

Drop the word void in this line and recompile

ok, I'll try that. By the way, it compiles successfullly, its just that when I press 1 or 2 it doesn't call after();

if (coul==1 || coul==2)

needs to be changed b/c your not working with numbers
your working with a basic string

if(coul == "1" || coul == "2")

This should fix it

btw, I'm using Dev-C++. How is it a basic string when it is declared as an integer?

The linker should link conio.c to your program if you include conio.h, so I would try dumping that too.

I agree with you, coul is of type int, and you should not try to equate that with a literal string.

I would try bringing the function body for after() into main() and comment out the function definition and function call. I would do this first without an if statement checking coul. If that compiles and runs then I would place the code for the function body of after() in an if statement such as you have. If that compiles and runs appropriately, then I would try it again with everything uncommented. If it still doesn't work, then I would scream foul.