[C] Caricamento lista ordinata da file

[Scara95]

             pNodo->next = *l; //salvo il vecchio nodo testa
            *l = pNodo; //valorizzo il nodo testa con il nuovo nodo inserito
            (*l)->next = pNodo; //collego il nuovo nodo testa con il vecchio

Stai creando un ciclo: pNodo punterà sempre e solo a pNodo e perderai il resto della lista.

            if(strcmp(prodotto.codP, pCorrente->info.codP) < 0){//inserimento non in coda alla lista

Non è vero perché tecnicamente potrebbe essere <= (anche se non nel tuo caso)

Puoi cambiare questo blocco di codice

while (pCorrente->next != NULL && strcmp(prodotto.codP, pCorrente->info.codP) > 0){//scansiono la lista finchè il nodo corrente punta ad un elemento non NULL e finchè il prodotto da inserire non ha un codice prodotto inferiore a quello del nodo corrente
                printf("\nEntrato4\n");
                pPrec = pCorrente;
                pCorrente = pCorrente->next;//assegno a pCorrente il riferimento del nodo successivo
            }
            /* concatenazione del nuovo record */
            if(strcmp(prodotto.codP, pCorrente->info.codP) < 0){//inserimento non in coda alla lista
                printf("\nEntrato5\n");
                pNodo->next = pCorrente;//collegamento tra il nuovo nodo e il suo successivo
                pPrec->next = pNodo;//collegamento tra il nodo precedente e quello nuovo
            }
            else if(pCorrente->next == NULL) {//inserimento in coda alla lista
                printf("\nEntrato6\n");
                pCorrente->next = pNodo;//collegamento della coda della lista al nuovo nodo
            }

con questo molto più semplice

while (pCorrente != NULL && strcmp(prodotto.codP, pCorrente->info.codP) > 0){
                pPrec = pCorrente;
                pCorrente = pCorrente->next;//assegno a pCorrente il riferimento del nodo successivo
            }
            pNodo->next = pCorrente;//collegamento tra il nuovo nodo e il suo successivo
            pPrec->next = pNodo;//collegamento tra il nodo precedente e quello nuovo

[mbistato]

Ciao Scara95 e grazie per aver risposto.

             pNodo->next = *l; //salvo il vecchio nodo testa
            *l = pNodo; //valorizzo il nodo testa con il nuovo nodo inserito
            (*l)->next = pNodo; //collego il nuovo nodo testa con il vecchio

Stai creando un ciclo: pNodo punterà sempre e solo a pNodo e perderai il resto della lista.

Su questo pezzo di codice non mi sono soffermato perchè nel mio caso non viene mai eseguito. Se ho capito correttamente, stai dicendo che con l'istruzione *l = pNodo perderò il riferimento del puntatore testa, il quale invece punterà a pNodo, dico bene?
Ma allora posso correggere scrivendo

pNodo->next = *l;
(*l)->info = prodotto;
(*l)->next = pNodo;

?


Per il resto è tutto chiaro.

[Scara95]

No, sto dicendo che pNodo->next punterà a pNodo e quindi creerai un ciclo infinito.

[mbistato]

Ho sostituito questo codice

pNodo->next = *l; //salvo il vecchio nodo testa
*l = pNodo; //valorizzo il nodo testa con il nuovo nodo inserito
			(*l)->next = pNodo;//collego il nuovo nodo testa con il vecchio

con questo

pNodo->next = *l;
(*l)->info = prodotto;
(*l)->next = pNodo;

Inoltre, apportando le altre modifiche da te suggerite il programma va in crash al secondo ciclo while e pCorrente non viene correttamente inizializzato con il nodo testa tramite l'istruzione pCorrente = *l;