Internal Beep Program

[agent766]

Here's basically a simple program I made to REALLY annoy my friends. 10kHz is awesome at annoying people

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

int main() {
do {
     cout << "Frequency: ";
     cin >> x;
     cout << endl << "Duration: ";
     cin >> y;

     Beep(x,y);
     System("CLS");
} while(true);
}

 

[Swan]

Here's basically a simple program I made to REALLY annoy my friends. 10kHz is awesome at annoying people

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

int main() {
do {
     cout << "Frequency: ";
     cin >> x;
     cout << endl << "Duration: ";
     cin >> y;

     Beep(x,y);
     System("CLS");
} while(true);
}

Mind telling me where you declared variables 'x' and 'y'?

Also, you could just use escape characters to achieve the same thing.

printf("\a");

omits a system beep.

 

[theory_]

By convention, his main should return something too. agent766, should also use k&r style scoping. A better control flow statement might have just been using a while statement(less typing). And variable x, and y are not declared.

 

[Swan]

By convention, his main should return something too. agent766, should also use k&r style scoping. A better control flow statement might have just been using a while statement(less typing). And variable x, and y are not declared.

Ehh, do-while has a subtle difference though. It actually executes once beforehand regardless of the condition set in the while loop. But I agree for the most part.

 

[Teh_Uber_Codex]

Thanks SWAN.

~Uber

 

[xedon]

Try this;

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

using namespace std;

int i;
int freqa;
int lengtha;
int lengtht;
int waita;
int waitt;
int timesa;




int main() {

cout << "How long would you like it to play for in seconds: ";
cin >> lengtht;
cout << "What frequency would you like to play at (about 20-200): ";
cin >> freqa;
cout << "How many times would you like this to be played: ";
cin >> timesa;
cout << "How long should we wait in between each beep in seconds: ";
cin >> waitt;

lengtha = lengtht*1000;
waita = waitt*1000;
for(i= 0; i < timesa; i++) {
Beep(freqa,lengtha);
Sleep(waita);
}

}

 

[d great one]

very long declaration of int .

You could just state

int i,freqa,lengtha,lengtht,waita,waitt,timesa;