Hey guys. Im a bit perplexed! Im trying to set the rate of blinking of an LED (in an optocoupler) by reading what comes through the serial port. Here is my code;
#define OPTO 13 // optocoupler pin
int val =0; // variable to store the serial
int pause =0; // variable for our blinks
void setup(){
Serial.begin(300);
pinMode (OPTO, OUTPUT);
}
void loop (){
while (Serial.available() ==0 ); // Arduino waits to recieve data
int val = Serial.read()-('0'); // only reads the first byte of data
int pause = map (val,0,9, 15, 100); // hence maximum of 9
pause = constrain (pause , 15, 100);
digitalWrite(OPTO, HIGH);
delay(pause);
digitalWrite (OPTO, LOW);
delay(pause);
Serial.println (pause);
}
The problem is that it only blinks the LED at the specified rate once, and then the serial goes back to 0.
Any help would be smashing thanks!
Step away from Arduino for a minute and think about what you need to do. Sometimes I actually write it out (on the back of a cocktail napkin at the NAME OF WATERING HOLE DELETED).
Often I write it in pseudo-code rather than my native language (American-English, as it happens.) The point is that I try to express it in logical terms before I actually write code.
(Instead of scanning the booze-stained napkin from this particular session at the lounge, I'll post my take on it as text):
[color=blue]The loop() function is executed repeatedly:
BEGIN loop function
IF there is a character available THEN
Read the character and calculate the "pause" value
END IF
Turn on LED
pause
Turn off LED
pause
END loop function
[/color]
Now if that makes sense, compare with your code. If it doesn't make sense, ask again.
Sound sound advice indeed. Still though i am having trouble. Here's what i tried;
#define OPTO 13 // optocoupler pin
int val =0; // variable to store the serial
int pause =0; // variable for our blinks
void setup(){
Serial.begin(300);
pinMode (OPTO, OUTPUT);
}
void loop (){
if (Serial.available() > 0) {
int val = Serial.read()-('0'); // only reads the first byte of data
int pause = map (val,0,9, 15, 100); // hence maximum of 9
pause = constrain (pause , 15, 100);
}
digitalWrite(OPTO, HIGH);
delay(pause);
digitalWrite (OPTO, LOW);
delay(pause);
Serial.println (pause);
}
Ive now uploaded this and at least the serial moniter is making more sense now. However the Led is still not behaving.
#define OPTO 13 // optocoupler pin
int val =0; // variable to store the serial
int pause =0; // variable for our blinks
void setup(){
Serial.begin(9600);
pinMode (OPTO, OUTPUT);
}
void loop (){
while (Serial.available() ==0 );
if (Serial.available() >0 ) {
int val = Serial.read()-('0'); // only reads the first byte of data
pause = map (val,0,9, 15, 100); // hence maximum of 9
pause = constrain (pause , 15, 100);
Serial.flush();
}
digitalWrite(OPTO, HIGH);
delay(pause);
digitalWrite (OPTO, LOW);
delay(pause);
Serial.println (pause);
}
The led just flashes once, do need to make a seperate void and call it with an if?
Right ive got it working by adding a do.. while statement.
#define OPTO 13 // optocoupler pin
int val =0; // variable to store the serial
int pause =0; // variable for our blinks
void setup(){
Serial.begin(9600);
pinMode (OPTO, OUTPUT);
}
void loop (){
while (Serial.available() ==0 );
if (Serial.available() >0 ) {
int val = Serial.read()-('0'); // only reads the first byte of data
pause = map (val,0,9, 15, 100); // hence maximum of 9
pause = constrain (pause , 15, 100);
Serial.flush();
}
do
{
digitalWrite(OPTO, HIGH);
delay(pause);
digitalWrite (OPTO, LOW);
delay(pause);// check the sensors
} while (Serial.available() ==0 );
Serial.println (pause);
}
However i dont think this is what you guys were getting at, so if theres a much more economic way of achieving the same results i would appreciate any insight. There is also some odd behaviour... When i input into the serial monitor, i have to input the number i want twice before the arduino changes the value of "pause". any ideas?
i couldnt get it to work the way described in reply # 1. I was clutching at straws when i added the while loop, and it happened to work. Although it does look very clumsy. Could you explicitly tell me the code involved to make it work as described in reply #1 as i may have made a silly mistake somewhere when i tried my implementation of it?
#define OPTO 13 // optocoupler pin
int pause; // variable for our blinks
void setup()
{
Serial.begin(9600);
pinMode (OPTO, OUTPUT);
}
void loop ()
{
if (Serial.available() >0 ) {
int val = Serial.read(); // only reads the first byte of data
pause = map (val, '0', '9', 15, 100); // hence maximum of 9
pause = constrain (pause , 15, 100); // only needed if you don't get '0' to '9'
}
digitalWrite(OPTO, HIGH);
delay(pause);
digitalWrite (OPTO, LOW);
delay(pause);// check the sensors
Serial.println (pause);
}
RIght that works, and was in fact the second thing i tried, the problem was declaring pause as an int in the map function in the loop. Thanks for the halp!.
if (Serial.available() >0 ) {
int val = Serial.read(); // only reads the first byte of data
pause = map (val, '0', '9', 15, 100); // hence maximum of 9
pause = constrain (pause , 15, 100); // only needed if you don't get '0' to '9'
}
You might want to modify this code to properly deal with invalid input. Suppose the serial port is sent a 'B'. That is not in the range '0' to '9'. As written, the code will cause pause to be assigned either 15 or 100, depending on which invalid character is sent.
I'd prefer to simply ignore any invalid characters. Your mileage may vary.