beep system

Reply to beep system on Mon, 17 Mar 2008 13:30:33 GMT

tingle — Mon, 17 Mar 2008 13:30:33 GMT

hi ich programmiere gerade eine beep/keyboard ähnliches teil zum üben es funtz zwar aber aus irgenteingrund daurt es immer before ich wieder ein sound machen kann... hier is das programm, was mach ich falsch oder warum is das so?:

int main()
{
	if (kbhit() && (getch()==27))
	{
		return 0;
	}
	if (kbhit() && (getch()==59))
	{
		Beep(200,150);
	}
	if (kbhit() && (getch()==60))
	{
		Beep(400,150);
	}
	if (kbhit() && (getch()==61))
	{
		Beep(600,150);
	}
	return main();
}

Reply to beep system on Mon, 17 Mar 2008 13:39:05 GMT

asc — Mon, 17 Mar 2008 13:39:05 GMT

tingle schrieb:

...

1. Nutze schleifen, hier sind rekursive Aufrufe fehl am Platz.
2. main sollte niemals im Programmcode aufgerufen werden.
3. Wird in einem Singlethread-Programm jeder Aufruf linear ausgeführt (Sprich ein Aufruf blockiert bis er fertig ist den eigentlichen Programmfluß). Du solltest dir dirs vor allem mal bezüglich deiner vielen Funktionsaufrufe in deinen ifs überlegen (zumal hier ein switch ohnehin besser wäre). Spiel mal im Gedanken durch was es für dein Programm bedeutet...

cu André

Reply to beep system on Mon, 17 Mar 2008 13:55:28 GMT

unskilled — Mon, 17 Mar 2008 13:55:28 GMT

void meinthread (void *parameter)
{
  Beep (((std::pair *) parameter)->first, ((std::pair *) parameter)->second);
}
//----------------------------------------

int main ()
{
std::pair  Ton;
while (kbhit ())
{
switch (getch ())
  {
      case 'C':
        Ton.first = tonhoehe;
        Ton.second = tonlaenge;
        _beginthread (NULL, meinthread, (void *) &bla, 0); //bin mir hier nicht mehr ganz sicher ^^
//außerdem wird es auch sehr viel bessere varianten geben... vll sind if else konstrukte auch besser - da kann man wenigstens variablen drin deklarieren
        break;
//etc
      default:
        return EXIT_SUCCESS;
  }
};
}

Reply to beep system on Mon, 17 Mar 2008 15:32:17 GMT

Tingle_ — Mon, 17 Mar 2008 15:32:17 GMT

ah danke dann mach ichs so ^^ noch ne frage ich will das alles übern numpad machen und nicht über F1,F2,F3, ect... was sind die zahlen bzw wo find ich die zahlen für die keys?

Reply to beep system on Mon, 17 Mar 2008 15:50:14 GMT

unskilled — Mon, 17 Mar 2008 15:50:14 GMT

in der msdn glaub ich...
ansonsten einfach so:

while (kbhit ())
{
std::cout << getch () std::endl;
Sleep (100);
};

