C++ loop not working

[Firepathedian]

I would like to have my program as a .exe file that is clickable and will run without closing quickly. I want it to where if I put an input in, it runs, then starts over at the beginning and doesn't close until I press '0'. How do I accomplish this?
--------------------------------------------------------------------------------

#include "stdafx.h"
#include <iostream>
#include <stdlib.h>
using namespace std;

int main()
{
	double ftemp;
	double ctemp;
	double ktemp;
	double select = -1;

	cout << "Valid options 1, 2, 3, 4, 5, or 6." << endl;
	while (select !=-1)
	
		cout << "Please select from the following (0 to quit): " <<endl;
		cout << "1) Fahrenheit-to-Celsius" << endl;
		cout << "2) Celsius-to-Fahrenheit" << endl;
		cout << "3) Kelvin-to-Celsius" << endl;
		cout << "4) Celcius-to-Kelvin" << endl;
		cout << "5) Kelvin-to-Fahrenheit" << endl;
		cout << "6) Fahrenheit-to-Kelvin" << endl;
		cout << "Enter: ";
		cin >> select;
		if (select == 1)
		{
			cout << "Enter temperature in Fahrenheit to convert to degrees Celsius: ";
			cin >> ftemp;
			ctemp = (ftemp-32) * 5 / 9;
			cout << "Equivalent in Celsius is: " << ctemp << endl ;
		}
		if (select ==2)
		{
			cout <<" Enter temperature in Celsius to convert to degrees Fahrenheit: ";
			cin >> ctemp;
			ftemp = ctemp*9/5 + 32;
			cout << "Equivalent in Fahrenheit is: " << ftemp << endl;
		}
		if (select ==3)
		{
			cout << "Enter temperature in Kelvin to convert to Celsius: ";
			cin >> ktemp;
			ctemp = (ktemp-273.15);
			cout << "Equivalent in Celsius is: " << ctemp << endl;
		}
		if (select ==4)
		{
			cout << "Enter Celsius to convert to Kelvin: ";
			cin >> ctemp;
			ktemp=ctemp+273.15;
			cout << "Equivalent in Kelvin is: " << ktemp << endl;
		}
		if
		
			(select ==5)
		{	cout << "Enter Kelvin to convert to Fahrenheit: ";
			cin >> ktemp;
			ftemp= (ktemp-459.67);
			cout << "Equivalent in Fahrenheit:" << ftemp << endl;
		}
		if
			(select ==6)
		{
			cout << "Enter Fahrenheit to convert to Kelvin: ";
			cin >> ftemp;
			ktemp= (ftemp+459.67);
			cout << "Equivalent in Kelvin:" << ktemp << endl;
		}
		


		if (select < 0 || select > 6) // invalid if either of these is true
		
		do {
			cout << "Valid options 1, 2, 3, 4, 5, or 6." << endl;
		}
		while (select <0 || select > 6) {
cout << select << ",":
		--select;
		}
	return 0;
}

[nvrmnd]

unsigned  i = 0
do {
    //...
    if( i == 1 ) {}
    else if( i == 2 ) {}
    // ...
    else if( i > 6 ) {}

} while( i != 0 );

I would like to have my program as a .exe file that is clickable

That is an advance one, if you want you can take a look how to do so :
http://www.benryves.com/tutorials/winconsole/

But that's not recommended, you can use instead graphics lib like sfml

[Firepathedian]

I'm not making a game, I'm just trying to convert temperatures back and forth. Is there a way to make it runnable from a .exe file.

[Lowest0ne]

First:

        double select = -1;

	cout << "Valid options 1, 2, 3, 4, 5, or 6." << endl;
	while (select !=-1)

? Doesn't look right. Change that double to an int.

I guess you're using VS. You have a project folder, and there is an exe somewhere in there. You would be able to click on that to get your program to run.

[Firepathedian]

The reason why there are double's is because I would like a decimal set to 2 places. When I click on the .exe file and put in a value, it creates an output, then closes quickly, and I don't have time to see it.
Update to last comment: I didn't realize you meant that last double, my bad. I did change that 'double' to 'int,' but it still closes too quickly.

