Wszystko

Najnowsze

Archiwum

cat_tea
  • 7
Czy tylko ja uważam, że pisanie komentarzy na code review obecnie straciło jakikolwiek sens?

Kiedyś był to mentoring, gościu poprawiał i na przyszłość już robił lepiej. Nas mniej głowa bolała jak trzeba było potem utrzymywać ten kod.

Dzisiaj - jest to pisanie komuś promptów, z których nic dodatkowego nie wyniknie na przyszłość. Ani taki typek kodu sam nie napisał, ani sam go nie poprawi, a AI też się niczego z komentarzy nie
  • 9
  • Odpowiedz

  • Otrzymuj powiadomienia
    o nowych komentarzach
PinezkaSwiata
  • 3
@cat_tea: ja się nawet ostatnio spotkałem że ktoś robił CR dzięki AI, a druga osoba potem poprawiala to co te AI znalazło, używając AI. Nikt z ich zapewne nie wie co dokładnie napisał i jak to było sprawdzone ale CR odfajkowane, pora na cs'a
  • Odpowiedz
suvroc
suvroc
  • 2
Serdecznie zapraszam do mojego najnowszego projektu.
Adwentowe Wyzwanie Security Code Review
To zbiór 24 codziennych lekcji z zakresu weryfikacji bezpieczeństwa kodu źródłowego wraz z zadaniami.

📅 Kiedy? 1-24 grudnia.
💰 Cena? Za darmo.
  • 4
  • Odpowiedz

  • Otrzymuj powiadomienia
    o nowych komentarzach
a.....e
a.....e
  • 3
Proszę, u mnie nie potrzebujesz nawet 80%. Jaką instytucją jesteś, że ten certyfikat ma mieć jakąkolwiek wartość? Powiedz, że chcesz listę mailingową żeby opchnąć jakiegoś ebooka naiwnym, to będzie chociaż uczciwie.
a.....e - Proszę, u mnie nie potrzebujesz nawet 80%. Jaką instytucją jesteś, że ten c...

źródło: 1000002135

Pobierz
  • Odpowiedz
suvroc
suvroc
  • 0
Ciekawe. Pierwszego linku nie znałem. Przyjrzę się dokładniej tej formie.

Choć próbowałem kiedyś robić pełne i zawsze dostępne programy szkoleniowe, ale ludzie tego wtedy nie kończyli (dojście do połowy już było sukcesem).
Dlatego taka forma mailowa to próba wdrożenia gamifikacji w proces nauki. A certyfikat/dyplom czy stempel z ziemniaka ma motywować (bo pamiętaj, że każdy inaczej rozumie wartość takiego certyfikatu), żeby regularnie wykonywać codzienne ćwiczenie.

Dałoby się to zrobić też w formie
  • Odpowiedz
zwei
  • 4
@ozzi91: chcesz, żeby twój program był używany przez kogoś. Jest to program działający tylko na windowsie. Ludzie używają innych systemów. Ale to tam już jak wolisz.

Osobiście podzieliłbym tak:

- klasy typu DocxFile, XlsxFile, Application (w sensie *Wniosek*) w osobnych modułach
- potem dziedziczące (dziedzieczenie zazwyczaj jest złe, ale nie zawsze – tu pasuje)
  • Odpowiedz
Arogancky
  • 1
@ozzi91: jeszcze sprawdzaj czy ścieżka istnieje: os.path.isdir() a zamiast helperow fizycznych to możesz próbować mieć coś w pamięci jeśli danych nie jest tona
  • Odpowiedz
janek_
  • 0
jak to zrobić, żeby było SOLID?
#python #programowanie #naukaprogramowania #codereview

mam sobie dwie klasy "rzecz" i "kontener na rzeczy" - jaki jest najlepszy sposób by każda z rzeczy mogła "powiedzieć", gdzie się znajduje? (zakładamy, że każda rzecz jest w jakimś konkretnym jednym kontenerku)? jak to najładniej zaprojektować?

mam taki przyklad: https://pastebin.com/JgLYYbur i proszę o opinie :)
(sorry za screenshot, wykop źle formatuje kod)
janek_ - jak to zrobić, żeby było SOLID? #python #programowanie #naukaprogramowania ...

źródło: comment_16603016323CHy3TqCSksDMaTNJsbAWq.jpg

Pobierz
  • 8
  • Odpowiedz

  • Otrzymuj powiadomienia
    o nowych komentarzach
PaaD
  • 1
@janek_: Hasła: agregacja oraz kompozycja - potem z dobrego zrozumienia agregacji można zrozumieć skąd bierze się agregat w DDD

W dużym skrócie wszystko zależy od tego czy obiekt może żyć / mieć własną tożsamość bez obiektu rodzica.

Przykładowo:
- książka może spokojnie istnieć
  • Odpowiedz
joolekk
joolekk
joolekk
via Wykop Mobilny (Android)
  • 5
@Rst00: co do pierwszego punktu się nie zgadzam. Bardzo często ustawienia dotyczące języka są napisane właśnie w danym języku, mamy więc listę w stylu: English, Deutsch, Francois, polski

chiński, japoński, rosyjski, ukraiński, itp też są zapisane własnym alfabetem
  • Odpowiedz
ProGraMajster
ProGraMajster
  • 3
