View Single Post
Old 05-09-2005, 11:57   #4
NA01
Senior Member
 
L'Avatar di NA01
 
Iscritto dal: Jun 2003
Città: Genova
Messaggi: 5676
Codice:
 maux=mauxsucc;
???
che è maxsucc?

Codice:
 maux=aux->m_next;
c'è qualche casino con gli * e &. su maux uno è di troppo, e è anche sbaglitao il nome della struttura.
la dichiari come misura e la usi come misure.
perchè poi passi l'indirizzo del puntatore alla testa invece che il puntatore?
potrebbe servirti per modificare la testa, ma in una funzione che deve solo leggere non ha senso e gli dà "troppi privilegi"

poi hai due variabili non utilizzate, ma suppongo che ti servano una volta risolto il problema.

vediamo se ho capito che deve fare la funzione:
hai la tua lista di sensori, cerchi il tuo sensore e poi vuoi stampare tutte le misure che sono state memorizzate nella lista di quello specifico sensore.
giusto?

se si allora il primo errore che vedo è
Codice:
  maux= aux->m_next;
lì aux è la testa della tua lista (se risolvi il problema degli *), non è il sensore che vuoi....
maux lo dovresti sistemare appena hai trovato il tuo sensore, non all'inizio

poi c'è quel mauxsucc.
suppongo dovrebbe essere una cosa del tipo
maux = maux->succ;

poi sula fine del ciclio interno metti un passaggio che scorre la lista sul prossimo elemento.
quindi c'è un else e scorri l'elemento...
perchè non lo metti semplicemente fuori dall'if?
intanto sia che vada bene sia che vada male il controllo di strcmp quello lo fai dopoo il blocco dell'if, quindi puoi semplicemente lasciarlo fuori senza alcuna condizione.

ciao!
NA01 è offline   Rispondi citando il messaggio o parte di esso