Progettazione software per cubo led 4x4x4

[Galaxi93]

Nel main chiami 2 volte

Codice: Seleziona tutto

fill_empty_led(1,200);

Non dovrebbe essere una volta con stato=1 e una volta con stato = 0 ?

Yes.
L'ho riscritto di fretta e mi sono dimenticato...

[posta10100]

Adesso funziona o ancora no?

[c1b8]

Ma adesso funziona?

Giuro che non ho mai visto questa sintassi:

Codice: Seleziona tutto

if (x==0, y==0, z==0)

le virgole stanno per && o per || ?

[posta10100]

E' vero... mi era sfuggito!

Ma il compilatore lo accetta?

Mai visto nulla di simile!

[Galaxi93]

Ma adesso funziona?

Giuro che non ho mai visto questa sintassi:

Codice: Seleziona tutto

if (x==0, y==0, z==0)

le virgole stanno per && o per || ?

E' vero... mi era sfuggito!

Ma il compilatore lo accetta?

Mai visto nulla di simile!

Si lo accetta, perché? Pensavo che fosse una cosa normalissima...

comunque ho riprovato ancora, ma non funziona lo stesso...

[posta10100]

Prova a sostituire

Codice: Seleziona tutto

if (x==0, y==0, z==0)

con

Codice: Seleziona tutto

if (x==0 && y==0 && z==0)

[Galaxi93]

Ma in ogni caso, funziona lo stesso. La funzione acc_led è la più sicura, fidati. Ci ho fatto tannte di quelle prove e giochi di luce (sono riuscito a riprodurre l'effetto "pioggia" che si vede nel video).
Tra tutte le funzioni, quella va bene.

[posta10100]

Prova con questo main:

Codice: Seleziona tutto


void main(void)
     {
     porta=0;
     portb=0;
     trisa=0;
     trisb=0;
    for (i = 0; i < 8; i++) {
         if (stato == 0) {
             map[i] = 255;
         } else {
             map[i] = 0;
         }
     }
     while (1)
           {
               cambiaStato(1, 1, 1, 1);
           }
     }
}


Questo codice dovrebbe accendere un singolo led... se funziona allora la cambiaStato è ok...

[c1b8]

Per me non è corretta la random, prova a cambiarla come segue:

Codice: Seleziona tutto

//Genera un numero casuale da 0 a 15 e da 0 a 3.
char random()
     {
     random_16=rand() % 16;         //Crea un numero casuale compreso tra 0 e 16.
     random_4=random_16 % 4;         //Per risparmiare prezioso spazio.
     return(random_16, random_4);  //Ritorna le due variabili.
     }


[Galaxi93]

Prova con questo main:

Codice: Seleziona tutto


void main(void)
{
porta=0;
portb=0;
trisa=0;
trisb=0;
for (i = 0; i < 8; i++) {
if (stato == 0) {
map[i] = 255;
} else {
map[i] = 0;
}
}
while (1)
{
cambiaStato(1, 1, 1, 1);
}
}
}


Questo codice dovrebbe accendere un singolo led... se funziona allora la cambiaStato è ok...

Ora è troppo tardi, domani lo provo.

Per me non è corretta la random, prova a cambiarla come segue:

Codice: Seleziona tutto

//Genera un numero casuale da 0 a 15 e da 0 a 3.
char random()
{
random_16=rand() % 16; //Crea un numero casuale compreso tra 0 e 16.
random_4=random_16 % 4; //Per risparmiare prezioso spazio.
return(random_16, random_4); //Ritorna le due variabili.
}


Ma dovrebbe funzionare anche come era prima, perché dividendo i numeri da 0 a 3 mi da 0, da 4 a 7 da 1, da 8 a 11 da 2 e da 12 a 15 da 3. Mi pare giusto no?