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

499

    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. Mitä yhteistä on sulla ja kaivatulla?

      Onko teillä samantyyppinen olemus tai luonne? Vai muistuttaako vartalonne toisiaan? Tai kasvot? Entä pukeutuminen? Onko
      Ikävä
      99
      1546
    2. Kerran oltiin mekin kahdestaan

      Saanko sanoa, että minua kyllä vähän huvitti jälkeenpäin? Olit kuin vapiseva jänis ketun vieressä valmis loikkimaan paik
      Ikävä
      13
      1216
    3. Ozzy Osbourne on kuollut

      Kevyet mullat ja R.I.P. Ei soi enää Paranoid.
      Maailman menoa
      230
      1200
    4. Persut yritti estää somalinaista väittelemästä lääketieteen tohtoriksi

      Yrittivät saada hänet lähihoitajakouluun, vaikka pärjäsi hyvin koulussa. Nyt hän on yliopistossa väitellyt lääketieteen
      Maailman menoa
      267
      1068
    5. Oikeasti, voitais mennä kahville

      ja jutella niin kuin normi-ihmiset. Kyllä toivon sitä. En pelkää kohdata enkä istua sun kanssa samassa pöydässä. Miten o
      Ikävä
      81
      989
    6. Naiset, naiset

      Älkää antako painonne nousta. Naisella ylipaino ei ole hyvä.💝
      Ikävä
      141
      970
    7. Jos nainen harrastaa seksiä

      Useiden kanssa, miten se eroaa miesten mielestä siitä, jos miehellä on ollut useita s kumppaneita? Oletan että kaikki mi
      Ikävä
      179
      963
    8. Pihlaja-puulle

      Illat on mulle pahimpia. En tiiä miks se olis vaan parasta yöstä toiseen nukkua sun kanssa ja herätä sun kasvoihin. Jos
      Ikävä
      15
      949
    9. No mitä nyt

      Ollaanko vaiko eikö olla? Miehelle.
      Ikävä
      74
      819
    10. Yritin keväällä unohtaa sinut nainen

      mutta siinä kävi niin, että ikävä tuli korkoineen takaisin. Niin että mikä taikavoima sinulla on minuun, sitä joutuu mie
      Ikävä
      33
      812
    Aihe