C alkeet ongelma while lauseen kanssa

keijokoodari12

Eli tarkoitus olisi tehdä ohjelma, joka kysyy oppilaiden koearvosanoja (kokonaisluvut 4-10 käytössä) ja laskee syötettyjen arvosanojen keskiarvon. Ohjelman tulee ottaa vastaan arvosanoja, kunnes arvosanojen syöttö lopetetaan negatiivisella kokonaisluvulla. Lopuksi ohjelma tulostaa arvosanojen lukumäärän sekä lasketun keskiarvon, joka tulostetaan kahden desimaalin tarkkuudella.

Ohjelman tulisi tulostaa seuraavasti:
Esimerkkitulostus

Ohjelma laskee syötettyjen arvosanojen keskiarvon.
Lopetus negatiivisella kokonaisluvulla.
Anna arvosana (4-10):4
Anna arvosana (4-10):5
Anna arvosana (4-10):6
Anna arvosana (4-10):9
Anna arvosana (4-10):7
Anna arvosana (4-10):-1
Ohjelmaan syötetty 5 arvosanaa.
Arvosanojen keskiarvo: 6.20

En saa tekemääni koodia toimimaan enkä oikein ymmärrä mikä siinä on vialla.
Voi myös olla että olen ihan pihalla tässä hommassa. Epätoivosesti yrittänyt maalaisjärjellä ratkoa tätä.

Koodi:
###########
#include

int main()
{
int i = 0;
int luku, ka, summa, maara;

printf("Ohjelma laskee syötettyjen arvosanojen keskiarvon.\nLopetus negatiivisella kokonaisluvulla.\n");
while(luku >= 0) {
printf("Anna arvosana (4-10):");
scanf("%d",&luku);
summa = luku;
i ;

}
maara = i - 1;
ka = summa / maara;
printf("Ohjelmaan syötetty %d arvosanaa.\n", maara);
printf("Arvosanojen keskiarvo: %.2d", ka);

return 0;
}
###########

11

