Thread: Point Out Mistake!!!

Well you could start by posting the code you actually compiled, rather than the random stuff you thought you had which doesn't compile.

with what compiler?

and why do you want turbo c 3?

bloodshed and microsoft both have a free to use IDE that work well and output error messages as to whats wrong with your code. you can download the IDE produced by bloodshed from here: http://prdownloads.sourceforge.net/d....9.2_setup.exe or from microsoft here: http://msdn.microsoft.com/vstudio/ex...ualc/download/

> i have complied it....
Well your compiler is crap.

printf("Enter the Numbers a,b,c,d);
This is missing a "

scanf("%d %d %d %f" , &a,&b,&c,&d0;
You forgot to hold down the shift key to make that 0 a )

Code:

int main()

should be

Code:

int main(void)

Code:

 printf("enter basic salary");

should be

Code:

 printf("enter basic salary\n");

Notice I put in '\n'. Under ANSI C, failing to include '\n' in the printf() function could lead to undefined behavior.

Code:

scanf("%f",&bs);

This should be re-written like:

Code:

if ( scanf("%f",&bs) ==1)

scanf() blows. But if you must use it, check for the return value. In this case, when when I check for the return value, the function would discard input like letters after the numbers. It's not perfect, but it's sort of respectable.

Okay, the hangover is getting worse. At the end of your program, put in the line of code

Code:

return 0;

As for the windows closing, I do believe this is an Operating Specific/compiler Specific Issue. Check your compiler to see if there is some kind of option to keep the window open after the program executes.

For the window closing it depends on the Operating System. What OS are you on?
~Sven

Main can also return other things. Whatever it returns it returns to the operating system. 0 usually signals success. So saying

Code:

return 0;

tells your operating system that your program successfully ended.

To make your program stay on the screen...
You can use getchar() to pause and wait for the user to press enter. You do not need to assign this to anything. Just add a line at the end of the program that is

Code:

getchar();

Or if using Dev-C++ (which I think you are) you can use getch() This is defined in stdio at least in Dev-C++. It does not display any text that you type in but it does pause your program until the user hits any key. The key does not need to be enter it can be anything. Like getchar() You just add a line at the end of your program that is

Code:

getch();

No arguments or assignments, just the funtion. Hope this helps.
~Sven

I wouldn't recommend getch(), it's not a standard function. You could printf() "Press enter to quit" and add a getchar() on the end. You could say "press enter to continue" and add this anywhere in your program, really.

The program is a commandline program, and was made to run from the command line. Thus the immediate exit.