[k.....l]

oceni ktoś? pierwszy taki projekt

https://github.com/kamil991/php_companies/

#programowanie #programista #informatyka

[siranen]

@kam1ll: git pull

[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 SQL

[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 = $conn->prepare(...);

$stmt->bind_param(...);

$stmt->execute();

W każdym przypadku realizacji zapytania, do którego wmiksowywane są wartości od użytkownika/API/zmiennych.

Szczegóły użycia bind_param poznasz dzięki zastosowaniu techniki RTFM. W sensie wspominam o tym, bo warto znać technikę RTFM :D

[k.....l]

@patrolez: czyli, że mam to wprowadzić w kod? co znaczą te rzeczy po "->"

[k.....l]

dzieki
@patrolez:

[MarcinOrlowski]

@kam1ll: przeczytałem Readme i dalej nie wiem ci to robi

[k.....l]

@MarcinOrlowski: w sensie

[k.....l]

@MarcinOrlowski: tam mial być krotki opis, poprawie to jeszcze żeby było mniej

[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 :)

[k.....l]

@MarcinOrlowski: dobra wymyślę cos bardziej mówiącego, dzięki!!!

[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 takich

[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

[k.....l]

@Jurix: kurna jak to wciąga

[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ę

[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 klas",

[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 ( ͡° ͜ʖ ͡°)

[k.....l]

@patrolez: ale jest to dobrze napisane? choćby bez tego obiektowego? zaktualizowałem kod i readme przed chwilą

[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 spod