Urgent need help!!!!!
I need help with a program for school. I keep getting a weird volume when I turn the radio on, and frequency wont work. it has to go from am, fm, aux, to mp3. AM frequency (530 kHz-1610kHz, going up by 10kHz). FM frequency (88.1MHz- 180MHz, going up by 100kHz). Volume up and down and a clock.
Last edited by josh223456; 03-29-2013 at 11:46 PM.
Topic Titles
Urgency
I'll leave you to read the rest of that FAQ
If you dance barefoot on the broken glass of undefined behaviour, you've got to expect the occasional cut.
If at first you don't succeed, try writing your phone number on the exam paper.
The first thing I would suggest is figuring out which language you're supposed to be learning.
Because at the moment, you have a train-wreck called C/C++.
The reason you get all sorts of garbage printed is you're abusing printf!Code:$ g++ -Wall -c foo.cpp foo.cpp: In member function ‘void cRadio::powerbutton()’: foo.cpp:54:27: warning: format ‘%d’ expects a matching ‘int’ argument [-Wformat] foo.cpp:57:39: warning: too many arguments for format [-Wformat-extra-args] foo.cpp:60:39: warning: too many arguments for format [-Wformat-extra-args] foo.cpp:63:40: warning: too many arguments for format [-Wformat-extra-args] foo.cpp:66:40: warning: too many arguments for format [-Wformat-extra-args] foo.cpp: In member function ‘void cRadio::setmode()’: foo.cpp:181:39: warning: too many arguments for format [-Wformat-extra-args] foo.cpp:184:38: warning: too many arguments for format [-Wformat-extra-args] foo.cpp:187:37: warning: too many arguments for format [-Wformat-extra-args] foo.cpp:190:38: warning: too many arguments for format [-Wformat-extra-args] foo.cpp: At global scope: foo.cpp:200:14: error: ‘::main’ must return ‘int’
If you used something like
cout << "Volume = " << volume << endl;
there would be a lot less chance of screwing up.
The other problem I see is that when you switch mode which involves either FM or AM, then you need to reset freq to something appropriate for that band.
Maybe say
int lastFMfreq;
int lastAMfreq;
If you dance barefoot on the broken glass of undefined behaviour, you've got to expect the occasional cut.
If at first you don't succeed, try writing your phone number on the exam paper.
could you show me how to make it work cause im very new to programming, and have been stuck on this for hours.
Given that you used the word "Urgent" in your subject line, I might consider answering your question in a week from next Tuesday. That reflects the actual urgency that is warranted.
As per one of the links that Salem provided in post #2, flagging your post as urgent is no more than a rude and selfish attempt to gain special attention.
No. Read the homework policy.Originally Posted by josh223456
If you are really struggling with this code, perhaps you a bit on over your head? If so, it might be better to start off with a simpler project.
printf - C++ Reference
Not the way I would do it.
Shorter and clearer in my view.Code:if (power==false){ } else
Tim S.Code:if (power!=false){
Last edited by stahta01; 03-30-2013 at 06:29 PM.
"...a computer is a stupid machine with the ability to do incredibly smart things, while computer programmers are smart people with the ability to do incredibly stupid things. They are,in short, a perfect match.." Bill Bryson
Also here.
