Wszystko

Najnowsze

Archiwum

Surykat
  • 4
Popełniłem kolejny wpis na bloga, tym razem w temacie rewizji/przeglądu kodu (Code Review) i wskazówek oraz dobrych praktyk, dzięki którym robi to nam w ekipie mega robotę. Chętnie poznam inne podejścia i tipsy. Dzielę się, bo może komuś pomoże

https://wearecogworks.com/blog/7-tips-for-more-effective-code-reviews

#naukaprogramowania #programowanie #codereview
  • 2
  • Odpowiedz

  • Otrzymuj powiadomienia
    o nowych komentarzach
dzejro
  • 5
@TheBloody: zmienne po polsku - do w-------a, brak podziału na jakieś pliki, czy klasy, nie mówiąc juz o oddzieleniu tworzenia widoku od logiki kalkulatora. Mówiąc najprościej pomieszanie z poplątaniem - do przepisania.
  • Odpowiedz
kao3991
  • 6
@TheBloody:
1. # Importowanie biblioteki tkinter i skrócenie jej nazwy dla ułatwienia dalszego wywoływania, tzw "alias"- ten i inne podobne komentarze są raczej zbędne - z kodu wynika co to jest i jest to czytelne bez komentarza.
2. nazewnictwo.
a) o wiele lepiej używać angielskiego. Trochę gorzej używać polskiego. najgorzej - mieszać to bez uzasadnienia (przy czym uzasadnienie to np. nazwy własne których wygodnie użyć, a nie widzimisię)
b) nazywaj rzeczy
  • Odpowiedz
djfoxer
  • 0
Code review – dlaczego warto praktykować i jak idealnie zaplanować przegląd kodu

Przemyślenia o code review, czym jest, po co je robić i na co warto zwrócić uwagę plus trochę statystyk i dobrych rad. Bazując na własnych doświadczeniach oraz badaniach przeprowadzonych pośród developerów w Microsofcie.

#programowanie #programista15k #codereview
djfoxer - Code review – dlaczego warto praktykować i jak idealnie zaplanować przegląd...

źródło: comment_6dJAk5WrEr1gBQsGAgQP8sR44rnziu3B.jpg

Pobierz
  • 2
  • Odpowiedz

  • Otrzymuj powiadomienia
    o nowych komentarzach
that_ugly_dude
  • 4
takie trochę nadmiarowe, dużo słów, a mało treści

dodałbym jeszcze jakis prosty schemacik jak wygląda flow od commitu do merge'a, po drodze testy automatyczne + właśnie review

- "Weryfikacja kodu (code review)" - to chyba nie jest właściwie tłumaczenie
- "Code review to również idealny sposób na sprawdzenie defektów kodu. W ten sposób można przeprowadzić testy statyczne, " - ocena kodu to "sposob przeprowadzenia analizy statycznej"?
  • Odpowiedz
GrubyCzarny
GrubyCzarny
  • 0
Siema mirki,prośba o pomoc.
Mam za zadanie przeanalizować ten kod,i wytknąć wszelkie uwagi.
Wymaganie jest proste,z podanego Stringa wypisujemy wszystkie słowa i liczbę ich wystąpień
https://pastebin.com/vJApbEVH

Na razie wymyśliłem że
  • 8
  • Odpowiedz

  • Otrzymuj powiadomienia
    o nowych komentarzach
63274682374
63274682374
  • 2
@GrubyCzarny: Zacząć przede wszystkim trzeba od tego, że algorytm jest błędny.
1. Przy podziale na słowa nie uwzględnia znaków interpunkcyjnych
2. W equals porównuje również count co samo w sobie błędem nie jest ale ten sam equals używany jest do wyszukiwania na liście i słowa występujące 2 i więcej razy znalezione już nie będą.

Dalej, jeśli ten algorytm miałby zostać, to:
1. Word word = new Word(values[i], 1); powinno być w else zamiast
  • Odpowiedz
Wujek_Jaskra
Wujek_Jaskra
  • 2
Wrzucam jeszcze raz ( ͡° ͜ʖ ͡°)

Cześć,
miesiąc temu zacząłem naukę #python i popełniłem poniższy mini program.
Czy ktoś mógłby zerknąć i wytknąć
  • 4
  • Odpowiedz

  • Otrzymuj powiadomienia
    o nowych komentarzach
qbek
qbek
  • 6
@Wujek_Jaskra: czasami używasz polskiego, czasami angielskiego, ujednolić to
o zmiennych nazwanych aa już nic nie powiem ;-)

