[C] Ciclo di scansione lista

[{-_-}]

stamani stavo studiando le code su un libro, abbastanza importante e famoso, però a un certo punto ho notato una cosa strana.
allora chiedo a voi se sono scemo io, cosa probabile o è un palese errore del testo.

nell'inserimento di un nuovo elemento in una coda, il libro mi dice giustamente che è necessario scandire la lista e dopo effettuare l'inserimento in coda.

mi scrive questo codice per la scansione:

puntTmp = puntatoreTesta;
if (puntTmp != NULL){ //se la lista ha degli elementi
while (puntTmp->puntSucc!=NULL) do 
puntTmp = puntTmp->puntSucc;

...poi passa alla creazione del nuovo elemento
}
else
{
crea il primo elemento
}

la riga in rosso non è che mi convica molto, non dovrebbe essere qualcosa del tipo :

do 
puntTmp = puntTmp->puntSucc;
while (puntTmp->puntSucc!=NULL); 

?


Grazie

[MacApp]

in rosso si controlla che al primo passaggio puntTmp->puntSucc!=NULL;
in blu NON si controlla che al primo passaggio puntTmp->puntSucc!=NULL;

[Ifrit]

cosa ci fa il "do" dopo gli argomenti del while?

[{-_-}]

Originariamente inviato da Ifrit
cosa ci fa il "do" dopo gli argomenti del while?

è esattamente questo il problema, il libro me lo scirve così, ho provato a correggerlo (in blu), ma non sono sicuro di averlo corretto bene, per questo l'ho postato.

mi sembra strano che un libro così famoso alla terza edizione abbia fatto un errore del genere

[{-_-}]

Originariamente inviato da MacApp
in rosso si controlla che al primo passaggio puntTmp->puntSucc!=NULL;
in blu NON si controlla che al primo passaggio puntTmp->puntSucc!=NULL;

il controllo del primo passaggio mi sembra lo faccia l'if , ma il mio problema è quel do dopo il while...il libro me lo scirve così

[MacApp]

Originariamente inviato da {-_-}
ma il mio problema è quel do dopo il while...il libro me lo scirve così

Non è detto che sia errato, per saperlo bisognerebbe vedere cosa c'è scritto dopo il rosso.
Ad esempio il seguente è corretto:

//
// Compiled with:
// i686-apple-darwin9-gcc-4.0.1 (GCC) 4.0.1 (Apple Inc. build 5465)
//
#include <stdio.h>

int main (){
    while (1) do 
    printf ("ciao;\n");
    while (1);
    return 0;
} 

[{-_-}]

questo è il codice completo. anch'io ho subito pensato ci deve essere un while dopo , ma invece niente di niente

puntTmp = puntatoreTesta;
if (puntTmp != NULL){ //se la lista ha degli elementi
while (puntTmp->puntSucc!=NULL) do 
puntTmp = puntTmp->puntSucc;
//poi passa alla creazione del nuovo elemento
puntTmp->puntSucc= (struct elem *) malloc (sizeof(struct elem));
puntTmp= puntTmp->puntSucc;
puntTmp->info = ele;
puntTmp->puntSucc = NULL;
}
else
{
puntatoreTesta = (struct elem *) malloc (sizeof(struct elem));
puntatoreTesta->info = ele;
puntTmp->puntSucc = NULL;
}

[Ifrit]

ascolta qui c'è qualkosa di troppo strano, fai una cosa, postaci tutto il programma e soprattutto le strutture,
da come sembra quel while oltre a non servire a nulla e'anche scritto in modo da non far compilare il programma....

[{-_-}]

quello che ho postato è l'unica parte di codice scritta sul libro.

cmq alla fine ho risolto, prima pensavo di aggiustare quel codice sistemando il do-while sintatitcamente errato, poi ho visto che non andava bene, allora ho messo direttamente un while e tutto funziona alla grande

puntTmp = puntatoreTesta;
if (puntTmp != NULL){                           //se la lista ha degli elementi
      while (puntTmp->puntSucc!=NULL)   //è sufficiente eliminare questo do, ovviamente errato
      puntTmp = puntTmp->puntSucc;
                                                     //poi passa alla creazione del nuovo elemento
      puntTmp->puntSucc= (struct elem *) malloc (sizeof(struct elem));
      puntTmp= puntTmp->puntSucc;
      puntTmp->info = ele;
      puntTmp->puntSucc = NULL;
    }
else
{
      puntatoreTesta = (struct elem *) malloc (sizeof(struct elem));
      puntatoreTesta->info = ele;
      puntTmp->puntSucc = NULL;
}

ora credo che passerò all'ordinamento di una lista, dopo un meritato riposo

grazie a tutti voi