[Mats]

Place a cin.ignore() at the end. That will do for now for stopping the thing closing. You can then hit return (or another key) to exit.

[Firepathedian]

It's not doing anything. Here's my code:

#include "stdafx.h"
#include <iostream>
#include <stdlib.h>
using namespace std;

int main()
{
	double ftemp;
	double ctemp;
	double ktemp;
	int select = -1;

	cout << "Valid options 1, 2, 3, 4, 5, or 6." << endl;
	while (select !=-1)
	
		cout << "Please select from the following (0 to quit): " <<endl;
		cout << "1) Fahrenheit-to-Celsius" << endl;
		cout << "2) Celsius-to-Fahrenheit" << endl;
		cout << "3) Kelvin-to-Celsius" << endl;
		cout << "4) Celcius-to-Kelvin" << endl;
		cout << "5) Kelvin-to-Fahrenheit" << endl;
		cout << "6) Fahrenheit-to-Kelvin" << endl;
		cout << "Enter: ";
		cin >> select;
		if (select == 1)
		{
			cout << "Enter temperature in Fahrenheit to convert to degrees Celsius: ";
			cin >> ftemp;
			ctemp = (ftemp-32) * 5 / 9;
			cout << "Equivalent in Celsius is: " << ctemp << endl ;
		}
		if (select ==2)
		{
			cout <<" Enter temperature in Celsius to convert to degrees Fahrenheit: ";
			cin >> ctemp;
			ftemp = ctemp*9/5 + 32;
			cout << "Equivalent in Fahrenheit is: " << ftemp << endl;
		}
		if (select ==3)
		{
			cout << "Enter temperature in Kelvin to convert to Celsius: ";
			cin >> ktemp;
			ctemp = (ktemp-273.15);
			cout << "Equivalent in Celsius is: " << ctemp << endl;
		}
		if (select ==4)
		{
			cout << "Enter Celsius to convert to Kelvin: ";
			cin >> ctemp;
			ktemp=ctemp+273.15;
			cout << "Equivalent in Kelvin is: " << ktemp << endl;
		}
		if
		
			(select ==5)
		{	cout << "Enter Kelvin to convert to Fahrenheit: ";
			cin >> ktemp;
			ftemp= (ktemp-459.67);
			cout << "Equivalent in Fahrenheit:" << ftemp << endl;
		}
		if
			(select ==6)
		{
			cout << "Enter Fahrenheit to convert to Kelvin: ";
			cin >> ftemp;
			ktemp= (ftemp+459.67);
			cout << "Equivalent in Kelvin:" << ktemp << endl;
		}
		


		if (select < 0 || select > 6) // invalid if either of these is true
		
		do {
			cout << "Valid options 1, 2, 3, 4, 5, or 6." << endl;
		}
		
		
		
	return 0;
cin.ignore()}

I've tried moving the cin.ignore() around at the bottom of the code, and no matter where I put it, it doesn't work. Before and after the:

                if (select < 0 || select > 6) // invalid if either of these is true
		
		do {
			cout << "Valid options 1, 2, 3, 4, 5, or 6." << endl;
		}

[nvrmnd]

#include <iostream>
using namespace std;

int main()
{
    double ftemp;
    double ctemp;
    double ktemp;
    int select = -1; // int ! not double
    
    while (select != 0)
    {
        cout << "Please select from the following (0 to quit): " <<endl;
        cout << "1) Fahrenheit-to-Celsius" << endl;
        cout << "2) Celsius-to-Fahrenheit" << endl;
        cout << "3) Kelvin-to-Celsius" << endl;
        cout << "4) Celcius-to-Kelvin" << endl;
        cout << "5) Kelvin-to-Fahrenheit" << endl;
        cout << "6) Fahrenheit-to-Kelvin" << endl;
        cout << "Enter: ";
        cin >> select;
        
        if (select == 1)
        {
            // ...
        }
        else if (select ==2)
        {
            // ...
        }
        else if (select ==3)
        {
            // ...
        }
        else if (select ==4)
        {
            // ...
        }
        else if (select ==5)
        {
            // ...
        }
        else if (select ==6)
        {
            // ...
        }
        else if (select < 0 || select > 6)
        {
            do
            {
                cout << "Valid options 1, 2, 3, 4, 5, or 6." << endl;
                cin >> select;
            }
            while( select < 0 || select > 6 );
        }
    }

    cin.get(); // wait for a keypress
    return 0;
}

