[C] Calcolo del numero maggiore con ciclo

[Zorgatone]

Salve a tutti ho scritto il seguente codice

int main(void)
{
	float n, maggiore;

	printf("\n************************************************\n");
	printf("* Programma per il calcolo del numero maggiore *\n");
	printf("*                 BY ZORGATONE                 *\n");
	printf("************************************************\n\n");

	maggiore = 0.00f;

	printf("Immettere i numeri da confrontare (0 per smettere): ");
	while (n != 0)
	{
		scanf("%f", &n);
		if (maggiore < n) {maggiore = n;};
	};
	printf("Il numero maggiore è %.2f.\n", &maggiore);
}

Ma quando compilo mi da un errore (credo trascurabile) e poi eseguendo il programma non mi restituisce il valore maggiore ma sempre e soltanto 0.

zorgatone@zorgatone-laptop:~/Programmazione In C/Esercizi$ gcc -o sez6prog1_comp.out sez6prog1.c
sez6prog1.c: In function ‘main’:
sez6prog1.c:18: warning: format ‘%.2f’ expects type ‘double’, but argument 2 has type ‘float *’
zorgatone@zorgatone-laptop:~/Programmazione In C/Esercizi$ ./sez6prog1_comp.out
************************************************
* Programma per il calcolo del numero maggiore *
*                 BY ZORGATONE                 *
************************************************

Immettere i numeri da confrontare (0 per smettere): 1 2 3 4 5 2 3 0
Il numero maggiore è 0.00.
zorgatone@zorgatone-laptop:~/Programmazione In C/Esercizi$

[MacApp]

• utilizzi il contenuto di "n" senza che tu gli abbia prima dato un valore: comportamento indefinito;
• il warning che ti da il compilatore, non è trascurabile: te lo dice chiaramente cosa non va;
• il ";" dopo la graffa chiusa del while, potrà crearti solo problemi;
• main deve restituire un intero.

[Zorgatone]

Originariamente inviato da MacApp

• main deve restituire un intero.

Ok ho aggiunto un return 0;

Originariamente inviato da MacApp

• utilizzi il contenuto di "n" senza che tu gli abbia prima dato un valore: comportamento indefinito;

• Ma il valore di n non è senza valore, lo prende dalla scanf no? O mi sbaglio?
  
  

  Originariamente inviato da MacApp

• il ";" dopo la graffa chiusa del while, potrà crearti solo problemi;

[MacApp]

Originariamente inviato da Zorgatone
Ok ho aggiunto un return 0;
Ma il valore di n non è senza valore, lo prende dalla scanf no? O mi sbaglio?

la scanf è dopo while (n != 0).

Originariamente inviato da Zorgatone
Guarda cosa succede se cambio la dichiarazione delle variabili che è in float in double:

sez6prog1.c:17: warning: format ‘%.2f’ expects type ‘double’, but argument 2 has type ‘double *’

non è chiaro quello che ti dice il compilatore? vuole un double, ma tu gli passi un double *!
;-)

[Zorgatone]

Originariamente inviato da MacApp
la scanf è dopo while (n != 0).

non è chiaro quello che ti dice il compilatore? vuole un double, ma tu gli passi un double *!
;-)

OK meglio cambiare in do {...} while (n != 0);
No non mi è chiaro,
ma scusa cos'è un double *?? Io non gli ho dato un double normale?? Puoi rileggere il codice completo aggiornato???
E poi ultima cosa è giusto usare su scanf e printf "%f" ??? (Scusa ma sono proprio agli inizi... )

EDIT: mi da questo errore...

zorgatone@zorgatone-laptop:~/Programmazione In C/Esercizi$ gcc -o sez6prog1_comp.out sez6prog1.c
sez6prog1.c: In function ‘main’:
sez6prog1.c:17: warning: format ‘%f’ expects type ‘float *’, but argument 2 has type ‘double *’
zorgatone@zorgatone-laptop:~/Programmazione In C/Esercizi$

[MacApp]

Originariamente inviato da Zorgatone
OK meglio cambiare in do {...} while (n != 0);
Ma scusa cos'è un double *?? Io non gli ho dato un double normale?? Puoi rileggere il codice completo aggiornato???

con "&maggiore" ottieni l'indirizzo di maggiore, che appunto è un double *.
http://it.wikipedia.org/wiki/Puntatore_(programmazione)

per quanto riguarda l'errore del ";" non posso sapere non avendo il codice sorgente che te lo procura. Quello che hai postato in precedenza non soffre di quell'errore.
;-)

EDIT: ok hai editato il messaggio precedente, quindi quest'ultima frase non ha più senso nel contesto ;-)

[Zorgatone]

Originariamente inviato da MacApp
con "&maggiore" ottieni l'indirizzo di maggiore, che appunto è un double *.
http://it.wikipedia.org/wiki/Puntatore_(programmazione)

per quanto riguarda l'errore del ";" non posso sapere non avendo il codice sorgente che te lo procura. Quello che hai postato in precedenza non soffre di quell'errore.
;-)

Beh non ci ho capito molto i puntatori non li ho ancora studiati (ho appena iniziato i vari cicli)...
Qui posto il codice com'è stato modificato finora (non ho proprio capito che modifica mi hai consigliato di apportare)

#include <stdio.h>

int main(void)
{
	double n, maggiore;

	printf("************************************************\n");
	printf("* Programma per il calcolo del numero maggiore *\n");
	printf("*                 BY ZORGATONE                 *\n");
	printf("************************************************\n\n");

	maggiore = 0.00f;

	printf("Immettere i numeri da confrontare (0 per smettere): ");
	do
	{
		scanf("%f", &n);
		if (maggiore < n)
		{
			maggiore = n;
		}
	} while (n != 0);
	printf("Il numero maggiore è %.2f.\n", &maggiore);
	return 0;
}

[MacApp]

ma scusa: per quale motivo scrivi:

	printf("Il numero maggiore è %.2f.\n", &maggiore);

cosa intendi fare con la "&"?

[Zorgatone]

Originariamente inviato da MacApp
ma scusa: per quale motivo scrivi:

	printf("Il numero maggiore è %.2f.\n", &maggiore);

cosa intendi fare con la "&"?

Uops non me ne ero accorto... ora ho tolto il &.....

rimangono gli errori del compilatore e se apro l'eseguibile non esce più dal ciclo quando immetto lo zero (sarà perchè ho corretto il while con un do while???)

[Zorgatone]

Incredibilmente ha iniziato a funzionareeeeeeeeeeeeee!!!!!
Il codice alla fine è

#include <stdio.h>

int main(void)
{
	float n, maggiore;

	printf("************************************************\n");
	printf("* Programma per il calcolo del numero maggiore *\n");
	printf("*                 BY ZORGATONE                 *\n");
	printf("************************************************\n\n");

	maggiore = 0.00f;

	printf("Immettere i numeri da confrontare (0 per smettere): ");
	do
	{
		scanf("%f", &n);
		if (maggiore < n)
		{
			maggiore = n;
		}
	} while (n != 0);
	printf("Il numero maggiore è %.2f.\n", maggiore);
	return 0;
}

GRazie mille per l'aiuto!!!!!!!