onko tässä jotain väärin

tai "epämääräisesti"

#include
#include

int main(void)
{

char etunimi[15]; // etunimi voi olla 15 merkkiä
char sukunimi[20]; // sukunimi 20 merkkiä
char tiedosto_nimi[12];

printf("Ohjelma tallentaa etunimesi ja sukunimesi tiedostoon.\n");
printf("Syötä etunimesi:");
scanf("%s",&etunimi[0]);
printf("Syötä sukunimesi:");
scanf("%s",&sukunimi[0]);
printf("Tiedosto johon haluat tallentaa nimesi:");
scanf("%s",&tiedosto_nimi[0]);


FILE *avaus;
char *tiedoston_nimi = tiedosto_nimi;
char *avaus_moodi = "w";
avaus = fopen(tiedoston_nimi, avaus_moodi);

if(avaus == NULL)
{
printf("Tiedoston avauksessa on tapahtunut virhe!");

return 0;
}
else
{
fprintf(avaus, "%s %s", etunimi, sukunimi);
fclose(avaus);
printf("\nTallennus onnistui!");    
}

getche();
return 0;

}

11

915

    Vastaukset

    Anonyymi (Kirjaudu / Rekisteröidy)
    5000
    • dx2

      Voisi ainakin kääntäjä tykätä parempaa jos poistat tuon rivin:

      char *tiedoston_nimi = tiedosto_nimi;

      koska ei ole hyvä idea määritellä montaa samannimistä muuttujaa. Ja jos käytät ANSI-C-kääntäjää, niin joudut myös siirtämään nämä rivit heti tuon tiedosto_nimi[12]-rivin jälkeen:

      FILE *avaus;
      char *avaus_moodi = "w";

      Itse myös poistaisin tuon rivin kun se ei ole ANSI-C:tä:

      getche();

      ja samalla myös sen -rivin. Joskin kyllä tuo getche() voi auttaa tulostuksen näkemistä joissain epämääräisissä kääntäjissä, jossa konsoli-ikkuna sulkeutuu heti ohjelman päättymisen jälkeen.

      • käytössä

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

        Voishan siihen tietenkin vissiin laittaa pause tai jotain muuta.


      • jatkoa

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


      • voi...
        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ä?

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


      • jossain muussa ehkä tapahtuu
        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ä?

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


      • TRRY
        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.


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

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


      • pitääpä pähkäillä ja kokeilla
        ...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.)

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

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

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


      • juuu...
        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"?

        "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ä
        ..vielä kirjoitti:

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

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

        Ööö... 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


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

    Luetuimmat keskustelut

    1. YLE Äänekosken kaupunginjohtaja saa ankaraa arvostelua

      Kaupungin johtaja saa ankaraa kritiikkiä äkkiväärästä henkilöstöjohtamisestaan. Uusin häirintäilmoitus päivätty 15 kesä
      Äänekoski
      83
      1611
    2. Euroopan lämpöennätys, 48,8, astetta, on mitattu Italian Sisiliassa

      Joko hitaampikin ymmärtää. Se on aivan liikaa. Ilmastonmuutos on totta Euroopassakin.
      Maailman menoa
      269
      1531
    3. Asiakas iski kaupassa varastelua tehneen kanveesiin.

      https://www.iltalehti.fi/kotimaa/a/33a85463-e4d5-45ed-8014-db51fe8079ec Oikein. Näin sitä pitää. Kyllä kaupoissa valtava
      Maailman menoa
      268
      1263
    4. Martina lähdössä Ibizalle

      Eikä Eskokaan tiennyt matkasta. Nyt ollaan jännän äärellä.
      Kotimaiset julkkisjuorut
      169
      1252
    5. Avustikset peruttu.

      Aettokosken ampuraan rahat otettu poekkeen valtiolle.
      Suomussalmi
      56
      857
    6. 66
      844
    7. Jos ei tiedä mitä toisesta haluaa

      Älä missään nimessä anna mitään merkkejä kiinnostuksesta. Ole haluamatta mitään. Täytyy ajatella toistakin. Ei kukaan em
      Ikävä
      64
      837
    8. Miksi mies tuntee näin?

      Eli olen mies ja ihastuin naiseen. Tykkään hänestä ja koskaan hän ei ole ollut minulle ilkeä. Silti ajoittain tunnen kui
      Ikävä
      40
      821
    9. Määpä tiijän että rakastat

      Minua nimittäin. Samoin hei! Olet mun vastakappaleeni.
      Ikävä
      40
      807
    10. Se nainen näyttää hyvältä vaikka painaisi 150kg

      parempi vaan jos on vähän muhkeammassa kunnossa 🤤
      Ikävä
      44
      781
    Aihe