Passaggio array da funzione

International Italiano Software
1

ciao a tutti, stavo cercando di venirmene fuori con un codice un po' piu' elegante nel senso, in questo caso ho solo 6 volori nell'array ma se ho 200 valori? come ho scritto il codice non ha senso..... idee su come si possa fare?
grazie

int hour,minute,second=0;
int a,b,c,d,e,f=0;
void setup() {
  Serial.begin(9600);    
  divArray(&hour,&minute,&second,&a,&b,&c,&d,&e,&f);
  Serial.print(a);
  Serial.println(b);
  Serial.print(c);  
  Serial.println(d);
  Serial.print(e);
  Serial.println(f);    
}
void loop() {
}
void divArray(int *hour,int *minute,int *second,int *a,int *b, int *c,int *d,int *e, int *f){
 *hour=12;
 *minute=37;
 *second=54;
  int Zarray[6]={0,0,0,0,0,0};
      Zarray[5] = *hour/10;
      Zarray[4] = *hour%10;
      Zarray[3] = *minute/10;
      Zarray[2] = *minute%10;
      Zarray[1] = *second/10;
      Zarray[0] = *second%10;   
      *a=Zarray[5];
      *b=Zarray[4];
      *c=Zarray[3];
      *d=Zarray[2];
      *e=Zarray[1];
      *f=Zarray[0];      
}
2

cosa stai cercando di fare esattamente?

3

un orologio a nixie.... pero' in questo momento stavo divagando e cercando altre soluzioni.

4

ciao superciccio,

diciamo che non è importante il progetto fisico in se ma cosa vuoi fare con il codice...quindi poco importa che sia un orologio nixie...quello che interessa è che tipo di "manipolazione" vuoi fare con gli array!?...dal tuo codice di esempio si capisce poco, almeno per me, di quello che vuoi fare...sembrerebbe che avessi delle variabili globali singole che vorresti passare ad altre variabili globali singole tramite una funzione...per me ha poco senso sta cosa.

5

si, ha poco senso perche' e' la modifica di un pezzo di codice, preso cosi' serve a poco.
vorrei portare i valori dell'array dalla funzione.

6

Cioè restituire l'intero array ?

7

si esatto

8

Gli array sono passati per indirizzo

Non serve fare nulla

9

intendi cosi?:

int hour,minute,second,aaa=0;

void setup() {
  Serial.begin(9600);    
  divArray(&hour,&minute,&second,&aaa);
  Serial.print(aaa);
  
}
void loop() {
}
void divArray(int *hour,int *minute,int *second,int *aaa){
 *hour=12;
 *minute=37;
 *second=54;
  int aaa[6]={0,0,0,0,0,0};
      aaa[5] = *hour/10;
      aaa[4] = *hour%10;
      aaa[3] = *minute/10;
      aaa[2] = *minute%10;
      aaa[1] = *second/10;
      aaa[0] = *second%10;        
}
10

No, intendo che lo passi come argomento

Nella funzione lo modifichi

E nel chiamante te lo ritrovi modificato

Ma queste sono le basi del C...

11

Di sicuro così non va bene, minimo devi dichiarare l'array globale, es:

int hour;
int minute;
int second; 
// sta brutta abitudine di mettere tutto in una riga, senza leggere che nei libri lo si fa per esigenze tipografiche. Quindi una dichiarazione per riga. 
#define AAA_LEN 7
int aaa[AAA_LEN];

Dentro la funzione questa riga però:

int aaa[6]={0,0,0,0,0,0};

dichiara e inizializza un array di int di nome aaa grande 6 elementi indicizzati da 0÷5, array che è locale allocato nello stack che viene ripulito quando la funzione termina.

Non ti so suggerire del codice, c'è troppa confusione da parte tua. Non capisco perché usare puntatori hour, minute e second, per di più sono variabili globali visibile da dentro la funzione.

Ciao.

12

continuo a non capire cosa vuoi fare...cosa ti serve.

13

grazie ci studiero' sopra

14

Su gli array e puntatori ho scritto questi articoli:

PS: ricorda che come contenitori di dati oltre agli array il C/C++ ci mette a disposizione la parola chiave struct.

Ciao.

15

