{"id":291,"date":"2017-11-27T17:30:45","date_gmt":"2017-11-27T16:30:45","guid":{"rendered":"http:\/\/glancu.com\/blog\/?p=291"},"modified":"2018-06-15T22:04:34","modified_gmt":"2018-06-15T20:04:34","slug":"czym-jest-i-po-co-code-review","status":"publish","type":"post","link":"https:\/\/glancu.com\/blog\/czym-jest-i-po-co-code-review\/","title":{"rendered":"Czym jest i po co code-review?"},"content":{"rendered":"

Cze\u015b\u0107! Dzisiaj opowiem Ci czym jest i do czego s\u0142u\u017cy code-review. Zapraszam!<\/p>\n

<\/p>\n

Czym jest code-review?<\/h3>\n

Zanim zaczn\u0119 opisywa\u0107 po co warto stosowa\u0107 code-review, musisz dowiedzie\u0107 si\u0119, czym on w og\u00f3le jest.<\/p>\n

Wed\u0142ug wikipedii: „Inspekcja kodu”<\/b>\u00a0\u2013 praktyka w dziedzinie\u00a0in\u017cynierii oprogramowania\u00a0maj\u0105ca na celu wykrycie i poprawienie b\u0142\u0119d\u00f3w pope\u0142nionych w kodzie w czasie fazy pisania oprogramowania, a co za tym idzie, poprawienie jako\u015bci tworzonego produktu. Praktyka ta polega na przegl\u0105dzie kodu napisanego przez programist\u0119 przez inn\u0105 osob\u0119 przed w\u0142\u0105czeniem kodu do\u00a0systemu kontroli wersji\u00a0i przekazaniem go do testowania.<\/p>\n

Czyli w skr\u00f3cie, code-review s\u0142u\u017cy do tego, \u017ceby kod, kt\u00f3ry piszemy sprawdzi\u0107\u00a0\u015bwie\u017cym okiem, czy kod nie zawiera b\u0142\u0119d\u00f3w logicznych, jest zbyt nieczytelny, lub w jakim\u015b przypadku mo\u017ce spowodowa\u0107 awari\u0119 (testowanie oprogramowania).<\/p>\n

Dbanie o dobry kod<\/h3>\n

Je\u017celi chcesz by\u0107 dobry\/dobra w programowaniu, to musisz dba\u0107 o sw\u00f3j kod. Je\u017celi nie do ko\u0144ca wiesz, co zrobi\u0107, \u017ceby ten kod by\u0142 lepszy, to (wed\u0142ug mnie) warto po prostu poprosi\u0107 koleg\u0119 z pracy, kt\u00f3ry jest bardziej do\u015bwiadczony. Dzi\u0119ki temu, \u017ce poka\u017cemy kod drugiemu programi\u015bcie mo\u017cemy dowiedzie\u0107 si\u0119, \u017ce dana funkcja jest niepotrzebna, a ifa mo\u017cemy troch\u0119 zmieni\u0107. Co Ci to da? Na pewno b\u0119dziesz bardziej uwa\u017cny\/uwa\u017cna w przysz\u0142o\u015bci na wszystkie funkcje, kt\u00f3re piszesz, oraz dba\u0142 o to, \u017ceby warunek dok\u0142adnie sprawdza\u0142 to, czego oczekujesz.<\/p>\n

G\u0142upie b\u0142\u0119dy<\/h3>\n

Jeste\u015bmy lud\u017ami, tak\u017ce ka\u017cdy mo\u017ce si\u0119 pomyli\u0107. R\u00f3wnie\u017c programi\u015bci. Co konkretnie mam na my\u015bli? Na przyk\u0142ad zamiast ustawienia warto\u015bci true, je\u017celi data publikacji jest wcze\u015bniejsza od aktualnej daty, to ustawimy false. Jaki jest tego efekt? Niezbyt dobry, poniewa\u017c przez to post, kt\u00f3ry ustawi\u0142e\u015b\/ustawi\u0142a\u015b do automatycznej publikacji no niestety, nie ujrzy \u015bwiat\u0142a dziennego.<\/p>\n

Innym przyk\u0142adem mo\u017ce by\u0107 np. z\u0142y typ dla konkretnego pola w bazie danych. Po co mamy zdefiniowa\u0107 typ text, skoro mo\u017cemy zdefiniowa\u0107 typ varchar? Dzi\u0119ki temu oszcz\u0119dzimy troch\u0119 miejsca. Kto\u015b mo\u017ce powiedzie\u0107 „ale to tylko kilka bajt\u00f3w”. Owszem. Dla kilku rekord\u00f3w to faktycznie mo\u017ce by\u0107 znikoma ilo\u015b\u0107, ale co w przypadku, gdy tych rekord\u00f3w b\u0119dzie bardzo du\u017co? Baza danych b\u0119dzie wa\u017cy\u0142a du\u017co wi\u0119cej, przez z\u0142y dob\u00f3r typ\u00f3w.<\/p>\n

Zmienne<\/h3>\n

Je\u017celi programujesz, to wiesz zapewne jak bardzo wa\u017cne jest nazywanie zmiennych. Oczywi\u015bcie to, \u017ce powinny si\u0119 one nazywa\u0107 po angielsku, chyba nie trzeba m\u00f3wi\u0107? \ud83d\ude09 Warto te\u017c nauczy\u0107 si\u0119 dobrze je wykorzystywa\u0107. Na przyk\u0142ad, je\u017celi mamy do wy\u015bwietlenia wszystkie artyku\u0142 to nie napiszemy zmiennej pokazArtykuly, tylko raczej showArticle. Dzi\u0119ki temu b\u0119dzie czytelniej, poniewa\u017c po pierwsze, kod jest po angielsku, wi\u0119c i zmienna taka powinna by\u0107, a po drugie wiemy, \u017ce dostajemy w\u0142a\u015bnie artyku\u0142.<\/p>\n

Komentarze<\/h3>\n

Ja osobi\u015bcie, kiedy\u015b w og\u00f3le nie u\u017cywa\u0142em komentarzy w swoim kodzie. Uwa\u017ca\u0142em, \u017ce jest to bez sensu, zajmuje tylko niepotrzebnie miejsce, no i kod jest mniej czytelny. Bo opr\u00f3cz funkcji, mam tylko jakie\u015b durne komentarze. Generalnie w pracy mo\u017cesz mie\u0107 wym\u00f3g tworzenia komentarzy dla ka\u017cdej metody, klasy, czy zmiennej. S\u0105 oczywi\u015bcie do tego r\u00f3\u017cne „u\u0142atwiacze”, kt\u00f3re zrobi\u0105 to za Ciebie. Wszystko zale\u017cy z jakiego oprogramowania korzystasz \ud83d\ude09<\/p>\n

Testy jednostkowe<\/h3>\n

O testach jednostkowych jeszcze na swoim blogu nie pisa\u0142em, co nie oznacza, \u017ce taki post si\u0119 nie pojawi. Na pewno pr\u0119dzej, czy p\u00f3\u017aniej, zawita on na moim blogu jako osoby post. P\u00f3ki co powiem, \u017ce je\u017celi nie umiesz tworzy\u0107 test\u00f3w jednostkowych to warto si\u0119 tego nauczy\u0107. Tutaj r\u00f3wnie\u017c piszemy kod, i on tak\u017ce mo\u017ce zosta\u0107 oceniony przez drug\u0105 osob\u0119. Dzi\u0119ki temu, by\u0107 mo\u017ce, nauczysz si\u0119 jak lepiej testowa\u0107 sw\u00f3j kod.<\/p>\n

Podsumowanie<\/h3>\n

Jak wida\u0107 wy\u017cej, warto pokaza\u0107 sw\u00f3j kod drugiej osobie. Owszem, b\u0119dziesz nara\u017cony\/nara\u017cona na krytyk\u0119, niekoniecznie pozytywn\u0105, i nie wszystkie mog\u0105 wi\u0105za\u0107 si\u0119 z pozytywn\u0105 odpowiedzi\u0105. Ale dzi\u0119ki temu jeste\u015b w stanie nauczy\u0107 si\u0119, jak pisa\u0107 kod lepiej, i dlaczego dan\u0105 funkcj\u0119 warto przenie\u015b\u0107 w inne miejsce.<\/p>\n

To ju\u017c wszystko, co przygotowa\u0142em na dzi\u015b. Mam nadziej\u0119, \u017ce dzi\u0119ki temu, nie b\u0119dziesz ba\u0142(a) si\u0119 pokaza\u0107 swojego kodu innym. Standardowo, zapraszam Ci\u0119 na moje socialmedia, dzi\u0119ki kt\u00f3rym b\u0119dziesz informowany\/a o nowych postach. Cze\u015b\u0107!<\/p>\n<\/div>","protected":false},"excerpt":{"rendered":"

Cze\u015b\u0107! Dzisiaj opowiem Ci czym jest i do czego s\u0142u\u017cy code-review. Zapraszam!<\/div>\n","protected":false},"author":31241,"featured_media":294,"comment_status":"open","ping_status":"open","sticky":false,"template":"","format":"standard","meta":{"_mi_skip_tracking":false,"_exactmetrics_sitenote_active":false,"_exactmetrics_sitenote_note":"","_exactmetrics_sitenote_category":0,"footnotes":""},"categories":[261],"tags":[54,117,116,84,88,60,53,118,115],"class_list":["post-291","post","type-post","status-publish","format-standard","has-post-thumbnail","hentry","category-web","tag-co","tag-code","tag-code-review","tag-czym","tag-i","tag-jest","tag-po","tag-review","tag-stosowac"],"better_featured_image":{"id":294,"alt_text":"","caption":"","description":"","media_type":"image","media_details":{"width":1920,"height":1032,"file":"2017\/10\/pexels-photo-270348.jpg","sizes":{"thumbnail":{"file":"pexels-photo-270348-440x290.jpg","width":440,"height":290,"mime-type":"image\/jpeg","source_url":"http:\/\/glancu.com\/blog\/wp-content\/uploads\/2017\/10\/pexels-photo-270348-440x290.jpg"},"medium":{"file":"pexels-photo-270348-300x161.jpg","width":300,"height":161,"mime-type":"image\/jpeg","source_url":"http:\/\/glancu.com\/blog\/wp-content\/uploads\/2017\/10\/pexels-photo-270348-300x161.jpg"},"medium_large":{"file":"pexels-photo-270348-768x413.jpg","width":768,"height":413,"mime-type":"image\/jpeg","source_url":"http:\/\/glancu.com\/blog\/wp-content\/uploads\/2017\/10\/pexels-photo-270348-768x413.jpg"},"large":{"file":"pexels-photo-270348-1100x591.jpg","width":1100,"height":591,"mime-type":"image\/jpeg","source_url":"http:\/\/glancu.com\/blog\/wp-content\/uploads\/2017\/10\/pexels-photo-270348-1100x591.jpg"},"gazana_hd":{"file":"pexels-photo-270348-1920x1032.jpg","width":1920,"height":1032,"mime-type":"image\/jpeg","source_url":"http:\/\/glancu.com\/blog\/wp-content\/uploads\/2017\/10\/pexels-photo-270348-1920x1032.jpg"},"gazana_standard":{"file":"pexels-photo-270348-1100x640.jpg","width":1100,"height":640,"mime-type":"image\/jpeg","source_url":"http:\/\/glancu.com\/blog\/wp-content\/uploads\/2017\/10\/pexels-photo-270348-1100x640.jpg"},"gazana_square":{"file":"pexels-photo-270348-1100x1032.jpg","width":1100,"height":1032,"mime-type":"image\/jpeg","source_url":"http:\/\/glancu.com\/blog\/wp-content\/uploads\/2017\/10\/pexels-photo-270348-1100x1032.jpg"},"gazana_grid":{"file":"pexels-photo-270348-530x380.jpg","width":530,"height":380,"mime-type":"image\/jpeg","source_url":"http:\/\/glancu.com\/blog\/wp-content\/uploads\/2017\/10\/pexels-photo-270348-530x380.jpg"},"gazana_list":{"file":"pexels-photo-270348-530x640.jpg","width":530,"height":640,"mime-type":"image\/jpeg","source_url":"http:\/\/glancu.com\/blog\/wp-content\/uploads\/2017\/10\/pexels-photo-270348-530x640.jpg"},"gazana_masonry":{"file":"pexels-photo-270348-530x285.jpg","width":530,"height":285,"mime-type":"image\/jpeg","source_url":"http:\/\/glancu.com\/blog\/wp-content\/uploads\/2017\/10\/pexels-photo-270348-530x285.jpg"},"gazana_mini":{"file":"pexels-photo-270348-100x100.jpg","width":100,"height":100,"mime-type":"image\/jpeg","source_url":"http:\/\/glancu.com\/blog\/wp-content\/uploads\/2017\/10\/pexels-photo-270348-100x100.jpg"}},"image_meta":{"aperture":"0","credit":"","camera":"","caption":"","created_timestamp":"0","copyright":"","focal_length":"0","iso":"0","shutter_speed":"0","title":"","orientation":"1","keywords":[]}},"post":291,"source_url":"http:\/\/glancu.com\/blog\/wp-content\/uploads\/2017\/10\/pexels-photo-270348.jpg"},"aioseo_notices":[],"aioseo_head":"\n\t\t\n\t\n\t\n\t\n\t\n\t\n\t\t\n\t\t\n\t\t\n\t\t\n\t\t\n\t\t\n\t\t\n\t\t\n\t\t\n\t\t\n\t\t\n\t\t