[C] Semplice esercizio con uso del while

[Composition86]

/*Determinazione dei Km fatti ad ogni pieno di benzina*/
#include <stdio.h>
#include <stdlib.h>

main()
{
int contatore, km, litri, kmallitro, kmallitrotot, media;

contatore = 0;

printf ("Inserire i kilometri percorsi con ogni pieno (-1 per uscire: \n");
scanf ("%d", &km);
printf ("Inserire i litri di benzina utilizzati: \n");
scanf ("%d", &litri);
kmallitro = km / litri;
printf ("Il numero di km al litro percorsi e': %d \n", kmallitro);

while (km != -1) {
printf ("Inserire i kilometri percorsi con ogni pieno (-1 per uscire: \n");
scanf ("%d", &km);
printf ("Inserire i litri di benzina utilizzati: \n");
scanf ("%d", &litri);
kmallitro = km / litri;
printf ("Il numero di km al litro percorsi e': %d \n", kmallitro);
kmallitrotot = kmallitrotot + kmallitro;
contatore = contatore + 1;
}

if (contatore != 0) {
media = kmallitrotot / contatore;
printf ("La media dei km con un pieno e': %d \n", media);
}
else
printf ("Non e' stato inserito nessun dato. \n");

system("pause");
return 0;
}

Sto facendo questo semplice esercizio, il cui obiettivo è calcolare i km/litro ed infinie trovare il valore medio di tutti i km/litro.

Ecco i miei dubbi:
-Prima del "while", se non rimetto tutta la parte in cui si richiede l'inserimento dei dati ed il calcolo, il programma non funziona correttamente, perchè?
-Se digito il valore sentinella "-1" (mediante il quale dovrei uscire dal programma), questo viene inserito come un comune dato e quindi non termina l'iterazione.
-La media finale non funziona.

Mi potreste fare un po' di chiarezza? Grazie.

[k0nt3]

intanto metti il codice tra i tag CODE e /CODE perchè se lo metti nei tag QUOTE non si indenta

[k0nt3]

ecco qualche spiegazione:
- se non rimetti la parte dentro il while prima del while, il codice non funziona perchè le variabili non sono inizializzate. devi inizializzare tutte le variabili a 0, altrimenti il comportamento del programma non è predicibile

- questo perchè il controllo sul valore di km viene fatto all'inizio di ogni ciclo, quindi una volta che assegni il valore -1 alla variabile km il ciclo continua e si interrompe al ciclo successivo

- se inizializzi le variabili dovrebbe funzionare

ps. usa getchar() al posto di system("pause").
system("pause") non è standard e inoltre è brutto

[Composition86]

/*Determinazione dei Km fatti ad ogni pieno di benzina*/
#include <stdio.h>
#include <stdlib.h>

main()
{
   int contatore, km, litri, kmallitro, kmallitrotot, media;

   contatore = 0;
   
      printf ("Inserire i kilometri percorsi con ogni pieno (-1 per uscire: \n");
      scanf ("%d", &km);
      printf ("Inserire i litri di benzina utilizzati: \n");
      scanf ("%d", &litri);
      kmallitro = km / litri;
      printf ("Il numero di km al litro percorsi e': %d \n", kmallitro);

   while (km  != -1) {
      printf ("Inserire i kilometri percorsi con ogni pieno (-1 per uscire: \n");
      scanf ("%d", &km);
      printf ("Inserire i litri di benzina utilizzati: \n");
      scanf ("%d", &litri);
      kmallitro = km / litri;
      printf ("Il numero di km al litro percorsi e': %d \n", kmallitro);
      kmallitrotot = kmallitrotot + kmallitro;
      contatore = contatore + 1;
      }

   if (contatore != 0) {
      media = kmallitrotot / contatore;
      printf ("La media dei km con un pieno e': %d \n", media);
   }
   else
      printf ("Non e' stato inserito nessun dato. \n");

   system("pause");
   return 0;
}

Si vero, meglio così.

[Composition86]

Originariamente inviato da k0nt3

ecco qualche spiegazione:
- se non rimetti la parte dentro il while prima del while, il codice non funziona perchè le variabili non sono inizializzate. devi inizializzare tutte le variabili a 0, altrimenti il comportamento del programma non è predicibile

- questo perchè il controllo sul valore di km viene fatto all'inizio di ogni ciclo, quindi una volta che assegni il valore -1 alla variabile km il ciclo continua e si interrompe al ciclo successivo

- se inizializzi le variabili dovrebbe funzionare

ps. usa getchar() al posto di system("pause").
system("pause") non è standard e inoltre è brutto

