{"id":380,"date":"2020-05-20T12:51:49","date_gmt":"2020-05-20T10:51:49","guid":{"rendered":"https:\/\/arasoftware.pl\/?p=380"},"modified":"2021-10-21T10:34:21","modified_gmt":"2021-10-21T08:34:21","slug":"5-bledow-podczas-code-review","status":"publish","type":"post","link":"http:\/\/arasoftware.pl\/en\/5-bledow-podczas-code-review\/","title":{"rendered":"5 B\u0142\u0119d\u00f3w Podczas Code Review"},"content":{"rendered":"<span class=\"rt-reading-time\" style=\"display: block;\"><span class=\"rt-label rt-prefix\">Czas czytania: <\/span> <span class=\"rt-time\">5<\/span> <span class=\"rt-label rt-postfix\">minut<\/span><\/span><p class=\"wp-block-paragraph\">Code review jest bardzo wa&#380;ne. Podnosi jako&#347;&#263; kodu, poprawia komunikacj&#281; i rozpowszechnia wiedz&#281; o projekcie w zespole. S&#261; to tylko nieliczne warto&#347;ci dodane, a jest ich znacznie wi&#281;cej. Dobry proces sprawdzania kodu przed commit&rsquo;em, ma te&#380; warto&#347;&#263; edukacyjn&#261;. Osoba sprawdzaj&#261;ca kod (reviewer) z zasady powinna posiada&#263; co najmniej takie samo albo i wi&#281;ksze do&#347;wiadczenie ni&#380; osoba, kt&oacute;ra ten kod udost&#281;pnia do weryfikacji. Co za tym idzie, istnieje prawdopodobie&#324;stwo, &#380;e reviewer ma wi&#281;ksz&#261; wiedz&#281; i mo&#380;e si&#281; ni&#261; podzieli&#263; z autorem kodu podczas code review.<\/p>\n\n\n<p class=\"wp-block-paragraph\">Z pozoru wydaje si&#281;, &#380;e jest to trywialny proces. Programista pisz&#281; kod, udost&#281;pnia go innemu programi&#347;cie do sprawdzenia, ten nanosi swoje uwagi, poprawka kodu i commit! Czy na pewno jest to takie proste? Czy zastanawia&#322;e&#347;(a&#347;) si&#281; kiedy&#347;, dlaczego code review w Twojej firmie jest traktowane po macoszemu? Dlaczego trwa d&#322;ugo? Albo czemu pomimo praktykowania tego procesu jako&#347;&#263; kodu nie wzrasta a ilo&#347;&#263; bug&oacute;w nie maleje?<\/p>\n\n\n<p class=\"wp-block-paragraph\">Oto 5 cz&#281;sto pope&#322;nianych b&#322;&#281;d&oacute;w przez programist&oacute;w podczas procesu code review. Zobacz jak mo&#380;esz znacz&#261;co skr&oacute;ci&#263; czas ca&#322;ego procesu i jak uchroni&#263; si&#281; przed bezustannym poprawianiem tego samego kodu.<\/p>\n\n\n<h2 class=\"wp-block-heading wp-block-heading\" id=\"1-brak-przedstawienia-problemu-wymagac584-biznesowych\">1. Brak przedstawienia problemu, wymaga&#324; biznesowych<\/h2>\n\n\n<p class=\"wp-block-paragraph\">Niewiele os&oacute;b dostrzega ten problem podczas code review. Cz&#281;sto ca&#322;y proces weryfikacji zmian polega na przejrzeniu kodu i je&#347;li nie ma tam nic niepokoj&#261;cego, to reviewer daje zielone &#347;wiat&#322;o autorowi. Niestety cz&#281;sto zdarza si&#281; tak, &#380;e po, mimo i&#380; kod nie posiada ra&#380;&#261;cych b&#322;&#281;d&oacute;w, kt&oacute;re s&#261; widoczne na pierwszy rzut oka, to nie zawsze jest w stanie realizowa&#263; postawione przed nim wymagania biznesowe. Przedstawienie problemu, kt&oacute;ry ma by&#263; rozwi&#261;zany poprzez konkretn&#261; implementacj&#281;, jest kluczowe w tym procesie. Reviewer dzi&#281;ki temu ma szerszy obraz tego co powinno zosta&#263; zaimplementowane przez autora. Kr&oacute;tkie, nawet kilku zdaniowe, wprowadzenie podczas rozmowy, podlinkowanie numeru ticketu lub opisu wymagania biznesowego w komentarzu commit&rsquo;a mo&#380;e znacz&#261;co u&#322;atwi&#263; ca&#322;y proces. Reviewer nie musi si&#281; zastanawia&#263; &bdquo;co autor mia&#322; na my&#347;li&rdquo; pisz&#261;c ten kod. Mo&#380;e te&#380; zasugerowa&#263; inne, lepsze rozwi&#261;zanie, kt&oacute;re b&#281;dzie lepiej pasowa&#322;o do realizacji danej funkcjonalno&#347;ci lub poprawi bug w lepszy, &bdquo;trwalszy&rdquo; spos&oacute;b.<\/p>\n\n\n<h2 class=\"wp-block-heading wp-block-heading\" id=\"2-brak-podziac582u-na-sprawdzenie-jakoc59bci-kodu-i-pokrycie-wymagac584-biznesowych\">2. Brak podzia&#322;u na sprawdzenie jako&#347;ci kodu i pokrycie wymaga&#324; biznesowych<\/h2>\n\n\n<p class=\"wp-block-paragraph\">Skoro ju&#380; jeste&#347;my przy wymaganiach biznesowych, to warto te&#380; wspomnie&#263;, &#380;e code review powinno by&#263; podzielone na dwie cz&#281;&#347;ci. Pierwsza z nich to sprawdzenie, czy dany kod b&#281;dzie realizowa&#322; za&#322;o&#380;one wymagania biznesowe Druga, czy jest poprawny i nie ma w nim po prostu zwyczajnych bug&oacute;w. Ten punkt faktycznie pokrywa si&#281; nie co z poprzednim, ale tutaj chcia&#322;bym zwr&oacute;ci&#263; Twoj&#261; uwag&#281; na to, &#380;e poprawny kod pod wzgl&#281;dem technicznym nie zawsze b&#281;dzie poprawny pod wzgl&#281;dem biznesowym. Pewnie nie raz zdarzy&#322;o Ci si&#281; dosta&#263; tego samego taska ponownie, z komentarzem &bdquo;co&#347; nie dzia&#322;a jak powinno&rdquo; albo &bdquo;to nie tak mia&#322;o wygl&#261;da&#263;&rdquo;. Pomimo &#380;e Tw&oacute;j commit przeszed&#322; przez code review bez problem&oacute;w (albo wszystkie zosta&#322;y poprawione), to musisz ponownie pochyli&#263; si&#281; nad tym samym kodem. <\/p>\n\n\n<p class=\"wp-block-paragraph\">Podzia&#322; code review na dwie cz&#281;&#347;ci znacz&#261;co zmniejsza ryzyko powrotu tego samego taska do sprintu. Taki proces mo&#380;e by&#263; przeprowadzany przez jedn&#261; osob&#281; w charakterze reviewer&rsquo;a, ale najlepiej jak by by&#322;y to dwie inne osoby. Jedna z nich z wi&#281;ksz&#261; wiedz&#261; techniczn&#261; a druga ze znajomo&#347;ci&#261; wymaga&#324; biznesowych. Zatwierdzenie Twoich zmian przez te dwie osoby podniesie nie tylko jako&#347;&#263; kodu, ale te&#380; i zgodno&#347;&#263; z wymaganiami. <\/p>\n\n\n\n\n\n<h2 class=\"wp-block-heading wp-block-heading\" id=\"3-niepoprawne-review-testc3b3w\">\n<br>3. Niepoprawne review test&oacute;w<\/h2>\n\n\n<p class=\"wp-block-paragraph\">Temat rzeka. Kiedy&#347;, gdy by&#322;em na pocz&#261;tku swojej &#8222;kariery programistycznej&#8221;, pierwszego dnia w nowej firmie wykona&#322;em checkout kodu i pu&#347;ci&#322;em build. Po chwili oczywi&#347;cie build si&#281; wywali&#322; z powodu test&oacute;w. Zapyta&#322;em siedz&#261;cego obok mnie koleg&#281;, o co chodzi i dlaczego testy nie przechodz&#261;? Odpowied&#378; by&#322;a zaskakuj&#261;ca, odpar&#322; &bdquo;Ah tak, musisz doda&#263; -DskipTests bo te testy i tak nie dzia&#322;aj&#261;&rdquo; &ndash; dla ciekawskich polecenie &bdquo;-DskipTests&rdquo; w Maven pomija wykonywanie test&oacute;w podczas buildu. Tak, to by&#322; ci&#281;&#380;ki okres. Oprogramowanie to by&#322;o bardzo niskiej jako&#347;ci.<\/p>\n\n\n<p class=\"wp-block-paragraph\">Wracaj&#261;c do tematu. Wiele razy widzia&#322;em, i Ty pewnie te&#380;, commit&rsquo;y nowych funkcjonalno&#347;ci lub fixy istniej&#261;cych bez dopisywania nowych albo poprawy istniej&#261;cych test&oacute;w. Przej&#347;cie przez code review takiej zmiany wydaje si&#281; wr&#281;cz niedorzeczne. Dlaczego pisz&#261;c kod od nowa nie jest on pokryty testami? Nie da si&#281; tego przetestowa&#263;? Nie umiesz? Nie chce Ci si&#281;, bo wiesz, &#380;e i tak przejdzie przez code review? To mo&#380;e oznacza&#263; tylko tyle &#380;e, albo &#378;le napisa&#322;e&#347;(a&#347;) dan&#261; funkcjonalno&#347;&#263;, albo &#380;e proces code review w Twojej firmie istnieje tylko po to, &#380;eby dzia&#322; HR m&oacute;g&#322; si&#281; chwali&#263; na rekrutacjach, &#380;e macie co&#347; takiego.<\/p>\n\n\n<p class=\"wp-block-paragraph\">Kolejnym problemem z code review test&oacute;w jest w&#322;a&#347;nie ich code review &#128522; Wiem, mas&#322;o ma&#347;lane. Czasami zdarza si&#281;, &#380;e testy s&#261; traktowane nie co mniej rygorystycznie ni&#380; kod produkcyjny. Co oczywi&#347;cie jest kompletnie bezpodstawne, poniewa&#380; te testy uruchamiaj&#261; i testuj&#261; kod produkcyjny! Dlaczego wi&#281;c maj&#261; by&#263; napisane gorzej? Niepoprawnie napisany test, pomimo tego, &#380;e przechodzi, to nie testuje danej metody czy funkcjonalno&#347;ci w odpowiedni spos&oacute;b. Stwarza to pole do wyst&#281;powania bug&oacute;w w kodzie, a co za tym idzie kolejne taski, kt&oacute;re trzeba szybko zafixowa&#263; w najnowszym releasie. <\/p>\n\n\n<p class=\"wp-block-paragraph\">Id&#378;my dalej! Niepoprawnie napisany test nie oznacza zawsze, &#380;e nie pokrywa w dostatecznym stopniu danej metody czy funkcjonalno&#347;ci. Mo&#380;e te&#380; oznacza&#263;, &#380;e dany test jest napisany zbyt gorliwie. Nie daje mo&#380;liwo&#347;ci refetoringu testowanego kodu i wytwarza zjawisko &bdquo;betonowania&rdquo; kodu &ndash; braku mo&#380;liwo&#347;ci refactoryzacji kodu bez gruntownych zmian w testach. Takie przypadki powinny by&#263; wy&#322;apywane podczas code review, aby projekt by&#322; rozwijalny. W innym przypadku cz&#281;sto zdarza si&#281;, &#380;e ma&#322;a zmiana lub refactoring powoduje lawinowe zmiany w du&#380;ej liczbie test&oacute;w, co zwi&#281;ksza wydatki z bud&#380;etu projektu. &nbsp;&nbsp;<\/p>\n\n\n<h2 class=\"wp-block-heading wp-block-heading\" id=\"4-za-duc5bco-zmian-w-jednej-sesji-code-review\">4. Za du&#380;o zmian w jednej sesji code review<\/h2>\n\n\n<p class=\"wp-block-paragraph\">Statystyczny cz&#322;owiek &#8211; jak by nie by&#322;o programi&#347;ci te&#380; si&#281; do tej grupy zaliczaj&#261; &#8211; przestaje si&#281; skupia&#263; na wykonywanej czynno&#347;ci po oko&#322;o 15 minutach. P&oacute;&#378;niej umys&#322; zmierza w kierunku innych my&#347;li typu &bdquo;Co b&#281;dzie dzisiaj na obiad?&rdquo; albo &bdquo;Chyba musz&#281; do &#322;azienki&rdquo;. Co za tym idzie, zbyt d&#322;ugie code review traci na jako&#347;ci. Rozproszony umys&#322; reviewera nie jest w stanie dzia&#322;a&#263; na pe&#322;nych obrotach przez trwaj&#261;ce godzin&#281; albo i wi&#281;cej przegl&#261;danie zmian w kodzie i jednoczesne &#347;ledzenie czy wszystkie s&#261; poprawne. Czas po&#347;wi&#281;cony na sprawdzenie jednej linii kodu skraca si&#281; diametralnie, je&#347;li code review trwa za d&#322;ugo. Reviewer dostaje &#8222;magicznych zdolno&#347;ci&#8221; sprawdzania nawet dziesi&#261;tek linii kodu w ci&#261;gu sekundy! <\/p>\n\n\n<p class=\"wp-block-paragraph\">Ok, czasami nie da si&#281; zrobi&#263; code review w mniej ni&#380; 15 min, poniewa&#380; dana funkcjonalno&#347;&#263; jest na tyle skomplikowana lub rozbudowana. Czy to przypadkiem nie oznacza, &#380;e ta funkcjonalno&#347;&#263; powinna by&#263; podzielona na bardziej atomowe cz&#281;&#347;ci? <\/p>\n\n\n<h2 class=\"wp-block-heading wp-block-heading\" id=\"5-niepoprawne-przygotowanie-kodu-do-review\">5. Niepoprawne przygotowanie kodu do review<\/h2>\n\n\n<p class=\"wp-block-paragraph\">Problemem jest te&#380; brak odpowiedniego przygotowania zmian przed udost&#281;pnieniem do code review. Kod napisany niesp&oacute;jnie z og&oacute;lnie przyj&#281;tymi praktykami w zespole lub projekcie staje si&#281; mniej czytelny. Warto zwr&oacute;ci&#263; uwag&#281; na formatowanie kodu oraz jego zgodno&#347;&#263; z zasadami &#8222;clean code&#8221;. Trzymanie si&#281; tych utartych &#347;cie&#380;ek pozwoli Ci na zmniejszenie ilo&#347;ci poprawek zg&#322;aszanych przez reviewer&oacute;w do Twoich zmian. <\/p>\n\n\n<p class=\"wp-block-paragraph\">Dobr&#261; praktyk&#261; jest korzystanie z narz&#281;dzi do statycznej analizy kodu takich jak SonarQube czy CheckStyle. Dzi&#281;ki tym i innym podobnym, narz&#281;dziom b&#281;dziesz w stanie znacznie szybciej wy&#322;apa&#263; miejsca w kodzie, kt&oacute;re powinny zosta&#263;  poprawione.<\/p>\n\n\n<h2 class=\"wp-block-heading wp-block-heading\" id=\"zakoc584czenie\">Zako&#324;czenie<\/h2>\n\n\n<p class=\"wp-block-paragraph\">Ta lista jest moim subiektywnym przemy&#347;leniem na ten temat. Pewnie masz te&#380; swoje, wi&#281;c zach&#281;cam Ci&#281; do podzielenia si&#281; nimi w komentarzach.<\/p>\n\n\n<p class=\"wp-block-paragraph\">Standardowo zach&#281;cam te&#380; do udost&#281;pnienia tego postu w social mediach oraz polubienia fanpage Ara Software na Facebooku!<\/p>\n\n\n<p class=\"wp-block-paragraph\">Przydatne linki:<\/p>\n\n\n<p class=\"wp-block-paragraph\"><a rel=\"noreferrer noopener\" aria-label=\" (otwiera si&#281; na nowej zak&#322;adce)\" href=\"https:\/\/www.sonarqube.org\/\" target=\"_blank\">https:\/\/www.sonarqube.org<\/a><br><a rel=\"noreferrer noopener\" aria-label=\" (otwiera si&#281; na nowej zak&#322;adce)\" href=\"https:\/\/checkstyle.sourceforge.io\/\" target=\"_blank\">https:\/\/checkstyle.sourceforge.io\/<\/a> <\/p>","protected":false},"excerpt":{"rendered":"<p><span class=\"rt-reading-time\" style=\"display: block;\"><span class=\"rt-label rt-prefix\">Czas czytania: <\/span> <span class=\"rt-time\">5<\/span> <span class=\"rt-label rt-postfix\">minut<\/span><\/span> Code review jest bardzo wa&#380;ne. Podnosi jako&#347;&#263; kodu, poprawia komunikacj&#281; i rozpowszechnia wiedz&#281; o projekcie w zespole. S&#261; to tylko nieliczne warto&#347;ci dodane, a jest ich znacznie wi&#281;cej. Dobry proces sprawdzania kodu przed commit&rsquo;em, ma te&#380; warto&#347;&#263; edukacyjn&#261;. Osoba sprawdzaj&#261;ca kod (reviewer) z zasady powinna posiada&#263; co najmniej takie samo albo i wi&#281;ksze do&#347;wiadczenie ni&#380; &hellip;<\/p>","protected":false},"author":1,"featured_media":956,"comment_status":"closed","ping_status":"open","sticky":false,"template":"","format":"standard","meta":{"footnotes":""},"categories":[23,13],"tags":[20],"class_list":["post-380","post","type-post","status-publish","format-standard","has-post-thumbnail","hentry","category-pogadanki","category-programowanie","tag-code-review"],"yoast_head":"<!-- This site is optimized with the Yoast SEO plugin v20.11 - https:\/\/yoast.com\/wordpress\/plugins\/seo\/ -->\n<title>5 B\u0142\u0119d\u00f3w Podczas Code Review - Ara Software<\/title>\n<meta name=\"robots\" content=\"index, follow, max-snippet:-1, max-image-preview:large, max-video-preview:-1\" \/>\n<link rel=\"canonical\" href=\"http:\/\/arasoftware.pl\/en\/5-bledow-podczas-code-review\/\" \/>\n<meta property=\"og:locale\" content=\"en_US\" \/>\n<meta property=\"og:type\" content=\"article\" \/>\n<meta property=\"og:title\" content=\"5 B\u0142\u0119d\u00f3w Podczas Code Review - Ara Software\" \/>\n<meta property=\"og:description\" content=\"Czas czytania: 5 minut Code review jest bardzo wa&#380;ne. Podnosi jako&#347;&#263; kodu, poprawia komunikacj&#281; i rozpowszechnia wiedz&#281; o projekcie w zespole. S&#261; to tylko nieliczne warto&#347;ci dodane, a jest ich znacznie wi&#281;cej. Dobry proces sprawdzania kodu przed commit&rsquo;em, ma te&#380; warto&#347;&#263; edukacyjn&#261;. Osoba sprawdzaj&#261;ca kod (reviewer) z zasady powinna posiada&#263; co najmniej takie samo albo i wi&#281;ksze do&#347;wiadczenie ni&#380; &hellip;\" \/>\n<meta property=\"og:url\" content=\"http:\/\/arasoftware.pl\/en\/5-bledow-podczas-code-review\/\" \/>\n<meta property=\"og:site_name\" content=\"Ara Software\" \/>\n<meta property=\"article:published_time\" content=\"2020-05-20T10:51:49+00:00\" \/>\n<meta property=\"article:modified_time\" content=\"2021-10-21T08:34:21+00:00\" \/>\n<meta property=\"og:image\" content=\"https:\/\/arasoftware.pl\/wp-content\/uploads\/2020\/07\/codereview.png\" \/>\n\t<meta property=\"og:image:width\" content=\"1920\" \/>\n\t<meta property=\"og:image:height\" content=\"1080\" \/>\n\t<meta property=\"og:image:type\" content=\"image\/png\" \/>\n<meta name=\"author\" content=\"Radek Osak\" \/>\n<meta name=\"twitter:card\" content=\"summary_large_image\" \/>\n<meta name=\"twitter:label1\" content=\"Written by\" \/>\n\t<meta name=\"twitter:data1\" content=\"Radek Osak\" \/>\n\t<meta name=\"twitter:label2\" content=\"Est. reading time\" \/>\n\t<meta name=\"twitter:data2\" content=\"7 minutes\" \/>\n<script type=\"application\/ld+json\" class=\"yoast-schema-graph\">{\"@context\":\"https:\/\/schema.org\",\"@graph\":[{\"@type\":\"Article\",\"@id\":\"https:\/\/arasoftware.pl\/5-bledow-podczas-code-review\/#article\",\"isPartOf\":{\"@id\":\"https:\/\/arasoftware.pl\/5-bledow-podczas-code-review\/\"},\"author\":{\"name\":\"Radek Osak\",\"@id\":\"https:\/\/arasoftware.pl\/#\/schema\/person\/a13fe68bd27e762ef255547a88956902\"},\"headline\":\"5 B\u0142\u0119d\u00f3w Podczas Code Review\",\"datePublished\":\"2020-05-20T10:51:49+00:00\",\"dateModified\":\"2021-10-21T08:34:21+00:00\",\"mainEntityOfPage\":{\"@id\":\"https:\/\/arasoftware.pl\/5-bledow-podczas-code-review\/\"},\"wordCount\":1372,\"publisher\":{\"@id\":\"https:\/\/arasoftware.pl\/#organization\"},\"keywords\":[\"Code Review\"],\"articleSection\":[\"Pogadanki\",\"Programowanie\"],\"inLanguage\":\"en-US\"},{\"@type\":\"WebPage\",\"@id\":\"https:\/\/arasoftware.pl\/5-bledow-podczas-code-review\/\",\"url\":\"https:\/\/arasoftware.pl\/5-bledow-podczas-code-review\/\",\"name\":\"5 B\u0142\u0119d\u00f3w Podczas Code Review - Ara Software\",\"isPartOf\":{\"@id\":\"https:\/\/arasoftware.pl\/#website\"},\"datePublished\":\"2020-05-20T10:51:49+00:00\",\"dateModified\":\"2021-10-21T08:34:21+00:00\",\"breadcrumb\":{\"@id\":\"https:\/\/arasoftware.pl\/5-bledow-podczas-code-review\/#breadcrumb\"},\"inLanguage\":\"en-US\",\"potentialAction\":[{\"@type\":\"ReadAction\",\"target\":[\"https:\/\/arasoftware.pl\/5-bledow-podczas-code-review\/\"]}]},{\"@type\":\"BreadcrumbList\",\"@id\":\"https:\/\/arasoftware.pl\/5-bledow-podczas-code-review\/#breadcrumb\",\"itemListElement\":[{\"@type\":\"ListItem\",\"position\":1,\"name\":\"Strona g\u0142\u00f3wna\",\"item\":\"https:\/\/arasoftware.pl\/\"},{\"@type\":\"ListItem\",\"position\":2,\"name\":\"5 B\u0142\u0119d\u00f3w Podczas Code Review\"}]},{\"@type\":\"WebSite\",\"@id\":\"https:\/\/arasoftware.pl\/#website\",\"url\":\"https:\/\/arasoftware.pl\/\",\"name\":\"Ara Software\",\"description\":\"\",\"publisher\":{\"@id\":\"https:\/\/arasoftware.pl\/#organization\"},\"potentialAction\":[{\"@type\":\"SearchAction\",\"target\":{\"@type\":\"EntryPoint\",\"urlTemplate\":\"https:\/\/arasoftware.pl\/?s={search_term_string}\"},\"query-input\":\"required name=search_term_string\"}],\"inLanguage\":\"en-US\"},{\"@type\":\"Organization\",\"@id\":\"https:\/\/arasoftware.pl\/#organization\",\"name\":\"Ara Software\",\"url\":\"https:\/\/arasoftware.pl\/\",\"logo\":{\"@type\":\"ImageObject\",\"inLanguage\":\"en-US\",\"@id\":\"https:\/\/arasoftware.pl\/#\/schema\/logo\/image\/\",\"url\":\"https:\/\/arasoftware.pl\/wp-content\/uploads\/2023\/06\/ara-logo-light-1.svg\",\"contentUrl\":\"https:\/\/arasoftware.pl\/wp-content\/uploads\/2023\/06\/ara-logo-light-1.svg\",\"width\":317,\"height\":55,\"caption\":\"Ara Software\"},\"image\":{\"@id\":\"https:\/\/arasoftware.pl\/#\/schema\/logo\/image\/\"}},{\"@type\":\"Person\",\"@id\":\"https:\/\/arasoftware.pl\/#\/schema\/person\/a13fe68bd27e762ef255547a88956902\",\"name\":\"Radek Osak\",\"image\":{\"@type\":\"ImageObject\",\"inLanguage\":\"en-US\",\"@id\":\"https:\/\/arasoftware.pl\/#\/schema\/person\/image\/\",\"url\":\"https:\/\/secure.gravatar.com\/avatar\/2a5a97fd24033c3df7dff799dc95774b?s=96&d=mm&r=g\",\"contentUrl\":\"https:\/\/secure.gravatar.com\/avatar\/2a5a97fd24033c3df7dff799dc95774b?s=96&d=mm&r=g\",\"caption\":\"Radek Osak\"},\"url\":\"http:\/\/arasoftware.pl\/en\/author\/radek030790gmail-com\/\"}]}<\/script>\n<!-- \/ Yoast SEO plugin. -->","yoast_head_json":{"title":"5 B\u0142\u0119d\u00f3w Podczas Code Review - Ara Software","robots":{"index":"index","follow":"follow","max-snippet":"max-snippet:-1","max-image-preview":"max-image-preview:large","max-video-preview":"max-video-preview:-1"},"canonical":"http:\/\/arasoftware.pl\/en\/5-bledow-podczas-code-review\/","og_locale":"en_US","og_type":"article","og_title":"5 B\u0142\u0119d\u00f3w Podczas Code Review - Ara Software","og_description":"Czas czytania: 5 minut Code review jest bardzo wa&#380;ne. Podnosi jako&#347;&#263; kodu, poprawia komunikacj&#281; i rozpowszechnia wiedz&#281; o projekcie w zespole. S&#261; to tylko nieliczne warto&#347;ci dodane, a jest ich znacznie wi&#281;cej. Dobry proces sprawdzania kodu przed commit&rsquo;em, ma te&#380; warto&#347;&#263; edukacyjn&#261;. Osoba sprawdzaj&#261;ca kod (reviewer) z zasady powinna posiada&#263; co najmniej takie samo albo i wi&#281;ksze do&#347;wiadczenie ni&#380; &hellip;","og_url":"http:\/\/arasoftware.pl\/en\/5-bledow-podczas-code-review\/","og_site_name":"Ara Software","article_published_time":"2020-05-20T10:51:49+00:00","article_modified_time":"2021-10-21T08:34:21+00:00","og_image":[{"width":1920,"height":1080,"url":"https:\/\/arasoftware.pl\/wp-content\/uploads\/2020\/07\/codereview.png","type":"image\/png"}],"author":"Radek Osak","twitter_card":"summary_large_image","twitter_misc":{"Written by":"Radek Osak","Est. reading time":"7 minutes"},"schema":{"@context":"https:\/\/schema.org","@graph":[{"@type":"Article","@id":"https:\/\/arasoftware.pl\/5-bledow-podczas-code-review\/#article","isPartOf":{"@id":"https:\/\/arasoftware.pl\/5-bledow-podczas-code-review\/"},"author":{"name":"Radek Osak","@id":"https:\/\/arasoftware.pl\/#\/schema\/person\/a13fe68bd27e762ef255547a88956902"},"headline":"5 B\u0142\u0119d\u00f3w Podczas Code Review","datePublished":"2020-05-20T10:51:49+00:00","dateModified":"2021-10-21T08:34:21+00:00","mainEntityOfPage":{"@id":"https:\/\/arasoftware.pl\/5-bledow-podczas-code-review\/"},"wordCount":1372,"publisher":{"@id":"https:\/\/arasoftware.pl\/#organization"},"keywords":["Code Review"],"articleSection":["Pogadanki","Programowanie"],"inLanguage":"en-US"},{"@type":"WebPage","@id":"https:\/\/arasoftware.pl\/5-bledow-podczas-code-review\/","url":"https:\/\/arasoftware.pl\/5-bledow-podczas-code-review\/","name":"5 B\u0142\u0119d\u00f3w Podczas Code Review - Ara Software","isPartOf":{"@id":"https:\/\/arasoftware.pl\/#website"},"datePublished":"2020-05-20T10:51:49+00:00","dateModified":"2021-10-21T08:34:21+00:00","breadcrumb":{"@id":"https:\/\/arasoftware.pl\/5-bledow-podczas-code-review\/#breadcrumb"},"inLanguage":"en-US","potentialAction":[{"@type":"ReadAction","target":["https:\/\/arasoftware.pl\/5-bledow-podczas-code-review\/"]}]},{"@type":"BreadcrumbList","@id":"https:\/\/arasoftware.pl\/5-bledow-podczas-code-review\/#breadcrumb","itemListElement":[{"@type":"ListItem","position":1,"name":"Strona g\u0142\u00f3wna","item":"https:\/\/arasoftware.pl\/"},{"@type":"ListItem","position":2,"name":"5 B\u0142\u0119d\u00f3w Podczas Code Review"}]},{"@type":"WebSite","@id":"https:\/\/arasoftware.pl\/#website","url":"https:\/\/arasoftware.pl\/","name":"Ara Software","description":"","publisher":{"@id":"https:\/\/arasoftware.pl\/#organization"},"potentialAction":[{"@type":"SearchAction","target":{"@type":"EntryPoint","urlTemplate":"https:\/\/arasoftware.pl\/?s={search_term_string}"},"query-input":"required name=search_term_string"}],"inLanguage":"en-US"},{"@type":"Organization","@id":"https:\/\/arasoftware.pl\/#organization","name":"Ara Software","url":"https:\/\/arasoftware.pl\/","logo":{"@type":"ImageObject","inLanguage":"en-US","@id":"https:\/\/arasoftware.pl\/#\/schema\/logo\/image\/","url":"https:\/\/arasoftware.pl\/wp-content\/uploads\/2023\/06\/ara-logo-light-1.svg","contentUrl":"https:\/\/arasoftware.pl\/wp-content\/uploads\/2023\/06\/ara-logo-light-1.svg","width":317,"height":55,"caption":"Ara Software"},"image":{"@id":"https:\/\/arasoftware.pl\/#\/schema\/logo\/image\/"}},{"@type":"Person","@id":"https:\/\/arasoftware.pl\/#\/schema\/person\/a13fe68bd27e762ef255547a88956902","name":"Radek Osak","image":{"@type":"ImageObject","inLanguage":"en-US","@id":"https:\/\/arasoftware.pl\/#\/schema\/person\/image\/","url":"https:\/\/secure.gravatar.com\/avatar\/2a5a97fd24033c3df7dff799dc95774b?s=96&d=mm&r=g","contentUrl":"https:\/\/secure.gravatar.com\/avatar\/2a5a97fd24033c3df7dff799dc95774b?s=96&d=mm&r=g","caption":"Radek Osak"},"url":"http:\/\/arasoftware.pl\/en\/author\/radek030790gmail-com\/"}]}},"_links":{"self":[{"href":"http:\/\/arasoftware.pl\/en\/wp-json\/wp\/v2\/posts\/380","targetHints":{"allow":["GET"]}}],"collection":[{"href":"http:\/\/arasoftware.pl\/en\/wp-json\/wp\/v2\/posts"}],"about":[{"href":"http:\/\/arasoftware.pl\/en\/wp-json\/wp\/v2\/types\/post"}],"author":[{"embeddable":true,"href":"http:\/\/arasoftware.pl\/en\/wp-json\/wp\/v2\/users\/1"}],"replies":[{"embeddable":true,"href":"http:\/\/arasoftware.pl\/en\/wp-json\/wp\/v2\/comments?post=380"}],"version-history":[{"count":0,"href":"http:\/\/arasoftware.pl\/en\/wp-json\/wp\/v2\/posts\/380\/revisions"}],"wp:featuredmedia":[{"embeddable":true,"href":"http:\/\/arasoftware.pl\/en\/wp-json\/wp\/v2\/media\/956"}],"wp:attachment":[{"href":"http:\/\/arasoftware.pl\/en\/wp-json\/wp\/v2\/media?parent=380"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"http:\/\/arasoftware.pl\/en\/wp-json\/wp\/v2\/categories?post=380"},{"taxonomy":"post_tag","embeddable":true,"href":"http:\/\/arasoftware.pl\/en\/wp-json\/wp\/v2\/tags?post=380"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}