Salve!
L'ideale per gestire una situazione simile sarebbe quello di utilizzare un interrupt alla pressione del pulsante, ma visto l'intento "didattico" volto a familiarizzare con lo switch-case, puoi anche farne a meno.
Come spesso accade, non c'è un'unica soluzione per risolvere il problema.
L'idea è quella di "congelare" in qualche modo l'avvenuta pressione del pulsante, attraverso una variabile (o più variabili).
Ti propongo una soluzione, forse non bellissima. Sicuramente si potrà snellire in qualche modo:
Code:
#include <msp430.h>
#define button P1IN & BIT6
void main(void) {
WDTCTL = WDTPW | WDTHOLD; // Disabilito il chiwawa
P2DIR |= BIT0 + BIT1 + BIT2 + BIT3 + BIT4; // Imposto le uscite
P2OUT = BIT0 + BIT1 + BIT2 + BIT3 + BIT4; // Tutti i LED spenti
unsigned char i = 0; // Questa variabile conta il numero di pressioni del pulsante
unsigned char stato_attuale = 0; // E' lo stato (pressione) attuale del pulsante
unsigned char stato_precedente = 0; // E' lo stato (pressione) precedente del pulsante
int delay = 0; // Variabile da utilizzare per il ritardo
while(1)
{
stato_attuale = button; // Memorizzo lo stato attuale del pulsante
for (delay = 0; delay < 10000; delay++); // Elimina questo ritardo se è già presente un anti-rimbalzo hw
// Qui entra in gioco il meccanismo relativo al controllo della pressione del pulsante:
if(!stato_attuale && stato_precedente == 0)
{
i = i+1; // Incremento i ad ogni pressione
}
stato_precedente = !stato_attuale; // Aggiorno lo stato del pulsante
switch (i)
{
case 1: // se i = a 1 accendi P2.0
P2OUT = (!BIT0 + BIT1 + BIT2 + BIT3 + BIT4);
break;
case 2: // se i = a 2 accendi P2.1
P2OUT = (BIT0 + !BIT1 + BIT2 + BIT3 + BIT4);
break;
case 3: // se i = a 3 accendi P2.2
P2OUT = (BIT0 + BIT1 + !BIT2 + BIT3 + BIT4);
break;
case 4: // se i = a 0 spegni tutto
P2OUT = (BIT0 + BIT1 + BIT2 + BIT3 + BIT4);
i=0; // Azzero la variabile i per ripetere la sequenza dello switch-case
break;
}
}
}
In pratica il meccanismo si basa sulla memorizzazione della pressione del pulsante attraverso la variabile stato_precedente.
Quando premi il pulsante, il codice da te scritto veniva letto diverse volte in una frazione di secondo, andando a "spazzolarsi" tutti i case dello switch. In questo caso, invece, blocchi l'esecuzione dei vari case perché blocchi l'incremento della variabile i. In che modo? Attraverso il valore logico alto della variabile stato_precedente, la quale non ti fa più accedere all'if contenente la variabile i. Quando rilasci il pulsante, la variabile stato_precedente torna al valore logico basso.
Nel case 4 puoi notare l'azzeramento della variabile i, in modo tale da ricominciare dal case 1 alla nuova pressione del pulsante.
Non ho modificato la parte relativa alle uscite perché non ho ben capito il modo in cui sono collegati i LED. In base a quello che hai scritto, al pin del microcontrollore dovrebbe essere connesso il catodo.
Per quanto riguarda il pulsante, assumo quanto chiarito nell'altro topic, ossia la presenza dei resistori di pull-up. In caso contrario, devi togliere il not logico all'if.
Saluti,
Cosimo
Tienici aggiornati.