Cześć.
Na wstępie bym prosił o odpowiedzi jak najszybciej ponieważ potrzebuje ich na jutro rano.

Otóż problem jest związany z nową pracą. Mamy napisać framework i stawiać na nim nasze systemy. Gotowe odpadają.
Koleś, który pracuje już tam 3 lata ma swój "prawie gotowy" framework. Na początku był plan, że dokańczamy w dwa tygodnie ten framework i zaczynamy pisać systemy.
Sprawdziłem kod i aż płakać mi się chciało. Może nie aż tak super źle, ale nic mnie nie zachwycało. Zaproponowałem aby dokończyć mój framework, który mam trochę zrobiony i jest na podobnym stadium co jego. I zaczęło się, że zamiast pracować to już dwa dni tylko dyskutujemy o tym. Więc chciałbym abyście zerknęli i powiedzieli co nieco na ten temat. Tylko proszę aby nie narzekać, że gotowe są lepsze itp. Mamy zrobić własny i już.

Mój: https://www.dropbox.com/sh/mj0kdxf2ijn9vcc/...LADB_2dRYa?dl=0
Jego: usunięte

U niego jest plan aby dodać namespace, ale tylko do kontrolerów, modeli i widoku, a może kiedyś do reszty - dla mnie bez sensu.
Ma on zrobione moduły, które u siebie już powoli zacząłem robić. U niego są w folderze 'libs'. Tak, w folderze z bibliotekami...

To jeszcze nie koniec.
Ja używam MVC tego typu: http://rachelappel.com/Media/Default/Windo...284/image_8.png
On: http://www.mindsilver.com/guide/docs/artic...gController.png
Czyli ma jeszcze jakieś połączenie pomiędzy View a Model. Jak dla mnie to zły pomysł aby widok miał dostęp do modelu. Zgadzam się aby były widgety czy tam pluginy, kto jaką nazwę woli, aby były w widokach, a te widgety/pluginy będą pobierał z modelu dane. Warto spojrzeć jeszcze, że on ma klasy View i nazwaView. Ja tego nie mam.
Jako argument dał, że w widoku wystarczy ładować mu dane i od razu wyświetlać.
Sprawdźcie te pliki:
usunięte
Spójrzcie na metody index(). Ładuje widok, a w widoku Model i wysyła do templatki. Fajnie, fajnie. Dobrze oddzielone i tak dalej.
A teraz mój argument, że to złe. Teraz sprawdźcie te dwa JEGO pliki:
usunięte
A dokładniej na metody one() - to wystarczy.
Po pierwsze, ładuje z Modelu dwa razy to samo. Zmienił to już (nie mam najnowszego kodu) i w widoku teraz przyjmuje z argumentu tamtą tablice. Czyli już pokazuje, że robi to samo co w moim frameworku.

Następna sprawa to baza danych. Ma on tabelę contents, która zawiera, uwaga, wszystkie treści! Artykuł, prawdopodobnie pierwszy post z tematu (czyli jest jako temat). Mówił mi, że KAŻDA treść tam będzie jako rekord żeby nie tworzyć za dużo tabelek na te dane. Tabelka comments, to samo. Aktualnie ma zrobione "komentarze" do forum, ale zgaduje, że tam też będą komentarze do newsów i innych podobnych modułów. Waszym zdaniem, ma to sens?
Jak dla mnie, jeśli są moduł to instalator powinien dodawać swoje tabele, a przy usuwaniu usuwać. Wszystkie CMS'y tak mają więc raczej jest to dobre.

I takie mniej ważne rzeczy to jego kod. Raz piszeTak, a raz_tak. Autoloader plików szuka tylko jako $nazwa.class.php. Nie widzi mi się aby pracować na jego kodzie.
Dlatego was proszę aby pomóc znaleźć argumenty aby zrobić to wszystko lepiej. Jeśli przekonam programistę to szef od razu się zgodzi. Są plany aby pracować na oby dwóch, ale skupić się bardziej na jego aby jak najszybciej zaczął pisać systemy. Ponoć jego framework jest gotowy aby bardzo szybko przenieść kod starego systemu na to i już działać. Nie wiem po co, ale tak gadał. Programista trzyma się twardo, chce swoje i już. Jeśli nie uda mi się załatwić dobrych argumentów i jutro rano będą znów długie dyskusje to nagle przerwę to i odpuszczę. Szkoda czasu wtedy. Ostatnia szansa jest więc.

