Mocna Kawa

Code-review, czyli przegląd kodu

napisane przez wiktor, 17:53 10-22-2007

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:

| View | Upload your own

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?

• Różne

Komentarzy (11)

1. Grzegorz napisał(a):
   22:52 10-22-2007

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

2. wiktor napisał(a):
   1:23 10-23-2007

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

3. Someone napisał(a):
   11:47 10-23-2007

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

   I powodzenia przy pisaniu.

4. wiktor napisał(a):
   11:56 10-23-2007

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

5. Rafał napisał(a):
   15:26 10-23-2007

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

6. wiktor napisał(a):
   15:37 10-23-2007

   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.

7. Piotrek napisał(a):
   8:30 10-24-2007

   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

8. Wiktor napisał(a):
   12:46 10-24-2007

   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ę ;))

9. Marcin napisał(a):
   0:26 10-26-2007

   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…

10. J. napisał(a):
    23:13 10-27-2007

    Pomysł oki, prezentacja nie do przyjecia.

11. Tomek napisał(a):
    20:07 10-29-2007

    To konca kwietnia chcesz napisac ? Dobre !

    No ale probuj…

Zostaw komentarz