k.....l
k.....l
0
  • Najlepsze
  • Wszystkie komentarze
patrolez
@kam1ll: MVC to nie jest, a styl pisania w PHP, że wtapia się logikę w renderowanie HTMLi jest techniką typową dla początkujących.

Ogólnie jak na początek jest nieźle. Nie kodzę w PHP od dawna, jednak może u ciebie wystąpić podatność bezpieczeństwa SQL Injection w momencie tak bezpośredniego wkładania $_POST do zapytań SQL:
https://stackoverflow.com/questions/16282103/php-mysqli-prevent-sql-injection
https://www.google.com/search?hl=pl&q=SQL%20injection%20mysqli&lr=lang_pl

W sensie - u ciebie ten regex niby to broni, ale nie używasz klasycznego/domyślnego snippetu zapobiegającemu
1
k.....l
k.....l
@Jurix: kurna jak to wciąga
1
szczesc_borze
@kam1ll: Nigdzie nie zastosujesz, bo nie wiesz jeszcze co to OOP. Musisz poznać i załapać ideę, wtedy dopiero coś sensownego napiszesz. Taka już kolej rzeczy ( ͡° ͜ʖ ͡°)
1
siranen
@kam1ll: git pull
0
patrolez
@kam1ll: domyślny snippet w przypadku mysqli przeciw SQL Injection to:
https://stackoverflow.com/a/30974143/12755962

Czyli kolejne/łańcuchowe wywoływanie:

$stmt
0
k.....l
k.....l
@patrolez: czyli, że mam to wprowadzić w kod? co znaczą te rzeczy po "->"
0
k.....l
k.....l
dzieki
@patrolez:
0
MarcinOrlowski
MarcinOrlowski
@kam1ll: przeczytałem Readme i dalej nie wiem ci to robi
0
k.....l
k.....l
@MarcinOrlowski: w sensie
0
k.....l
k.....l
@MarcinOrlowski: tam mial być krotki opis, poprawie to jeszcze żeby było mniej
0
MarcinOrlowski
MarcinOrlowski
@kam1ll: w sensie ze nie wiem "po co to". Ten twój opis to jak "man pages":

Głównym zastosowaniem tego skryptu jest przeszukanie bazy danych w celu sprawdzenia poprawności wpisanych danych w formularzu, aby zobaczyć, czy są one prawdą, czy fałszem.


dużo tekstu, mało treści, zrozumiałe tylko dla autora :)
0
k.....l
k.....l
@MarcinOrlowski: dobra wymyślę cos bardziej mówiącego, dzięki!!!
0
Jurix
Jurix
@kam1ll: Naucz się OOP asap.
W przypadku Twojego projektu można rozwodzić się nad słabo zaprojektowaną (a raczej wcale nie zaprojektowaną) architekturą, ifologią, słabą czytelnością kodu, mieszaniem logiki i widoku, brakiem stabilnego stylu pisania kodu, jednak prawdopodobnie dopiero zaczynasz i taka ilość wiedzy Cię przytłoczy. Opanuj podstawy OOP i zacznij pisać obiektowo, wybierz framework (Symfony lub Laravel) i jak to narzędzie narzuci Ci szereg dobrych rozwiązań to będzie z górki.

Z
0
k.....l
k.....l
@Jurix: możesz trochę inaczej? bo nie rozumiem + jeszcze, nie chce używać frameworka na razie bo imo bez sensu dla 2-3 inputów i submitu używać framework i tymbardziej do napisania prostej strony imo
0
k.....l
k.....l
@Jurix: gdzie bym mógł zastosować w ogóle to programowanie obiektowe w tym skrypcie? bo nie mam pojęcia jak tak patrzę
0
patrolez
@kam1ll: na początku faktycznie ciężko to zaobserwować.

Najlepiej to się uwypukla po poznaniu techniki programistycznej TDD (Test driven development) LUB podczas projektowania modelu aplikacji z wykorzystaniem jakiegoś frameworka ORM w technice "model-first" - pomocnym tematem do utrwalenia o co w tym wszystkim chodzi jest jeszcze zapoznanie się z tematem "Jak projektować OR modelować relacyjne bazy danych?" z wykorzystaniem narzędzi pokroju "MySQL Workbench".

https://www.google.com/search?hl=pl&q=jak%20projektowa%C4%87%20OR%20modelowa%C4%87%20bazy%20danych%3F%20%22MySQL%20Workbench%22&lr=lang_pl

Względem OOP jeszcze jest UML oraz "diagramy
0
k.....l
k.....l
@patrolez: ale jest to dobrze napisane? choćby bez tego obiektowego? zaktualizowałem kod i readme przed chwilą
0
Jurix
Jurix
@kam1ll: Moim zdaniem, jako osoby kładącej duży nacisk na czysty kod i trzymanie się standardów programistycznych ten kod nie jest dobry. Potrzeba lat praktyki żeby pisać dobry kod, więc jeśli ciągle będziesz się uczyć to z czasem będzie tylko lepiej.

Chodzi o to, żeby inny programista rozwijając Twój projekt mógł bez problemu połapać się o co chodzi w danym serwisie, a cały kod tworzony przez cały zespół programistów wyglądał jak
0