Nie ma to jak przeczytać z rana parę miłych słów o swoim kodzie. (⌐ ͡■ ͜ʖ ͡■)
#chwalesie #geektastic #programowanie #programista15k #codereview
#chwalesie #geektastic #programowanie #programista15k #codereview
Wszystko
Najnowsze
Archiwum
Warsztaty dla Początkujących z Jakości Kodu #5: Python
https://forum.pasja-informatyki.pl/477727/wirtualne-warsztaty-z-jakosci-kodu-dla-poczatkujacych Discord Chat: https://discord.gg/QAwfE5R Powered by Gynvael C...
z- 0
- #
- #
- #
- #
- #
@informatyk: gdzie to takie pochwały się dostaje? ( ͡° ͜ʖ ͡°) Pokaż ten piękny kod przy okazji ( ͡° ͜ʖ ͡°)
@informatyk: szkoda, że nie ma moich technologii, ale zdziwiłbym się jakby były.
#programowanie #heheszki #codereview
Na jakie najgłupsze fragmenty kodu trafiliście w życiu? Ja notorycznie u 'młodzików' po studiach widzę takie konstrukcje:
Na jakie najgłupsze fragmenty kodu trafiliście w życiu? Ja notorycznie u 'młodzików' po studiach widzę takie konstrukcje:
class ArrayManagement {
//
Komentarz usunięty przez moderatora
@nadmuchane_jaja: Tak z innej beczki widziałem jak podczas "kopiowania" zadania i zmiany nazw zmiennych i funkcji, gość zmienił super na ekstra ( ͡° ͜ʖ ͡°)
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
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
źródło: comment_6dJAk5WrEr1gBQsGAgQP8sR44rnziu3B.jpg
Pobierz
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"?
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"?
@that_ugly_dude: https://www.michaelagreiler.com/code-reviews-at-microsoft-how-to-code-review-at-a-large-software-company/
Dzięki za uwagi, pozdrawiam.
Dzięki za uwagi, pozdrawiam.
Code review. Dlaczego warto praktykować i jak idealnie zaplanować przegląd kodu?
Programowanie najczęściej wiąże się z tworzeniem kodu. Weryfikacją tego, jak deweloper poradził sobie z zadaniem, zajmuje się dział testów. Jednakże nie tylko wynik, ale także sposób w jaki osiągnięto dany rezultat jest istotne. W takim przypadku niezmiernie ważnym aspektem jest sprawdzenie kodu...
z- 0
- #
- #
- #
- #
Hej programistyczne świry, chciałby ktoś zrobić mi małe #codereview? Napisałem kilka testów w #nunit i #csharp (#dotnet dla zasięgu), ale nie wiem czy poziom refaktoryzacji jest zadowalający :C
https://pastebin.com/sJFhi1qa
( ͡° ͜ʖ ͡°)
https://pastebin.com/sJFhi1qa
( ͡° ͜ʖ ͡°)
@Hopsiup-siup: popatrzyłam co tam pisałam te trzy lata temu i widzę że coś p---------m bez składu, ale nie ma za co ( ͡° ͜ʖ ͡°)
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
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
@GrubyCzarny: brak hashcode, niepoprawny zwracany typ, klasa util posiadająca stan, niekompletny javadoc, niewydajny algorytm
@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.
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 zamiastPrzegląd kodu - dlaczego, jak to robić, jakich narzędzi używać?
Godzinne wystąpienie na temat przeglądów kodu (code reviews), dobry przelot przez temat dla początkujących i doświadczonych.
z- 0
- #
- #
- #
- #
- #
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ąć
Cześć,
miesiąc temu zacząłem naukę #python i popełniłem poniższy mini program.
Czy ktoś mógłby zerknąć i wytknąć
@Wujek_Jaskra: czasami używasz polskiego, czasami angielskiego, ujednolić to
o zmiennych nazwanych
korzystaj z jakiegoś IDE które będzie Cię strzelać po łapach za kiepskie formatowanie (np.
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, widthDzięki za zerknięcie i poświęcony czas :)
#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
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
źródło: comment_vLorOoPaF2uI3CLMbBss0vcuRl0hst67.jpg
Pobierz
@Seveir: Najpierw to zrób sobie language review i pozmieniaj te bolesne, wszechobecne ortografy... "expenses", nie "expences".
@Seveir:
tak na szybko:
-brak walidacji po angularowej stronie
- ux leży (warto poczytać ewentualnie popatrzeć na konkurencyjne strony),
-wywalasz stack co ci się dokładnie wysypało do klienta ->
https://expences-app.herokuapp.com/add-expenses dodaj + add all zwraca 500 + stack co
tak na szybko:
-brak walidacji po angularowej stronie
- ux leży (warto poczytać ewentualnie popatrzeć na konkurencyjne strony),
-wywalasz stack co ci się dokładnie wysypało do klienta ->
https://expences-app.herokuapp.com/add-expenses dodaj + add all zwraca 500 + stack co
@Niestabilny: reakcja jest na kod ludzi, a nie sam fakt istnienia codereview
@deccan: przecież działało bez tego, po co to zmieniać?
@enceladus71: D--a Debuging. W kernelu Linuksa też parę k**w poleciało przed „ocenzurowaniem”.
@enceladus71: W kodzie na PLC znalazłem kiedyś "parówka się zesrała".
#java #javapomoc <-- obserwuj proszę #codereview
https://gist.github.com/enseitankad0/ce1471b67069f91ece848b6a97d10222
Mogę prosić o code review? W sumie programik generuje losowe auta, wkłada je do pliku .dat i .txt a następnie czyta. To wszystko. Zastanawiam się czy coś dało się prościej porobić albo czy coś nie wygląda jak januszerka.
Pozdrawiam
https://gist.github.com/enseitankad0/ce1471b67069f91ece848b6a97d10222
Mogę prosić o code review? W sumie programik generuje losowe auta, wkłada je do pliku .dat i .txt a następnie czyta. To wszystko. Zastanawiam się czy coś dało się prościej porobić albo czy coś nie wygląda jak januszerka.
Pozdrawiam
@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" +
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" +
@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
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
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
[[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
#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
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
@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.
Dodatkowo w pierwszym kodzie nazwy zmiennych powinny być lepiej dobrane.
@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 :)
#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
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
źródło: comment_xOZ5XIsYX7Zu4pcdVZrgme9RRP8P6Xqb.jpg
Pobierz
@borrow: nie pisz if(statement==true) tylko samo if(statement) samo ide ci nawet podpowiada
@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..
Hej
Prośba o codereview. Piszę na zaliczenie grę w statki.
Opis ogólny co gdzie i jak:
Pakiety:
Prośba o codereview. Piszę na zaliczenie grę w statki.
Opis ogólny co gdzie i jak:
Pakiety:
@krasnoludkolo
Lombok. Meh.
Lombok. Meh.
@BosmanPociagowy: dzięki ;)
Code Review. 7 Mistakes You May Be Making
What and how to avoid during the code review process to make the most out of it.
z- 0
- #
- #
- #
- #
- #
- #
7 błędów code review
Code review może bardzo poprawić jakość kodu. Jest tylko jeden warunek - by wykorzystać ten potencjał, trzeba je odpowiednio przeprowadzić. Bez zbędnego gad...
z- 0
- #
- #
- #
- #
- #
- #
Jest to mój pierwszy program poprowadzony od początku, do końca
Pisałem w pythonie korzystając z biblioteki tkinter
https://pastecode.xyz/view/16c6a38f
#python #naukaprogramowania #programowanie
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