Podsumowując, prosiłbym o komentarz według punktów:
1. Porównanie frameworków.
2. O samo MVC, czy View powinno być klasą. O dostęp do modelu poprzez widok, czy może lepszy mój pomysł z widgetami/pluginami.
3. Dwie tabele zamiast więcej oddzielonych przez moduły.
4. Ogólnie przedstawić argumenty.

Próbowałem to jak najszybciej napisać bo dopiero co wróciłem więc może być trochę niejasności. Proszę o szybkie odpowiedzi, ewentualnie dopytać a spróbuję jak najszybciej odpowiadać. Przypominam, że jutro od rana już będę rozmawiał z programistą więc bym musiał mieć to wszystko na dziś. Programista tamten pracuje zdalnie jak by co.
Dzięki i pozdrawiam.

@edit1:
Zapomniałem dodać, że framework na być łatwy dla nowych, aby jak ktoś będzie na urlopie lub chory to ktoś z zewnątrz mógł w razie czego raz dwa coś poprawić.

Moje porównanie:



dalej nawet nie czytałem... skoro zastanawiacie się "czy warto dodać namespace"...


Nie ogarniam po co robić swój framework który będzie milion razy bardziej dziurawy, milion razy bardziej brzydki, milion raz mniej funkcjonalny, każda rozbudowa to będzie dodatkowy nakład nie potrzebnej pracy a dodatkowo niczego dobrego nikt się nie nauczy - takie jest moje zdanie.

Brać gotowy i zaoszczędzicie dużo pieniędzy, czasu i nerwów. Żaden argument mnie nie przekona.

Proponowałem to szefowi, ale mówiłem, że zamiast pisania płynnego będzie trochę nauki to powiedział, że mamy skończyć w ciągu 2 tygodni ten framework i już pracować.
Potrzebuję bardzo dobrych argumentów żeby przekonać programistę. On mówił, że mamy specyficzne projekty i musimy sami zrobić framework. Wszystkie CMSy mają tak być zrobione, że kopiuje się folder z modułem, aktywuje w ustawieniach i już wszystko działa.
Bardzo ciężki typ. Ja z chęcią chciałbym używać w robocie Symfony albo Laravel, ale potrzebuje do tego argumenty. Że szybciej się pracuje to go nie przekona.

I szkoda, że albo niejasno napisałem albo ty źle zrozumiałeś, ale namespace jak dla mnie obowiązkowo musi być, a mu kiedyś tam niedziałały więc nie dawał. Jak mu pomogłem zrobić aby działało to wtedy wyraził jakąś chęć. Spójrz na mój framework i wtedy pomyśl czy chce używać namespaców.

Dokładnie, też w tym momencie przestałem czytać.

To, że ktoś jest ofermą i nie umie korzystać z frameworków argumentując to tekstem: "mamy specyficzne projekty i musimy sami zrobić framework", albo próbuje pisać fw nie używając namespace. Sorry, ale ktoś taki miałby problem załapać się na juniora w poważnej firmie.

Tego nie trzeba argumentować, bo to jest poniżej poziomu dyskusji. To tak jak ktoś chciałby założyć plandekę nad plażą i oświetlać ją lampkami i może się przekona, żeby jednak odsłonić plandekę i dostarczyć światło słoneczne, ale musisz mu to uargumentować.


Edit: Zajrzałem w parę plików, które załączyłeś, jesteś pewny, że im chodzi o FRAMEWORK? To wygląda na kod jakiegoś CMSa, kiepskiej jakości.

Evinek

Moje zdanie jest takie: dokończcie ten jego framework, niech chłopak będzie zadowolony. Takiego typa nie przekonasz do niczego. Postaraj się wyciągnąć jak najwięcej z pracy w tej firmie (nie mówię, że tego dużo będzie) i po 10 miesiącach szukaj sobie nowej pracy z tą pozycją w CV. Brzmi to trochę jak jedna z moich byłych prac, chociaż bardzo wątpię, abyś tam był Nie ma co wdawać się w dyskusje z tego typu ludźmi. Uprzedź szefa o swoich obawach, aby podczas niepowodzenia nie zostało zwalone nic na Ciebie.

