Hirdetés

Új hozzászólás Aktív témák

  • Benmartin
    senior tag

    Ajjjaj ilyen rossz példával ne kábítsuk szegény tanulókat :P
    Hirtelen ennyi:

    1. te nem a sorok elejéről szedted le az első 4 karaktert
    2. az fscanf visszatérési értékét célszerű ellenőrizni, nem pedig a feof-ot
    3. %s formátummal nem olvasunk fájlneveket (sem) mert megáll a szóközöknél
    4. az fscanf-ben nincs hossz limit
    5. teljesen felesleges betölteni a memóriába a fájlt
    6. strcat-hoz hiányzik a megfelelő #include
    7. malloc-hoz hiányzik a megfelelő #include
    8. a malloc void*-ot ad vissza, csúnyán néznek azokra akik ezt cast-olják
    9. a lefoglalt memóriát nem szabadítod fel
    10. a változók globálisak nem lokálisak
    11. az ftell nem unsigned int-et ad vissza
    12. hibakezelés teljes hiánya

    1.) Tényleg elnéztem.
    2.) Meglehet, gyakorlatban azonban működik.
    3.) Jó, hát általában nem teszünk szóközt a fájlokba. :)
    4.) Azért van a MAX_FAJLNEV_HOSSZ.
    5.) Több megoldás is létezik.
    6.-7.) ANSI C fordítónak nem is kell.
    8.) Csúnyán néznek de működik, mondjuk akár ki is lehet venni.
    9.) Ez jogos.
    10.) Ebbe fölösleges belekötni.
    11.) long-ot, de ebből nem igen lesz probléma.
    12.) Nem is akartam hibakezelést írni. (gondolom magának írja)

Új hozzászólás Aktív témák