dotnetomaniak.pl

dotnetomaniak.pl - Artykuły z tagiem Code review

~ MJ Code review, czyli przeglądanie kodu innego programisty w zespole, uważam za jedną z najlepszych technik tworzenia dobrego kodu i osobistego rozwoju – niezależnie od tego, czy ktoś przegląda moją pracę, czy ja jego. Pamiętam jak dziś swoje pierwsze code review. Ukończyłem pisanie jednego dosyć sporego modułu. Przyzwyczajony do tego, że zawsze po prostu commitowałem swój kod, który działał, myślałem, że jestem prawdziwym artystą, a mój kod jest perfekcyjny. Och, jakże byłem głupi. Review zakończyło...

Code Review c.d. | maciejjędrzejewski

Sztuka programowania 123 dni, 18 godzin, 37 minut temu jedmac 250 rozwiń

Code review is an important part of development process. Various teams use different styles from “over the shoulder” to the ones based on tools like Gerrit. However, more and more Git servers (like GitLab) give merge request functionality, which makes incorporating code review process easier.

Dziel się z innymi:
[EN] Using Slack With .NET To Speed up Code Review Process | Future Processing

Sztuka programowania 226 dni, 20 godzin, 25 minut temu FutureProcessing 206 rozwiń

Szczęśliwy, trzynasty, odcinek to chwilowy odpoczynek od technikaliów. Tym razem wraz z Grzegorzem Rycajem serwujemy Wam dywagacje na popularny temat: agile. Grzegorz od wielu lat programuje i kieruje zespołami programistów. Prawdopodobnie wielu z Was niejednokrotnie miało okazję oglądać go na scenie, gdyż regularnie występuje na różnych eventach. MVP w kategorii Visual Studio ALM. 40-minutową rozmowę rozpoczynamy od historii agile. Nie zagłębiamy się jednak w teorię...

Źródło: devtalk.pl
Dziel się z innymi:
13 - O agile z Grzegorzem Rycajem | DevTalk

Inne 746 dni, 20 godzin, 4 minuty temu tuszcz 105 rozwiń

One can ask why we need to think of CR practices at all. Review of code written by another team member can be done without any preparation or introduction. It is true, but there are some aspects that we need to think of. In my opinion, we especially need to support the following values:Quality – Code Review should be done in the same way, no matter if you are hurrying, bored or it’s your 3rd hour of doing CR,Motivation – team has to see value in Code Review, and this value shouldn’t be lost because of ne...

[EN] Yet Another Code Review Best Practices | Future Processing

Sztuka programowania 822 dni, 12 godzin, 48 minut temu FutureProcessing 183 rozwiń

Ostatnio na facebookowym profilu .NET Developers Poland odbyła się ciekawa, choć krótka dyskusja na temat podejścia do przeglądu kodu (po polsku brzmi to okropnie, więc dalej będę używał code review). Przypomniała mi ona, że pewna dobra praktyka, bez której chyba nie mógłbym już pracować, nie jest wcale tak powszechnie stosowana. O niej chciałbym napisać i zachęcić wszystkich do wdrożenia jej u siebie w zespole. Na-ten-tychmiast! Chodzi mi o peer code review, czyli...

Źródło: jakubin.pl
Dziel się z innymi:
Peer code review

Narzędzia 882 dni, 7 godzin, 44 minuty temu Jakub Binkowski 277 rozwiń

Dzisiaj coś o Code Review, czyli przeglądaniu własnego kodu przez innego członka zespołu. Jak to działa? Wyjaśnię to na podstawie Scrum’a. Rozpoczyna się sprint, każdy backlog item ma w sobie różne taski. Każdy task ma wyestymowany czas potrzebny na wykonanie zadania, np. Handle operation documentsCreate database structure 4hPrepare model changes 2hCreate data structures 4hCode Review 2hRework 3h Jak widać, również Code Review oraz Rework jest wyestymowany. Jak wygląda to w praktyce? Robimy pełen dev...

