Thread: How can I make my app exit...

Hey, this is probably a simple thing to do. Basically, i'm doing a project for uni work. it is just a simple currency converter. I have managed to get the application so the data is checked and if it is not numercial, the user is alerted. But I would also like the program to exit when the user types exit.

Here is what I have so far

Code:

#include <stdlib.h>
#include <stdio.h>
#include <string.h>
void welcome(void); //displays welcome message and app info
void convert(void); //converts users input to $
int main()
{
	int count;
   welcome();
   count=0;
   while (count==0)
      {
		convert();
      }
}
void welcome(void)  //this bit welcomes the user and shows some info
{
	printf ("Currency Converter\nBy Kieran Symes\n24/10/2006\n");
}
void convert(void)
{
	char line[80];
	double input;  //users input
	char *endptr;
	int valid = 0;  //decides whether data is valid, this may not be required...
   double output; //output after calcs
	do
   {
		printf("\nPlease enter an amount you wish to convert without the &#163;.\n");
		fgets(line, 80, stdin);  //get data
		input = strtod(line, &endptr);
		if (endptr == line)
		{
			printf("\nYou have entered incorrect data.  Please ensure you type numbers only!\n\n");
			valid = 0;
		}
		else
		{
			valid = 1;
		}
	   } while (!valid);  //data is valid
     	output = input * 1.49;  //do the maths
   	printf("\n\&#163;%.2f is equal to %.2f euros.\n", input, output);  //print output
}

I think I can use strcmp but I am unsure how to use this correctly. If anybody would like to post an example with sample code, I would very much appreciate it. Also, where would I put this?

I am quite new to programming so that is why my code may not be very efficient etc.

thanks

Try this:

Code:

if (line == "exit") //And yes, this should take uppercase;i'm lazy
{
     printf("Exiting...");
     return 0;
}

What OS are you using?

If you return a number it should kill you app (as main is a function, returning a value ends the function).

Ofcourse you are. Replace this:

Code:

if (line == "exit") //And yes, this should take uppercase;i'm lazy
{
     printf("Exiting...");
     return 0;
}

with this:

Code:

if (line == "exit") //And yes, this should take uppercase;i'm lazy
{
     printf("Exiting...");
     return;
}

And try to understand these simple errors next time.

Whoops, I copied the code.

Code:

if (strcmp(line,"exit")==0) //And yes, this should take uppercase;i'm lazy
{
     printf("Exiting...");
     return;
}

I just meant that he probably had that "return 0;" in where it supposed to be just "return;".

Post your new code.