onko tässä jotain väärin

niin pakko jollain pysäyttää, siksi tuo getche().

Voishan siihen tietenkin vissiin laittaa pause tai jotain muuta.

char *tiedoston_nimi = tiedosto_nimi;

ei ole saman niminen muuttuja (?)

Voiko tuon tiedoston nimen (char *tiedoston_nimi)
viedä suoraan tuona tiedosto_nimi -muuttujana, jonka ohjelma kysyy käyttäjältä?

jatkoa kirjoitti:

char *tiedoston_nimi = tiedosto_nimi;

ei ole saman niminen muuttuja (?)

Voiko tuon tiedoston nimen (char *tiedoston_nimi)
viedä suoraan tuona tiedosto_nimi -muuttujana, jonka ohjelma kysyy käyttäjältä?

Lue lisää

Voi:
avaus = fopen(tiedosto_nimi, "w");

Koska taulukon nimi toimii myös osoittimena. Samasta syystä myös tämä käy:
scanf("%s", etunimi);

Tuolta löytyy asiaan jotain selitystä:
http://www.nic.funet.fi/c opas/osoittimet.html

Osoittimia pitää C:ssä tankata niin kauan että ne tosiaan osaa ja tajuu.

Mitäs muuten tapahtuu kun syöttää etunimeen yli 15 merkkiä?

voi... kirjoitti:

Voi:
avaus = fopen(tiedosto_nimi, "w");

Koska taulukon nimi toimii myös osoittimena. Samasta syystä myös tämä käy:
scanf("%s", etunimi);

Tuolta löytyy asiaan jotain selitystä:
http://www.nic.funet.fi/c opas/osoittimet.html

Osoittimia pitää C:ssä tankata niin kauan että ne tosiaan osaa ja tajuu.

Mitäs muuten tapahtuu kun syöttää etunimeen yli 15 merkkiä?

Lue lisää

Ei tapahdu mitään Dev-Cpp:ssä vaan ottaa ne ylimääräisetkin merkit talteen, kun kokeilin.

Joku toinen IDE ei välttämättä tykkää.

jossain muussa ehkä tapahtuu kirjoitti:

Ei tapahdu mitään Dev-Cpp:ssä vaan ottaa ne ylimääräisetkin merkit talteen, kun kokeilin.

Joku toinen IDE ei välttämättä tykkää.

Ensimmäinen kuolemansynti on scanf("%s", ...). scanf:lle pitää kertoa, kuinka monta merkkiä sille annetun osoittimen päässä on muistia käytettävissä tallennettavalle merkkijonolle. Jos tätä ei kerrota, data tallennetaan mahdollisesti muuhun käyttöön tarkoitetulle muistialueelle, ja seuraukset ovat arvaamattomat.

Toisin sanoen rivin "scanf("%s",&etunimi[0]);" pitäisi olla "scanf("%14s", etunimi);" (14 siksi, että 15. merkki pitää jättää merkkijonon lopputavulle.)

Toinen kuoleman synti, on scanf:n paluuarvojen tarkastamatta jättäminen. scanf palauttaa EOFin, jos syötettä ei saada luettua. Jos suoritat ohjelman komennolla

echo "Erkka" | ohjelma.exe

niin se yrittää tyytyväisenä avata määrittelemättömän nimistä tiedostoa (joskus saattaa onnistuakin!) tallentaakseen sinne Erkka-etunimen ja määrittelemättömän sukunimen.

Kolmas synti (ei ehkä enää kuoleman sellainen) on kirjoitettavan tiedoston nimen tarkastamatta jättäminen. Turvallisuussyistä olisi paikallaan varmistaa, onko annettu tiedosto olemassa. Käyttäjä voi "vahingossa" ylikirjoittaa omia tai jopa järjestelmän tiedostoja. Myös varatut tiedostonimet kuten Windowsissa CON, PRN, COM1 yms. voivat tuottaa yllättäviä tuloksia. *nix-ympäristössä puolestaan /dev/x:t ovat myös jännittäviä tiedostoja, ja esim. pipe-tiedostoon kirjoittaminen pysäyttää ohjelman kunnes pipe luetaan (tämä voi olla haluttuakin).

Neljäs synti on tulostaa "Tallennus onnistui" vaikkei asia olisikaan niin. Tämä taas johtuu siitä, ettei fprintf:nkään paluuarvoa tarkasteta. Tiedostoon kirjoittaminen ei välttämättä onnistu, vaikka se olisikin pystytty avaamaan write-tilaan. Esimerkiksi levyssä voi olla vikaa tai se voi olla täysi.

