Problem mit den C++ Funktionen

[PancakeBear]

Hallo Leute!

Ich habe heute angefangen mit C++ Coding und komme hier nicht weiter:

                                  

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



using namespace std; 


int Arbeiter; 
int checker; 
string username; 
string password; 


int main() 
{
    
    

    Arbeiter = 24; 

    system("title Administratorprogramm"); 
    printf("Guten Tag!\n"); 
    printf("Willkommen im Administratorprogramm!\n"); 
    printf("Arbeiter = %d", Arbeiter); 
    printf("\n"); 
    Sleep(3000); 
    system("cls");

    void user_input();
    {
    checker = 0;
    printf("Username:\n");   
    cout << endl;
    cin >> username;      
    cin  >> password; 
    if (username == "admin")
    {
        checker = checker + 1;

    }
    
    
    if(password  == "12345")
    {
    checker = checker + 1;    
    }
    

    if (checker = 2)
    {
    cout << "Erfolgreich eingeloggt!";
    getchar();
    }
    else 
    {
        system("cls");
        cout << "Username/Passwort falsch" << endl;
        cout << "Zurück zur Benutzereingabe..."<< endl;
        Sleep(2000);
        user_input();



    }
    }

Ist ein ganz einfacher Code der checken soll ob der Username und das Passwort richtig sind
Liegt es daran das ich die Funktion user_input falsch aufrufe, oder falsch erstellt habe? Oder sind da irgendwelche andere Fehler drin?

Vielen Dank ,

mfg Chillerz.

[Apex]

Hieß das nicht getch(); ?

[PancakeBear]

Danke für die Antwort

Also getchar(); müsste meiner Meinung nach richtig sein...

[Gaara]

du hast deine funktion in der main funktion deklariert. sowas geht nicht das müsste so aussehen

 void user_input()
{
code....
}

int main()
{
...code....
} 

außerdem hast du hinter void user_input() ein semikolon( ; ) was auch nicht sein darf.

auf weitere fehler hab ichs nicht geprüft.

dazu kommt dass du dir "checker" sparen kannst^^ besser wäre es wenn dus so machen würdest

if((benutzername == "admin") && (passwort == "12345"))
{

} 

[Sawyer]

Du hast eine Funktion in eine andere geschachtelt und ein paar Unnötige Semikolons. Hier eine Überarbeitete Version:

/* main.cpp */
#include <iostream> 
#include <windows.h> 
#include <string> 
using namespace std; 

// Definition
void user_input();

// Variablen
int Arbeiter = 24; 
int checker = 0; 
string username; 
string password; 

int main() 
{
    system("title Administratorprogramm"); 
    printf("Guten Tag!\n"); 
    printf("Willkommen im Administratorprogramm!\n"); 
    printf("Arbeiter = %d", Arbeiter); 
    printf("\n"); 
    Sleep(3000); 
    system("cls");

    // Aufruf der Passwortabfrage
    user_input();
}

void user_input()
{
    checker = 0;
    printf("Username:\n");   
    cout << endl;
    cin >> username;      
    cin >> password;

    if (username == "admin")
    {
        checker += 1;
    }
    
    if(password  == "12345")
    {
        checker += 1;    
    }

    if (checker == 2)
    {
        cout << "Erfolgreich eingeloggt!";
        getchar();
    }
    else 
    {
        system("cls");
        cout << "Username/Passwort falsch" << endl;
        cout << "Zurück zur Benutzereingabe..."<< endl;
        Sleep(2000);
        user_input();
    }
}

MfG, Sawyer

[PancakeBear]

Vielen Dank für die Antworten
Kann geclosed werden ,

mfg Chillerz.

[0x30]

system() zu benutzen ist sehr gefährlich... Vor allem wenn man nicht den Pfad mit angibt.

[blackberry]

Zudem ist es auch ziemlich schlechter Stil Veriablen als globale Variablen zu deklarieren.
Und das mit dem "checker" ist auch überflüssig, da man die Abfragen mit einem logischen und (&&) regeln könnte (was Gaare ja schon gesagt hat)...

Naja... um nicht weiter drauf rumzuhacken und um dem Wunsch des Threaderstellers nachzukommen:
/closed