Printable View
Code:goto loop1;
I just woke up, not sure if I missed anything.Code:#include <windows.h>
#include <iostream>
#include <string>
using namespace std;
string pass;
string username;
int main()
{
loop1:
cout << "Enter username\n\n";
getline(cin, username);
cout << "Enter password\n\n";
getline(cin, pass);
if (username == "lala" && password == "haha")
{
cout << "Welcome, put more shit here.\n\n";
}
else
{
cout << "Wrong, little shit head.\n\n";
Sleep (2000);
goto loop1;
}
return 0;
}
Most sources will tell you goto is very, very bad design and style and you should use real loop structures instead, and I wholly support this.
* The goto statement is never necessary; it can always be elimitated by rearranging the code.
* Use of the goto statement violates the rules of structured programming.
http://c.comsci.us/syntax/statement/goto.html
* The primary problem with goto is that it allows a programmer to cause the point of execution to jump around the code arbitrarily. This creates what is not-so-affectionately known as spaghetti code.
http://www.learncpp.com/cpp-tutorial...to-statements/
* It is good programming style to use the break, continue, and return statement in preference to goto whenever possible.
http://msdn.microsoft.com/en-us/libr...dz(VS.80).aspx
* Dijkstra argued that unrestricted GOTO statements should be abolished from higher-level languages because they complicated the task of analyzing and verifying the correctness of programs.
http://en.wikipedia.org/wiki/Goto#Cr..._of_goto_usage
Have a little more respect. He's explaining you some of the very basics of C/C++. If the only thing you're interested in is getting it to work, then quit programming, you won't be up for it. In programming, getting things to work is often easy, but getting it to work well, maintainable and efficient is the true challenge.Quote:
Originally Posted by alexz003
Also, the questions you ask are the most basic questions, they are the best documented / tutorialized questions on the internet. Use the search engine properly.
@Tman151
Argh, a goto statement! I see that as a bug honestly. C/C++ has much better alternatives, that keep your code readable.
FrostElite- when i do create an else statement it makes it to where if it is wrong it just closes for me. I want it to go back to the beginning to try again.
Negeta- I know that goto statements are the worst for C++ or any coding(except batch) in that sense, but i do need something that wil make it to where i can go back so i can have the person try again.
Daevius- I know that they are basic but whenever i looked it up it never went back to the beginning to have the user try again it just closes on me.
any help regarding this is greaty appreciated
Usually (if not always) tasks where you need to go back and repeat some steps are in essence loops. When going backwards in code you can easily avoid goto by using a for/while loop instead. A forward goto on the other hand is most often the case of an if-block.Quote:
Originally Posted by alexz003
so in order to make it go back to the beginning i wil have to use goto even though it isn't recommended?
No, a goto is never needed. Negata just said you should use a loop for that (a for, while or do while loop, which ever suits best).Quote:
Originally Posted by alexz003
right but none of those work without the application closing.
The goto is just fine for small programs.
However, using goto isn't the best choice for big or released programs.
Then you're obviously doing it wrong.. are you sure your conditions are correct? Look for information on for/while loops and how to apply them on the net.Quote:
Originally Posted by alexz003
To open an executable, use CreateProess Function
@Negeta Then can you show me how i can make it work?
@paulbub WinExec works alot better
I can't point out your error without seeing your current code.Quote:
Originally Posted by alexz003
I want it to loop back to the beginning if the username and password aren't right.Code:cout << "Please enter your username and password" << endl;
cout << "The program will hide in order to securely keep your password safe" << endl;
cout << "DO NOT CLICK ANYTHING WHEN APPLICATION HIDES IN ORDER TO MAKE THIS WORK" << endl;
cin >> username;
HWND hWnd = GetConsoleWindow();
ShowWindow( hWnd, SW_HIDE );
cin >> password;
system("cls");
hWnd = GetConsoleWindow();
ShowWindow( hWnd, SW_SHOW );
if(username == "alexz003" || username == "nonoowns", password == "xxxxxxxx")
{
system("cls");
MessageBox(0,"Login Correct!","Login", 0);
cout << "Applications or Games?" << endl;
cin >> choice;
system("cls");
if(choice == "Games")
{
cout << "Please type out the name of the game you wish to play:" << endl;
cout << "1) Redstone" << endl;
cout << "2) Wolfteam" << endl;
cout << "3) Runescape" << endl;
cout << "4) YourFunCo" << endl;
cout << "5) Conquer" << endl;
cout << "6) Sims" << endl;
cin >> gamename;
if (gamename == "Redstone")
{
system("cls");
WinExec("C:\\Program Files\\Red Stone\\Launcher.exe",SW_SHOWNORMAL);
cout << "w00t" << endl;
system("pause");
}
else if (gamename == "Wolfteam")
{
system("cls");
WinExec("C:\\Program Files\\SoftNyx\\Wolfteam\\NyxLauncher.exe",SW_SHOWNORMAL);
cout << "w00t" << endl;
system("pause");
}
else if (gamename == "Runescape")
{
system("cls");
WinExec("C:\\Program Files\\SwiftKit\\SwiftKit.exe",SW_SHOWNORMAL);
cout << "w00t" << endl;
system("pause");
}
else if (gamename == "YourFunCo")
{
system("cls");
WinExec("C:\\Program Files\\YourFunCo\\YourFunCo.exe",SW_SHOWNORMAL);
cout << "w00t" << endl;
system("pause");
}
else if (gamename == "Conquer")
{
system("cls");
WinExec("C:\\Program Files\\Conquer 2.0\\Conquer.exe",SW_SHOWNORMAL);
cout << "w00t" << endl;
system("pause");
}
else if (gamename == "Sims")
{
WinExec("C:\\Program Files\\Electronic Arts\\The Sims 3\\Game\\Bin\\Sims3Launcher.exe", SW_SHOWNORMAL);
cout << "w00t" << endl;
system("pause");
}
}
}
Use a while loop, and break if password is correct, you could also use a switch instead of else if.