Poucz się trochę obiektówki, trochę wzorców. Popisz coś po godzinach, i szukaj czegoś lepszego.

Pewnie zaraz mi się oberwie, ale trudno

Ja bym proponował w ogóle zacząć od początku. Twoje rozwiązanie jest kiepskie - jego fatalne. Na prawdę z tego niczego sensownego nie ulepicie. Pojawiają się jakieś klasy, ale to z OOP nie ma nic wspólnego. W obu przypadkach jest to coś, co wygląda na kod pisany przez kogoś pomiędzy średnim juniorem a kiepskim mid-level. A do tego to, co już napisał Damonsson - to nie jest framework. To jest CMS. Kompletnie dwa tematy. Można CMS oprzeć na frameworku, ale sam w sobie CMS nie jest i nie będzie frameworkiem.

Biorąc pod uwagę, że nie masz za bardzo wyjścia (jesteś nowy i nie masz odpowiedniego autorytetu/siły przebicia), to proponuję już szukać innej pracy. W tej nie nauczysz się kompletnie niczego dobrego, a Twój współpracownik będzie Cię ciągnął w dół.

Uciekaj. Najszybciej jak się da. I nie martw się, co powiesz kolejnemu pracodawcy. Uwież mi, że każdy rozsądnie myślący przedsiębiorca w 100% zrozumie Twoje zachowanie. A jeśli nie, to znaczy, że sam stosuje dokładnie takie same praktyki i lepiej Ci będzie gdzie indziej.

Ja to już przerabiałem. Nie licz na to, że coś się zmieni i będzie lepiej. Nie będzie. A ty tylko stracisz czas i nerwy. Nie warto.

Przeczytanie całej dokumentacji laravela + kilka przykładów to nawet nie jest jeden dzień. W tydzień większość podstaw byście opanowali i mogli stawiać swoje "systemy".

Co ja bym zrobił? Nawet po godzinach, na szybkiego chociażby w laraverze zrobiłbym przykładową wersję tego "systemu" i pokazał że czasowo jak i jakościowo wychodzi to lepiej. A zapewne da się ten "system" napisać w kilka dni, bo mało kiedy jakieś agencje interaktywne mają coś bardziej skomplikowanego, co by same ogarniały..

EDIT: tak sobie pomyślałem, że w czasach takich FW jak laravel, które mają bardzo niski próg wejścia i nie jest fatalnym frameworkiem, aż mnie dziwi że są ludzie którym chce się pisać FW od podstaw. Rozumiem jeszcze gdybyście wykorzystali jakieś już gotowe komponenty (via httpFoundation), ale serio..

Swoją drogą, masz pozwolenie na udostępnianie nie swojego kodu? Nie wiem jak wygląda twoja umowa (o ile w ogóle jest jakaś), ale raczej IMO się tak nie powinno robić...

jezus maria. powinni tego zabronić.

Argumenty o własnym FW są absurdalne ale nawet jeśli się tego trzymać to powinniście sobie coś złożyć z gotowych komponentów symfony + rożne biblioteki + silex/slim.

Obydwa systemy siebie warte, jeden i drugi dziurawy oraz ograniczony.
A obecnie są tak samo publiczne jak laravel czy inne fw.
Fw takie jak laravel/ko/ci/yii są banalne a nauka ich podstaw które są i u was trwa godzinę.

Przy korzystaniu z gotowego frameworka nie ma z tym żadnego problemu. Wasze jak będą miały komentarze to będzie już sukces, o dokumentacji lepiej nawet nie wspominać.

niestety nie zgodzę się,
jeśli pracuje się nad większymi projektami to ok nie ma problemu, ale jeśli są to "mniejsze CMS'y" to często klienci mają już swoje serwery gdzie są starsze wersje php. A serwera zmienić niemożna bo przecież ten jest dobry :/
Nie wspominają już o serwerze który miał domyślnie php4 i to wcale nie był dedyk.

To niestety cały czas jest problem przy małych projektach.

