[C++] Problem z if'em

Question

Witam mam problem, iż stworzyłem małe menu :

#include <windows.h>
#include <iostream>
#include <cstdlib>

using namespace std;
	
string komenda;
void menu()
{
	cout<<"Hackier Tools [Wersja 1.0]"<<endl;
	cout<<"Copyright (c) 2015 cio3kov Corporation. Wszelkie prawa zastrzezone."<<endl<<endl;
	cout<<"Hackier Tools>";
	cin>>komenda;
}

void erro()
{
	cout<<"nie ma takiej komendy!";
}


int main()
{

for(;;)
{
	/*----------------------------------------------*/
	system("COLOR A");
	/*----------------------------------------------*/



	
	menu();




	if(komenda=="/keylogger" || "keylogger")
    	MessageBox( NULL, "Usługa chwilowo niedostępna!", "Information", MB_ICONINFORMATION | MB_OK );
	else erro();
	
	if(komenda=="/information")
		MessageBox( NULL, "Usługa chwilowo niedostępna!", "Information", MB_ICONINFORMATION | MB_OK );	
	else erro();
	
	
	
	
	getchar(); 
    system("cls");
}
	return 0;

}

i gdy odpale programik i wpisze /information wyskakują dwa msgbox'y jeden z /keylogger a drugi z /information a powinen tylko z /information

Answer 1

komenda=="/keylogger" || "keylogger"

Źle sformułowałeś warunek. Powinieneś porównywać zmienne "osobno":

komenda=="/keylogger" || komeda == "keylogger"

 

Comments

dziękuję, a może nie wiesz jeszcze dlaczego gdy wpisuje /information i /keylogger wyskakuje mi oprócz komunikatu wyskakuje mi "nie ma takiej komendy z elsa?

---

ponieważ sprawdzasz czy komenda jest równa /keylogger 

niejest więc pisze ze nie jest potem sprawdza czy jest rowna /information i jest ok

---

napisz to tak

if(komenda=="/keylogger" || "keylogger")

        MessageBox( NULL, "Usługa chwilowo niedostępna!", "Information", MB_ICONINFORMATION | MB_OK );

    else  if(komenda=="/information")

        MessageBox( NULL, "Usługa chwilowo niedostępna!", "Information", MB_ICONINFORMATION | MB_OK );

    else erro();

---

Gdyż jeżeli wpiszesz np. /information to wykona się else z ifa z keyloggerem, czyli wywołanie erro. Najlepszym wyjściem z tej sytuacji jest użycie konstrukcji z else if:

if(komenda == "/keylogger" || komenda == "keylogger") { // Jeżeli ...
  // Akcja dla komendy keylogger
} else if(komenda == "information") { // W innym przypadku jeżeli ...
  // Akcja dla komendy information
} else { // W innym przypadku ...
  // W przeciwnym razie błąd
  erro();
}

 

---

Dziękuję   ;)

---

Proszę bardzo! :)

---

A jak chciałem dalej zrobić else if(...)

if(komenda=="/keylogger" || komenda=="keylogger")
	{
    MessageBox( NULL, "Usługa chwilowo niedostępna!", "Information", MB_ICONINFORMATION | MB_OK );
	}else if(komenda=="/info -computer" || komenda=="info -computer"){
		MessageBox( NULL, "Usługa chwilowo niedostępna!", "Information", MB_ICONINFORMATION | MB_OK );	
	}else if(komenda = "/regkey -new" || komenda=="regkey -new"){
		MessageBox( NULL, "Usługa chwilowo niedostępna!", "Information", MB_ICONINFORMATION | MB_OK );
	}else if(komenda=="/regkey -del" || komenda=="regkey -del"){
		MessageBox( NULL, "Usługa chwilowo niedostępna!", "Information", MB_ICONINFORMATION | MB_OK );
	}else if(komenda=="/info -program" || komenda=="info -program")
	
	else 
	{
	cout<<"nie ma takiej komendy!";	
	}

to wyskakuje mi błąd w kompilerze

 

Answer 2

if(komenda=="/keylogger" || komenda == "keylogger")

        MessageBox( NULL, "Usługa chwilowo niedostępna!", "Information", MB_ICONINFORMATION | MB_OK );

    else erro();

     

    if(komenda=="/information")

        MessageBox( NULL, "Usługa chwilowo niedostępna!", "Information", MB_ICONINFORMATION | MB_OK );

    else erro();

 

 

sprobuj tak

Comments

napisz to tak

if(komenda=="/keylogger" || "keylogger")

        MessageBox( NULL, "Usługa chwilowo niedostępna!", "Information", MB_ICONINFORMATION | MB_OK );

    else  if(komenda=="/information")

        MessageBox( NULL, "Usługa chwilowo niedostępna!", "Information", MB_ICONINFORMATION | MB_OK );

    else erro();