Absturz verhindern & zwei Dinge synchron ablaufen lassen

[B4n4n4]

Hallo ich bin es mal wieder

Meine Probleme heute?

1) Wenn ich eine integer Variable erstelle die durch "cin >> var;" eingegeben werden soll, ich allerdings einen Buchstaben eingebe führt das bei mir unweigerlich zu totalem Chaos!
Was mich dabei wundert ist das ich eine Schleife verwende die so lange läuft bis die eingegebene Zahl zwischen 1 und 7 liegt, und über if ausgibt das nur Zahlen von 1-7 eingegeben werden sollen.
Solange ich eine zu hohe/niedrige Zahl eingebe funktioniert das wunderbar, wird in einer integer Variable ein Zeichen nicht einfach als ASCII Zahl des entsprechenden Zeichens gespeichert?
Statt aber wie gewünscht zu funktionieren wird endlos "Bitte nur 1-7 eingeben!" ausgegeben.

Hier mal Quellcode:

int eingabe=0;
        while(eingabe<1 || eingabe>7)
        {
        cin >> eingabe;
        if(eingabe<1 || eingabe>7)
            cout << "Bitte nur 1-7 eingeben!" << endl;
        };

2) Mein Programm soll alle X Sekunden etwas tun. Dies habe ich durch "Sleep(X);" in einer Schleife realisieren können. Wie kann ich aber nun eine weitere Schleife gleichzeitig laufen lassen, die alle Y Sekunden etwas anderes tut? Oder lässt sich das ganze in einer Schleife realisieren? Die beiden Sekunden angaben sollen nämlich frei wählbar sein.

Freue mich auf hilfreiche Antworten (hoffentlich wieder von BB, unserem zynischen C Guru ),

B4n4n4

[inout]

Hi,

Das was du da bei 1) machst ist ziemlich gefährlich, du solltest nach jedem Einlesen immer den Status des Streams überprüfen (und ggf. zurücksetzen), da sonst einige unvorhergesehende Dinge passieren können.
Ih würde das hier vorschlagen (ich weiss es geht schöner, ohne Endlosschleife und so ):

{
    int eingabe;

    for(;;) {
        cout << "Bitte Zahl zwischen 1 und 7 eingeben: ";
        cin >> eingabe;
        if(cin.fail())
            cout << "Keine Zahl!" << endl;
        else if(eingabe < 1 || eingabe > 7)
            cout << "Keine Zahl zwischen 1 und 7!" << endl;
        else
        {
            cout << "Alles Klar!" << endl;
            break;
        }
        cin.clear();
        cin.ignore(INT_MAX, '\n');
    }
    cout << "Eingabe: " << eingabe << endl;
}

Bei 2) brauchst du dann wohl Threading.

[B4n4n4]

Vielen Dank für Antwort 1 !!

Zu 2:

Nur mal fragen ob ich das mit den Threads richtig verstanden habe.

DWORD WINAPI banane(LPVOID); //definiert den Thread

HANDLE hThread;
DWORD  dwThreadID;
hThread = CreateThread(NULL,0,banane,0,0,&dwThreadID);
CloseHandle(hThread); //ruft den Thread auf (bedeutet was?)

DWORD WINAPI banane(LPVOID data) //gibt dem Thread eine Funktion.
{
        cout << "Das kommt aus dem Thread!" << endl;
};

Muss ich den Thread also erst aufrufen bevor ich ihm Befehle zuweisen kann?

[AlterHacker]

DWORD WINAPI banane(LPVOID); //definiert die Banane-Funktion

HANDLE hThread;
DWORD  dwThreadID;
hThread = CreateThread(NULL,0,banane,0,0,&dwThreadID); //Erstellt den Thread.
CloseHandle(hThread); //Gibt das Handle frei und beendet nicht den Thread
DWORD WINAPI banane(LPVOID data) //Definiert den Funktionskörper von banane. ;)
{
        cout << "Das kommt aus dem Thread!" << endl;
};

[wacked]

Falsch.

#include <windows.h>
#include <iostream>
using namespace std;
DWORD WINAPI banane(LPVOID); //definiert den Thread
int main()
{
    HANDLE hThread;
    DWORD  dwThreadID, dwExitCode;

    hThread = CreateThread(NULL,0,banane,0,0,&dwThreadID);
    CloseHandle(hThread);
    cout << "CloseHandle(hThread)" << endl;
    Sleep(5000);
    GetExitCodeThread(hThread, &dwExitCode);
    if(GetLastError() != ERROR_INVALID_HANDLE)
        cout << "Well, that was unexpected" << endl;
    cout << " Exit code " << dwExitCode << endl;
}
DWORD WINAPI banane(LPVOID data) //gibt dem Thread eine Funktion.
{
    while(1)
    {
        cout << "Das kommt aus dem Thread!" << endl;
        Sleep(100);
    }
};

In diesem Bild sieht man das der Thread noch weiterläuft. (Außerdem sieht man direkt eine Gefahr des Multithreadings. Da die beiden couts "gleichzeitig" ablaufen wird der Text gemixt ausgegeben.)

CloseHandle() bewirkt nur dass man mit diesem Handle (Handle wie der Halter an der Tasse) nichts mehr anfangen kann. Wenn danach etwas mit diesem Handle gemacht werden soll (GetExitCodeThread()) merkt diese Funktion dass dieses Handle ungültig ist und bricht ab.

[Dj Hackmaster A]

Generell "sollte" man einen Thread nicht von außen terminieren. Wenn man das doch machen möchte, dann natürlich mit TerminateThread, das Handle hat selbstverständlich keine Auswirkung auf den Exekutionskontext des Threads.

[motion]

Du könntest den ganzen Multithreading Spaß auch mit Boost realisieren, ist glaub ich einfacher und vor allem Plattformunabhängig:
Highscore - Die Boost C++ Bibliotheken - Multithreading

[B4n4n4]

motion, tut mir wirklich Leid das man nur einmal "THX" drücken kann

Die "normalen" Threads haben zwar so langsam angefangen sich verstehen zu lassen, aber je einfacher umso besser

Freundlich Grüßt,

die Banane vom Dienst

@wacked

Dein Quellcode tut zwar genau das was er soll, wenn ich ihn allerdings kompiliere stürzt jedesmal ein Spiel mit folgender Fehler Meldung ab:

Access violation at adress 0047644A in module 'noaex.dat'. Read of address 00000000.