Moja rada taka, usiądzie do tego i przedyskutujcie czego tak naprawdę potrzebujecie. Nie porównujcie co i jak jest u mnie, a jak u ciebie tylko czego potrzebuję.
Następnie sprawdźcie jak to czego potrzebujecie robią inni i które rozwiązanie Wam odpowiada. Na tej podstawie może uda się sklecić coś "ciekawego".
Potraktujcie tego FW jako proste środowisko uruchomieniowe, czyli na start potrzebujecie MVC, ruting i coś do db. Reszta wyjdzie w praniu.



nie do końca, ja tak robię ale zakładam że dla widoku model jest read only. Oczywiście


jak dla mnie to nie. To znaczy tak, ale w innym znaczeniu.
sam widok jest klasą ale jest to jedna klasa, a nie dla każdego kontrolera osobna. Widok ma zebrać dane, obrobić je i wyrenderować. To wsio, nie widzę tu potrzeby tworzenia różnych klas widoków.


i na koniec taka uwaga co do "jego kodu", otóż w bibliotekach znalazłem to:
*PHPMailer
*smarty-3.1.27

sory ale nie, kto normalny używa w tych czasach smarków i mailera.

oraz uwaga do obu
podobno macie autoloadery więc po kiego wuja to

[PHP] pobierz, plaintext 
include_once 'controller/controller.php';
...
include_once 'controller/controller_modules.php';
[PHP] pobierz, plaintext 

znowu.. dalej nawet twojej wypowiedzi nie czytam. Pracuje dopiero rok na etacie, z klientami i jeszcze żaden nie miał takiego serwera. Abstrahujesz albo współpracujesz z idiotami - bez urazy... dedyk z php 5.7 do już 7zł / m-c. Nie przesadzaj, dobrze?

@Forti - są różni klienci. Ja kiedyś miewałem i potwierdzam że tacy istnieją. Tutaj za problem bym uznał brak umiejętności lub możliwości aby wymusić na kliencie zmianę hostingu, oraz jeśli jakiś dziwny typ to podziękowanie/danie ceny zaporowej. Wszystko zależy w jakiej agencji pracujesz.

nie przesadzam,
o ile 5.2 zdarza się coraz rzadziej to to 5.3 to norma. W tym tygodniu miałem taki przypadek użyłem [], a po przeniesieniu okazało się że dziwnymi błędami sypie. Szybkie wrzucenie phpinfo i co ? php5.3

A niech se będzie to 7zł, ba może być 5zł i co z tego ? już widzę to marudzenie "ale już zapłaciłem do końca roku", "no ale przecież miało działać" ...

a to że nie czytasz wypowiedzi do końca świadczy już tylko o Tobie

5.2/5.3 to nie php4. [] na array() szybko zmienisz.




no ok, wiem że są tacy. Są tacy co wykupią na rok na home.pl i potem przychodzą i trzeba im przez ftp wgrywać symfony.. wszystko sie zdarza. Są tacy co chcą portale społecznościowe tematyczne budować na wordpress / joomla. Takich uświadamiamy i najczęściej się dogadujemy. Branża jest jaka jest - ale na brak roboty to nikt narzekać nie powinien.. chyba że jest "słaby".

Ta praca jest tymczasowa. Na 100% będę zmieniał, ale pytanie to kiedy? Na pewno do czasu rozpoczęcia studiów i trochę w trakcie będę w tej firmie. Wtedy zacznę szukać nowej mając w CV pracę i studia.

W obu przypadkach Framework posiada już przykładowe kontrolery, modele i widoki.
U mnie to na pewno newsy. U kolegi były artykuły i forum. Nie wiem jak by musiał wyglądać kod abyście go pochwalili. U siebie bym musiał jeszcze zmienić niektóre klasy na abstrakcyjne i tak dalej. Też nierozumiem do końca waszych opinii na temat frameworka, że jest dziurawy i ograniczony jak tam prawie nic nie ma.