Inoltre NON ha senso dichiarare TUTTI i parametri puntatore.
I primi tre della funzione non hai necessità di modificarli dentro alla funzione, quindi quelli passali normalmente, senza puntatore.

P.S. specificate anche sempre su che Arduino state lavorando.

16

grazie a tutti per le preziose informazioni, penso di aver risolto, adesso funziona pero' il compilatore mi da un errore.

int hour=23;
int minute=11;
int second=54;
int myArr[5];
void setup() {
  Serial.begin(9600);    
  divArray(&hour,&minute,&second,myArr);
  Serial.print(myArr[5]);
  Serial.println(myArr[4]);
  Serial.print(myArr[3]);
  Serial.println(myArr[2]);
  Serial.print(myArr[1]);
  Serial.println(myArr[0]);    
}
void loop() {
}
int divArray(int *hour,int *minute,int *second,int *myArr){

   myArr[5];
      myArr[5] = *hour/10;
      myArr[4] = *hour%10;
      myArr[3] = *minute/10;
      myArr[2] = *minute%10;
      myArr[1] = *second/10;
      myArr[0] = *second%10;    
      return myArr;    
}

questo e' l'errore;

C:\Users\ciccio\Documents\Arduino\sketch_dec27a\sketch_dec27a.ino: In function 'int divArray(int*, int*, int*, int*)':
C:\Users\ciccio\Documents\Arduino\sketch_dec27a\sketch_dec27a.ino:26:14: warning: invalid conversion from 'int*' to 'int' [-fpermissive]
       return myArr;
              ^~~~~
17

Se non compila... non funziona !

Usa i puntatori solo dove serve. Poi sarebbe meglio usare byte, int è esagerato per numeri piccoli

...
  divArray(hour, minute, second, myArr);
...
void divArray(int hh, int mm, int ss, int* divArr) {
  divArr[5] = hh/10;
  divArr[4] = hh%10;
  divArr[3] = mm/10;
  divArr[2] = mm%10;  
  divArr[1] = ss/10;
  divArr[0] = ss%10;    
 }
18

Nel prototipo della funzione hai specificato che questa restituisce al chiamante un valore di tipo int, però l'ultima riga della funzione restituisci il puntatore myArr:

return myArr; // questo è un puntatore passato come argomento

Ora con i puntatori in questo specifico caso non ha senso restituirli al chiamante, ci basta specificare il puntatore nella lista degli argomenti.

Tagliando corto, elimina la riga return myArr; e non riceverai errori (mi pare) ma un warning che ti avvisa che la funzione non ha un return, ma il prototipo lo prevede.

Risolvi specificando che la funzione restituisce nulla al chiamante, cioè al posto di int ci scrivi void.

void divArray( int *hour, int *minute, int *second, int *myArr ){

PS: nota gli spazi dopo la virgola e dopo e prima delle parentesi tonde. Al compilatore non serve ciò ma a chi legge il codice si eccome se serve.

A me sembra che si voglia creare una funzione di inizializzazione a run-time, in gerene queste funzioni hanno init nel nome, es:
void initDateTime(int *hour, ... ecc.

Ciao.

19

grazie a tutti, adesso funziona! il compilatore continua a darmi un'errore pero' lo caompila e il risultato e' quello giusto.
un saluto, alle prossime.
A.

20

Se leggi bene, il compilatore non ha sollevato un errore, ma un warning:

warning: invalid conversion from 'int*' to 'int' [-fpermissive]

Ti avvisa che c'è qualcosa che non torna secondo lui.
Resta però il fatto che è un errore di programmazione.

Non sottovalutare i warning del compilatore, quasi certamente dietro un warning c'è un errore di programmazione che si presenta a run-time, cioè il risultato non è quello che ti aspetti.

Domanda: ma con l'array poi che ci devi fare?
No perché se lo devi scansionare in un ciclo for e allora va bene, diversamente ci sono le struct come contenitori di dati eterogenei, cioè puoi mischiare int, byte float ecc es:

struct myDateTime {
    byte hour; // per le ore basta il byte
    byte minutes; // per i minuti basta il byte
    byte seconds; // come sopra
    uint16_t year; // per l'anno ci voglio 2 byte
}

Ciao.