Anfänger braucht Hilfe mit C++

[Junge-06]

Hallo, liebe Programmierer.
Ich habe mich gestern mal rangesetzt und habe meine ersten Versuche mit C++ gestartet. Ich habe mir als Erstes zum Ziel gesetzt, ein "Klick Macro" zu schreiben. Das klappt alles soweit auch ganz gut, nur habe ich ein Problem, ich habe leider keine Ahnung wie ich das Programm, nach starten, wieder beenden bzw stoppen kann.

Gedacht ist es so:
Man startet die .exe, und wird mit "Bitte gebe die Zeit ein" begrüßt. Nun richtet man seine Maus dort hin, wo sie klicken soll und gibt die Zeit ein, nach wie viel Sekunden der Bot an diese Stelle klicken soll. Dann kann man optional diese Schritte wiederholen (je nachdem wie viele Klicks man benötigt). Starten tut man das ganze mit Eingabe von "0". Jetzt besteht das Problem, dass das Programm nicht stoppt bzw gestoppt werden kann.

An welche Stelle muss ich jetzt was schreiben, damit man das Programm in irgend einer Art und Weise stoppen bzw beenden kann?

Bin wie gesagt noch Anfänger und habe mir dieses Programm mit Hilfe von mehreren Videos auf Youtube zusammen geschustert, also die Antworten so einfach, wie nur möglich gestalten.

Danke im Vorraus.

PS: Mein Code:

 Spoiler

// Klickbot by Junge-06

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

using namespace std;

int main () {

double zeit;
POINT pos;
vector<int> vX;
vector<int> vY;
vector<double> vZeit;
int a = 0;

while ( zeit != 0){
cout << "Bitte gebe die Zeit ein: ";
cin >> zeit;
cout << endl;


while ( zeit < 0 ){
cout << "Bitte gebe die Zeit ein: ";
cin >> zeit;
cout << endl;
}
GetCursorPos( &pos );
vX.push_back( pos.x );
vY.push_back( pos.y );
vZeit.push_back( zeit );
}
vX.push_back( 0 );
vY.push_back( 0 );
vZeit.push_back( 0 );

while( 1 ) {
Sleep( (int) (vZeit[a]*1000));
SetCursorPos( vX[a], vY[a]);
mouse_event(MOUSEEVENTF_LEFTDOWN,0,0,0,0);
mouse_event(MOUSEEVENTF_LEFTUP,0,0,0,0);

a++;
if( vX[a] == 0 && vY[a] == 0 && vZeit[a] == 0) a = 0;
}
}

Mfg Junge-06

[Kosic]

Du hast das Problem dass du hier:
while( 1 ) {
Sleep( (int) (vZeit[a]*1000));
SetCursorPos( vX[a], vY[a]);
mouse_event(MOUSEEVENTF_LEFTDOWN,0,0,0,0);
mouse_event(MOUSEEVENTF_LEFTUP,0,0,0,0);

a++;
if( vX[a] == 0 && vY[a] == 0 && vZeit[a] == 0) a = 0;
}
eine Endlosschleife hast.
Eventuell kannst du eine Funktion in die Schleife einbinden welche Pro Sekunde eine Variable +1 rechnet. Und wenn diese == zeit ist, soll die Schleife auf false gesetzt werden. Somit geht es weiter im Code und kommt zu return 0; //Programm Ende

Mfg,
Kosic

[Mokiri15]

in der schleife eine bedingung
if(vergZeit > zeit){ break; }
springt dann ebenfalls aus der Endlosschleife

[MrSm!th]

Warum nicht gleich als Schleifenbedingung?

[Junge-06]

Originally Posted by MrSm!th

Warum nicht gleich als Schleifenbedingung?

Meinst du wie die Bedingung, die vor dir jemand gerade vorgeschlagen hat?

[Kosic]

Er meint so:

PHP Code:

int count = 0;
while( count <= zeit ) {
        Sleep( (int) (vZeit[a]*1000));
        SetCursorPos( vX[a], vY[a]);
        mouse_event(MOUSEEVENTF_LEFTDOWN,0,0,0,0);
        mouse_event(MOUSEEVENTF_LEFTUP,0,0,0,0);
                count++;
        a++;
        if( vX[a] == 0 && vY[a] == 0 && vZeit[a] == 0) a = 0;
    } 


Nur die If-Abfrage verstehe ich nicht, was die bringen soll.

Mfg,
Kosic

[Mokiri15]

Ob man es im IF oder in While macht ist ja hinfällig ;;

[Junge-06]

Originally Posted by Kosic

Nur die If-Abfrage verstehe ich nicht, was die bringen soll.

Laut Video(s) soll dieses "If" dafür sorgen, dass man mit Eingabe von "0" starten kann bzw., dass man nicht 0 als Zahl eingeben kann.

Mir ist gerade noch eine Möglichkeit eingefallen. An welche Stelle müsste ich was schreiben, damit die Console dauerhaft im Vordergrund ist?

Originally Posted by Kosic

Er meint so:

PHP Code:

int count = 0;
while( count <= zeit ) {
        Sleep( (int) (vZeit[a]*1000));
        SetCursorPos( vX[a], vY[a]);
        mouse_event(MOUSEEVENTF_LEFTDOWN,0,0,0,0);
        mouse_event(MOUSEEVENTF_LEFTUP,0,0,0,0);
                count++;
        a++;
        if( vX[a] == 0 && vY[a] == 0 && vZeit[a] == 0) a = 0;
    } 


Habe es jetzt auf diese Weise versucht, jedoch returnt er direkt nach der Eingabe von "0" und startet das Macro gar nicht erst - siehe

Was nun?

€: Bei Eingabe einer beliebigen Taste schließt sich das Programm.

[MrSm!th]

Originally Posted by Junge-06

Meinst du wie die Bedingung, die vor dir jemand gerade vorgeschlagen hat?

Eben nicht. Warum ein if + break? Dann doch gleich in den Schleifenkopf.

Originally Posted by Mokiri15

Ob man es im IF oder in While macht ist ja hinfällig ;;

Guter Stil undso

[Junge-06]

Originally Posted by MrSm!th

Eben nicht. Warum ein if + break? Dann doch gleich in den Schleifenkopf.

Wie genau füge ich das in den Schleifenkopf hinzu?

[Schlüsselbein]

Schau dir nochmal die stinknormale for-loop an und überlege dir, wo sich darin die Abbruchbedingung versteckt.

[Mokiri15]

in dm du die bedingung in While schreibst und nicht in if


@mrsmith
du siehst ja, dass er es nicht kann ;D

[MrSm!th]

Originally Posted by Junge-06

Wie genau füge ich das in den Schleifenkopf hinzu?

Ich würde vorschlagen, du lernst die Sprache richtig, bevor du dich mit Makros und dergleichen beschäftigst. Wenn du schnelle Erfolge in diese Richtung möchtest, nimm AutoIt.

#closed