Code-review, czyli przegląd kodu

Posted by wiktor on Oct 22, 2007 in Różne

Dzisiaj na seminarium magisterskim na MIMUWie przedstawiłem temat mojej pracy magisterskiej. Oto jej tytuł:

  • po polsku: “System wspomagający przegląd kodu w rozproszonych zespołach programistycznych”,
  • po angielsku: “Code review system for distributed developer teams”.

Poniżej możesz przeczytać opis pracy magisterskiej oraz obejrzeć slajdy z prezentacji:

Opis pracy magisterskiej:
Celem pracy magisterskiej jest stworzenie aplikacji sieciowej, która by znacząco ułatwiła wirtualne przeglądy kodu w rozproszonych zespołach programistycznych. Sam przegląd kodu będzie się odbywał poprzez przeglądarkę internetową, a użytkownik będzie widział graficzną reprezentację różnic zawartości plików. Razem z możliwością dyskusji nad dowolnym zaznaczonym fragmentem, a nawet poszczególnymi wierszami oraz z integracją z analizatorem statycznym kodu FindBugs będzie to podstawowa funkcjonalność aplikacji. Wprowadzone zostaną także możliwości przeglądania repozytorium poprzez WWW i wysyłania powiadomień o nowych zdarzeniach.

Co o tym myślicie?

13 Comments

Grzegorz
Oct 22, 2007 at 10:52 pm

U nas zwie się dodatkowo takei coś spalszczjąć kota-review z uwagi na “kocenie” przesłuchiwanego :D


 
wiktor
Oct 23, 2007 at 1:23 am

Kota-review :) a to dobre, piękne, piękne tłumaczenie :D


 
Someone
Oct 23, 2007 at 11:47 am

Hm, już gdzieś coś takiego widziałem… http://code.google.com/p/reviewboard/

I powodzenia przy pisaniu. :)


 
wiktor
Oct 23, 2007 at 11:56 am

Ooo to coś nowego, nie widziałem tego jeszcze. Dzięki za linka :)


 
Rafał
Oct 23, 2007 at 3:26 pm

przedstawiles taką prezentację na seminarium? wydaje mi sie, ze jej luzacki ton (“bajery”, “papierkowa robota”) nie bardzo pasuje na uczlenię i pokazywanie jej profesorowi.


 
wiktor
Oct 23, 2007 at 3:37 pm

Tak, przedstawiłem taką prezentację na seminarium. Pamiętaj, że wszystko zależy od tego, kto jest prowadzącym i jakie ma podejście do tego. Celem mojej mgr jest stworzenie aplikacji, a reszta to dla mnie jest na prawdę papierkowa robota, bo nie stanowi meritum mgr. Zgadzam się, że występuje w niej “luzacki ton” i jest on przeze mnie wprowadzony świadomie, aby publiczność nie zasypiała :) . Natomiast kiedy opowiadałem o moim pomyśle to na taki luz już sobie nie pozwalałem.


 
Piotrek
Oct 24, 2007 at 8:30 am

Po pierwsze: na 18 slajdzie jest “Dwie pary oczy…” rozumiem, że to literówka – mi wydaje się, że powinno być “oczu”.
Po drugie: rozmiary czcionek nie są jednolite. Mam wrażenie – może błędne – że część zdań na tym samym poziomie wypunktowania na jedyn slajdzie są pisane czcionką w rozmiarze X, a na innych X–. Jeśli nie możesz zmieścić treści na jednym slajdzie, to go podziel na kilka.
Po trzecie: podoba mi się temat i treść pracy :) Naprawdę jestem ciekaw co z tego wyjdzie.

Pytanie: Czy upublicznisz ten projekt?
Pozdrawiam
Piotrek


 
Wiktor
Oct 24, 2007 at 12:46 pm

Jak najbardziej mój projekt będzie upubliczniony. Mam nadzieje, że będę mógł go pokazać już na początku lutego (oby oby).

Dzięki za znalezienie literówki. Jest to mój błąd i jest on już poprawiony.

Co do czcionek to używam mniejszych czcionek w podpunktach (ich kolor jest szarawy). Jest to moje świadome działanie, może faktycznie na przyszłość powinienem to zmienić.

Dzięki!

(mam nadzieję, że slideshare zaraz uaktualni prezentację ;) )


 
Marcin
Oct 26, 2007 at 12:26 am

Szacunek co do prezentacji:) Fajnie, ze postanowiles z tym zawalczyc. Tak sie sklada, ze w obecnym projekcie robimy przeglady “nieblokujace” typu “na wyrywki”, bo najzwyczajniej nie ma czasu na doglebna analize kazdego checkina. I tak sie ciesze, ze udalo mi sie przekonac Klienta co do zalet tego typu rozwiazania (chociaz troche okrojonego). Jesli ktos zakomituje kod z razacymi bledami (nie chodzi mi w tym przypadku raczej o problemy typu nazewnictwo) to po review z wynikiem negatywnym poprawia je i umieszcza w repozytorium ponownie. W sumie to dziala. Mam problem z jednym czlonkiem zespolu, ktory kazda krytyke przyjmuje bardzo osobiscie i niepotrzebnie sie stresuje, marnujac czas na udawadnienie, ze to nie jego wina itp. Mysle, ze krotki rozdzial o sposobie postepowania w takich przypadkach bylby calkiem ciekawy. Z niecierpliwoscia czekam na dalsze postepy (a przede wszytkim wynikowy soft). Na jutrzejszej konferencji mnie nie bedzie (deadline :/ ), ale moze wieczorem…


 
J.
Oct 27, 2007 at 11:13 pm

Pomysł oki, prezentacja nie do przyjecia.


 
Tomek
Oct 29, 2007 at 8:07 pm

To konca kwietnia chcesz napisac ? Dobre !

No ale probuj… ;-)


 
Dominik
Mar 19, 2009 at 4:05 pm

Przejrzyj sobie narzędzia atlassiana, zwłaszcza:
http://www.atlassian.com/software/crucible/


 
Pawel Szulc
May 28, 2009 at 8:48 am

no i co klepnales to w koncu? :)


 

Reply

Copyright © 2010 Mocna Kawa All rights reserved. Theme by Laptop Geek.