[Firepathedian]

Does this look right? I've tried building it, and the .exe file still closes too quickly.

#include <iostream>
using namespace std;

int main()
{
    double ftemp;
    double ctemp;
    double ktemp;
    int select = -1; // int ! not double
    
    while (select != 0)
    {
        cout << "Please select from the following (0 to quit): " <<endl;
        cout << "1) Fahrenheit-to-Celsius" << endl;
        cout << "2) Celsius-to-Fahrenheit" << endl;
        cout << "3) Kelvin-to-Celsius" << endl;
        cout << "4) Celcius-to-Kelvin" << endl;
        cout << "5) Kelvin-to-Fahrenheit" << endl;
        cout << "6) Fahrenheit-to-Kelvin" << endl;
        cout << "Enter: ";
        cin >> select;

		if (select == 1)
		{
			cout << "Enter temperature in Fahrenheit to convert to degrees Celsius: ";
			cin >> ftemp;
			ctemp = (ftemp-32) * 5 / 9;
			cout << "Equivalent in Celsius is: " << ctemp << endl ;
		}
		else if (select ==2)
		{
			cout <<" Enter temperature in Celsius to convert to degrees Fahrenheit: ";
			cin >> ctemp;
			ftemp = ctemp*9/5 + 32;
			cout << "Equivalent in Fahrenheit is: " << ftemp << endl;
		}
		else if (select ==3)
		{
			cout << "Enter temperature in Kelvin to convert to Celsius: ";
			cin >> ktemp;
			ctemp = (ktemp-273.15);
			cout << "Equivalent in Celsius is: " << ctemp << endl;
		}
		else if (select ==4)
		{
			cout << "Enter Celsius to convert to Kelvin: ";
			cin >> ctemp;
			ktemp=ctemp+273.15;
			cout << "Equivalent in Kelvin is: " << ktemp << endl;
		}
		else if (select ==5)
		{	cout << "Enter Kelvin to convert to Fahrenheit: ";
			cin >> ktemp;
			ftemp= (ktemp-459.67);
			cout << "Equivalent in Fahrenheit:" << ftemp << endl;
		}
		else if (select ==6)
		{
			cout << "Enter Fahrenheit to convert to Kelvin: ";
			cin >> ftemp;
			ktemp= (ftemp+459.67);
			cout << "Equivalent in Kelvin:" << ktemp << endl;
		}
		else if (select < 0 || select > 6) // invalid if either of these is true
		{
            do
            {
                cout << "Valid options 1, 2, 3, 4, 5, or 6." << endl;
                cin >> select;
            }
            while( select < 0 || select > 6 );
       			}
	}
	cin.get(); // wait for a keypress
    return 0;
}

[Mats]

Just put cin.ignore(2) there is probably a '\n' left over.

[Firepathedian]

That didn't work either. I ran 'calculate code metrics for solution' under 'analyze' and it stated "Project: ConsoleApplication14
Configuration: Debug
Scope: None
Assembly: C:\Users\bill\Documents\Visual Studio 2012\Projects\ConsoleApplication14\Debug\ConsoleApplication14.exe
Maintainability Index:
Cyclomatic Complexity:
Depth of Inheritance:
Class Coupling:
Lines of Code:
Message: The project target file 'C:\Users\bill\Documents\Visual Studio 2012\Projects\ConsoleApplication14\Debug\ConsoleApplication14.exe' contains no managed code."
What does this mean?