dann merkste dir, was du drückst und was auf der konsole erscheint ^^ geht am schnellsten - allerdings wird das ja meist über defines bzw const int`s gemacht... also hast du dann nur den wert - das sollte sich zwar die nächsten tage nicht gleich ändern aber doof isses trotzdem ^^

Reply to beep system on Tue, 18 Mar 2008 07:45:49 GMT

Tingle_ — Tue, 18 Mar 2008 07:45:49 GMT

habs hingekriegt ^^ ma ne andere frage ich brauch ein if comparison das check ob das was die person eingibt eine zahl is oder nicht:

int main()
{
	cout << "-----------------------------------\n";
	cout << "---- Tingle's Frequency Tester ----\n";
	cout << "-------- Press Esc to Exit --------\n";
	cout << "-----------------------------------\n\n";
	int b;
	int a;
	b = 1;
	while (b=1)
	{
			cout << "Please enter a Frequency: ";
			cin >> a;
			Beep (a,300);
			cout << "\nBeep at " << a << " Frequency\n\n";
	}
    return 0;
}

ich will das da ein if is ob a eine nummer is oder ein buchstabe und wenn es was anderes is als eine nummer soll es eine nachricht bringen also ein error

Reply to beep system on Tue, 18 Mar 2008 10:33:15 GMT

Tingle_ — Tue, 18 Mar 2008 10:33:15 GMT

mm i got it but what can i do in the if statment to check if int a is a number because when ever i typ in letter my programm starts going wild

int main()
{
	cout << "-----------------------------------\n";
	cout << "---- Tingle's Frequency Tester ----\n";
	cout << "-----------------------------------\n\n";
	int b;
	int a;
	b = 1;
	while (b=1)
	{
			cout << "Enter a Frequency: ";
			cin >> a;
			if (a == )
			{
			Beep (a,300);
			cout << "\nBeep at " << a << " Frequency\n\n";
			}
			else
			{
				cout << "\nError :: Not a legal variable\n\n";
			}
	}
    return 0;
}

Reply to beep system on Tue, 18 Mar 2008 10:40:27 GMT

Fellhuhn — Tue, 18 Mar 2008 10:40:27 GMT

if (cin.fail()){
  cout << "An error occured!" << endl;
  cin.clear();
}

http://www.cppreference.com/cppio/index.html

Reply to beep system on Tue, 18 Mar 2008 10:50:26 GMT

tingle_ — Tue, 18 Mar 2008 10:50:26 GMT

thx it works but no it doesnt now the message comes but it loops in an error for some reson take a look:

#include "stdafx.h"
using namespace std;

int main()
{
	cout << "-----------------------------------\n";
	cout << "---- Tingle's Frequency Tester ----\n";
	cout << "-----------------------------------\n\n";
	int b;
	int a;
	b = 1;
	while (b=1)
	{
			cout << "Enter a Frequency: ";
			cin >> a;
			if (a >= 0)
			{
			Beep (a,300);
			cout << "\nBeep at " << a << " Frequency\n\n" << endl;
			}
			if (cin.fail())
			{
				cout << "Error, only numbers are legal\n\n" << endl;
				cin.clear();
			}
	}
    return 0;
}

Reply to beep system on Tue, 18 Mar 2008 10:51:36 GMT

tingle_ — Tue, 18 Mar 2008 10:51:36 GMT

ah nevermind i know why because it keeps the integer saved in a... how to i clear

Reply to beep system on Tue, 18 Mar 2008 11:24:48 GMT

Fellhuhn — Tue, 18 Mar 2008 11:24:48 GMT

#include 

using namespace std;

int main()
{
	cout << "-----------------------------------\n"
	     << "---- Tingle's Frequency Tester ----\n"
	     << "-----------------------------------\n\n";
	int a;
	while (true)
	{
			cout << "Enter a Frequency (-1 to quit): ";
			cin >> a;
			if (cin.fail())
			{
				cout << "Error, only numbers are legal\n\n" << endl;
				cin.clear();
			}
			else if (a >= 0)
			{
			  Beep (a,300);
			  cout << "\nBeep at " << a << " Frequency\n\n" << endl;
			}
                        else
                        { 
                          if (a == -1) break;
                          cout << "Only numbers greater or equal than zero allowed\n\n" << endl;
                        }
	}
}

Reply to beep system on Tue, 18 Mar 2008 11:34:50 GMT

tingle_ — Tue, 18 Mar 2008 11:34:50 GMT

nope as soon as you typ in a number it loops/crashes

Reply to beep system on Tue, 18 Mar 2008 11:52:32 GMT

tingle_ — Tue, 18 Mar 2008 11:52:32 GMT

um i mean character

Reply to beep system on Tue, 18 Mar 2008 12:07:46 GMT

Fellhuhn — Tue, 18 Mar 2008 12:07:46 GMT

cin.ignore(); may help. Or cin.ignore(1000);

Or something like string sth; cin >> sth; just to get rid of the rest. But it is unclean.

Reply to beep system on Tue, 18 Mar 2008 12:34:03 GMT

unskilled — Tue, 18 Mar 2008 12:34:03 GMT

@tingle_: deutsch zu schreiben wurde wohl auf die Dauer langweilig? ; )

Reply to beep system on Tue, 18 Mar 2008 12:42:09 GMT

tingle_ — Tue, 18 Mar 2008 12:42:09 GMT

lol^^ sry^^ ich bin bilangual auf gewachsen ich merks meinstens garnicht vorallem im i-net hehehe

Reply to beep system on Tue, 18 Mar 2008 13:23:08 GMT

unskilled — Tue, 18 Mar 2008 13:23:08 GMT

^^ np - mein englisch is gerad noch gut genug um das zu verstehen ;o) aber in nem deutschen forum isses wahrscheinlich besser, deutsch zu schreiben (man bekommt schneller antworten etc ^^)

Reply to beep system on Tue, 18 Mar 2008 14:38:02 GMT

tingle_ — Tue, 18 Mar 2008 14:38:02 GMT

^^, mal ne frage ich check das ganz net mit switch und so... könnt mir jemand ein template(skellet) her schreiben für das was ich machen will? das sind die tasten codes die töne erzeugen sollen:
(alles nummpads ^^)
0 = 48
1 = 49
2 = 50
3 = 51
4 = 52
5 = 53
6 = 54
7 = 55
8 = 56
9 = 57

Reply to beep system on Tue, 18 Mar 2008 15:27:42 GMT

tingle_ — Tue, 18 Mar 2008 15:27:42 GMT

ah egal ich habs ^^ funtzt ja supper... nur ne frage gibt es ein command das die aktuele zeile gelöscht wird bzw das was angezeigt wird? weil 20x \b funtzt nur in so fern wie nix länger is als das vorherige

Reply to beep system on Tue, 18 Mar 2008 16:12:26 GMT

unskilled — Tue, 18 Mar 2008 16:12:26 GMT

das was angezeigt wird

versteh die frage nicht -.-

std::cout << "text" << varialbe << /*...*/ <<  std::endl;

geht zum anzeigen eigtl immer ganz gut fand ich ^^
zum zeile löschen gibts mit sicherheit auch was - aber da müsstest du wahrscheinlich mal unter improved console gucken - hab ich kA von ^^

Reply to beep system on Wed, 19 Mar 2008 10:50:39 GMT

tingle_ — Wed, 19 Mar 2008 10:50:39 GMT

juhuu^^ hab das programm fertig sogar mitn intigierten lied "Popcorn" xD hier is download link und source:

http://filebeam.de/temp/Tingles_Sound_Pad.exe

#include "stdafx.h"
using namespace std;

#define C 65
#define D 73
#define E 82
#define F 87
#define G 99
#define A 110
#define H 123
#define Cs 70
#define Ds 78
#define Fs 92
#define Gs 104
#define As 117

int main()
{
	int a;
	int b;
	int c;
	b = 150;
	a = 1;
	c = 1;
	cout << "---------------------------" << endl;
	cout << "---- Tingles Sound Pad ----" << endl;
	cout << "---------- v3.05 ----------" << endl << endl;
	cout << "Keys:" << endl << endl;
	cout << "ESC - Exit" << endl << endl;
	cout << "F1 - Short Tone" << endl;
	cout << "F2 - Normal Tone" << endl;
	cout << "F3 - Long Tone" << endl;
	cout << "F4 - Very Long Tone" << endl;
	cout << "F5 - Extra Long Tone" << endl << endl;
	cout << "F6 - Octav 2 and 3" << endl;
	cout << "F7 - Octav 4 and 5" << endl;
	cout << "F8 - Octav 6 and 7" << endl << endl;
	cout << "F9 - Play Popcorn Song" << endl << endl << endl << endl;
	while (_kbhit() && a || 1)
	{
		switch(_getch ())
		{
		case 27:
			cout << "\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\bExiting... Good bye               ";
			Sleep(2000);
			return 0;
			break;
		case 59:
			b = 50;
			cout << "\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\bSet to Short Tone               ";
			break;
		case 60:
			b = 150;
			cout << "\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\bSet to Normal Tone               ";
			break;
		case 61:
			b = 250;
			cout << "\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\bSet to Long Tone               ";
			break;
		case 62:
			b = 350;
			cout << "\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\bSet to Very Long Tone               ";
			break;
		case 63:
			b = 450;
			cout << "\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\bSet to Extra Long Tone               ";
			break;
		case 64:
			c = 1;
			cout << "\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\bSet to Octav 2 and 3              ";
			break;
		case 65:
			c = 4;
			cout << "\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\bSet to Octav 4 and 5               ";
			break;
		case 66:
			c = 16;
			cout << "\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\bSet to Octav 6 and 7               ";
			break;
		case 67:
			cout << "\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\bPlaying Popcorn Song               ";
				Beep(H*c,b);
				Sleep(100);
				Beep(A*c,b);
				Sleep(100);
				Beep(H*c,b);
				Sleep(100);
				Beep(Fs*c,b);
				Sleep(100);
				Beep(D*c,b);
				Sleep(100);
				Beep(Fs*c,b);
				Sleep(100);
				Beep(62*c,b);
				Sleep(300);
				Beep(H*c,b);
				Sleep(100);
				Beep(A*c,b);
				Sleep(100);
				Beep(H*c,b);
				Sleep(100);
				Beep(Fs*c,b);
				Sleep(100);
				Beep(D*c,b);
				Sleep(100);
				Beep(Fs*c,b);
				Sleep(100);
				Beep(62*c,b);
				Sleep(300);
				Beep(H*c,b);
				Sleep(100);
				Beep(Cs*2*c,b);
				Sleep(100);
				Beep(D*2*c,b);
				Sleep(100);
				Beep(Cs*2*c,b);
				Sleep(100);
				Beep(D*2*c,b);
				Sleep(100);
				Beep(H*c,b);
				Sleep(100);
				Beep(Cs*2*c,b);
				Sleep(100);
				Beep(H*c,b);
				Sleep(100);
				Beep(Cs*2*c,b);
				Sleep(100);
				Beep(A*c,b);
				Sleep(100);
				Beep(H*c,b);
				Sleep(100);
				Beep(A*c,b);
				Sleep(100);
				Beep(Fs*c,b);
				Sleep(100);
				Beep(H*c,b);
				Sleep(100);
				cout << "\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\bPopcorn Song stoped               ";
				break;
		case 121:
			Beep(65*c,b);
			break;
		case 115:
			Beep(69*c,b);
			break;
		case 120:
			Beep(73*c,b);
			break;
		case 100:
			Beep(78*c,b);
			break;
		case 99:
			Beep(82*c,b);
			break;
		case 118:
			Beep(87*c,b);
			break;
		case 103:
			Beep(92*c,b);
			break;
		case 98:
			Beep(99*c,b);
			break;
		case 104:
			Beep(104*c,b);
			break;
		case 110:
			Beep(110*c,b);
			break;
		case 106:
			Beep(117*c,b);
			break;
		case 109:
			Beep(123*c,b);
			break;
		case 113:
			Beep(130*c,b);
			break;
		case 50:
			Beep(139*c,b);
			break;
		case 119:
			Beep(147*c,b);
			break;
		case 51:
			Beep(156*c,b);
			break;
		case 101:
			Beep(165*c,b);
			break;
		case 114:
			Beep(175*c,b);
			break;
		case 116:
			Beep(196*c,b);
			break;
		case 54:
			Beep(208*c,b);
			break;
		case 122:
			Beep(220*c,b);
			break;
		case 55:
			Beep(233*c,b);
			break;
		case 117:
			Beep(247*c,b);
			break;
		case 53:
			Beep(185*c,b);
			break;
		default:
			cout << "\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\bUnknow Key...               ";
		}
	}
	return 0;
}

Reply to beep system on Wed, 19 Mar 2008 14:26:40 GMT

unskilled — Wed, 19 Mar 2008 14:26:40 GMT

#define C 65
#define D 73
#define E 82
#define F 87
#define G 99
#define A 110
#define H 123
#define Cs 70
#define Ds 78
#define Fs 92
#define Gs 104
#define As 117
//geht, aber besser:
namespace Toene
{
const unsigned char C (65); //konstanter 8bit großer positiver int mit dem wert 65
/*rest...*/
}

int a;
int b;
int c;
b = 150;
a = 1;
c = 1;
//geht zwar, aber:
int a (1); //brauchst du gar nicht ^^
int b (150);
int c (1);

cout << "\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\bExiting... Good bye               ";
return 0;
//naja...
cout << endl << "Exiting... Good bye";
return 0;
//geht wohl nicht?
//ist auch KEINE gute lösung aber ich find sie besser, als die jetzige ^^:
SYSTEM ("CLS"); //konsole ist leer, aber besser, du guckst ma bei improved console vorbei
cout << "Exit... Bye";
return 0;

case 67:
            cout << "\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\bPlaying Popcorn Song               ";
                Beep(H*c,b);
                Sleep(100);
//...
                Sleep(100);
                cout << "\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\bPopcorn Song stoped               ";
                break;

wie wärs statt dessen so?

void PopcornSong (void)
{
                cout << "Playing Popcorn Song";
                Beep(H*c,b);
                Sleep(100);
//...
                Sleep(100);
                cout << "Popcorn Song stopped"; //doppel p dachte ich ^^
}

case 67:
  PopcornSong ();
  break;

ist doch ein wenig übersichtlicher, oder? ^^

while (_kbhit() && a || 1)
{
//entweder, ich habs übersehen, oder du nutzt nirgendwo a...
}
//=>
while (_kbhit ())
{
//...
}

noch bissl was allgemeines:

Beep (D*2*c,b); //sieht zwar vll ne schlecht aus, aber ich fänds so besser:

/*ich weiß jz nicht, was Beep als ersten Parameter erwartet - ich nehm mal DWORD an...*/

const DWORD getTone (const unsigned char ton, const unsigned char oktave)
/*
könnte man auch noch in den Namespace Toene mit packen - oder sollte ^^
ReturnValue: 1. Parameter von Beep
ton: eben der ton an sich ^^
oktave: relative Angabe zur "normalen" Oktave?
 (also 0 für normale Oktave, -1 für eine tiefer etc? kA, aber fände es so übersichtlicher)
*/
{
//ich kenn die rechnung nicht - hab keine Ahnung von allem, was mit Musik zu tun hat ; )
}

//also als Aufruf dann so:
Beep (getTone (Toene::C, act_okt), laenge);
//du nimmst also statt c den act_okt und b statt laenge

Reply to beep system on Thu, 20 Mar 2008 07:47:33 GMT

tingle_ — Thu, 20 Mar 2008 07:47:33 GMT

nuja ^^ funtzt ja alles :> man kann lustige melodien damit machen allerdings gibt es eine andere möglichkeit einen text bzw zeile zu ersetzen auser ganz viele \b zu benutzten

Reply to beep system on Thu, 20 Mar 2008 13:46:21 GMT

unskilled — Thu, 20 Mar 2008 13:46:21 GMT

Wie gesagt: Hier gibts nen forum namens improved console - frag dort mal nach ^^ Aber wäre es denn so schlimm, die Zeile dazulassen und dann einfach eine weiter unten zu schreiben?

Es funtzt zwar alles, aber ich find es nicht schön und doch ein wenig unübersichtlich

Nun ja - damit wars das wo erst mal ^^
bb