C alkeet ongelma while lauseen kanssa

No joo, tuo ikuinen luuppi ongelma on liian vaikea noin alkuvaiheessa.

Syy on scanf, joḱa ei tyhjennä stdin-virtaa:

http://c-faq.com/stdio/scanfjam.html

Itse asiassa scanf:ää ei pitäisi koskaan käyttää syötteen lukuun:

http://c-faq.com/stdio/scanfprobs.html

Ja fgets-ratkaisukin on huono:

http://www.delorie.com/gnu/docs/glibc/libc_185.html

Jos saat mallivastauksen, niin laitapa tänne.

"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ää.

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ää.

Lue lisää

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

"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ä...

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.