Dziel się z innymi:
Code Review – parę słów na temat | maciejjędrzejewski

Sztuka programowania 1123 dni, 19 godzin, 46 minut temu jedmac 187 rozwiń

Witajcie w Coding News – serii screencastów, w której omawiam najciekawsze wydarzenia i znaleziska minionego tygodnia.

Coding News 17.02.2014 | codingtv();

Web 1152 dni, 8 godzin, 6 minut temu lukaszgasior 152 rozwiń

Dzisiejszym wpisem odbiegnę trochę od typowego dla mojego bloga technicznego bełkotu, zamiast tego skupiając się dziś bardziej na zagadnieniu związanym z samym procesem (cyklem?) wytwarzania oprogramowania. W większości firm tworzących soft, w której przykłada się wagę do jakości tworzonego oprogramowania prowadzona jest praktyka Code-Review. W mojej osobistej karierze zawodowej, tak na prawdę zetknąłem się z tym zagadnieniem dopiero odkąd rozpocząłem pracę w Objectivity, a jako że pracuję już tutaj pra...

Tagi: Code review
Dziel się z innymi:
burczu programator - blog: Code-Review – jak i dlaczego?

Sztuka programowania 1161 dni, 18 godzin, 13 minut temu burczu 284 rozwiń

Przychodzi baba do lekarza. Mówi “panie doktorze, byłam już u doktora X i on zalecił mi Y“. Na co lekarz: “a-HA! znam X, to konował, nic nie umie, tak naprawdę to dopiero JA pani powiem co trzeba zrobić“. Baba jest od tej pory zakochana w swoim nowym doktorze, nie mając tak naprawdę pojęcia czy faktycznie jest tak dobry jak twierdzi. Chodzi po sąsiadac...

Dziel się z innymi:
Nie krytykuj | Maciej Aniserowicz o programowaniu

Inne 1509 dni, 1 godzinę, 33 minuty temu psz750 269 rozwiń

