Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: Ocena frameworka
Forum PHP.pl > Inne > Oceny
KKamil897
Witam.

Proszę o ocenę mojego mini frameworka, jeśli można go tak nazwać.

https://mega.co.nz/#!LAcVUYLC!dlfhm...to40MMOrMi47djU


Dziękuje!
com
następnym razem wrzuć to na jakiegoś gita czy svn, po pierwsze wtedy od razu mamy wgląd do kodu bez konieczności jego pobierania, a tobie też ułatwii wprowadzenie zmian, bo w końcu oceniamy kod nie efekt wink.gif

btw na starcie jest :
Cytat
Fatal error: Call to a member function query() on a non-object in ...\core\modules\Database.php on line 55
wink.gif
KKamil897
To przez to że nie masz bazy danych.
!*!
Wrzuć całość na https://github.com bo nie pobieram czegoś z lewych serwisów.
com
no to jak już dajesz do ściągnięcia to daj kompletne wink.gif

("świeczka") !*! mega to akurat stary Megaupload wink.gif

ponadto tak jak już powiedziałem i wspomniał !*! wrzuć to np na githuba wink.gif
KKamil897
Po półgodzinnym "rozkminianiu" githuba, udało mi się stworzyć projekt.


https://github.com/KKamil897/LunyGo

Dziękuje Wam bardzo, bardzo dobry pomysł z tym Gitem smile.gif
pedro84
Poczytaj o sobie i gicie, co to, z czym się je i jakie info w commitach się ustawia smile.gif

Co do kodu, widać, że to zabawy.

1. Dlaczego ograniczasz się tylko do jednego silnika bazy danych?
2. trzymaj się jednej konwencji formatowania kodu bo Ci sieka wychodzi w niektórych plikach.
3. "Klasa" database to już w ogóle pomyłka. Słyszałeś kiedyś o prepared statements w PDO?
KKamil897
1. Na początek będzie MySQL. Jakieś propozycje na dodanie popularniejszej bazy? postrego, mariadb?
3. Nie uważam tej klasy za pomyłkę. Może i PDO ma specjalne możliwości jak np. bindowanie, ale jakoś to do mnie nie przemawia.

pedro84
Cytat(KKamil897 @ 1.01.2014, 21:57:20 ) *
1. Na początek będzie MySQL. Jakieś propozycje na dodanie popularniejszej bazy? postrego, mariadb?
3. Nie uważam tej klasy za pomyłkę. Może i PDO ma specjalne możliwości jak np. bindowanie, ale jakoś to do mnie nie przemawia.

1. MariaDB jest kompatybilna z MySQL. W ogóle cała architektura pod tym względem leży.
3. To po co PDO? Przecież mysql_* byłoby prostsze...
!*!
1. wszystkie flagi dostępu masz ustawione na "public"
2. autoloader jest słaby, popatrz jak to jest zrobione w symfony2, prosto, i większe możliwości
3. uzupełniając wiedzę o autoloaderze rzuć okiem na standardy nazewnictwa PSR.
4. skoro masz tablicę z konfiguracją, to na co Ci dane do bazy w stałych?

Cytat
("świeczka") !*! mega to akurat stary Megaupload

I właśnie dlatego trzeba to omijać biggrin.gif
KKamil897
@up A jak zmienić na "private"? czy ja muszę mieć do tego konto ulepszone, bo z tego co czytam to tak jest.
usb2.0
na gicie priv repo platne sa,
7$ za miesiac
KKamil897
Czyli jak rozumiem każdy może edytować moje pliki?
!*!
Chodziło mi o flagi metod w kodzie klas ;)
usb2.0
kazdy moze zrobic sobie pulla na swojego kompa ofc
KKamil897
Okej, dzięki Panowie. !*! - masz może jeszcze jakieś zastrzeżenia/propozycje?
!*!
Pracuj dalej, jesteś na dobrej drodze.
KKamil897
Dziękuje bardzo Panowie!
ctom
tu masz prywaten repozytoria za free https://bitbucket.org/
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2024 Invision Power Services, Inc.