korzystaj z jakiegoś IDE które będzie Cię strzelać po łapach za kiepskie formatowanie (np.

self.tekst1 = Entry(okno, textvariable = self.var, width
  • Odpowiedz
Seveir
  • 3
#programowanie #java #spring #codereview

Kiedyś już prosiłem o code review mojego innego projektu i pamiętam, że dało mi to dużo wskazówek do przemyślenia. Dlatego zwracam się do Was lepsi programiści o zerknięcie na mój projekt API do zarządzania wydatkami: .

Tutaj sobie to implementuje używając Angulara, więc można się pobawić.

Takie
Seveir - #programowanie #java #spring #codereview Kiedyś już prosiłem o code revi...

źródło: comment_vLorOoPaF2uI3CLMbBss0vcuRl0hst67.jpg

Pobierz
  • 14
  • Odpowiedz

  • Otrzymuj powiadomienia
    o nowych komentarzach
bohme
bohme
  • 3
@borrow
1. Wszędzie gdzie masz komentarze w stylu: // 1. WRITE TO DAT FILE powinna być nowa metoda.
2. Po co robisz coś takiego: Integer k = Integer.valueOf(i);? zobacz jakim typem jest i
3. Po co klasa Garage implementuje interfejs Map? Przez to niepotrzebnie masz takie kwiatki jak metoda entrySet która zwraca nulla.
4. Formatowanie kodu!! Puste linie, wcięcia itp...
5. Taki catch niewiele daje: catch(IOException io) {System.out.println("IO Exception" +
  • Odpowiedz
A.....t
A.....t
  • 2
@borrow:
1. zamiast zapisu do takiego pliku txt mógłbyś pomyśleć o csv
2. zmienne w Car mogą być final
3. klasa Garage według mnie jakaś taka bez sensu - część operacji implementujących interfejs map działa poprawnie, część niepoprawnie, zastanowiłbym się czy rzeczywiście warto implementować mapę
4. Ta metoda printGarage po pierwsze nie printuje, po drugie zwraca reprezentacje tekstową pierwszego samochodu lub nulla - zła jest albo nazwa funkcji albo implementacja, a w ogóle
  • Odpowiedz
rtrv
  • 2
Mirki, uczę się #php i zrobiłem sobie zadanko zapostowane jakiś czas temu na mirko, starając się stosować w praktyce rzeczy, których ostatnio się nauczyłem. Mógłbym prosić kogoś mądrego o rzuceniem okiem na kod (nie ma tego dużo) i wytknięcie błędów / złych praktyk?

[[Link do githuba]](https://github.com/retrowaver/Transiter)

Uwagi: Starałem się stosować MVC (ale jakby bez View, bo tu sprowadza się tylko do wyświetlenia JSONa). Pierwszy raz ever pisałem testy
  • 18
  • Odpowiedz

  • Otrzymuj powiadomienia
    o nowych komentarzach
borrow
  • 1
#java #javapomoc #codereview

Czy ten kod jest poprawny? Jak dostanę pytanie żeby wprowadzić liczbę sekund i zamienić na format HH:MM:SS to jak oddam taki kod do będzie ok? Lepsza wersja 1 czy 2? Obie dają ten sam wynik

// input i is time in seconds
// it's being converted to format HH:MM:SS
  • 4
  • Odpowiedz

  • Otrzymuj powiadomienia
    o nowych komentarzach
63274682374
63274682374
  • 1
@borrow: Jeśli nie ma limitu na liczbę sekund na wejściu to kod1, jeśli na wejściu nie będzie więcej niż 86399s to kod 2

Dodatkowo w pierwszym kodzie nazwy zmiennych powinny być lepiej dobrane.
  • Odpowiedz
b.....f
b.....f
  • 1
@borrow: Jak to taka typowa rozmowa przy tablicy to napisz na szybko to pierwsze, bo nikomu nie chodzi o to żebyś pamiętał nazwy metod i bibliotek na pamięć - ale możesz wspomnieć, że w kodzie produkcyjnym użyłbyś gotowej metody z java.time :)
  • Odpowiedz
borrow
  • 1
#java #programowanie #codereview

Mam taką sytuację. Chodzi o hermetyzację.
1.Czy jest ona tutaj zaimplementowana poprawnie?
2.Czy lepiej skorzystać z drugiej metody z użyciem getterów i drukować wartości w klasie "Main" a nie w klasie "Printer"? Czy lepiej zostawić drukowanie informacji metodzie Printer?

Pomijając błędy takie jak, że numberOfPages nie powinno być w konstruktorze tylko zdefiniowane w mainie to jest ok? Jest rozwiązanie tego zadania, ale nie
borrow - #java #programowanie #codereview Mam taką sytuację. Chodzi o hermetyzacj...

źródło: comment_xOZ5XIsYX7Zu4pcdVZrgme9RRP8P6Xqb.jpg

Pobierz
  • 14
  • Odpowiedz

  • Otrzymuj powiadomienia
    o nowych komentarzach
P.....s
P.....s
  • 4
@Saly: @PoteznyNick: Przecież to jest jakieś zadanko z podstaw programowania na studiach czy czegoś takiego, nie mieszajcie mu w głowie na początku :) Mozę jeszcze ma robić to na intefejsach a później robić implementacje interfejsu do komunikacji przez wiersz poleceń xd..
  • Odpowiedz

Powiązane tagi