pic con memoria insufficiente

[lol21]

ok grazie dell'aiuto vi farò sapere
ma le variabili comunque non occupano RAM? io ho problemi di ROM

ciao
lol21

[frank]

Originalmente inviato da lol21

ok grazie dell'aiuto vi farò sapere
ma le variabili comunque non occupano RAM? io ho problemi di ROM

ciao
lol21

Se sono array statici non è detto, il compilatore potrebbe allocarle in ROM. Ma dovrebbe essere documentato nel manuale.

[lol21]

oooooooooooooooooooooooooooooooooooooook, vaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa aaaaaa bene

ora provo con il metodo degli array (si sono duro di testa ) poi magari riprovo coi cicli cercando anche altri esempi online perchè non ho mai fatto come hai detto tu

grazie a tutti intanto, vi farò sapere se funziona

ciao
lol21

[illez]

Ho guardato un po' il codice e devo dire che con un ciclo FOR non risolveresti: hai delle eccezioni da gestire che riguardano i casi 'somma', voglio dire i valori 3,5,6,9 che non sono diretta potenza di 2.
Probabilmente hai già una buona riduzione del codice raccogliendo tutti i casi in una funzione che richiamerai da tutto il codice.
ad esempio

unsigned int function check_porta()
{
switch(PORTA) {
case 1: return 1;
case 2: return 2;
case 3: return 9;
etc etc
}
}

Nel codice scriveresti, per esempio:
arciereb[freb] = check_porta();

Attenzione che comunque hai un paio di eccezioni, come qui:
case 6:
arciereb[freb] = 0;
freb = freb-1;
break;
in questo caso devi comunque controllare porta==6 e decrementare freb

Non conoscendo bene il funzionamento, non credo che si possa fare sempre dentro la funzione. Ma insomma, vedi tu...

[lol21]

per il case 6 infatti mettevo un if
il fatto che non siano potenze di 2 in realtà è perchè per alquni tasti uso un solo bit acceso per altri anche 2
ad esempio
case 1 (0b00000001)
case 2 (0b00000010)
case 4 (0b00000100)
...
case 3 (0b00000011) da quì non sono più potenze di 2

uso questo modo per non usare pin eccessivi.

grazie dell'aiuto, appena ho un po di tempo butto giù il codice completo

ciao
lol21

[illez]

Originalmente inviato da lol21

per il case 6 infatti mettevo un if
il fatto che non siano potenze di 2 in realtà è perchè per alquni tasti uso un solo bit acceso per altri anche 2
ad esempio
case 1 (0b00000001)
case 2 (0b00000010)
case 4 (0b00000100)
...
case 3 (0b00000011) da quì non sono più potenze di 2

uso questo modo per non usare pin eccessivi.

grazie dell'aiuto, appena ho un po di tempo butto giù il codice completo

ciao
lol21

I tasti quanti sono? Li hai messi in matrice?

[illez]

Originalmente inviato da illez

I tasti quanti sono? Li hai messi in matrice?

Non te lo avevo già suggerito?

Cerco qualcosa che ti possa aiutare, ma non è difficile!

[frank]

Originalmente inviato da lol21

magari riprovo coi cicli cercando anche altri esempi online perchè non ho mai fatto come hai detto tu

Guarda qui:

// MAXPAGLIONI e' il numero massimo di paglioni
int mask, i;
if (PORTA) 
        for (mask=1, i=1; mask < (1 << MAXPAGLIONI); mask = mask << 1, i++) 
	        if (PORTA & mask) break;
// al termine del ciclo, i contiene il numero del paglione colpito

Questo funziona se, come nel tuo codice, può essere colpito un solo paglione per volta. Se è possibile che vengano colpiti più bersagli insieme, bisogna modificare le cose.

[frank]

Anzi, meglio ancora e più efficiente:

// MAXPAGLIONI e' il numero massimo di paglioni
int mask, i;
if (PORTA) 
        for (mask=1, i=1; i < MAXPAGLIONI+1; mask = mask << 1, i++) 
	        if (PORTA & mask) break;
// al termine del ciclo, i contiene il numero del paglione colpito

[illez]

Originalmente inviato da frank

Anzi, meglio ancora e più efficiente:

// MAXPAGLIONI e' il numero massimo di paglioni
int mask, i;
if (PORTA) 
        for (mask=1, i=1; i < MAXPAGLIONI+1; mask = mask << 1, i++) 
            if (PORTA & mask) break;
// al termine del ciclo, i contiene il numero del paglione colpito

Ci sono le eccezioni in alcuni casi (vedi 3, 5 etc)
i < MAXPAGLIONI+1; è inefficiente. Fai una somma ad ogni ciclo.
piuttosto:
i <= MAXPAGLIONI;