problema con c

[Nhio]

Ho fatto il seguente programma usando Dev c++ solo che non mi funziona e non capisco perchè...

#include <stdio.h>

int main (void)
{
int x,y,somma,differenza;
char option;
printf("inserisci il primo numero");
scanf("%d",&x);
printf("inserisci il secondo numero");
scanf("%d",&y);
printf("digita s per sommare i numeri oppure d per fare la differenza");
scanf("%c",&option);
if (option=='s')
{
somma=x+y;
printf("la somma è %d",somma);
}
else
{
differenza=x-y;
printf("la differenza è %d",differenza);
}
scanf(" ");
return 0;
}


lo compila senza errori ma non funziona

[okay]

Originariamente inviato da Nhio

Ho fatto il seguente programma usando Dev c++ solo che non mi funziona e non capisco perchè...

#include <stdio.h>

int main (void)
{
int x,y,somma,differenza;
char option;
printf("inserisci il primo numero");
scanf("%d",&x);
printf("inserisci il secondo numero");
scanf("%d",&y);
printf("digita s per sommare i numeri oppure d per fare la differenza");
scanf("%c",&option);
if (option=='s')
{
somma=x+y;
printf("la somma è %d",somma);
}
else
{
differenza=x-y;
printf("la differenza è %d",differenza);
}
scanf(" ");
return 0;
}


lo compila senza errori ma non funziona

errore:
if (option=='s')

[Nhio]

mi dici anche come correggere? cosa c'è che non va? io arrivo da java e sto facendo fatica con la grammatica di c...

[Paco De Luciaaaaa]

Originariamente inviato da okay

errore:
if (option=='s')

scusa ma ne sei certo?

va benissimo

[Paco De Luciaaaaa]

Originariamente inviato da Nhio

Ho fatto il seguente programma usando Dev c++ solo che non mi funziona e non capisco perchè...

#include <stdio.h>

int main (void)
{
int x,y,somma,differenza;
char option;
printf("inserisci il primo numero");
scanf("%d",&x);
printf("inserisci il secondo numero");
scanf("%d",&y);
printf("digita s per sommare i numeri oppure d per fare la differenza");
scanf("%c",&option);
if (option=='s')
{
somma=x+y;
printf("la somma è %d",somma);
}
else
{
differenza=x-y;
printf("la differenza è %d",differenza);
}
scanf(" ");
return 0;
}


lo compila senza errori ma non funziona

l'errore sta nelle scanf... quando la seconda scanf acquisisce il secondo numero nel buffer di tastiera rimane il carattere '\n' dato dall'invio che viene pescato poi dalla terza scanf per il carattere: ergo option sarà sempre il carattere '\n'

Per correggere basta sostituire questo:

scanf("%d%*c",&y);

che forza l'acquisizione del carattere '\n'.

Prova.

[okay]

Originariamente inviato da Paco De Luciaaaaa

scusa ma ne sei certo?

va benissimo

scusa ma data l'ora... ho il cervello un pò bruciato...

Ho dato un'occhio veloce al code.

Mi sembrava la comparazione di char option=="stringa" con "stringa" in cui si usa (stringa==0). Invece ora che rileggo è il carattere 's'...

Cmq è un carattere equivalente a stringa quindi devi acquisire una Stringa con %s

La soluzione eccola:

scanf("%s",&option);

[Nhio]

scanf("%s",&option);


perfetto adesso funziona!


grazie 1000 nn ci sarei mai arrivato!!!

[Paco De Luciaaaaa]

Originariamente inviato da okay

scusa ma data l'ora... ho il cervello un pò bruciato...

Ho dato un'occhio veloce al code.

Mi sembrava la comparazione di char option=="stringa" con "stringa" in cui si usa (stringa==0). Invece ora che rileggo è il carattere 's'...

Cmq è un carattere equivalente a stringa quindi devi acquisire una Stringa con %s

La soluzione eccola:

scanf("%s",&option);

Sintatticamente è corretta ma non per come ha dichiarato option: ricorda che c'è una bella differenza tra un char e una stringa. il %s in quel caso funzionerebbe solo se da tastiera si immette un carattere+INVIO... prova a scrivere a run time
rpippo\n e vedi cosa succede.

Ripeto funziona ma è molto sensibile a bug da tastiera.

P.S. rivedendo la scanf ci sarebbe un altro errore: con le stringhe non si può operare il reference &.

scanf("%s",option);

[/quote]

[Gino+89+]

Fai così:

#include <stdio.h>

int main ()
{
int x,y,somma,differenza;
char option;

printf("Inserisci il primo numero:");
scanf("%d",&x);
while(getchar()!='\n');

printf("\nInserisci il secondo numero:");
scanf("%d",&y);
while(getchar()!='\n');

printf("\nDigita s per sommare i numeri, oppure d per fare la differenza:");
scanf("%c",&option);
while(getchar()!='\n');

if (option=='s')
{
somma=x+y;
printf("\nLa somma e' %d",somma);
}
else
{
differenza=x-y;
printf("\nLa differenza e' %d",differenza);
}

getchar();
return 0;
}
(devo ancora capire come si può indentare il codice in sto forum...)

P.S.: Non usare mai nomi come x o y per questo tipo di varaibili

Saluti

[Paco De Luciaaaaa]

Originariamente inviato da Gino+89+

Fai così:

#include <stdio.h>

int main ()
{
int x,y,somma,differenza;
char option;

printf("Inserisci il primo numero:");
scanf("%d",&x);
while(getchar()!='\n');

printf("\nInserisci il secondo numero:");
scanf("%d",&y);
while(getchar()!='\n');

printf("\nDigita s per sommare i numeri, oppure d per fare la differenza:");
scanf("%c",&option);
while(getchar()!='\n');

if (option=='s')
{
somma=x+y;
printf("\nLa somma e' %d",somma);
}
else
{
differenza=x-y;
printf("\nLa differenza e' %d",differenza);
}

getchar();
return 0;
}
(devo ancora capire come si può indentare il codice in sto forum...)

P.S.: Non usare mai nomi come x o y per questo tipo di varaibili

Saluti

Corretto ma non ottimizzato perchè inutilmente usi un while e una chiamata alla procedura getchar quando puoi fare la stessa cosa nella scanf...

Ripeto basta usare l'operatore "*" nelle scanf...

[Gino+89+]

Più che ottimizzazione del codice sono scelte di stile da parte del programmatore...comunque si, e potevo pure (e forse era meglio) definire option come int per risparmiare 3 byte

[Paco De Luciaaaaa]

Originariamente inviato da Gino+89+

Più che ottimizzazione del codice sono scelte di stile da parte del programmatore...comunque si, e potevo pure (e forse era meglio) definire option come int per risparmiare 3 byte

Stai dicendo che un char occupa 4 byte?

Mi sa che devi rivederti i tipi in c... cmq va bene così. Fidati nn cambiare.

[Gino+89+]

Opsss è verpo 1 byte char e 4 byte int ...Errore di distrazione infedeli

[DanieleC88]

Originariamente inviato da Nhio

scanf("%s",&option);

Si, come no. Metti qualche carattere extra e ti becchi un bel SEGMENTATION FAULT.