Już kiedyś pisałem jak obsługiwać prawidłowe wyjątki ale dzisiaj jeszcze raz chciałbym rozwinąć temat. Zacznijmy od:privatestring GetData(int id) { string result=null; try { result = _service.GetData(id); } catch(Exception e) { } return result; } Jest to oczywiście skrajnie złe rozwiązanie ponieważ wszystko ignorujemy. Na szczęście programiści rzadko popełniają powyższy błąd. Niestety dużo częściej popełnianym błędem jest:privatestring GetData(int id) { stri...

Dziel się z innymi:
Piotr Zieliński » Code review: zła obsługa wyjątków

Sztuka programowania 1575 dni, 3 godziny, 10 minut temu pzielinski 166 rozwiń

Rozważmy, następujący kawałek kodu:struct Color { publicstaticexplicitoperator Color(short value) { // jakas tam konwersja - nieistotne dla przykladureturnnew Color {R = value}; } publicshort R, G, B; } Co w nim nie tak? Operatory nie są wspierane przez wszystkie języki. Podczas kompilacji wszystkie operatory zamieniane są na zwykłe metody. Na przykład operator rzutowania implicit zostanie zamieniony na op_Explicit oraz oznaczony  przez CLR specjalnym atrybutem specialname:.class seque...

Dziel się z innymi:
Piotr Zieliński » Code review: operator rzutowania

Sztuka programowania 1665 dni, 17 godzin, 38 minut temu pzielinski 146 rozwiń

Kod:if(anyText.ToLower()=="tekst") { Console.WriteLine("Zmienne takie same"); } Kod ma na celu sprawdzenie czy jakaś zmienna jest równa danemu strumieniowi znaków. Nie chcemy brać pod uwagę wielkości liter więc dlatego używamy funkcji ToLower. Zatem if zwróci true gdy anyText jest równy “tekst” lub “TEKST” itp. Rozwiązanie ma jedną wadę – tworzony jest nowy, tymczasowy string po wywołaniu metody ToLower. W powyższym przykładzie będziemy mieli zatem 3 obiekty: anyText, obiekt dla “tekst” oraz obie...

Tagi: C#, Code review
Dziel się z innymi:
Piotr Zieliński » Code review: porównywanie string’ów

Inne 1692 dni, 20 godzin, 25 minut temu pzielinski 201 rozwiń

Dziś trochę o matematycznej walidacji kodu czyli o metrykach.  LCOM (lack  cohesion of methods) umożliwia obliczenie jak bardzo nasze metody są spójne. Do obliczania tej wartości istnieją gotowe narzędzia takie jak nDepend, ale aby w pełni zrozumieć znaczenie metryki obliczymy sami jej wartość dla prostej klasy. Powinno dążyć się do jak największej spójności a tym samym do jak najniższej wartości LCOM. Przed wyjaśnieniem czym jest metryka LCOM wprowadźmy następujące pojęcia: M – zbiór metod danej klasy...

Tagi: C#, Code review
Dziel się z innymi:
Piotr Zieliński » Code review: Badanie spójności kodu

Sztuka programowania 1737 dni, 22 godziny, 4 minuty temu pzielinski 219 rozwiń

Rozważmy następujący kod:IEnumerable

Tagi: Code review, Linq
Dziel się z innymi:
Piotr Zieliński » Code review: Pułapka z LINQ

Inne 1904 dni, 8 godzin, 3 minuty temu pzielinski 446 rozwiń

W postach oznaczonych “Code review” zamierzam przedstawiam dobre i złe praktyki pisania kodu. Posty przeważnie będą składały się z fragmentu kodu i opisu co jest w nim źle. Dziś spójrzmy na:StreamReader streamReader =new StreamReader(File.Open(path, FileMode.Open)); string header = streamReader.ReadLine(); string dataPoints = streamReader.ReadLine(); string[] dataPointArray = dataPoints.Split(newchar[]{';’});streamReader.Close(); Co w tym jest jest nie tak? W przypadku gdy np. ReadLine wyrzuci wyjątek, ...

Tagi: Code review
Dziel się z innymi:
Piotr Zieliński » Code review: słowo kluczowe using

Inne 1908 dni, 4 godziny, 59 minut temu pzielinski 407 rozwiń

Czy na pewno code review to strata czasu? Cóż może być nudniejsze i bardziej kosztowne niż przeglądanie cudzego kodu źródłowego? Chyba już tylko poprawnianie w nim błędów. Jeszcze kilka lat temu koncepcja przeglądania mojego kodu przez kolegów z zespołu była dla mnie totalną abstrakcją. Aczkolwiek, z czasem zrozumiałem, że w tym szaleństwie jest metoda. W kilku punktach chciałbym przedstawić benefity, które może wprowadzić Code Review do Twojego ekosystemu projektowego drogi Czytelniku. Edukacja M...

Dziel się z innymi:
Code review, lubię to!

Inne 1918 dni, 19 godzin, 48 minut temu paduda 337 rozwiń

Szkolenia SecurITum

kwiecień

26.
CodeEurope
Kraków

maj

17.
InfoShare
Gdańsk
23.
CodeEurope
Wrocław
Zobacz wszystkie

Najaktywniejsi

1

macko (32 816,53)

2

http://pawlos.blo... (31 603,45)

3

pzielinski (27 178,29)

4

gordon_shumway (21 178,87)

5

paduda (20 336,33)

6

psz750 (13 018,14)

7

rroszczyk (10 383,88)

8

Damian (9 041,08)

9

danielplawgo (7 235,99)

10

arek (6 807,91)

11

burczu (6 214,22)

12

PaSkol (5 393,84)

13

lukaszgasior (4 097,38)

14

jj09 (3 578,06)

15

jedmac (3 313,39)

16

http://jakub-flor... (3 224,66)

17

CaMeL (2 954,87)

18

spetz (2 783,21)

19

mnikolajuk (2 596,93)

20

lkurzyniec (2 590,68)