http://forums.leaflabs.com/topic.php?id=851 A place to share, learn, and grow... en-US Fri, 22 Jan 2016 00:18:56 +0000 http://bbpress.org/?v=1.0.2 q http://forums.leaflabs.com/search.php http://forums.leaflabs.com/topic.php?id=851#post-5137 Sat, 11 Jun 2011 08:31:23 +0000 robodude666 5137@http://forums.leaflabs.com/ <p><code>isButtonPressed()</code> is simply poorly named and its functionality is therefore misleading. If you were to use Objective-C's verbose naming scheme it should be called <code>wasButtonPressedAndThenReleased()</code>. In other words:</p> <pre><code>uint8 isButtonPressed() { if (digitalRead(BOARD_BUTTON_PIN)) { delay(BUTTON_DEBOUNCE_DELAY); while (digitalRead(BOARD_BUTTON_PIN)) ; return true; } return false; }</code></pre> <p>It checks to see if the button was pressed. It then waits a debounce period (1ms), and waits for the button to be released. You may want to try this function for your application:</p> <pre><code>bool isButtonDown() { if (digitalRead(BOARD_BUTTON_PIN)) { delay(1); // 1ms debounce perioud return digitalRead(BOARD_BUTTON_PIN) ? true : false; } return false; }</code></pre> <p>This <code>isButtonDown()</code> function simply checks to see if the button is down after a debounce period. With this new function, your code should work as expected:</p> <pre><code>void loop() { if(isButtonDown()) // Is the Button down? { toggleLED(); // Toggle the LED while(isButtonDown()); // Wait for button to be released } }</code></pre> <p>The last <code>while(isButtonDown());</code> is required or else <code>isButtonDown()</code> will continuously return <code>true</code> if the button remains down. The while statement, as you previously expected it to, waits for the button to be released. In addition, because of this new "release" check no debounce delay is required in your application itself.</p> <p>-robodude666 </p> http://forums.leaflabs.com/topic.php?id=851#post-5133 Sat, 11 Jun 2011 00:30:56 +0000 Darth Maker 5133@http://forums.leaflabs.com/ <p>The button example program does not work properly. While you hold down the button, the LED oscillates off and on rapidly. I suppose it is possible that that is proper operation, it would make more sense to have the switch to the opposite state when the button is pressed. </p> <p>This is true in release 0.0.11, using a brand spanking new Maple Mini.</p> <p>This should be fixed with the simple addition of a while(isButtonPressed()); statement inside the if block. However, this does not seem to help, even with a 1ms delay added for debounce.</p> <p>Ok, I added a 10ms delay, and it seems to work properly now, except that the toggle is happening when I release the button, instead on press like it should.</p> <p><code>void setup() {<br /> // Initialize the built-in LED pin as an output:<br /> pinMode(BOARD_LED_PIN, OUTPUT);<br /> // Initialize the built-in button (labeled BUT) as an input:<br /> pinMode(BOARD_BUTTON_PIN, INPUT);<br /> }</p> <p>void loop() {<br /> // Check if the button is pressed.<br /> if (isButtonPressed())<br /> {<br /> // If so, turn the LED from on to off, or from off to on:<br /> toggleLED();</p> <p> while(isButtonPressed());<br /> delay(10);<br /> }<br /> }</code> </p>