Ocena frameworka Opcje

[KKamil897]

Witam.

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

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


Dziękuje!

Ten post edytował KKamil897 1.01.2014, 10:15:23

[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 (IMG:style_emoticons/default/wink.gif)

btw na starcie jest :

Fatal error: Call to a member function query() on a non-object in ...\core\modules\Database.php on line 55

(IMG:style_emoticons/default/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 (IMG:style_emoticons/default/wink.gif)

("świeczka") !*! mega to akurat stary Megaupload (IMG:style_emoticons/default/wink.gif)

ponadto tak jak już powiedziałem i wspomniał !*! wrzuć to np na githuba (IMG:style_emoticons/default/wink.gif)

Ten post edytował com 1.01.2014, 19:00:24

[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 (IMG:style_emoticons/default/smile.gif)

Ten post edytował KKamil897 1.01.2014, 20:40:54

[pedro84]

Poczytaj o sobie i gicie, co to, z czym się je i jakie info w commitach się ustawia (IMG:style_emoticons/default/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.



Ten post edytował KKamil897 1.01.2014, 21:59:48

[pedro84]

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?

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

I właśnie dlatego trzeba to omijać (IMG:style_emoticons/default/biggrin.gif)

Ten post edytował !*! 2.01.2014, 10:10:06

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