518

    Vastaukset

    Anonyymi (Kirjaudu / Rekisteröidy)
    5000
    • eternal_loop

      Tuossa on aika monta virhettä:

      ka pitäisi olla float.
      Summaa ei alustettu nollaksi.
      Muunnos jotain näin: ka = (float) summa / maara;
      Ja silloin %.2d olisi tietysti %.2f

      Lisäksi summaa lasketaan vaikka luku on muuta kuin 4-10 esim -1 tai 3.

      Turha i-muuttuja mielellään pois ja mitä tapahtuu jos kirjoittaa vaikka pari kirjainta... ei kai jää ikuiseen silmukkaan.

    • keijokoodari12

      Kiitos paljon avusta. Sain sen nyt toimimaan niin kuin piti.

      • eternal_loop

        Jos saat mallivastauksen, niin laitapa tänne.


    • sdfsfsfsdfsfsfds

      Tosiaan C:n standardikirjastoa ei todellakana ole suunniteltu konsolikäyttöliittymien luomiseen joten tämän kaltaiset koulutehtävät ovat hieman idioottimaisia.

      Datan syöttäminen pitäisi toteuttaa enemminkin argumentteina.


      Ncurses ja muut curses kirjastot ovat sitten varsinaisten käyttöliittymien tekoon.

      • eternal_loop

        "Datan syöttäminen pitäisi toteuttaa enemminkin argumentteina."

        Nimenomaan ja jos tehtävä vielä olisi: "Tee ohjelma joka laskee argumenttiensa keskiarvon".

        Ratkaisu:

        #include

        int main( int argc, char *argv[] )
        {
            int i, count = 0;
            float num, sum = 0.0;

            for ( i = 1; i < argc; i ) {
                if ( sscanf(argv[i], "%f", &num) == 1 ) {
                   sum = num;
                   count ;
                }
            }

            printf( "Lukuja löytyi %d kpl\n", count );
            printf( "Keskiarvo: %.2f\n", sum / count );

            return 0;
        }

        Ohjelma on näin selkeämpi, näppärämpi käyttää, monipuolisempi ja yleisempi ratkaisu. Ja tietysti bugiton (luultavasti).

        Toimii oikein mm. näillä syötteillä:
        ./keskiarvo 4 5 6
        ./keskiarvo 4.9999 5 jdja 6
        ./keskiarvo `cat keskiarvo.c` #tiedoston syöttö

        Ohoh, tuohan muistuttaa jo oikeaa komentoriviohjelmaa. Ja siihen kun vielä myöhemmin lisää optioina alarajan ja ylärajan niin nuubi-koodari tuntisi että tulipa opittua jotain järkevää.


      • Jepajee...
        eternal_loop kirjoitti:

        "Datan syöttäminen pitäisi toteuttaa enemminkin argumentteina."

        Nimenomaan ja jos tehtävä vielä olisi: "Tee ohjelma joka laskee argumenttiensa keskiarvon".

        Ratkaisu:

        #include

        int main( int argc, char *argv[] )
        {
            int i, count = 0;
            float num, sum = 0.0;

            for ( i = 1; i < argc; i ) {
                if ( sscanf(argv[i], "%f", &num) == 1 ) {
                   sum = num;
                   count ;
                }
            }

            printf( "Lukuja löytyi %d kpl\n", count );
            printf( "Keskiarvo: %.2f\n", sum / count );

            return 0;
        }

        Ohjelma on näin selkeämpi, näppärämpi käyttää, monipuolisempi ja yleisempi ratkaisu. Ja tietysti bugiton (luultavasti).

        Toimii oikein mm. näillä syötteillä:
        ./keskiarvo 4 5 6
        ./keskiarvo 4.9999 5 jdja 6
        ./keskiarvo `cat keskiarvo.c` #tiedoston syöttö

        Ohoh, tuohan muistuttaa jo oikeaa komentoriviohjelmaa. Ja siihen kun vielä myöhemmin lisää optioina alarajan ja ylärajan niin nuubi-koodari tuntisi että tulipa opittua jotain järkevää.

        Todellakin juuri näin. Tämä on oikea ratkaisutapa standardikirjastolla.


    • 23423423

      Tosiaan tämän tehtävän on laatinut joko henkilö joka ei tiedä mitää C-ohjelmoinnista tai sitten henkilö joka on vittuillakseen pistänyt kurssilaiselle moisen sudenkuopan ratkaistavaksi.

      Paras ratkaisu olisi muuttaa koko datan haku systeemi tai sitten käyttää curses kirjastoa.

      • eternal_loop

        "Tosiaan tämän tehtävän on laatinut joko henkilö joka ei tiedä mitää C-ohjelmoinnista tai sitten henkilö joka on vittuillakseen pistänyt kurssilaiselle moisen sudenkuopan ratkaistavaksi."

        Toivokaamme hartaasti jälkimmäistä...


    • 22 22 22 22 22

      tuohan on ihan normaali ohjelmoinnin peruskurssin tehtäviä, joka todennäköisesti tehdään siinä vaiheessa kun opiskellaan while-lausetta.

      • dsfsfsdfs

        Onko oikein vaatia aloittelijalta kiertämään inhottavia ongelmia jotka syntyvät siitä kun käsketään käyttää peruskirjastoa sellaiseen tarkoitukseen mihin sitä ei ole lainkaan suunniteltu.


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

    Luetuimmat keskustelut

    1. Voiko normaali ihminen ryhtyä vasemmistolaiseksi?

      Tätä jäin pohdiskelemaan.
      Maailman menoa
      202
      4208
    2. SDP haluaa 40 000 nettomaahanmuuttajaa

      SDP:n Suunnanmuutos-vaihtoehtobudjetissa, käy ilmi, että demarit itse asiassa vaativat räjähdysmäistä ”työperäisen” maah
      Maailman menoa
      146
      3778
    3. Orpo: Velkajarrua vastustavaa puoluetta vaikea ajatella hallitukseen

      No Minja Koskelan kommunistipuolue jäi ulos tuosta. Kaikki eduskuntapuolueet vasemmistoliittoa lukuun ottamatta sopivat
      Maailman menoa
      84
      3132
    4. Hienoa! Eduskunta luopui käteisen käytöstä

      Nyt tuo sama muutos pitää saada myös muuhun yhteiskuntaan. Käteistähän ei tarvitse tänä päivänä enää kuin rikolliset.
      Maailman menoa
      43
      1579
    5. Ikävä sinua mies

      Vuosia kuluu, mutta tunteet ei ole hävinnyt. Tasoittuneet toki, kun ei olla nähty. Järki palannut päähän kuitenkin. Se i
      Ikävä
      19
      1418
    6. Sulla on avaimet ja keinot

      Jos haluat jatkaa tutustumista. Itse olen niin jäässä etten pysty tekemään enää mitään. Pidempi keppi johon on helpompi
      Ikävä
      25
      1365
    7. Mikä tämä henkilö mahtaa touhuta Parkanossa

      Kamalaa https://www.ylasatakunta.fi/teksti/pirkanmaan-karajaoikeus-vangitsi-koiran-tappamisesta-epaillyn-6.68.127794.b58
      Parkano
      32
      1357
    8. Orpo loukkaantui fasismiin viittaavasta sanavalinnasta

      Mutta miksi loukkaantui? Orpohan on tehnyt yhteistyötä fasistien kanssa jo vuonna 2019, siis jo neljä vuotta ennen loukk
      Maailman menoa
      27
      1341
    9. Kiinnostaa - ei kiinnosta - kiinnostaapas

      Selittäkää hämmentyneelle miksi miehiä ei ikinä kiinnosta silloin, kun sitä olisi itsekin kiinnostunut? Sitten kun siirt
      Sinkut
      116
      1130
    10. Martina haluaa Marbellaan

      Martinan tekisi mieli ottaa lennot Marbellaan, jossa näkisisi kauniita ja hyväntuulisia ihmisiä. No sitten pitää matkust
      Kotimaiset julkkisjuorut
      215
      1053
    Aihe