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

• instantwater addikt

  #7519

  #7510 hiperFizikus

  2019-10-28 08:44:40

  Új Válasz

  Ú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

  

  instantwater

  addikt

  #7510 hiperFizikus

  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.

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