Post
#1
|
|
|
Grupa: Zarejestrowani Postów: 7 Pomógł: 0 Dołączył: 5.04.2017 Ostrzeżenie: (0%)
|
Hej,
Ogólnie od jakiegoś czasu pracuję jako programista, ale jako że nie mam nikogo kto by mi zrobił jakikolwiek codereview albo pomógł powiedział co poprawić to stwierdziłem że zapytam tutaj (IMG:style_emoticons/default/smile.gif) Jak każdy szanujący się programista staram się rozwijać samodzielnie itp. Postanowiłem napisać własną paczkę do Laravela, i bardzo bym prosił nie tyle o ocene bo mam świadomość tego że nie koniecznie pięknie jest napisane, samemu mi się nie podoba to raczej wiadomości co mógłbym zmienić poprawić no i oczywiście jak. Podsyłam link https://github.com/19lyczkul91/CommentBox ponadto zależy mi też żeby ktoś mi doradził (tutorial lub po prostu napisał) jak pisać testy jednostkowe. Pod jak nie mam na myśli coś w stylu "O Panie tu musisz wywołać tą metodę i zawsze ma Ci zwracać to i to" tylko raczej coś w stylu tu powinieneś napisać testy, tu nie. Konkretne moje pytanie czy to metod store/update itp powinno się robić testy ? I jeśli tak to jak zrobić np do metody store test żeby nie utworzyło nowego elementu ? nie wiem co jeszcze jak wymyślę to dodam. |
|
|
|
starf commentbox - ocena paczki, co poprawić 6.04.2017, 09:14:39
nospor Bbyloby naprawde milo, jakbys jeszcze napisal co t... 6.04.2017, 09:44:08
starf No tak, paczka ma za zadanie dać możliwość dodawan... 6.04.2017, 11:45:57
nospor Cytatbyć RestApi więc chyba ExceptionHandler nam o... 6.04.2017, 12:14:32
starf to żebym dobrze zrozumiał, jeśli zrobimy tak jak p... 6.04.2017, 13:35:27
nospor Tak, ale ten wyjatek moze byc wylapany w Exception... 6.04.2017, 14:21:40
starf Czyli żebym dobrze zrozumiał, nie przejmuję się ja... 6.04.2017, 14:26:07
nospor Wybacz, wlasnie wrocilem z tzw "welcome drink... 6.04.2017, 14:33:34
starf A i widzisz i teraz rozumiem o co chodzi pyknę mo... 6.04.2017, 14:39:16
nospor Cytat"do standardowych metod w kontrolerach n... 6.04.2017, 14:46:23
starf w laravelu też mam testy na akcje ale... za ostatn... 6.04.2017, 16:00:37
r4xz To ja dorzucę jeszcze swoje rady/uwagi/spostrzeżen... 7.04.2017, 21:05:54
starf poprawiłem trochę z tego co pisałeś, nie rozumiem ... 8.04.2017, 13:35:07
viking Nie znam co prawda LV ale co się stanie jak https:... 8.04.2017, 13:40:53
nospor @viking jak cos bedzie nie tak to poleci wyjatek a... 8.04.2017, 19:43:11
viking A co wróci wtedy do klienta? Na ok jest json z dan... 8.04.2017, 20:12:19
nospor Nie wiem co zroci laravel. Na pewno nie zwroci 200... 8.04.2017, 20:15:12 ![]() ![]() |
|
Aktualny czas: 21.12.2025 - 15:09 |