Cześć!
Od jakiegoś dłuższego czasu piszę w csharp i tak się zastanawiam jak u mnie z jakością kodu...
Byłby ktoś chętny przejrzeć mój projekt?
Chętnie bym przyjmie jakieś rady itp.

Link do projektu -> link
  • 26
  • Odpowiedz

  • Otrzymuj powiadomienia
    o nowych komentarzach
PotatoZiemniak
PotatoZiemniak
PotatoZiemniak
via Wykop Mobilny (Android)
  • 5
@ProGraMajster: try z pustymi catch, a inne z jakimś dzikim wypisywaniem do konsoli, inne nazwy klasy a inne pliku (form1.cs), część nazw metod po polsku a część po angielsku, troche zduplikowanego kodu też masz.
  • Odpowiedz
radek1122012
radek1122012
  • 3
#python #django #codereview

Cześć. Właśnie skończyłem pisać projekt. Jest to prosta aplikacja quizowa napisana w Django. Byłbym wdzięczny, gdyby ktoś zechciał spojrzeć na kod i podpowiedzieć jakie błędy popełniam. Jest to moja druga, prawie w pełni napisana przeze mnie, aplikacja w Django, więc proszę o wyrozumiałość. Z góry dziękuję za wszelkie uwagi.

Link do repozytorium: https://github.com/rcybulski1122012/django_quiz_app
  • 3
  • Odpowiedz

  • Otrzymuj powiadomienia
    o nowych komentarzach
Xaaq
  • 2
@login_zajety_sic:
1. Linijki 17-22 - masz zduplikowaną linijkę zwracającą ścieżkę, można to zrobić mniej więcej tak
2. Jeśli chcesz połączyć stringi ze zmiennymi to używaj Pythonowego formatowania stringów - a najlepiej f-stringów.
3. Jeśli robisz operacje na ścieżkach do plików to używaj os.path.join() lub Path z modułu pathlib - one zajmą się wstawieniem slashy lub backslashy (zależnie od systemu), więc nie będziesz musiał ich ręcznie wstawiać i escape'ować.
4. Komentarze, które mówią oczywiste
  • Odpowiedz
informatyk
  • 1
@login_zajety_sic: osobiście jeszcze trochę popracowałbym tu na modularyzacją. Więcej mniejszych funkcji, dzięki czemu kod staje się samoopisowy i bardziej czytelny, a przy tym możesz unikać zbędnych komentarzy.

Dobrymi (choć nie jedynymi) kandydatami na osobne funkcje są np. bloki z linii 17-22 oraz 58-99.

Lepiej będzie też wyglądało jeśli początek wykonywania programu będzie jasno określony (oczywiście przed doSomething powinno być wcięcie, tylko wykop je
  • Odpowiedz
PC27
  • 1
Hej, potrzebuję waszej wiedzy ( ͡° ͜ʖ ͡°)!

Mianowicie zdaje mi się, że zaszła zmiana na stronie internetowej rządowej i tekst się różni.
Czy można to sprawdzić? Chcę rozwiać wątpliwości.

Stary link z wczoraj
  • 4
  • Odpowiedz

  • Otrzymuj powiadomienia
    o nowych komentarzach
MarcinOrlowski
MarcinOrlowski
  • 2
OctoPrint i KDE: napisałem widget do monitorowania stanu drukarki 3d sterowanej przez #octoprint, jej postępów w drukowaniu oraz podglądania obrazu z webcama bezpośrednio na desktopie #kde. Plasmoid można osadzić zarówno w panelu lub też używać jako widget na desktopie.

Instrukcja, obrazki jak to wygląda w działąniu oraz oczywiście pliki do pobrania są na stronie projektu: https://github.com/marcinorlowski/octoprint-monitor

PS: dodaje też #programowanie -> może ktoś ma
MarcinOrlowski - OctoPrint i KDE: napisałem widget do monitorowania stanu drukarki 3d...

źródło: comment_1606809852Y8tdZpW5vO8O1YYDHxFPdO.jpg

Pobierz
  • Odpowiedz

  • Otrzymuj powiadomienia
    o nowych komentarzach
cumpot
  • 0
ostatnio robiłem zadanie rekrutacyjne z #symfony ale od razu im powiedziałem, że nie mam komercyjnego doświadczenia z tym frameworkiem i tak średnio go znam. No i teraz chcą ze mną omówić ten projekt :D

Czy ktoś z #programista15k wrzuciłby okiem na ten controller https://pastebin.pl/view/23ff6413 i podpowiedział gdzie przenieść nadmierną logikę z niego?

Dodatkowe pytanie: W feedbacku dostałem Brak bardziej zaawansowanej architektury, użycia wzorców projektowych, interfejsów itd. W projekcie posiadam jeden
  • 19
  • Odpowiedz

  • Otrzymuj powiadomienia
    o nowych komentarzach
croppz
  • 1
btw. ta walidacja powinna byc w $form->isValid(), czyli formularz powinien ci to zrobic.


@Klopsztanga: tak właściwie to powinna być ogarnięta przez api platform albo fos rest bundle na podstawie przyjmowanego dto i jego constraintów, nie ma tutaj potrzeby żeby bezpośrednio dotykać requesta albo się foremkami bawić ¯\_(ツ)_/¯
  • Odpowiedz

Powiązane tagi