Näiden syntien lisäksi siistimpää (niponipo) olisi käyttää stdlibin getcharia ja return EXIT_SUCCESS:ia kuin jonkin epästandardin conio:n getche:a ja return 0:aa.

TRRY kirjoitti:

Ensimmäinen kuolemansynti on scanf("%s", ...). scanf:lle pitää kertoa, kuinka monta merkkiä sille annetun osoittimen päässä on muistia käytettävissä tallennettavalle merkkijonolle. Jos tätä ei kerrota, data tallennetaan mahdollisesti muuhun käyttöön tarkoitetulle muistialueelle, ja seuraukset ovat arvaamattomat.

Toisin sanoen rivin "scanf("%s",&etunimi[0]);" pitäisi olla "scanf("%14s", etunimi);" (14 siksi, että 15. merkki pitää jättää merkkijonon lopputavulle.)

Toinen kuoleman synti, on scanf:n paluuarvojen tarkastamatta jättäminen. scanf palauttaa EOFin, jos syötettä ei saada luettua. Jos suoritat ohjelman komennolla

echo "Erkka" | ohjelma.exe

niin se yrittää tyytyväisenä avata määrittelemättömän nimistä tiedostoa (joskus saattaa onnistuakin!) tallentaakseen sinne Erkka-etunimen ja määrittelemättömän sukunimen.

Kolmas synti (ei ehkä enää kuoleman sellainen) on kirjoitettavan tiedoston nimen tarkastamatta jättäminen. Turvallisuussyistä olisi paikallaan varmistaa, onko annettu tiedosto olemassa. Käyttäjä voi "vahingossa" ylikirjoittaa omia tai jopa järjestelmän tiedostoja. Myös varatut tiedostonimet kuten Windowsissa CON, PRN, COM1 yms. voivat tuottaa yllättäviä tuloksia. *nix-ympäristössä puolestaan /dev/x:t ovat myös jännittäviä tiedostoja, ja esim. pipe-tiedostoon kirjoittaminen pysäyttää ohjelman kunnes pipe luetaan (tämä voi olla haluttuakin).

Neljäs synti on tulostaa "Tallennus onnistui" vaikkei asia olisikaan niin. Tämä taas johtuu siitä, ettei fprintf:nkään paluuarvoa tarkasteta. Tiedostoon kirjoittaminen ei välttämättä onnistu, vaikka se olisikin pystytty avaamaan write-tilaan. Esimerkiksi levyssä voi olla vikaa tai se voi olla täysi.

Näiden syntien lisäksi siistimpää (niponipo) olisi käyttää stdlibin getcharia ja return EXIT_SUCCESS:ia kuin jonkin epästandardin conio:n getche:a ja return 0:aa.

Lue lisää

Vielä ainakin yksi ongelma eli stdin-puskuriin jää merkkejä jos niitä syötetään enemmän kuin scanf hakee, joten se pitää tyhjentää (ehkäpä) näin:

while((c = getchar()) != '\n' && c != EOF);

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

Koska scanf:n käyttö menee näin monimutkaiseksi ja riskaabeliksi, niin järkevää olisi kai käyttää valmista kirjastoa apuna. Jotain tollaisia:

http://tiswww.case.edu/php/chet/readline/rltop.html
http://www.astro.caltech.edu/~mcs/tecla/index.html
http://www.thrysoee.dk/editline/

Samalla saa mukaan muita 'herkkuja'.

Scanf:stä tietoa:
http://en.wikipedia.org/wiki/Scanf

(hmm.. wikipedian mukaan GNU readline on buginen ja rajoittunut.)

...vielä kirjoitti:

Vielä ainakin yksi ongelma eli stdin-puskuriin jää merkkejä jos niitä syötetään enemmän kuin scanf hakee, joten se pitää tyhjentää (ehkäpä) näin:

while((c = getchar()) != '\n' && c != EOF);

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

Koska scanf:n käyttö menee näin monimutkaiseksi ja riskaabeliksi, niin järkevää olisi kai käyttää valmista kirjastoa apuna. Jotain tollaisia:

http://tiswww.case.edu/php/chet/readline/rltop.html
http://www.astro.caltech.edu/~mcs/tecla/index.html
http://www.thrysoee.dk/editline/

Samalla saa mukaan muita 'herkkuja'.

