APUA!!! Switch

Moi!
Olin vasta aloittamassa opintojakson ymmärtääkseni lopulta ohjelmoinnin perusteet. Ja nyt olen täysin hämmentynyt enkä ymmärrä, mitä tässä asiassa tehdä. En ole koskaan aiemmin ollut mukana ohjelmoinnissa. Rehellisesti sanottuna minulla ei ole vielä helppoa ymmärtää kaikkia näitä algoritmeja, mutta minulla on halu ymmärtää tätä aluetta. Siksi pyydän apua virheideni korjaamiseen ja siten materiaalin ymmärtämiseen.
Kiitos jo etukäteen!


Tentävä:
Tee yksinkertainen laskinohjelma, joka laskee kahden luvun erotuksen, summan tai tulon. Ohjelma kysyy ensin mikä laskutoimitus suoritetaan. Tämän jälkeen ohjelma kysyy luvut, suorittaa laskutoimituksen ja tulostaa saadun tuloksen. Ohjelman valinta on tehtävä käyttämällä switch()-lausetta.

Esimerkkituloste:
1: vähennyslasku
2: yhteenlasku
3: kertolasku
Valitse toiminto:3
Anna eka luku:3
Anna toka luku:54
3*54=162

Yritin näin (ja oli ilmoitus "Virhe: Ohjelmasi ei kääntynyt"):

#include <stdio.h>

int main()

{
int eka_luku, toka_luku, tulos1=eka_luku-toka_luku, tulos2=eka_luku*toka_luku, tulos3=eka_luku*toka_luku, ;
eka_luku=3;
toka_luku=54;
kertolasku=162;


printf("1: vähennyslasku\n");
scanf;

printf("2: yhteenlasku\n");
scanf;

printf("3: kertolasku\n");
scanf;

printf("Valitse toiminto:3\n");
scanf;

char op;

printf("Anna eka luku: 3\n");
scanf;

printf("Anna toka luku: 54\n");
scanf;

switch (op)

{
case 1:
{
printf("tulos1=eka_luku-toka_luku\n");
break;

}
case 2:
{
printf("tulos2=eka_luku toka_luku\n");
break;
}
case 3:
{
printf("tulos3=eka_luku*toka_luku\n");
break;
}

}

printf("3*54=162\n");
scanf;


return 0;
}

17

