Witam
Aktualnie uczę się OOP w php i w tym celu stworzyłem o taki projekt
https://github.com/opcmyk/opbook2
jeśli chodzi o ten projekt jest w nim zawarte parę klas oraz użycie js z jquery(ajax)
bardzo proszę o sprawdzenie i napisaniu co jest do poprawy, oczywiście jestem przygotowany do napisania od nowa całego projektu wraz z waszymi uwagami, projekt nie jest całkowicie skończony i pewnie zawiera dużo błędów.
Mam dwie wiadomosci, a poniewaz jestem jedyna osoba na tym forum, ktora dla zasady Cie nie sponiewiera, wiec masz szczescie ze wypowiadam sie pierwszy :D.
Pierwsza jest zla :D
Twoj kod to masakra :D i moznaby na ten temat napisac magisterke :D
Druga jest dobra :D
Znajdujesz sie we wlasciwym momencie, zeby zaczac naprawde uczyc sie kodzic w php :D, potrzebujesz jedynie mentora.
Ile masz lat?
Jak dlugo bawisz sie z php?
Po co w ogole to robisz?
Do połączenia z bazą możesz używać Singletona.
Połowa klas pustą albo z zakomentowanym kodem - wyczyść to.
Poczytaj o PSR w PHP zarówno pod kątem autoloadera jak i pod kątem nazewnictwa klas i plików.
Popracuj nad nazewnictwem zmiennych. Kod ma być samo dokumentujący.
Np.
foreach ($book as $i)
{
}
Sugeruje, że iterujesz po jakichs elementach książki, a widać, że iterujesz po tablicy obiektów.
@babel100:
Dzięki że raczyłeś spojrzeć na to i napisać coś.
Może kod jest masakryczny i można o nim napisać pracę jak nie programować :D , nie mówię że nie, ale no trzeba to przyjąć do wiadomości i zabrać się na poprawienie błędów i skupić na dalszej edukacji.
mam 20 lat, z php bawię się chwilę, ciężko określić ile dokładnie, po co to robię?, ponieważ lubię programować, sprawia mi taką "frajdę" o ile można to tak nazwać, nie mam z byt dużego doświadczenia ani wiedzy , i nie ukrywam że planuję z tym powiązać swoją przyszłość, i właśnie dlatego postanowiłem rzucić się na głęboką wodę i napisać coś będąc zielonym.
@jurek1980:
czytając na temat singletona, dużo osób bardziej polecało wstrzykiwanie zależności, nw czy u mnie w kodzie coś takiego istnieje, tak połowa klas pusta ponieważ jak wspomniałem wcześniej projekt nie jest dokończony i klasy te powstały i nie zostały uzupełnione w kod, ponieważ chciałem wiedzieć jak reszta kodu wygląda według was na obecną chwilę, poczytam o psr oraz wprowadzę to w życie, także dziękuje że raczyłeś spojrzeć na ten kod i coś napisać.
@opcmyk:
Nie wziales tego co napisalem do siebie, jedynie spokojnie przyjales do wiadomosci. Z mojego punktu widzenia bardzo dobrze to o Tobie swiadczy :D. Jesli zdecydujesz, ze chcesz sie wziac za php na powaznie i bedzie to przemyslana decyzja, daj znac :D
Najpierw poznaj DI i Singletona. Napisz trochę kodu. Poczytaj np. https://blog.gordon-oheim.biz/2011-01-17-Why-Singletons-have-no-use-in-PHP/
U Ciebie nie widziałem nigdzie DI, ale może gdzieś przeoczyłem jakiś konstruktor.
Próbuj i ucz się. Pytania się będą pojawić to wiedza jest w necie i zawsze możesz wszystko skonsultować tu, na forum.
jurek1980 napisał(a):
[...] Singletona [...]
Wpiszcie w Google'a na przykład "why singleton pattern is bad"...
@koszalek-opalek: ja wiem. Ale ciągle stosuję się go do loggerow a na etapie nauki trzeba poznać co to jest.
opcmyk