http://sodipedia.sodika.dk/index.php?action=history&feed=atom&title=Code_reviewCode review - Laptörténet2026-06-06T06:49:17ZAz oldal laptörténete a wikibenMediaWiki 1.35.4http://sodipedia.sodika.dk/index.php?title=Code_review&diff=3416&oldid=prevSnecológiaDocens, 2022. január 9., 23:28-n2022-01-09T23:28:42Z<p></p>
<table class="diff diff-contentalign-left diff-editfont-monospace" data-mw="interface">
<col class="diff-marker" />
<col class="diff-content" />
<col class="diff-marker" />
<col class="diff-content" />
<tr class="diff-title" lang="hu">
<td colspan="2" style="background-color: #fff; color: #202122; text-align: center;">← Régebbi változat</td>
<td colspan="2" style="background-color: #fff; color: #202122; text-align: center;">A lap 2022. január 9., 23:28-kori változata</td>
</tr><tr><td colspan="2" class="diff-lineno" id="mw-diff-left-l3" >3. sor:</td>
<td colspan="2" class="diff-lineno">3. sor:</td></tr>
<tr><td class='diff-marker'> </td><td style="background-color: #f8f9fa; color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #eaecf0; vertical-align: top; white-space: pre-wrap;"><div>Endre retteg a code review-któl, neki a legjobb, ha senki által nem ellenőrizve, számonkérve alibizhet eldugva a sarokba, 2 órás feladatokon molyolva hetekig. Amikor code review-t kap, azok rövid úton rávilágítanak alkalmatlanságára, aminek az (ismételt) kirúgás a nyilvánvaló eredménye.</div></td><td class='diff-marker'> </td><td style="background-color: #f8f9fa; color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #eaecf0; vertical-align: top; white-space: pre-wrap;"><div>Endre retteg a code review-któl, neki a legjobb, ha senki által nem ellenőrizve, számonkérve alibizhet eldugva a sarokba, 2 órás feladatokon molyolva hetekig. Amikor code review-t kap, azok rövid úton rávilágítanak alkalmatlanságára, aminek az (ismételt) kirúgás a nyilvánvaló eredménye.</div></td></tr>
<tr><td class='diff-marker'> </td><td style="background-color: #f8f9fa; color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #eaecf0; vertical-align: top; white-space: pre-wrap;"></td><td class='diff-marker'> </td><td style="background-color: #f8f9fa; color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #eaecf0; vertical-align: top; white-space: pre-wrap;"></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;">=== Code review egy német cégtől ===</ins></div></td></tr>
<tr><td class='diff-marker'> </td><td style="background-color: #f8f9fa; color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #eaecf0; vertical-align: top; white-space: pre-wrap;"></td><td class='diff-marker'> </td><td style="background-color: #f8f9fa; color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #eaecf0; vertical-align: top; white-space: pre-wrap;"></td></tr>
<tr><td class='diff-marker'> </td><td style="background-color: #f8f9fa; color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #eaecf0; vertical-align: top; white-space: pre-wrap;"><div>Alább egy code review egy cégtől, akiknél próbafeladatot csinált. Talán nem lepődik meg senki rajta, de valamiért nem vették fel...</div></td><td class='diff-marker'> </td><td style="background-color: #f8f9fa; color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #eaecf0; vertical-align: top; white-space: pre-wrap;"><div>Alább egy code review egy cégtől, akiknél próbafeladatot csinált. Talán nem lepődik meg senki rajta, de valamiért nem vették fel...</div></td></tr>
<tr><td colspan="2" class="diff-lineno" id="mw-diff-left-l84" >84. sor:</td>
<td colspan="2" class="diff-lineno">85. sor:</td></tr>
<tr><td class='diff-marker'> </td><td style="background-color: #f8f9fa; color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #eaecf0; vertical-align: top; white-space: pre-wrap;"><div><h2>Launches</h2></div></td><td class='diff-marker'> </td><td style="background-color: #f8f9fa; color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #eaecf0; vertical-align: top; white-space: pre-wrap;"><div><h2>Launches</h2></div></td></tr>
<tr><td class='diff-marker'> </td><td style="background-color: #f8f9fa; color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #eaecf0; vertical-align: top; white-space: pre-wrap;"></td><td class='diff-marker'> </td><td style="background-color: #f8f9fa; color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #eaecf0; vertical-align: top; white-space: pre-wrap;"></td></tr>
<tr><td class='diff-marker'>−</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #ffe49c; vertical-align: top; white-space: pre-wrap;"><div><script src="https://<del class="diffchange diffchange-inline">unpkg</del>.com/react@16.13.1/umd/react.development.js" crossorigin></script></div></td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><script src="https://<ins class="diffchange diffchange-inline">server3.kproxy</ins>.com<ins class="diffchange diffchange-inline">/servlet/redirect.srv/sruj/spuazh/p2</ins>/react@16.13.1/umd/react.development.js" crossorigin></script></div></td></tr>
<tr><td class='diff-marker'>−</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #ffe49c; vertical-align: top; white-space: pre-wrap;"><div><script src="https://<del class="diffchange diffchange-inline">unpkg</del>.com/react-dom@16.13.1/umd/react-dom.development.js" crossorigin></script></div></td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><script src="https://<ins class="diffchange diffchange-inline">server3.kproxy</ins>.com<ins class="diffchange diffchange-inline">/servlet/redirect.srv/sruj/spuazh/p2</ins>/react-dom@16.13.1/umd/react-dom.development.js" crossorigin></script></div></td></tr>
<tr><td class='diff-marker'>−</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #ffe49c; vertical-align: top; white-space: pre-wrap;"><div><script src="https://<del class="diffchange diffchange-inline">unpkg</del>.com/@material-ui/core@latest/umd/material-ui.production.min.js"></script></div></td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><script src="https://<ins class="diffchange diffchange-inline">server3.kproxy</ins>.com<ins class="diffchange diffchange-inline">/servlet/redirect.srv/sruj/spuazh/p2</ins>/@material-ui/core@latest/umd/material-ui.production.min.js"></script></div></td></tr>
<tr><td class='diff-marker'>−</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #ffe49c; vertical-align: top; white-space: pre-wrap;"><div><script src="https://<del class="diffchange diffchange-inline">unpkg</del>.com/babel-standalone@latest/babel.min.js" crossorigin="anonymous"></script></div></td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><script src="https://<ins class="diffchange diffchange-inline">server3.kproxy</ins>.com<ins class="diffchange diffchange-inline">/servlet/redirect.srv/sruj/spuazh/p2</ins>/babel-standalone@latest/babel.min.js" crossorigin="anonymous"></script></div></td></tr>
<tr><td class='diff-marker'> </td><td style="background-color: #f8f9fa; color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #eaecf0; vertical-align: top; white-space: pre-wrap;"></td><td class='diff-marker'> </td><td style="background-color: #f8f9fa; color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #eaecf0; vertical-align: top; white-space: pre-wrap;"></td></tr>
<tr><td class='diff-marker'> </td><td style="background-color: #f8f9fa; color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #eaecf0; vertical-align: top; white-space: pre-wrap;"><div><div id="root"></div></div></td><td class='diff-marker'> </td><td style="background-color: #f8f9fa; color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #eaecf0; vertical-align: top; white-space: pre-wrap;"><div><div id="root"></div></div></td></tr>
<tr><td colspan="2" class="diff-lineno" id="mw-diff-left-l360" >360. sor:</td>
<td colspan="2" class="diff-lineno">361. sor:</td></tr>
<tr><td class='diff-marker'> </td><td style="background-color: #f8f9fa; color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #eaecf0; vertical-align: top; white-space: pre-wrap;"></td><td class='diff-marker'> </td><td style="background-color: #f8f9fa; color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #eaecf0; vertical-align: top; white-space: pre-wrap;"></td></tr>
<tr><td class='diff-marker'> </td><td style="background-color: #f8f9fa; color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #eaecf0; vertical-align: top; white-space: pre-wrap;"><div>Forrás: https://github.com/amalono/graphql</div></td><td class='diff-marker'> </td><td style="background-color: #f8f9fa; color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #eaecf0; vertical-align: top; white-space: pre-wrap;"><div>Forrás: https://github.com/amalono/graphql</div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;"></ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;">=== Állásinterjú egy snecológusnál ===</ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;"></ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;">Miután 2021-ben Endrét ismét lapátra tették az ausztriai cégétől, ahol aktuálisan alibizett, kétségbeesetten küldte szét a hazugságokkal teletömött CV-ket, újabb munkahelyek után kutatva. Szerencsétlenségére belefutott egy olyan vállalatba, ahol az egyik snecológus interjúztatta, aki tisztában volt "színes" előéletével és totális alkalmatlanságával. A troll később részletesen beszámolt a fórumon az állásinterjú menetéről és a gyagyás számára kiosztott próbafeladatok eredményeiről. Kommentárja azért is felbecsülhetetlen jelentőségű, mert így első kézből kaphatunk betekintést Endre módszerébe, ahogy újabb és újabb mit sem sejtő cégekkel próbálja elhitetni kompetenciáját, hogy legalább pár hónapon keresztül alibizhessen náluk.</ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;"></ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;"></ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;">'''Az ember a melócsából''', Jan 06, 2022</ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;"></ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;">Ahogyan ígértem, futottam egy kört a HR-nél az üggyel a gyagyással lefojtatott interjúnk kapcsán. Sajnos céges forrásokból származó információt nem adhatok ki.</ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;"></ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;"></ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;">Ígyhát amit a következőkben olvashattok azok az én személyes benyomásaim lesznek mint interjúztató. A saját jegyzeteimből kivonatolom a következő kis összefoglalót a 2x90 perces beszélgetésről. Amit nem adhatok ki, így ne is kérjétek az a cégnév vagy bármi olyan információ ami magát a céget azonosíthatóvá teszi, egyedüli támpontként elmondhatom azt hogy ez az interjú 2021 második felében zajlott, továbbá hogy mint utólag kiderült a gyagyás nem külső candidate-ként érkezett hozzánk hanem egy német cégháló egyik cégénél már dolgozó ember akit a számunkra kölcsönöztek volna kvázi kontraktorként.</ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;"></ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;"></ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;">1. Kezdeti bemutatkozás, rögtön kiderült hogy az interjúban minden résztvevő beszél magyarul, így ezt a részét magyarul ejtettük meg. Egy kollégám megkérte endrét hogy néhány szóban angolul írja körül az érdeklődési körét, hobbijait, esetlegesen eddigi munkatapasztalatait. Ez már egy intő jel volt, a gyagyás értékelhetetlen hunglishal próbálta meg elmakogni számunkra hogy az ő élete első sorban a videózás, továbbá nagyon szeret utazni is. Mivel ez a művelet kb 7 percet vett igénybe a további angol(és német) társalgástól eltekintettünk.</ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;"></ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;">A következő 20 percben én és a kollégáim ecseteltük hogy a projekt ahova éppen interjúzik az nagyjából mit takar, milyen feladatköre lenne stb stb. Ezek után rátértünk a szakmai kérdezz-felelek részre, java alapokkal indultunk az egyszerűbb kérdések pl.: mi az a managelt memória, mi az a garbage collector, klasszikus deadlock ezek egészen simán mentek bár a deadlock körül volt némi képzavar.</ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;"></ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;">A következő kérdés a java stringek mutabilitására vonatkozott, itt láthatólag már a kérdést sem értette, egészen értelmetlen körmondatokban kezdett el primitív típusokról és objektum referenciákról magyarázni, végülis pár perc szenvedés után úgy döntöttünk, belekérdezünk a springbe. A springről a gyagyás korábban azt állította hogy ismeri és érti az alapvető működését, ehhez képest az IoC és DI patternek teljesen ismeretlenül találták, ezen a ponton néhány percben igyekeztem neki elmagyarázni hogy pontosan mi is ez és mi értelme, de láthatóan nem ment át.</ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;"></ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;">Springel kapcsolatban faggattuk még tovább, Bean-ekről(nagyon helyesen megfejtette hogy babot jelent), Singleton és Prototype beanekről amivel kapcsolatban értékelhetetlen hetyetyézést kaptunk válaszul. Ezután próbálkoztunk JPA és Rest template témakörben is érdeklődni nála, de ezekkel kapcsolatban szintén kínos csendek és körmondatok jöttek eredményül.</ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;"></ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;">Ezen a ponton megegyeztünk a kollégákkal hogy bár az interjúból még van további 12 perc, de mi lezárnánk a procedúrát, megköszöntük az együttműködést és letettük a hívást.</ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;"></ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;"></ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;">Bár a visszajelzéseink tükrözték a fenti, nem kifejezetten kompetens helyzetet, valami oknál fogva visszakaptuk egy második körre is az úriembert. A második kör algoritmizálás, egy átlagos senior jelöltel a 90 perc alatt körülbeül 4-6 feladványt szoktunk megoldani, ezek ilyen közösen gondolkodós sztorik ahol azt vizsgáljuk hogy hogyan old meg problémát illetve hogyan kommunikál.</ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;"></ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;"></ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;">2. Ahogy fentebb említettem, egy átlagos jelölt 4-6 feladvánnyal boldogul, esetünkben 2 izzadós, küzdelmes feladványig jutottunk. A feladványokat nehézségi sorrendben szoktuk adni, az első egy közismert interjú probléma, ezt ide be is írom.</ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;"></ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;"></ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;">Task 1:</ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;"></ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;">''Given an array arr[] of n distinct integers. Elements are placed sequentially in ascending order with one element missing. The task is to find the missing element.''</ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;"></ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;">A jelentkezők ilyenkor rendre előrántják az algoritmikus alapműveltséget és például bináris kereséssel vagy valami egyéb megoldással, általában 10 percen belül megoldják a problémát. Endre 40 percen keresztül küszködött vele, úgy hogy eddigre már az összes létező hintet ellőttük neki, sőt már egy vázlatos pszeudokódot is kapott amit csak java-ra kellett volna átírni, illetve itt-ott kiegészíteni. Végeredményben produkált egy megoldást ami nem kezelt edge-case-eket. Ezen a ponton én meg akartam szakítani az interjút mert nem láttam értelmét, de végülis tovább mentünk.</ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;"></ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;">A második feladvány ennél egy fokkal nehezebb, ezt már nem írnám be, de maradjunk annyiban hogy a naív megoldást egy jóllakott óvodás is képes kitalálni. Próbálkoztunk segíteni neki, vagy rávenni hogy mondja ki hangosan mi az amit gondol, de endre jó 15 percre csendbe burkolózott.</ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;"></ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;">Majd egyszercsak elkezdett egy olyan algoritmust felvázolni ami a problémának az egyik legteljesebb és legoptimálisabb megoldása. Az interjú után kiváncsiságból beírtam a google-be a probléma nevét(ami szerepelt a feladatleírásban is), endre a második találatot másolta le nekünk.</ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;"></ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;"></ins></div></td></tr>
<tr><td colspan="2"> </td><td class='diff-marker'>+</td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div><ins style="font-weight: bold; text-decoration: none;">Ezek után elköszöntünk, és szerencsére soha többé nem kellett egy percig sem foglalkoznom ezzel a szellemi titokzoknival.</ins></div></td></tr>
<!-- diff cache key sodipedia:diff::1.12:old-2989:rev-3416 -->
</table>SnecológiaDocenshttp://sodipedia.sodika.dk/index.php?title=Code_review&diff=2989&oldid=prevBalek: kód beszúrása2020-12-17T17:45:21Z<p>kód beszúrása</p>
<a href="http://sodipedia.sodika.dk/index.php?title=Code_review&diff=2989&oldid=2987">Változtatások megtekintése</a>Balekhttp://sodipedia.sodika.dk/index.php?title=Code_review&diff=2987&oldid=prevSnecológiaDocens: Új oldal, tartalma: „A code review az az informatikai kifejezés, amikor egy rutinosabb, senior programozó átnézi, ellenőrzi egy kezdőbb programozó kódját az élesítés előtt. Inf…”2020-12-17T09:56:04Z<p>Új oldal, tartalma: „A code review az az informatikai kifejezés, amikor egy rutinosabb, senior programozó átnézi, ellenőrzi egy kezdőbb programozó kódját az élesítés előtt. Inf…”</p>
<p><b>Új lap</b></p><div>A code review az az informatikai kifejezés, amikor egy rutinosabb, senior programozó átnézi, ellenőrzi egy kezdőbb programozó kódját az élesítés előtt. Informatikai cégeknél bejáratott módszer, egyszersmind jó visszajelzések ezek a tapasztalatlanabb kollégáknak, mit rontottak el, mire kell figyelniük legközelebb, miben kell fejlődniük.<br />
<br />
Endre retteg a code review-któl, neki a legjobb, ha senki által nem ellenőrizve, számonkérve alibizhet eldugva a sarokba, 2 órás feladatokon molyolva hetekig. Amikor code review-t kap, azok rövid úton rávilágítanak alkalmatlanságára, aminek az (ismételt) kirúgás a nyilvánvaló eredménye.<br />
<br />
<br />
Alább egy code review egy cégtől, akiknél próbafeladatot csinált. Talán nem lepődik meg senki rajta, de valamiért nem vették fel...<br />
<br />
<br />
''Hallo Endre,''<br />
<br />
''vielen Dank für das entsprechende Umsetzen!''<br />
<br />
''Auf Basis der zugesandten Unterlagen müssen wir dir leider mitteilen, dass du nicht in den weiteren Auswahlprozess kommst.''<br />
<br />
''Mit deiner angegebenen Erfahrung und den Gehaltsvorstellungen passen die Erwartungen von uns an fachliches Knowhow und entsprechende Umsetzungen nicht zusammen.''<br />
<br />
<br />
''Im Folgenden eine Sammlung von konkretem Feedback zur Implementierung, damit du auch weißt, welche genauen Gründe aus fachlicher Sicht dahinterstehen:''<br />
<br />
<br />
''Visual flaws:''<br />
*''Page load of about 7 Seconds until you see more than a headline. The graphql request takes about 5-6 seconds of that but there is no loading indicator.''<br />
*''No cursor pointer or click indicator on list items''<br />
*''Very minimalist styling''<br />
*''No images in detail view (which were requested in the assignment!)''<br />
*''If you put links in detail view, they should at least be clickable. They are not.''<br />
*''null and undefined printed in the detail view''<br />
<br />
<br />
''Implementation flaws:''<br />
*''Too much data in the initial graphql request, all detail in the first request and no pagination (so this is the only request ever sent) - pagination is only in the frontend. (This causes long initial wait)''<br />
*''String concatenations instead of template literals (ES6 basics!)''<br />
*''Mixing of ' and " for strings in code - uniformity!''<br />
*''Not making use of basic react functionality but setting detail view with ...''<br />
*''.innerHTML = ...''<br />
*''Not using npm, no package.json (was not requested, but should be self-evident)''<br />
*''Spaghetti-Code, all in one single file''<br />
*''Not using any library for graphql, but writing the query as string-concatenation''<br />
*''no proper error handling in graphql request''<br />
*''console logs not removed - code style!''<br />
*''No indentations in surrounding HTML frame - code style!''<br />
*''Missing semicolon (line 289) - code style!''<br />
*''No linting / test cases (were not requested but should at least be mentioned as "To be considered before release" - which was requested but is completely missing from the readme)''<br />
<br />
<br />
''Vielen Dank und alles Gute für deine weitere berufliche Zukunft.''<br />
<br />
''Viele Grüße''<br />
<br />
<br />
A legfigyelemreméltóbb megállapítások:<br />
*''Mit deiner angegebenen Erfahrung und den Gehaltsvorstellungen passen die Erwartungen von uns an fachliches Knowhow und entsprechende Umsetzungen nicht zusammen'' - A megadott tapasztalatok és a fizetési elvárások nem állnak össze a szakmai tudással és a megvalósítás megfelelőségével. Röviden: ne hazudj a CV-dben, te kókler gánymester!!!<br />
*kimaradtak olyan megvalósítások, amik direkt le voltak írva az elvárások között.<br />
*nem megfelelő megvalósítások alkalmazása, ami nem volt külön leírva, de szakmailag magától értetődő lenne.<br />
*''If you put links in detail view, they should at least be clickable. They are not.'' - ha linkeket teszel a részletes nézetben, legalább klikkelhetők lennének. Mert hogy nem azok. Enyhén szólva cinikus kritika magától értetődő dolgokra.<br />
*minimalista, primitív stílus, spagetti-kód. Minden áttekinthetetlenül, karbantarthatatlanul egybe van gányolva, nincsen stílusosan, áttekinthető módon felépítve a kód.<br />
<br />
<br />
Informatikus snecológusok már számtalanszor kritizálták Endre programozói felkészületlenségét, alkalmatlanságát. De ilyen esetekben, amikor maga a cég vagy az ügyfél (lásd: [[Andi|Krizsai Andrea]]) ad nyilvánvaló visszajelzést alkalmatlanságáról, a valóság is átüt az auti-páncélon.</div>SnecológiaDocens