2066

    Vastaukset

    Anonyymi (Kirjaudu / Rekisteröidy)
    5000
    • Anonyymi

      Mahtaa siinä ainakin tämä osa:

      printf("3*54=162\n");
      scanf;

      olla aivan liikaa ja turhaa.

      Sillä ohjelman kai pitäisi tulostaa syötettyjen lukujen perusteella tehtyjen laskutoimitusten tuloksia.

    • Anonyymi

      Ainakin scanf-funktiota käytetään väärin. Tee se tähän tyyliin:

      int i = 0;
      printf("Anna luku: ");
      scanf("%d", &i);

      Tuossa siis luodaan muuttuja i alkuarvolla nolla. Sitten pyydetään luku. Lopuksi scanf-funktiolla luetaan numero (%d) ja sijoitetaan se muuttujaan i (merkintään funktiossa &i).

      Sitten sulla oli joku turha char -juttu siellä.

      Muuttujat kannattaa luoda selkeyden vuoksi tuossa ohjelman alussa niin että teet ne kaikki omilla riveillään ja alustat ne nollaksi. Eli suunnilleen näin:

      int eka_luku = 0;
      int toka_luku = 0;
      int vastuus = 0;

      Sitten switch-rakenteen sisällä teet laskun jossa vastauksen arvo muuttuu eli esim pluslaskussa:

      vastaus = eka_luku toka_luku;

      Sitten swtichin jälkeen vasta aina tulostetaan vastaus:

      printf("Vastaus on: %d", vastaus);


      ...ja hei, hienoa että opiskelet ohjelmointia ja vieläpä C-kielellä! Tsemppiä!

      • Anonyymi

        Kiitos!!!


    • Anonyymi

      Luitko tuossa edellä antamani vinkit (klo 13:33)?

      Jos ei auennut vielä niin korjasin tuon sinun koodin toimivaksi. En laita sitä tähän kirjoitettuna että et voi suoraan kopioida vastausta tehtävään. Laitan kuitenkin kuvan mistä voit katsoa yhden toimivan toteutuksen malliksi.

      https://i.postimg.cc/yYpPNfyB/laskuri.jpg

      Itse koodi on siis vasemmassa reunassa tummalla pohjalla. Oikeassa reunassa on näkyvissä miten ohjelma kääntyi ilman virheilmoituksia gcc -kääntäjällä komentoriviltä linuxissa ja että kaikki laskutoimitukset toimivat oikein.


      Kiinnitä erityisesti huomiota:

      1:
      Muuttujien alustaminen alussa on tehty selkeys edellä. Lisäksi ei mielellään ikinä jätetä muttujaa alustamatta kun se luodaan. Eli siksi niihin kaikkiin laitetaan nollat alkuun.

      2:
      Scanf on funktio. Eli siinä pitää olla ne sulut ja lisäksi se tarvitsee parametrejä. Eka parametri, joka on hipsujen sisällä, kertoo vastaanotettavan tiedon muodon. Prosentti-d tarkoittaa numeromuotoa. Sitten se toinen parametri on viittaus muuttuaan johon se vastaanotettava tieto tallennetaan. &-merkki tekee siitä viittauksen kyseisen muuttujan muistialueeseen.

      3:
      Laskutoimitukset tehdään switch-case -rakenteen sisällä ja laskujen lopputulosta käytetään vasta rakenteen jälkeen. Näin ei tarvitse kutsua printf -funktiota kuin vain yhdestä paikasta eikä jokaisessa haarassa.

      • Anonyymi

        Kiitoksia paljon!


    • Anonyymi

      Eipä mitään! Paljon tsemppiä opiskeluun!

    • Anonyymi

      Halusin tuoda mukaan vielä yhden erilaisen tavan käsitellä tätä samaa asiaa. Tämä perinteinen tapa että käyttäjältä tivataan asioita, jotka ohjelma voi päätellä syötteestä ei aina oikein innosta puolin eikä toisin.

      Tässä käyttäjä syöttää kaavan, ja kaavasta päätellään mikä tehtävä tulee suorittaa, kuva selventää asiaa: https://s7.gifyu.com/images/s24kysymys.png

      Pistin myös koodin tuonne ( https://pastebin.com/fkE4iQs6 ), koska mukana on taas näitä väritettyjä tulosteita. Asetan vielä kysymyksen tähän.

      Tuossa kaava tulee antaa niin ettei sitä jaeta välilyönnein ( 234-23 ) mitä muutoksia tulee tehdä, että haarautumiset onnistuvat oikein vaikka käyttäjä syöttäisi kaavan näin ( 234 - 23 ), eli erittelee operandit ja operaatorit välilyönnein, ja siitä huolimatta haarautumiset menee oikein.

      • Anonyymi

        Lisätään vielä, että, minä tiedän että, tämä ( switch (op) {} ) lohko on turhaan, koska ne laskut olisi voitu suorittaa jo niissä if -lauseissa joissa muuttujalle op haettiin arvoa. Halusin sen kuitenkin säilyttää, koska asiaa tähän asti käsiteltiin juuri switch() haarautumisien vuoksi.


      • Anonyymi
        Anonyymi kirjoitti:

        Lisätään vielä, että, minä tiedän että, tämä ( switch (op) {} ) lohko on turhaan, koska ne laskut olisi voitu suorittaa jo niissä if -lauseissa joissa muuttujalle op haettiin arvoa. Halusin sen kuitenkin säilyttää, koska asiaa tähän asti käsiteltiin juuri switch() haarautumisien vuoksi.

        Tämähän oli oikein tervetullut ratkaisuversio! Oma "ohjelmoinnin perusteet" kurssin silloin joskus sisälsi tämän saman tehtävän mutta en muista että kukaan olisi tehnyt tälläistä kaava-versiota. Kai se opetus vähän ohjasi siihen anna-eka-luku -tyyppiseen ajatteluun.


      • Anonyymi
        Anonyymi kirjoitti:

        Tämähän oli oikein tervetullut ratkaisuversio! Oma "ohjelmoinnin perusteet" kurssin silloin joskus sisälsi tämän saman tehtävän mutta en muista että kukaan olisi tehnyt tälläistä kaava-versiota. Kai se opetus vähän ohjasi siihen anna-eka-luku -tyyppiseen ajatteluun.

        Itseppä yllytit, nyt sitten versio jossa mukana myös tuo lasketa, ja vielä niin ettei välilyönnit tee kiusaa operaattorien ja operandien välissä.
        Kuva https://s7.gifyu.com/images/oneversion.png

        Tässä versiossa on jo luovuttu tuosta switch() lajittelusta, mutta uutena mukana esimerkki kuinka tuosta käyttäjältä saadusta merkkijonosta poimitaan laskennassa tarvittavat luvut.
        Koodi: https://pastebin.com/GrNDrLAe


      • Anonyymi
        Anonyymi kirjoitti:

        Itseppä yllytit, nyt sitten versio jossa mukana myös tuo lasketa, ja vielä niin ettei välilyönnit tee kiusaa operaattorien ja operandien välissä.
        Kuva https://s7.gifyu.com/images/oneversion.png

        Tässä versiossa on jo luovuttu tuosta switch() lajittelusta, mutta uutena mukana esimerkki kuinka tuosta käyttäjältä saadusta merkkijonosta poimitaan laskennassa tarvittavat luvut.
        Koodi: https://pastebin.com/GrNDrLAe

        gcc antaa seuraavat varoitukset:

        warning: implicit declaration of function ‘gets’; did you mean ‘fgets’? [-Wimplicit-function-declaration]
        gets(kaava);
        ^~~~

        warning: the `gets' function is dangerous and should not be used.


        ...eli siis gets() on käytännössä altis puskurinylivuodolle. fgets() ei ole mutta vähän hankalampi käyttää. Jännä että myös scanf() on periaatteessa altis puskuriylivuodolle mutta siitä ei tule varoituksia.

        Muuten kyllä on ihan kiva toteutus, joskaan ei kyllä varmaankaan aloittelijalle enää kovin ymmärrettävä. Nykyisin kun kääntäjät osaavat optimoida niin hyvin, on mielestäni järkevää panostaa nimenomaan luettavuuteen ja ymmärtämisen helppouteen lähdekoodissa.


      • Anonyymi
        Anonyymi kirjoitti:

        gcc antaa seuraavat varoitukset:

        warning: implicit declaration of function ‘gets’; did you mean ‘fgets’? [-Wimplicit-function-declaration]
        gets(kaava);
        ^~~~

        warning: the `gets' function is dangerous and should not be used.


        ...eli siis gets() on käytännössä altis puskurinylivuodolle. fgets() ei ole mutta vähän hankalampi käyttää. Jännä että myös scanf() on periaatteessa altis puskuriylivuodolle mutta siitä ei tule varoituksia.

        Muuten kyllä on ihan kiva toteutus, joskaan ei kyllä varmaankaan aloittelijalle enää kovin ymmärrettävä. Nykyisin kun kääntäjät osaavat optimoida niin hyvin, on mielestäni järkevää panostaa nimenomaan luettavuuteen ja ymmärtämisen helppouteen lähdekoodissa.

        Olet taas oikeassa, muuta se rivi (23) tälläiseksi:
        fgets(kaava, sizeof(kaava), stdin);


      • Anonyymi
        Anonyymi kirjoitti:

        gcc antaa seuraavat varoitukset:

        warning: implicit declaration of function ‘gets’; did you mean ‘fgets’? [-Wimplicit-function-declaration]
        gets(kaava);
        ^~~~

        warning: the `gets' function is dangerous and should not be used.


        ...eli siis gets() on käytännössä altis puskurinylivuodolle. fgets() ei ole mutta vähän hankalampi käyttää. Jännä että myös scanf() on periaatteessa altis puskuriylivuodolle mutta siitä ei tule varoituksia.

        Muuten kyllä on ihan kiva toteutus, joskaan ei kyllä varmaankaan aloittelijalle enää kovin ymmärrettävä. Nykyisin kun kääntäjät osaavat optimoida niin hyvin, on mielestäni järkevää panostaa nimenomaan luettavuuteen ja ymmärtämisen helppouteen lähdekoodissa.

        Kävin pastebin versionkin korjaamassa.


      • Anonyymi
        Anonyymi kirjoitti:

        gcc antaa seuraavat varoitukset:

        warning: implicit declaration of function ‘gets’; did you mean ‘fgets’? [-Wimplicit-function-declaration]
        gets(kaava);
        ^~~~

        warning: the `gets' function is dangerous and should not be used.


        ...eli siis gets() on käytännössä altis puskurinylivuodolle. fgets() ei ole mutta vähän hankalampi käyttää. Jännä että myös scanf() on periaatteessa altis puskuriylivuodolle mutta siitä ei tule varoituksia.

        Muuten kyllä on ihan kiva toteutus, joskaan ei kyllä varmaankaan aloittelijalle enää kovin ymmärrettävä. Nykyisin kun kääntäjät osaavat optimoida niin hyvin, on mielestäni järkevää panostaa nimenomaan luettavuuteen ja ymmärtämisen helppouteen lähdekoodissa.

        Luettavuutta tuskin voi parantaa, muuten kuin värityksen poistolla, ja palaamalla takaisin siihen että tinkaa käyttäjältä parametrit erikseen. Eli näön ja käytettävyyden kustannuksella luettavuus paranee, ja asiat näyttää yksinkertaisimmilta.


    • Anonyymi

      Tuskin tälläkään palstalla kirjoittelevista löytyy yhtään sellaista, joka ei olisi jossakin vaiheessa ollut kiinnostunut C ohjelmointikielestä.

      • Anonyymi

        Joo, siis kyllähän se on edelleen vähän semmoinen status-juttukin jos osaa kirjoittaa C-kielellä ohjelmia. Enkä oikein usko että työpaikat C-osaajilta on loppumassa lähiaikoina.

        Lämmöllä muistelen niitä iltoja kun aikoinaan niitä kaikkia harjoituksia taistelin läpi.


      • Anonyymi
        Anonyymi kirjoitti:

        Joo, siis kyllähän se on edelleen vähän semmoinen status-juttukin jos osaa kirjoittaa C-kielellä ohjelmia. Enkä oikein usko että työpaikat C-osaajilta on loppumassa lähiaikoina.

        Lämmöllä muistelen niitä iltoja kun aikoinaan niitä kaikkia harjoituksia taistelin läpi.

        Joko ne raivostuttavat illat on unohtunut, aina ei asiat luista niin kuin pitäisi tai oletti.


    Ketjusta on poistettu 0 sääntöjenvastaista viestiä.

    Luetuimmat keskustelut

    1. Kun väestö ikääntyy ja veronmaksajat vähenee, mitä sitten vasemmistolaiset?

      Maahanmuutto ei vaan ole ratkaisu väestön ikääntymiseen. Maahanmuutto lykkää ja hidastaa väestön ikääntymistä ja työv
      Maailman menoa
      200
      4363
    2. "Mitä sä nainen tuot sitten pöytään" ?

      Jos mies provaidaa ja suojelee... Pitääkö miesten kysyä tuollaisia?
      Ikävä
      100
      2902
    3. Ekologinen kommunismi tulee voittamaan fossiilikapitalismin

      Kiina on mahtitekijä uusiutuvien energialähteiden kehityksessä, ja Trump osoitus viimeisestä öljyn perään itkemisestä, m
      Maailman menoa
      21
      2845
    4. Minja jytkyttää vas.liiton kannatusta ylöspäin

      Alkaa raavaat duunarimiehetkin palaamaan vasemmistoliiton kannattajiksi. Eduskunnassahan on vain kaksi työntekijöiden p
      Maailman menoa
      92
      2654
    5. Aktivistinainen pysäytti ICE-agentin luodin päällään USA:ssa!

      Video ampumistilanteesta: https://edition.cnn.com/2026/01/07/us/video/ice-shooting-minneapolis-digvid "Media: ICE:n am
      Maailman menoa
      152
      2347
    6. 6 kW saunan lämmityksestä kohta 10 euron lisämaksu / kerta

      Kokoomuslainen sähköyhtiöiden hallitsema Energiavirasto ehdottaa 5 kW:n rajaa, jonka ylittämisestä tulee lisämaksu. Tark
      Maailman menoa
      57
      2295
    7. Oikeistopuolueiden kannatus vain 37,8 %, vasemmiston 43,0 %

      Keskustaan jää 17,4 prosenttia ja loput ovat sitten mitä ovat. Mutta selvästikin Suomen kansa on vasemmalle kallellaan.
      Maailman menoa
      33
      2107
    8. Tiedän ettei

      Meistä mitään tule. Toinen oli sinulle tärkeämpi
      Ikävä
      20
      2072
    9. Mahonselän jäät - Saaristokunta Lieksa brutaalisti kriisin partaalla!

      Lieksan loppuvuoden hyvän kehityksen jälkeen ei olisi uskonut että palstan ahkerista kommentoijista huolimatta matkailu
      Lieksa
      137
      2001
    10. Laitetaan nyt kirjaimet kohdilleen

      kuka rakastaa ja ketä ?
      Ikävä
      87
      1999
    Aihe