Wszystko

Wszystkie

Archiwum

ozzi91
ozzi91
  • 1
Mireczki
#programowanie #codereview (to chyba lekka przesada, ale i tak wołam). #python #naukaprogramowania #programista15k

Zacznijmy może od opisania USER STORY.
1. Użytkownik wybiera ścieżkę folderu, w którym chce, by program zadziałał.
2. Wybiera, do jakich instytucji chce skierować wnioski.
3. Uzupełnia określony plik, by móc na ich podstawie zbudować wnioski.
4. Program drukuje wygenerowany wniosek lub jeżeli trzeba, to wysyła mailem.

Pseudokod mojej aplikacji - niekoniecznie jest zakodowane w ten sposób.
Zapytaj
zwei
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) po Application typu TauronApplication(Appllication)
To są tak zwane modele. Modele reprezentują jakieś rzeczywiste rzeczy i
ozzi91
ozzi91
  • 0
#python #programowanie i może #codereview ?

Poddaje się, przy pomocy kolegi @Calka_internowana oraz @Krolik udało się stworzyć coś bardzo fajnego dla mnie ale zacząłem nowe featury dodawać i się wysypało. Tutaj link do pastebina: https://pastebin.com/jR4iid1p

Kod błędu jest nastepujacy:

Podaj w postaci cyfry do kogo chesz wygenerować wniosek np 1 lub wpisz lista jeżeli nie wiesz jaka cyfra odpowiada wnioskowi lub exit jeżeli chcesz wyjśc z programu: 3
sciezka pierwszego wygenerowanego pliku
janek_
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
PaaD
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ć bez regału
- zlecenie giełdowe nie ma sensu bez giełdy

Gdyby być mega purystą architektonicznym można by się zastanowić czy w przypadku z OP nie wydzielić osobnej klasy,
AnonimoweMirkoWyznania
AnonimoweMirkoWyznania
  • 0
#anonimowemirkowyznania
Cześć, wykona ktoś dla mnie code review aplikacji, która jest zadaniem rekrutacyjnym?

git

Opis polecenia na obrazku:

Na drugim branchu załączyłem dodatkowe funkcjonalności: edycja rekordów oraz podawanie ceny netto wraz z wyborem stawki podatkowej.
Wygląda to tak:
zrzut ekranu

#codereview #csharp #naukaprogramowania

---
Kliknij tutaj, aby odpowiedzieć w tym wątku anonimowo
Kliknij tutaj, aby wysłać OPowi anonimową wiadomość prywatną
ID: #62a21308b025d493b59dde8b
Post dodany za pomocą skryptu AnonimoweMirkoWyznania ( https://mirkowyznania.eu ) Zaakceptował:
AnonimoweMirkoWyznania - #anonimowemirkowyznania Cześć, wykona ktoś dla mnie code re...

źródło: comment_1654950350Z0lKhkFec9sbzPESNnlK0l.jpg

Pobierz
Rst00
Rst00
  • 5
@BRmedia:

1. Trzymaj się jednego języka - angielskiego. Co to ma być za ustawianie "polski"? Powinno być "polish"
2. Rzeczy typu magical number koniecznie do osobnych stałych a najlepiej do pliku gdzie masz wszystkie globalne ustawienia. Takie + 15 w kodzie nic nie mówi
3. W BooksList ten return z Bookiem wygląda źle i różni się tylko jednym propsem - ja to bym ogarnął jednym returnem z dynamicznym propsem albo wyrzucił
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
login_zajety_sic
login_zajety_sic
  • 0
#python #programowanie #codereview

Zwracam się do Was z prośbą o pomoc w uporządkowaniu kodu który naklepałem w Pythonie,

Skrypt ma odczytywać wskazany przez usera plik .csv i na jego podstawie utworzyć makro do systemu które będzie zapisane na pendrive.

jestem początkujący, skrypt działa tak jak chce w obecnej formie, docelowo chce go wypuścić w postaci pojedynczego pliku .exe (py2exe) co by jego obsługa nie wymagała uruchamiania wiersza poleceń ale chciałbym poprawić jeszcze
loginzajetysic - #python #programowanie #codereview Zwracam się do Was z prośbą o ...

źródło: comment_1615315065xpCZCu2sQ58Vm4oE3KquC1.jpg

Pobierz
Xaaq
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
informatyk
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 ignoruje):

if __name__ == "__main__":
doSomething()
a nie
PC27
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 co nie działa: https://www.urpl.gov.pl/pl/informacja-prezesa-urz%C4%99du-z-dnia-9-pa%C5%BAdziernika-2018-r-w-sprawie-dopuszczenia-do-obrotu-0?fbclid=IwAR0WidEjZDxO5EboJS9R1Xtgz7FAbp0IQ5d7fkCFKzKU3yi7gmJdswk-t_A

Obecny link działający: http://www.urpl.gov.pl/pl/informacja-prezesa-urz%C4%99du-z-dnia-9-pa%C5%BAdziernika-2018-r-w-sprawie-dopuszczenia-do-obrotu-0

Dziękuję za pomoc.

Hashtagi do ekspertów #internet #stronywww #codereview #hacking #hackingnews #programowanie #programista15k
AjentSzprot
AjentSzprot
  • 2
@PC27: widocznie wygasł cert i nie łapie ssla. Gdybyś jednak kiedyś chciał zobaczyć jak strona wyglądała w przeszłości, to możesz użyć https://web.archive.org/. Tam wklejasz podstawowy link który Ciebie interesuje, czyli np. w tym przypadku http://www.urpl.gov.pl/pl/informacja-prezesa-urz%C4%99du-z-dnia-9-pa%C5%BAdziernika-2018-r-w-sprawie-dopuszczenia-do-obrotu-0 i wybierasz któryś z zapisanych snapshotów
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 doświadczenie z QML, QT czy pisaniem
MarcinOrlowski - OctoPrint i KDE: napisałem widget do monitorowania stanu drukarki 3d...

źródło: comment_1606809852Y8tdZpW5vO8O1YYDHxFPdO.jpg

Pobierz
cumpot
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 form type,
croppz
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ć ¯\_(ツ)_/¯

Powiązane tagi