![]() |
![]() ![]() |
![]() |
![]() ![]()
Post
#1
|
|
Grupa: Zarejestrowani Postów: 111 Pomógł: 1 Dołączył: 24.12.2013 Ostrzeżenie: (0%) ![]() ![]() |
Hej,
Tym razem do oceny mam klase zapewniajaca dostep do baz danych. Kod dostepny jest pod adresem: http://wklej.org/hash/ae8995ddf21/ Jak zwykle z niecierpliwoscia wyczekuje krytyki ![]() Pozdrawiam serdecznie. |
|
|
![]()
Post
#2
|
|
![]() Grupa: Moderatorzy Postów: 36 482 Pomógł: 6303 Dołączył: 27.12.2004 ![]() |
namespace Framework\Platform;
Jak na moj gust to zbyt ogolnikowe i moze sie zdarzyc ze ktos jeszcze wpadl na podobna genialna nazwe i nagle bedzie zonk. Czemu wlasciwosci so private a metody protected? Nie uzywaj singletona. Teraz to jest "passe" ![]() Nie blokuj nam wyjatkow bazy. To ze ty ich nie uzywasz, nie znaczy ze inni nie. Wyjatki to teraz standard a ty je nam wylaczasz. Nie zmuszaj nas do pobierania danych tylko jako FETCH_ASSOC. Od czasu do czasu NUMERIC jest bardziej wskazane. -------------------- "Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista "Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer |
|
|
![]()
Post
#3
|
|
![]() Grupa: Zarejestrowani Postów: 8 068 Pomógł: 1414 Dołączył: 26.10.2005 Ostrzeżenie: (0%) ![]() ![]() |
- brakuje fluent interface
- fetch... 98% kodu to duplikacja. - ustawiasz na sztywne wszystkie opcje ale przy fetch nie dajesz możliwości zmian co czyni te metody bezuzytecznymi. Ogólnie nie widzę sensu tej klasy. Same ograniczenia. |
|
|
![]()
Post
#4
|
|
Grupa: Zarejestrowani Postów: 556 Pomógł: 40 Dołączył: 20.07.2012 Skąd: Warszawa Ostrzeżenie: (0%) ![]() ![]() |
|
|
|
![]()
Post
#5
|
|
![]() Grupa: Zarejestrowani Postów: 3 033 Pomógł: 366 Dołączył: 24.05.2012 Ostrzeżenie: (0%) ![]() ![]() |
nospor ten namespace to raczej wynika z tego iż to jest fragment jakiegoś większego systemu, bo kolega już się chwalił kilkoma innymi rozwiązaniami, które jakby złożyć to zapewne tworzą jedną całość, ale fakt mało twórczy
![]() Mnie razi bardzo i źle się czyta kod w takiej konwencji jak została przyjęta. Cytat abstrahując od tematu -> może ktoś mnie oświecić jaka zbrodnię popełnia singleton? - global - trudno testowalny - w przypadku gdy chcemy wykorzystać jakiś fragment systemu, może wymagać od nas wyjęcie z jednego potem pól aplikacji, a potrzebowaliśmy tylko jedną funkcjonalność dla nowego systemu itp |
|
|
![]()
Post
#6
|
|
Grupa: Zarejestrowani Postów: 111 Pomógł: 1 Dołączył: 24.12.2013 Ostrzeżenie: (0%) ![]() ![]() |
Poprawilem to na co pozwolil mi czas, tj:
* metody sa private * nie wymuszam FETCH_ASSOC Reszta zajme sie jak bedzie troche wiecej czasu - http://wklej.org/hash/9ad80953845/ Znalazlem chwile aby usprawnic DBAL (http://wklej.org/hash/ca1ae75911a/) Co prawda nie poprawilem jeszcze wszystkiego na co zwrociliscie moja uwage, ale to co do tej pory zaprezentowalem przeszlo mala metamorfoze ![]() Calosc podzielilem przede wszystkim na 3 klasy, dzieki czemu dane mozna pobrac na 2 sposoby:
lub
Metoda fetchAllRows oraz inne zdefiniowane w klasie Common przyjmuja teraz opcjonalny parametr pozwalajacy na zmiane sposobu w jaki zwracane sa dane. Co sadzicie o tej zmianie? Ten post edytował q.michal 22.04.2016, 21:30:55 |
|
|
![]()
Post
#7
|
|
![]() Grupa: Moderatorzy Postów: 36 482 Pomógł: 6303 Dołączył: 27.12.2004 ![]() |
Cytat Metoda fetchAllRows oraz inne zdefiniowane w klasie Common przyjmuja teraz opcjonalny parametr pozwalajacy na zmiane sposobu w jaki zwracane sa dane. No to miedzy innymi kazalismy ci wlasnie poprawic, wiec co niby mamy myslec?
Co sadzicie o tej zmianie? -------------------- "Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista "Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer |
|
|
![]()
Post
#8
|
|
Grupa: Zarejestrowani Postów: 111 Pomógł: 1 Dołączył: 24.12.2013 Ostrzeżenie: (0%) ![]() ![]() |
Chodzilo mi bardziej o rozbicie klasy na 2 czesci.
|
|
|
![]()
Post
#9
|
|
![]() Grupa: Moderatorzy Postów: 36 482 Pomógł: 6303 Dołączył: 27.12.2004 ![]() |
W celu ze powodu? I na jakie znowu dwie czesci?
-------------------- "Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista "Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer |
|
|
![]() ![]() |
![]() |
Wersja Lo-Fi | Aktualny czas: 20.06.2024 - 08:48 |