void (__cdecl *)(int)' in 'DWORD

[xazorstix]

Hey e*pvpers,

Ich habe gerade mit C++ angefangen. Nun wollte ich einen Timer machen, in den man die Zeit eingibt und der dann diese Zeit abwartet(Sleep) und danach den Warnton(\a) ausgibt.

Dies hier ist mein Code:

 Spoiler

#include <iostream>
#include <string>
#include <windows.h>
using namespace std;

int sectime, mintime, htime;

void sectim(int n)
{
	sectime = n*1000;
	Sleep(sectim);
}

void mintim(int n)
{
	mintime = n*60000;
	Sleep(mintim);
}

void htim(int n)
{
	htime = n*3600000;
	Sleep(htim);
}

int main()
{
	string t;
	int n;

	cout << "Wanna give the time in seconds, minutes or hours ?" << endl;
	cin >> t;

	cout << "Please set the time" << endl;
	cin >> n;

	if(t == "sec", "seconds")
		sectim(n);
	else if(t == "min", "minutes")
		mintim(n);
	else if(t == "h", "hours")
		htim(n);

	return 0;
}

Dies hier ist der Fehlercode

 Spoiler

1>a:\dokumente\c++\timer\timer.cpp(11) : error C2664: 'Sleep': Konvertierung des Parameters 1 von 'void (__cdecl *)(int)' in 'DWORD' nicht möglich
1> Es gibt keinen Kontext, in dem diese Konvertierung möglich ist

Bin natürlich auch für Anregungen am Code selber offen.




mfg
xazorstix

[ms​]

Du gibst immer den Funktionsnamen anstatt des Variablennamens als Argument an die Sleep-Funktion. Außerdem solltest du wo es möglich ist nur lokale Variablen benutzen, da globale Variablen böse sind.

[yihaaa]

VBA vorbelastet.

void sectim( int n )
{
	Sleep( n * 1000 );
}

MfG

[Nightblizard]

Das Problem ist ganz einfach, dass du nicht den Namen der Variablen an Sleep übergist, sondern die Funktionen. Und deine Funktionen sind in dem Fall nunmal vom typen void (__cdecl*)(int), was der Compiler jedoch nicht implizit nach unsigned long (DWORD) casten kann.

Anregungen:
Globale Variablen vermeiden.
Globales using namespace vermeiden
Gebe deinen Variablen aussagekräftige Namen. n und t sagen dir absolut gar nichts.

Was ich cool finde:
Wie du den String auf mehrere Möglichkeiten mit dem Komma Operator prüfst. Sieht man nicht sehr häufig!

[ms​]

Originally Posted by Nightblizard

Was ich cool finde:
Wie du den String auf mehrere Möglichkeiten mit dem Komma Operator prüfst. Sieht man nicht sehr häufig!

Ist...

if(t == "sec", "seconds")

... nicht gleichwertig mit ...?

if("seconds")

Ich dachte immer der Komma-Operator evaluiert zu dem letzten Ausdruck und verwirft den Rest.

[Nightblizard]

Originally Posted by Metin2Spieler97

Ist...

if(t == "sec", "seconds")

... nicht gleichwertig mit ...?

if(t == "seconds")

Ich dachte immer der Komma-Operator evaluiert zu dem letzten Ausdruck und verwirft den Rest.

Wir haben beide unrecht.
Der Code macht folgendes:

mov ecx, offset string "seconds"
test ecx,ecx
je DEADBEEFh ;überspringt den if block

Der if-Block wird also immer aufgerufen!


Edit:
Ah, du hast es geändert.

[ms​]

Originally Posted by Nightblizard

Edit:
Ah, du hast es geändert.

Mir ist im Nachhinein erst aufgefallen, dass der == Operator vor dem Komma kommt. :>

[xazorstix]

DANKE für eure Hilfe, ich habe die Funktion in Sleep durch die Variablen ersetzt.
Dazu habe ich noch einen Warnton eingebaut, welcher ertönt wenn Sleep(time) vorüber ist.

Habe auch eure anderen Verbesserungen mit eingebracht.

 Spoiler

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


void sectime(int n)
{
	using namespace std;
	int sectim;
	sectim = n*1000;
	Sleep(sectim);
	cout << "\a";
}

void mintime(int n)
{
	using namespace std;
	int mintim;
	mintim = n*60000;
	Sleep(mintim);
	cout << "\a";
}

void htime(int n)
{
	using namespace std;
	int htim;
	htim = n*3600000;
	Sleep(htim);
	cout << "\a";
}

int main()
{
	using namespace std;
	string timetype;
	int time;

	cout << "Wanna give the time in seconds, minutes or hours ?" << endl;
	cin >> timetype;

	cout << "Please set the time" << endl;
	cin >> time;

	if(timetype == "sec", "seconds")
		sectime(time);
	else if(timetype == "min", "minutes")
		mintime(time);
	else if(timetype == "h", "hours")
		htime(time);

	return 0;
}

[Nightblizard]

Du kannst die Eingabe so nicht prüfen. Der Compiler macht aus diesem Code

if(timetype == "sec", "seconds")

folgendes:

timetype == sec;
if("seconds") //prüft, ob der angegebene String ungleich null ist und das ist er, weshalb dieser Teil immer aufgerufen wird

Du kannst also so nicht prüfen, da egal ist was du eingibst, es wird immer ausgeführt!
Wie es du machen solltest:

if(tyetype == "sec" || timetype == "seconds")

[xazorstix]

Wär ja auch zu schön gewesen

 Spoiler

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


void sectime(int n)
{
	using namespace std;
	int sectim;
	sectim = n*1000;
	Sleep(sectim);
	cout << "\a";
}

void mintime(int n)
{
	using namespace std;
	int mintim;
	mintim = n*60000;
	Sleep(mintim);
	cout << "\a";
}

void htime(int n)
{
	using namespace std;
	int htim;
	htim = n*3600000;
	Sleep(htim);
	cout << "\a";
}

int main()
{
	using namespace std;
	string timetype;
	int time;

	cout << "Wanna give the time in seconds, minutes or hours ?" << endl;
	cin >> timetype;

	cout << "Please set the time" << endl;
	cin >> time;

	if(timetype == "sec" || timetype == "seconds")		
		sectime(time);
	else if(timetype == "min" || timetype == "minutes")		
		mintime(time);
	else if(timetype == "h" || timetype == "hours")		
		htime(time);			

	return 0;
}

[MrSm!th]

Es ist auch nicht gerade schlau, die Namen der Funktionen den Namen der Variablen so ähnlich zu wählen (sectime, sectim).