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

520

    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. SDP jo 100 % suositumpi kuin persut

      Kertoo Hesarin uusin kannatuskysely. Demareiden kannatus on miltei tuplat verrattuna persuihinl. Suomen kansa ei selväst
      Maailman menoa
      287
      14608
    2. Valtavasti suomalaisia asunnottomina, mutta ei yhtään somalia

      tai muuta kehitysmaalaista. Mites tässä näin kävi? Tiedän hyvin, että esim. somaleita lentää ulos mm. Hekan asunnoista,
      Maailman menoa
      156
      4102
    3. PS ylivoimainen nousija myös HS:n gallupissa, SDP laskee taas

      https://www.verkkouutiset.fi/a/hs-gallup-sdpn-suosio-laskee-ps-nousussa/#0a7d2507 Ylivoimainen viime kuukausien nousija
      Maailman menoa
      132
      3846
    4. Totuuspuolueen johtaja Jaana "prinsessa Leia" Kavonius on vangittu

      Kavonius määrättiin jo keväällä 2024 poissaolevana vangittavaksi todennäköisin syin epäiltynä 13 vainoamisesta ja 24 kun
      Maailman menoa
      434
      1784
    5. Kovia syytöksiä Stefan Thermaninsta.

      Jättänyt taas maksamatta kohuliikemies, hupparit ja muita ostamiaan tavaroita. On soiteltu liikkeestä ja Stefan iskenyt
      Kotimaiset julkkisjuorut
      171
      1757
    6. En minä kyllä enää odota sinua

      Olet siellä sen harmaan kanssa. Niin, annoit minun nähdä lämpimät tunteesi siitä huolimatta. Se merkitsi kyllä paljon. O
      Ikävä
      35
      1689
    7. Jos joku luulee että kaikki käy

      Sanon vain tämän. Minun kanssani ei neuvotella. Minun kanssani eletään tasavertaisesti. Jos se on liikaa, niin ovi rinn
      Ikävä
      49
      1356
    8. Sakin hivutus - ilmiö

      Miten tuollainen tuollainen ilmiö kuin ”sakin hivutus” syntyy? Mitä syitä ilmiön syntymiseen tarvitaan? Onko sakissa jok
      80 plus
      66
      1132
    9. Joka kolmas työtön on työkyvytön

      Viime vuonna työnhakuvelvoitteen ulkopuolella oli noin kolmannes työttömistä työnhakijoista. Huhheijaa, mihin suomalais
      Maailman menoa
      125
      1127
    10. Ei ole rohkeutta tulla jututtamaan

      Voidaan me nähdä ja tervehtiä, sitäkin harvoin, mutta iso kynnys on edes mennä lähelle ja kysyä kuulumisia. Ymmärrät var
      Tunteet
      8
      1121
    Aihe