2023-08-14 Alarm Code Review

 

⚠️ Šīs nav vienīgās vietas kodā!!! SALABO VISAS, kuras tu esi kodējis un arī, ja redzi citu pieļautas kļūdas, šī ir jau 3 reize, tas nav normāli, ka tu nelasi un neizdari code reviews!

 

  1. Aizliegts rakstīt inline vairāk kā vienu check, ja ir vairāk kā viens, jāraksta temp mainīgie ar saprotamiem nosakumiem - SALABOT VISĀS vietās tavās komponentēs

image-20230814224843045

Salabot un lai šādi vairs nebūtu nekur!!!!!

image-20230814225142803

 

  1. Vēljoprojām nav izdarīts, ko jau prasu no tevis 2 pēdējos code reviews - obligāti salabo styles! aizliegts kopēt styles, itsevišķi font families, font sizes, colors utt. Jāizmanto kopējie stili ar jēgpilniem nosaukumiem!

image-20230814224957827

 

  1. Ar apakašvītru apzīmē PRIVATE mainīgos nevis scope mainīgos, pie tam DOES NOT MAKE SENSE šeit map funkcija aizvākt to lūdzu

image-20230814225333028

image-20230814225350353

 

  1. Nebija notestēts kods, editing existing mission - nedrīkst teikt, ka kaut kas ir pabeigts, kad esi tikai vienreiz notestējis - uzreiz kā palaižu nekas nestrādā kā vajag, lai tā nenotiktu reset simulator un pārbaudi ar tukšiem datiem, pārbaudi pēc tam ar iepriekšējiem datiem pa jaunu atverot image-20230814225546508 image-20230814225737931

 

  1. Atkal trešo reizi! NEDRIKST KOMENTĒT kodu, lai tā vairs nekur nebūtu

  2. image-20230814231816086image-20230814225955492

image-20230814230240474

image-20230814230558349

  1. Attālumi starp pogām neatbilst dizainam image-20230814230352627

 

  1. Sakārtot haotiskus zIndex - šādi nedrīkst bez nekādas loģikas palielināt random vērtības image-20230814230515881

 

  1. AZILIEGTS atstāt neizmantotu kodu. IZMANTO WebStorm “Inspect Code” funkciju - OBLIGATI un salabo bugs!!! image-20230814230715420

 

  1. Kāpēc ir square brackets - does not make SENSE. Iemācies lūdzu ko tās nozīmē image-20230814230822622

image-20230814231110155

 

  1. Sakārtot visur kodā Tabs!!! image-20230814231043576

 

  1. Sakārtot Views šobrīd BEZJĒDZIGI Daudz views image-20230814231252563

 

  1. AZILIEGTS MILZĪGUS inline IF rakstīt - šeit totāli nelasāms ko nozīmē šis else, kas atrodas pa vidu JSX failā - sadalīt pa mainīgajiem un funkcijām image-20230814231358366

 

  1. Es jau brīdināju tevi šis nav VIEW code - izlasi velreiz kā strādā MODEL VIEW CONTROLER, nevar šādu Controller / model code iekodē UI!! image-20230814231556491

 

  1. Pārtaisīt bez map, parastā funkciju kodā nevis JSX lūdzu pēc iespējas MAZAK izmantot map ar callback!!! image-20230814231751182

 

  1. Viss jābūt camelCase image-20230814232138123

 

  1. Pārāk ilgi atrast kur ir otrā puse šim macaroni, rezultātā nevaru salabot bugs image-20230814232953351