Problem mit den C++ Funktionen

Hallo Leute!

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

Code:

---

                                 

#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.

Hieß das nicht getch(); ?

Danke für die Antwort :)

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

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

PHP-Code:

---

 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

PHP-Code:

---

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

} 


---

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

Code:

---

/* 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

Vielen Dank für die Antworten :)
Kann geclosed werden ,

mfg Chillerz.

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

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