Progettazione software per cubo led 4x4x4

[c1b8]

mmm per approfittare del discorso di c1b8 {beh +/-}

Ciao,
mi piacerebbe sapere anche la tua di opinione, perché {beh +/-}?
In cosa la pensi diversamente?

[phylum]

il +/- era inteso in un altro senso : Extreme Programming o meglio la regola PP e' intesa (almeno che io ricordi)
mettiamo di star scrivendo un programma
io scrivo tu leggi quello che scrivo e dai il tuo contributo
poi ci scambiamo di posto



il +/- era perche' nn stavo interpretando propriamente quel ruolo, ma una fattispecie (molto far )
ho scritto un codice che (se nn ho commesso errori di distrazione e/o fraintendimenti) riproduce quello che Galaxy voleva fare col suo usando una logica diversa,
in sostanza appoggiavo sia il fatto che hai ricordato questa tecnica sia cercare di dimostrare di quanto possa essere benefica

[posta10100]

ho dato un occhiata veloce al tuo codice, e ho visto subito una cosa che mi crea un dubbio: ma la funzione getstato, non mi ritorna 0 se è spento e un numero diverso da 0 se è acceso?

Giusto... la mia memoria comincia a perdere colpi...

Codice: Seleziona tutto

        //Funzione che date le coordinate x, y, z, ne cambia lo stato solo se è diverso
        //dallo stato dato alla funzione e se lo cambia, restituisce 1 altrimenti 0.

        char cambia_if(char x, char y, char z, char stato)
             {
             char stato_led =getStato(x,y,z);
             if (stato_led) {
                 stato_led = 1;
             }
             char cambiato = 0;
             if (stato != statoLed) { // se lo stato è diverso
                 cambiaStato(x, y, z, stato); //lo cambio
                 return 1; // e restituisco 1
             }
             return 0; // altrimenti restituisco 0
        }



Così è a posto!

Per il resto ho capito tutto, tranne quel map[i] = !stato*255; che cosa fa quello che ho scritto io?

A guardarlo bene dovrebbe funzionare (avevo interpretato la moltiplicazione come uno shift verso sinistra,ma in questo caso non è quella la funzione) solo che probabilmente è più lento in esecuzione dovendo fare la moltiplicazione....

L'hai provato? Funziona?

[posta10100]

Come mi ha fatto gentilmente notare Stella,

Codice: Seleziona tutto

if (stato != statoLed) { // se lo stato è diverso

deve diventare

Codice: Seleziona tutto

if (stato != stato_led) { // se lo stato è diverso

l'abitudine fa brutti scherzi... in genere i nomi li lascio attaccati usando una maiuscola...

[Galaxi93]

Come mi ha fatto gentilmente notare Stella,

Codice: Seleziona tutto

if (stato != statoLed) { // se lo stato è diverso

deve diventare

Codice: Seleziona tutto

if (stato != stato_led) { // se lo stato è diverso

l'abitudine fa brutti scherzi... in genere i nomi li lascio attaccati usando una maiuscola...

Non ti preoccupare, il compilatore me lo a fatto notare (con una decina di errori ).


Provato!!...non funziona....
Proprio il nulla, meno di zero. Ci deve essere qualcosa di "indefinito" perché se provo ad accendere e spegnere più volte il circuito, mi capitano dei led accesi casualmente...

Dopo gli do una bella occhiata, e cerco l'errore, se lo trovo... (cavolo mi è venuto un mal di testa incredibile, sarà perché sono sul computer da tutto il giorno?? ).

[c1b8]

Una cosa che non ho capito, alla fine hai utilizzato la

Codice: Seleziona tutto

map[i] = !stato*255;

o il codice inserito da posta10100?

Quella istruzione (map[i] = !stato*255;) non può funzionare!
1) se stato=0 quanto fa !stato*255? fa 255*255 che è ok.
2) ma se stato=1 allora vale 254*255 che fa 2. Quindi non ok

Inoltre la random immagino imposti la random_4, ma con che valori?
da 0 a 3 o da 1 a 4? il secondo range non va bene, deve essere da 0 a 3.

[Galaxi93]

Si, ho utilizzato il codice di posta10100 e la random l'ho cambiata che va da 0 a 3.
A proposito della random, come mai occuma moltissimo spazio (quasi un terzo della memoria dal 16f84)???

Ah poi adesso che sto rileggendo bene il codice, nella funzione cambia_if, la variabile "cambiato" non serve più, dato che è stata sostituita da due return.

[posta10100]

Ah poi adesso che sto rileggendo bene il codice, nella funzione cambia_if, la variabile "cambiato" non serve più, dato che è stata sostituita da due return.

Il codice è già abbastanza complesso di suo quindi tutto ciò che non serve va cancellato...
Potresti postare l'intero codice?

[Galaxi93]

Ho trovato un errore:

Codice: Seleziona tutto

for (i = 0; i < 8; i++) {
             if (stato == 0) {
                 map[i] = 0;
             } else {
                 map[i] = 255;
             }

é stato invertito lo 0 e il 255 poiché se stato è uguale a 0, deve accendere tutto il cubo, poiché, poi, uno alla volta, spegne tutti i led, e viceversa con stato=1. Quindi diventa:

Codice: Seleziona tutto

for (i = 0; i < 8; i++) {
             if (stato == 0) {
                 map[i] = 255;
             } else {
                 map[i] = 0;
             }

In ogni caso la situazione non cambia, siamo ritornati come prima (ovvero il codice che avevo scritto io). I led si comportano in maniera anomala senza nessuna apparente logica.

Il codice è già abbastanza complesso di suo quindi tutto ciò che non serve va cancellato...
Potresti postare l'intero codice?

Tutto, tutto??

[phylum]

beh se ti viene complicato farlo qui puoi usare pastebin.com