4 buttons - combination

Projects General Guidance
1

I already searched the whole forum but do not find the same problem:

I have 4 buttons: they work each separate, but there should be a combination:

for example:
But 1: LED 1 = on
But 2: LED 2 = on
But 3: LED 3 = on
But 4: LED 4 = on

But 1 and But 2 tighter: set the brightness off the leds... BUT don't turn LED 1 an LED 2 on..

this is what I have so far (without the combination):

//Button 1 is pressed
if ((but1read == HIGH) && ((millis() - but1LastPress) > buttonPressDelay)) {
but1LastPress = millis();
doButton1();
}else if ((but1read == LOW) && ((millis() - but1LastPress) > buttonPressDelay ))
but1LastPress = 0; // reset

//Button 2 is pressed
if ((but2read == HIGH) && ((millis() - but2LastPress) > buttonPressDelay)) {
but2LastPress = millis();
doButton2();
}
else if ((but2read == LOW) && ((millis() - but2LastPress) > buttonPressDelay ))
but2LastPress = 0; // reset

//Button 3 is pressed
if ((but3read == HIGH) && ((millis() - but3LastPress) > buttonPressDelay)) {
but3LastPress = millis();
doButton3();
}
else if ((but3read == LOW) && ((millis() - but3LastPress) > buttonPressDelay ))
but3LastPress = 0; // reset

//Button 4 is pressed
if ((but4read == HIGH) && ((millis() - but4LastPress) > buttonPressDelay)) {
but4LastPress = millis();
doButton4();
}
else if ((but4read == LOW) && ((millis() - but4LastPress) > buttonPressDelay ))
but4LastPress = 0; // reset

2

this is what I have so far

No it is not.
You have other lines in your code.
Please post ALL your code and use the code tags to do it. That is the icon that looks like a scroll with < > symbols over it.

3

You need to think about this more carefully.

If button 1 is pressed, how do you know if the user intends to turn on LED1 OR he's about to also press button 2 (to adjust something)

It's impossible to press two buttons at EXACTLY the same moment.

I'd suggest if you need to look at the logic of when the buttons are released instead. But you are creating an interface that is over complicated. There are better ways of doing this.

4

KenF:
There are better ways of doing this.

any suggestions?

5

Start off by stating exactly what you want to do.
At the moment it just sounds like a programming exercise for an assignment.

I would read all the buttons into four bits of an int variable and then use a switch statement to define what to do with each of the 16 combinations.

This should show you how to do this:-
http://www.thebox.myzen.co.uk/Tutorial/Arrays.html