Smarty on mówi, że potrzebne. Mówił mi, że jeśli on daje dostęp do PHP to ludzie od szablonów wyprawiają cuda z PHP. Zaproponowałem Twig, ale on chce Smarty.
Co do autoloadera to u mnie jest autoloader na wszystko według namespaców. Wydaje mi się coś, że jakoś mało uważnie czytaliście mój post. Mój kod jest jeszcze dostępny, jego usunąłem.
Te 4 przykłady były jego. Spójrzcie jeszcze raz na mój framework i upewnijcie się czy dobrze porównywaliście.

Ogólnie spodziewałem się takich odpowiedzi. Tylko nie sądziłem, że mój kod jest tak okropny. Nie będę pokazywał na jakim kodzie pracuje aktualnie, ale oczy wypala.

Decyzja moja jest taka, że odpuszczam. Jeśli i tak nie zabawie tam długo to nie ma sensu zmieniać jego nawyków. Dzięki za odpowiedzi.

Jedyną słuszną decyzją jest skorzystanie z gotowego, istniejącego frameworka np. Symfony2

Zawsze jest sens. Tamtą firmę zamkną, albo klient z niej zrezygnuje, a z racji że to będzie to samo miasto, klient będzie szukać kogoś lokalnego i znowu może ten kod trafić do ciebie.

Kiedyś też myślałem że potrzebny mi papier, że się uczyłem i że gdzieś tam już pracowałem. Cóż, dwa lata się przemęczyłem w takiej firmie, gdzie właśnie tak się produkowało "kod", tylko po to żeby mieć papier. Aktualnie nigdzie nawet nie pokazywałem tego papieru, bo wszędzie gdzie próbowałem się dostać, wstyd było pokazać że brałem udział w tworzeniu takiego "kodu". Więc te 2 lata w większości były zmarnowane..



Cóż, jak już ktoś wyżej napisał, oba "fw" są siebie warte. Przykładowo czy zdajesz sobie sprawę z tego że każdy model tworzy nowe połączenie do bazy danych? Pomijając to że konfiguracja jest zahardcodowana w wielu miejscach i zmiana ustawień wiążę się z zmianą w kilku miejscach. A brak komentarzy to już w ogóle nie do pomyślenia, zwłaszcza że sporo edytorów generuje takie podstawowe komentarze skrótem klawiaturowym.. Po co Ci IDE, skoro i tak musisz mniej więcej wiedzieć co gdzie jest zwracane ?



No to nie uświadamiajmy klientów, niech dalej używają IE6, niech dalej używają php 5.2 albo nawet php4; bo przecież oni się na tym znajdą, a przychodzą do nas tylko dlatego, bo nie mają czasu się grzebać w kodzie.. Takie podejście właśnie sprawia że na wielu hostingach jest wciąż php 5.2 które już dawno nie jest nawet wspierane. Jeżeli ktoś nie posiada odpowiednich argumentów do tego żeby przekonać drugą osobę do swojej racji, to moim zdaniem sam tak na prawdę nie wie dlaczego powinno się przejść na nowszą wersję, czy skorzystać z innego rozwiązania. Są jeszcze sytuacje beznadziejne, gdzie żaden argument niczego nie zmienia, ale takie "sytuacje" powinno się raczej omijać..

Pozatym jak goście robią przestrzenie nazw, to 5.2 raczej to nie jest, więc do którejś wersji symfony, czy kompomentów symfony, można by już skorzystać. Nie wspominając tego że taki Slim jest dużo lepszym wyborem dla nich, niż pisanie własnego FW. Też nie jest to duży FW, ma raptem kilkanaście klas, więc łatwo będzie im się wdrążyć w ten FW, które zresztą jest banalny.

Serio, zostawcie oboje swoje "frameworki" i użyjcie czegoś gotowego, co ma dokumentacje, co ma wsparcie, pod co będziecie mogli podłączyć jakieś inne gotowe moduły dość łatwo. Może przynajmniej dzięki temu oboje złapiecie jakieś dobre praktyki, bo tak to niestety lipa..

W kwestii wybory własny vs istniejący framework zostało już chyba wszystko powiedziane więc nie będę tego powielał, a jedynie się pod tym podpiszę.

