Tämän ohjelman tarkoitus on ensin valita laskutoimitus ja sitten syöttää kaksi lukua se laskee tuloksen. Mikä tässä on vikana?
#include
#include
int main ()
{
int eka_luku, toka_luku;
printf ("1: vähennyslasku\n2: yhteenlasku\n3: kertolasku\nValitse laskutoimitus:");
scanf ("%d", &eka_luku, toka_luku);
switch (eka_luku, toka_luku)
{
case 1:
{
printf ("Anna eka luku\n");
scanf ("%d", &eka_luku);
printf ("Anna toka luku\n");
scanf ("%d", &toka_luku);
vahennus = eka_luku - toka_luku;
break;
}
case 2:
{
printf ("Anna eka luku\n");
scanf ("%d", &eka_luku);
printf ("Anna toka luku\n");
scanf ("%d", &toka_luku);
vahennys = eka_luku toka_luku;
break;
}
case 3:
{
printf ("Anna eka luku\n");
scanf ("%d", &eka_luku);
printf ("Anna toka luku\n");
scanf ("%d", &toka_luku);
tulo = eka_luku * toka_luku;
break;
}
default:
printf ("Antamasi sy\x99te on arvoalueen ulkopuolella.\n");
}
system ("pause");
return 0;
}
#include
#include
int main ()
{
int eka_luku, toka_luku;
printf ("1: vähennyslasku\n2: yhteenlasku\n3: kertolasku\nValitse laskutoimitus:");
scanf ("%d", &eka_luku, toka_luku);
switch (eka_luku, toka_luku)
{
case 1:
{
printf ("Anna eka luku\n");
scanf ("%d", &eka_luku);
printf ("Anna toka luku\n");
scanf ("%d", &toka_luku);
vahennus = eka_luku - toka_luku;
break;
}
case 2:
{
printf ("Anna eka luku\n");
scanf ("%d", &eka_luku);
printf ("Anna toka luku\n");
scanf ("%d", &toka_luku);
vahennys = eka_luku toka_luku;
break;
}
case 3:
{
printf ("Anna eka luku\n");
scanf ("%d", &eka_luku);
printf ("Anna toka luku\n");
scanf ("%d", &toka_luku);
tulo = eka_luku * toka_luku;
break;
}
default:
printf ("Antamasi sy\x99te on arvoalueen ulkopuolella.\n");
}
system ("pause");
return 0;
}
Vastaukset 7
Vanhimmat
yksi ensimmäisistä riveistä:
scanf ("%d", &eka_luku, toka_luku);
mitähän tuon on tarkoitus tehdä?
yrität tallentaa desmiaaliluvun kahteen eri muuttujaan vai mitä häh? se ei kyllä toimi noin. ja eikös sun tuossa kohdassa pitäs enneminki tallentaa se haluttu laskutoimitus johonkin ihan eri muuttujaan? vai olenko nyt ihan hakuteillä
Ei tuo voi toimia.
Laskutoimitusta ei edes kysytä ja sitten switch..case rakennetta ei osata käyttää.
Suotta on monta kertaa tuo lukujen kysely.
double eka_luku, toka_luku, tulos;
int laskutoimitus;
printf ("Anna kaksi lukua:");
scanf ("%d", &eka_luku, toka_luku);
printf ("1: vähennyslasku\n2: yhteenlasku\n3: kertolasku\nValitse laskutoimitus:");
scanf ("%d", &laskutoimitus);
switch (laskutoimitus)
{
case 1:
{
tulos = eka_luku - toka_luku;
break;
}
case 2:
{
tulos = eka_luku + toka_luku;
break;
}
case 3:
{
tulos = eka_luku * toka_luku;
break;
}
default:
printf ("Antamasi sy\x99te on arvoalueen ulkopuolella.\n");
return -1;
}
//tässä vaiheessa voit tehdä tuloksella tulos mitä itse haluat
system ("pause");
return 0;
}
double eka_luku, toka_luku, tulos;
int laskutoimitus;
printf ("Anna kaksi lukua:");
scanf ("%d", &eka_luku, toka_luku);
doublea ei lueta "%d" eikä scanf osaa sijoitaa kahteen muutujaan noilla tiedoilla.
eka_luku ja ja toka_luku voivat olla inttejä kunhan kertoessa tehdään castaus.
Kopsasin sen alkuperäisen koodin, ja korjasin ne periaatteelliset virheet (kaksi muuttujaa switch-lauseessa, muuttujien kysely useampaan kertaan switch-lauseen sisällä, eri tulosmuuttujia laskutoimituksesta riippuen jne).
Pikkuseikkoihin kuten lukuformaatteihin en jaksannut puuttua!
Muutin muuttujatyyppiä liukuluvuksi koska en ymmärrä miksi rajottaa laskentaa kokonaisluvuksi (kyseessähän on pääohjelma, ei funktio joka nimeenomaan pitäisi palauttaa kokonaislukua!).
"eka_luku ja ja toka_luku voivat olla inttejä kunhan kertoessa tehdään castaus. "???
Nyt en ymmärrä lainkaan mitä höpiset? Kyllähän kahdella kokonasilyvulla voidaan tehdä kertolasku ilman mitään muutujatyypin muuttamista!
int a, b, int_tulos;
double x, y, real_tulos;
int_tulos= a*b; //täysin oikein
real_tulos=(double)(a*b); //oikein mutta ei tarpeellinen
int_tulos=(int)(x*y); // tavallisesti käytetään funtiota (esim floor()) pyristysvirheen hoitamiseen
Kahden kokonaisluvun tulo on aina kokonaisluku. Kahden kokonaisluvun jakolaskussa lopputulos on lähes aina realiluku, eli jos kaikki muuttujat lasku-lausekkeessa ovat tyyppiä int (tai byte tai long...) tapahtuu pyöristysvirhe.
Seuraavan kerran kun vastaat kysymykseen, älä viisastele ja kerro mitä mielestäsi on väärin. Samalla vaivalla voit saman tien kertoa miten sen pitää olla! (eli jos tiedät!)
...
float eka_luku, toka_luku, tulos;
...
scanf ("%f %f", &eka_luku, &toka_luku);
...
siinähän tuli tekstiä. itseasiassa scanf kikkailu on vakavampi koska siitä kääntäjä ei valita.
olet oikeassa tuossa kertolasku asiassa.