2. Fórumok
  3. Szoftverfejlesztés
  4. C++ programozás

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

  • #1197 Jester01 veterán Brianiac #1196
    Új Válasz #1197
    Új hozzászólás
    Összes hozzászólása itt Válaszok az összes hozzászólására itt Válaszok erre a hozzászólásra
    Privát üzenet küldése

    Jester01

    veterán

    válasz Brianiac #1196 üzenetére

    A lényegi kérdéshez nem tudok érdemben hozzászólni, de azért a kódhoz van észrevételem:

    1) system("pause") ez a "kedvencünk" itt a fórumon, ezért azonnali főbelövés jár. Ráadásul te még a szükséges headert sem hívtad be, így le sem fordul.
    2) setlocale (LC_ALL, "Hun"); ez nem tudom milyen rendszeren működik, de a szabályos locale nevek leginkább hu_HU és hasonlók. Persze visszatérési érték ellenőrzése nélkül ezt nem veszed észre. Egyébként is illik inkább a felhasználói beállításokat átvenni, továbbá célszerű akkor már wide karakteret használni.
    3) Ha már iostream-et használsz, akkor a \n csúnya (persze azért működik). Helyette ajánlott az endl manipulátor.
    4) A pont és kör középpont távolságnégyzetet illene kitenni egy változóba (ahogy a sugárnégyzetet is). A fordító persze úgyis kioptimalizálja, de ez így nem annyira olvasható.
    5) hibakezelés nincs, a kiírt szöveg pedig megtévesztő az elvárt formátumot illetően
    6) (ez nagyon halvány észrevétel) logikátlan a kör sugarát a végén bekérni, érdemesebb együtt tartani a kör jellemzőit
    7) plusz pont lehetne, ha ráutaló magatartással jeleznéd, hogy ismered a lebegőpontos számítások veszélyét (egyenlőséget általában nem szabad ellenőrizni)
    8) a cmath-t fölöslegesen hívod be, nem használsz belőle semmit.
    9) formailag személy szerint hiányolok néhány szóközt innen-onnan, az olvashatóság jegyében

    Négyzetre emelni leginkább úgy célszerű ahogy csináltad (azaz szorzással), legfeljebb csinálhatsz neki külön függvényt.

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