Chciałbym Ci jednak kolego Evinek zwrócić uwagę na jedną kwestię: nie kozacz za bardzo. Skoro poruszyłeś ten temat na forum to zapewne i w pracy mocno obstawiałeś przy swoim rozwiązaniu. Zapewniam Cię, że nie jest mile widziane gdy przychodzi świeżak i zaczyna wprowadzać swoje wizje krytykując przy tym inne. U nas pojawił się nowy analityk biznesowy, na dzień dobry chciał zrobić rewolucję w oprogramowaniu (analityk?...). Chciał wywalić i zmienić wszystko od środowiska programistycznego przez system kontroli wersji i bugtracker, a skończywszy na pakiecie biurowym. Popracował niespełna 2 tygodnie.

Weź przy tym pod uwagę, że w długofalowych projektach (a takim będzie zapewne ten framework) nie liczą się nowinki, a skuteczność funkcjonowania. Klienta tak naprawdę średnio obchodzi jak system wygląda od środka - chce po prostu żeby działało prawidłowo i było wykonane w terminie. Patrząc na Twój framework (który, jak sam określiłeś, nie jest zbyt rozbudowany) to pozwolę sobie wytknąć kilka miejsc, które rzucają mi się w oczy. Wiem, że nie prosisz o ocenę, ale chcę nieco ostudzić Twoje zapędy w gloryfikowaniu tego, co zrobiłeś:
- Bootstrap - jesteś w stanie zrobić szablon własnej strony błędu 404?
- Controller - w konstruktorze tworzysz obiekt widoku, a co z kontrolerem, który nie będzie produkował widoku? czy metoda loadModel faktycznie tam powinna być?
- Database - na sztywno ustawione UTF-8,
- Model - dane do połączenia na sztwyno zaszyte w kodzie, co z modelami, które nie opierają się na bazie danych?
- Validation - dodawanie nowych reguł to ingerenacja w metodę init()? metoda validAll() zwraca na sztywno komunikat błędu, metoda getError() rzuci ostrzeżeniem w przypadku braku klucza w tablicy.
Jak widzisz pomimo niewielkiej zawartości frameworka trochę się tego zebrało. Postaraj się zachować więcej wstrzemięźliwości w przepychaniu własnych pomysłów. Możliwe, że framework tego kolegi pozornie wygląda na gorszy, ale nie ma np. powyższych wpadek i po prostu lepiej sprawdza się / będzie sprawdzał się w praktyce.

Tak więc moim zdaniem: uszka troszkę po sobie, masz fajny pomysł czy konkretną uwagę - zgłaszaj, ale nie zadzieraj nosa zbyt wysoko.

@Evinek - Wchodzisz do firmy z marszu i już Ci coś w niej nie pasuje? Koledzy i szef na pewno Cie pokochają ;) poza tym może nie wiesz o wszystkim. Własny FW w firmie to po prostu dodatkowe źródło dochodu, więc argumenty za i przeciw są może i na miejscu, a nawet ten FW może być gorszy niż cały kod z działu przedszkole, ale jak można na tym zarobić to nikt normalny z tego nie zrezygnuje.

jedyne czego potrzebujecie to Composer i jakis microframework np silex, reszta to juz w zaleznosci od potrzeb

a co do wyboru, to wybral bym np ten ktory ma wiecej unit testow napisanych,

Odpowiedź phpion chyba podsumowała wszystko. Kod nie jest najświeższy, ale nawet nie zauważyłem takich podstawowych błędów. Tak jak pisałem, decyzja już zapadła.
Akurat z szefem jest bardzo prosto się dogadać więc nie ma problemu. Ja daje pomysły, a on się zastanawia. Proponowałem też mu aby zmienić system zarządzania projektami, bo ja jako nowy, nie potrafiłem bardzo szybko odnaleźć się w używanym w firmie. Zaproponowałem Redmine, pogadaliśmy i doszliśmy razem do wniosku, że nie warto. Zostaliśmy przy Odoo.

Doświadczenie mi pokazało, że jak baza będzie chu... to potem będą tylko same problemy.
Oczywiście, że jeśli to zostanie zbudowane na FW to też nie gwarantuje sukcesu. Wszystko zależy od tego jak to się rozwija. Bo spagetti to można w byle czym zrobić.

a co do frameworkow to Uncle Bob ma swoje zdanie https://vimeo.com/43612849