Problemi ordinamento. [ c ]

[tridi9]

do{
	 non_scambio =  true;
for (i = 0; i < k; i++)
	{
	 if (articoli_letti[i].prezzo > articoli_letti[i+1]prezzo)
		{
		 non_scambio = false;
		 temp= articoli_letti[i];
		 articoli_letti[i] = articoli_letti[i+1];
		 articoli_letti[i]=temp;
		}
	 }

}while (! non_scambio );

Ciao ragazzi.Sto impazzendo a fare sto esercizio.
Non capisco, ma perchè mi cicla all'infinito ?
Devo ordinare il mio array che è composto da una

struct libro{
int cod;
float prezzo;
char titolo[20];
char autore[20];
};


Uso lo stesso codice per ordinare ma in usando il campo cod.
Uno è un int e funziona l'altro è un float e non funziona.
Ho provato a fare un cambio di tipo ma niente.:master:
Avete qualche idea ?

[seafarer]

prova
do{
non_scambio = true;
for (i = 0; i < k; i++)
{
if (articoli_letti[i].prezzo > articoli_letti[i+1].prezzo)
{
non_scambio = false;
temp= articoli_letti[i];
articoli_letti[i] = articoli_letti[i+1];
articoli_letti[i+1]=temp;
}
}

}
while (! non_scambio );

[tridi9]

Grazie ma nulla da fare.

[seafarer]

fai così, prendi un foglio di carta, simula il programma e vedi xkè looppa..

[r0x]

for (i = 0; i < k; i++)

Posto che k sia la lunghezza dell`array, devi arrivare a k-2, non a k-1, altrimenti con i+1 oltrepassi i limiti, prendendo in considerazione dei valori misteriosi. Quindi:

for (i = 0; i < k - 1; i++)

Tentar non nuoce. :bubu:

[tridi9]

Caspita ero quasi convinto che andasse.
Mi sembra davvero inspiegabile.
Dal debug mi sembra che è la condizione del while che non viene soddisfatta, il che mi lascai davvero davvero sbogottito :master:

[Navimel]

do{
non_scambio = true;
for (i = 0; i < k; i++)
{
if (articoli_letti[i].prezzo > articoli_letti[i+1]prezzo)
{
non_scambio = false;
temp= articoli_letti[i];
articoli_letti[i] = articoli_letti[i+1];
articoli_letti[i]=temp;
}
}

}while (!false );

prova così

[tridi9]

OK ora ci provo

[r0x]

}while (!false );

[Navimel]

correggo:

do{ 
non_scambio = true; 
for (i = 0; i < k; i++) 
{ 
if (articoli_letti[i].prezzo > articoli_letti[i+1]prezzo) 
{ 
non_scambio = false; 
temp= articoli_letti[i]; 
articoli_letti[i] = articoli_letti[i+1]; 
articoli_letti[i]=temp; 
} 
} 

}while (non_scambio != false )

r0x la prossima volta invece di piangere, correggi, mi avresti risparmiato un post......