Hirdetés

Keresés

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

  • válasz hiperFizikus #7510 üzenetére

    Remélem nem programozóként dolgozol főállásban, mert ezt a kódot nem engedném mergelni a master branchbe.

    A program minőségét nem a sorok számában mérik.

    Egy rövidebb program ami ugyanazt csinálja, mint egy hosszú program sokkal jobb, feltéve, hogy a rövidebb jobban olvasható, és nem egy összenyomorított x=z=abc típusú olvashatatlan valami.

    Továbbá abban mérik a minőséget, hogy milyen egyszerű módosítani a kódot.
    A te 5-6 szint mélyen egymásba ágyazott ifeid és ciklusaid pont az ellenkezőjét mutatják.

    Fel kellene darabolnod több függvényre a kódot, hogy könnyebben megérthető legyen mit csinál, könnyebben lehessen irányítani, hogy mi fusson le és hányszor.
    És akkor talán nem kellene random helyekre extra ifeket befűznöd.
    Illetve függvények használatakor jónéhány ciklusiterációt megspórolhatsz a korai visszatérés használatával, ha megvan az eredmény vagy invalid az input.

    A programozás egy elég rugalmas dolog, de vannak alap szabályok amkket a saját érdekedben be kell tartanod.

    Ha 3 hónap múlva ránézel erre a kódra azt sem fogod tudni mit csinál, mert olvashatatlan.

    Nincs olyan, hogyha kedved tartja akkor 5000 sor egy fájl.
    Ha van egy tapasztalt főnököd, és rendszeres code review, akkor ezt nem engedi be a rendszerbe, mert, ha te felmondasz, nincs ember aki ezt tovább viszi, újraírni pedig rengeteg pénz és idő a cégnek.

    Rövid, beszédes nevű függvények, max 20-30 sor, és a kódbázis legyen feldarabolva külön fájlokra, max 150 sor per fájl.
    Ezek a számok nem szentírás, de az 5000 soros fájl az elfogadhatatlan.

    Ha tényleg segítséget kérsz, akkor vegyél vissza ebből a próféta stílusból, mert így nem fogsz messzire jutni.

  • cattus

    addikt

    válasz hiperFizikus #7510 üzenetére

    Elképesztő. Elágazást használtál egy programban. Meghajlok kódoló tudásod előtt.

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