Intanto ti ringrazio, fra un po' provo e vedo cosa ne esce.
il system("pause") è un comando che sto usando temporaneamente per visualizzare il programma (praticamente appiccicato senza sapere cosa sia), appena arrivo a studiare la getchar() lo cambio!

[k0nt3]

come non detto getchar() poi ha problemi quando usi la scanf... vai pure avanti con il tuo passo

[Composition86]

Ho provato a fare un po' di cambiamenti, purtroppo non ho ottenuto nulla.

Il risultato è sempre che se inserisco il valore sentinella "-1" (a tutte e due le variabili, visto che ci sono, ho provato anche con l'OR nella condizione del "while") il programma esegue la divisione con tale numero (ovviamente non richiesta), ricomincia poi un altro ciclo e così via.

[Composition86]

Originariamente inviato da k0nt3

ecco qualche spiegazione:
- questo perchè il controllo sul valore di km viene fatto all'inizio di ogni ciclo, quindi una volta che assegni il valore -1 alla variabile km il ciclo continua e si interrompe al ciclo successivo

while (km  != -1) {
      printf ("Inserire i kilometri percorsi con ogni pieno (-1 per uscire: \n");
      scanf ("%d", &km);
      printf ("Inserire i litri di benzina utilizzati: \n");
      scanf ("%d", &litri);
      kmallitro = km / litri;
      printf ("Il numero di km al litro percorsi e': %d \n", kmallitro);
      kmallitrotot = kmallitrotot + kmallitro;
      contatore = contatore + 1;
      }

Credo di aver intuito cosa vuoi dire, ma mi sfugge un particolare: io inserisco "-1" a km che poi viene assegnato tramite scanf ad una certa locazione di memoria.
Il controllo su questa variabile viene effettivamente fatto quando il ciclo ricomincia.
Io invece vorrei che venisse fatto subito, al primo tentativo ed evitare (se possibile) di riscrivere tutta la parte iniziale ridondante (ho provato ad inizializzare tutte le variabili). Ti ringrazio per l'aiuto.

[k0nt3]

ad esempio potresti fare una roba del genere:

while (km != -1) {
        printf ("Inserire i kilometri percorsi con ogni pieno (-1 per uscire): \n");
        scanf ("%d", &km);
        if(km >= 0) {
            printf ("Inserire i litri di benzina utilizzati: \n");
            scanf ("%d", &litri);
            kmallitro = km / litri;
            printf ("Il numero di km al litro percorsi e': %d \n", kmallitro);
            kmallitrotot = kmallitrotot + kmallitro;
            contatore = contatore + 1;
        }
    }

che calcola i km al litro solo se la variabile km contiene un numero maggiore di 0.
in questo modo quando inserisci -1 la condizione nell'if non è verificata e il ciclo termina subito

[Composition86]

Originariamente inviato da k0nt3

ad esempio potresti fare una roba del genere:

while (km != -1) {
        printf ("Inserire i kilometri percorsi con ogni pieno (-1 per uscire): \n");
        scanf ("%d", &km);
        if(km >= 0) {
            printf ("Inserire i litri di benzina utilizzati: \n");
            scanf ("%d", &litri);
            kmallitro = km / litri;
            printf ("Il numero di km al litro percorsi e': %d \n", kmallitro);
            kmallitrotot = kmallitrotot + kmallitro;
            contatore = contatore + 1;
        }
    }

che calcola i km al litro solo se la variabile km contiene un numero maggiore di 0.
in questo modo quando inserisci -1 la condizione nell'if non è verificata e il ciclo termina subito

Si l'effetto è lo stesso, ma a questo punto la condizione del while è una inutile aggiunta.
E poi, attenendomi all'esercizio del libro, volevo avere un unico valore per cui terminare il ciclo: adesso la condizione dell' "if" per i numeri positivi funziona, ma se invece mi trovo a dover fare un programma dove le variabili possono assumere anche valori negativi, tale condizione non sarebbe più funzionale.

[k0nt3]

beh non è proprio inutile la condizione del while..
per come l'ho scritto io funziona così:
- se il numero è >= 0 calcola i km al litro
- se il numero è negativo non calcola niente ma va avanti a chiedere numeri
- se il numero è -1 termina il ciclo

se vuoi trovare una soluzione più elegante mi sa che devi cambiare il modo con cui dici al programma di terminare il ciclo. ma dipende da quello che devi fare

[Composition86]

Ti ringrazio ancora.
Mi devo solo attenere all'esercizio, per il momento mi interessa solo capire quello che faccio, non avere una soluzione elegante. Quindi va bene come hai fatto tu. Penso di poterl archiviare l'esercizio.