Simple program I made - Question about code

Well there's your mistake. The op had a big problem with his conditions earlier, so if I was going to post an answer in code, I would do my best to make sure the conditions worked. Yours don't, so I made a comment.

Or you could refactor away all the junk and just go with:

Code:

#include <iostream>

int main() {
    std::cout<<"Megan Fox is the hottest girl!"<<std::endl;
}

Originally Posted by Tux0r

Or you could refactor away all the junk and just go with:

Code:

#include <iostream>

int main() {
    std::cout<<"Megan Fox is the hottest girl!"<<std::endl;
}

you forgot to return 0

Originally Posted by ಠ_ಠ

you forgot to return 0

No he didn't.

Originally Posted by bithub

No he didn't.

prove it

C++ does not require an explicit return statement from main.

Originally Posted by c++ standard

3.6.1 Main function

A return statement in main has the effect of leaving the main function
(destroying any objects with automatic storage duration) and calling
exit with the return value as the argument. If control reaches the
end of main without encountering a return statement, the effect is
that of executing
return 0;

Originally Posted by Aliaks

May I suggest using a do while loop?

Code:

#include <iostream>

using namespace std;

int main()
{
	int HottestGirl = 0;

	cout << "Who is the hottest girl?\n";
	cout << "1)Claudia Lynx\n";
	cout << "2)Megan Fox\n";
	cout << "3)Audrina Patridge\n";
	cout << "\n";

//------------------------------------------------------------------------------------------------------------------------// START   //
    do {                                                                                                                  // Do While -
        cin >> HottestGirl;                                                                                               // Input
        if (HottestGirl == 1) {                                                                                           // Set Case If
            cout << "\nClaudia Lynx is the hottest girl!\n\n";
        }
        else if (HottestGirl == 2) {                                                                                      // Set Case If
            cout << "\nMegan Fox is the hottest girl!\n\n";
        }
        else if (HottestGirl == 3) {                                                                                      // Set Case If
            cout << "\nAudrina Patridge is the hottest girl!\n\n";
        }
        else {                                                                                                            // If none above then do
            cout << "Invalid Selection\n\n";
        }
    } while (HottestGirl == 0);                                                                                           // Do the above while HottestGirl equals 0


}

hmm... I understand that except for the last part where it says:

Code:

    } while (HottestGirl == 0);

What is this there for?
Also, one thing this doesn't do is display "Selection: " before the user inputting the selection. Is there a way to fix this or no? It's not that it's so important, the program means nothing, I am just trying to learn

You need to put effort into getting a book, because loops is one basic thing all programmers must know.
If you don't have one, I would recommend getting Accelerated C++.
This loop simply says: loop WHILE HottestGirl equal 0.