Scanf:stä tietoa:
http://en.wikipedia.org/wiki/Scanf

(hmm.. wikipedian mukaan GNU readline on buginen ja rajoittunut.)

Lue lisää

Ok, pitääpä pähkäillä noita kommentteja ja neuvoja. Tosiaan vielä perusteita opiskelemassa niin ei ihan vielä kaikki hallinnassa. Tosin aina välillä tulee niitä hetkiä joilloin itsekin tajuaa " entäs jos..." ja kun ei tiedä oikein noita mahdollisuuksia niin jää tekemättä. Eikä kyllä aikakaan riittäisi pelkästään C:n harjoituksille, valitettavasti.

Hitto kun olisi sellainen opettaja/kurssi, jossa käytäisiin just näin läpi tehtäviä. Oppilaat kirjoittaisivat koodin, sitten niistä käytäisiin läpi selkeitä virheitä, arveluttavia toteutustapoja tyyliin "mitä tapahtuu" jos. Sitten noiden korjaamisen tapooihin annettaisiin vähän vinkkiä.

Onko muuten olemassa kirjaa, jossa olisi esitelty tyyliin "EI MISSÄÄN TAPAUKSESSA NÄIN"?

...vielä kirjoitti:

Vielä ainakin yksi ongelma eli stdin-puskuriin jää merkkejä jos niitä syötetään enemmän kuin scanf hakee, joten se pitää tyhjentää (ehkäpä) näin:

while((c = getchar()) != '\n' && c != EOF);

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

Koska scanf:n käyttö menee näin monimutkaiseksi ja riskaabeliksi, niin järkevää olisi kai käyttää valmista kirjastoa apuna. Jotain tollaisia:

http://tiswww.case.edu/php/chet/readline/rltop.html
http://www.astro.caltech.edu/~mcs/tecla/index.html
http://www.thrysoee.dk/editline/

Samalla saa mukaan muita 'herkkuja'.

Scanf:stä tietoa:
http://en.wikipedia.org/wiki/Scanf

(hmm.. wikipedian mukaan GNU readline on buginen ja rajoittunut.)

Lue lisää

On tietysti vielä tämä:
http://en.wikipedia.org/wiki/Fgets

Joka tapauksessa scanf:ää EI KANNATA KÄYTTÄÄ stdin-virran lukemiseen.

pitääpä pähkäillä ja kokeilla kirjoitti:

Ok, pitääpä pähkäillä noita kommentteja ja neuvoja. Tosiaan vielä perusteita opiskelemassa niin ei ihan vielä kaikki hallinnassa. Tosin aina välillä tulee niitä hetkiä joilloin itsekin tajuaa " entäs jos..." ja kun ei tiedä oikein noita mahdollisuuksia niin jää tekemättä. Eikä kyllä aikakaan riittäisi pelkästään C:n harjoituksille, valitettavasti.

Hitto kun olisi sellainen opettaja/kurssi, jossa käytäisiin just näin läpi tehtäviä. Oppilaat kirjoittaisivat koodin, sitten niistä käytäisiin läpi selkeitä virheitä, arveluttavia toteutustapoja tyyliin "mitä tapahtuu" jos. Sitten noiden korjaamisen tapooihin annettaisiin vähän vinkkiä.

Onko muuten olemassa kirjaa, jossa olisi esitelty tyyliin "EI MISSÄÄN TAPAUKSESSA NÄIN"?

Lue lisää

"Onko muuten olemassa kirjaa, jossa olisi esitelty tyyliin "EI MISSÄÄN TAPAUKSESSA NÄIN"?"

En tiedä kirjasta mutta C FAQ:
http://c-faq.com/index.html

Paras kirja suomeksi lienee: C ohjelmointi Kernighan, Brian W. ; Ritchie, Dennis M.

..vielä kirjoitti:

On tietysti vielä tämä:
http://en.wikipedia.org/wiki/Fgets

Joka tapauksessa scanf:ää EI KANNATA KÄYTTÄÄ stdin-virran lukemiseen.

Lue lisää

Ööö... komento takaisin. Eihän tossa fgets:ssä ole mitään käytännön eroa tuohon scanf("s", str); Puskuriin jää kamaa molemmissa.

Jos tota aiemmin mainittua readline-juttua ei voi käyttää niin on getline (Dev-Cpp kai mukana):
http://crasseux.com/books/ctutorial/getline.html

Myös numerot pitää käpistellä getline/readline sscanf:
http://crasseux.com/books/ctutorial/sscanf.html