Witam
Poproszę o przebadanie stronki Poradni Psych-Ped we Włocławku. Skrypty, grafika... mojego autorstwa.
Poradnia Psychologiczno-Pedagogiczna we Włocławku
Źródła, skrypt : pobierz

Ustawienia skryptu :
Konfiguracja smarta w : Smarty.class.php (nie będę się rozpisywał, przypuszczam, że osoba która się za to bierze, wie o czym mówimy)
Konfiguracja mysql w configs/mysql-cfg.php
Struktura bazy : sql.sql
Meta w nagłówku : utf-8(linux) - weź to pod uwagę przy tworzeniu tabel!

Będę wdzięczny za pomoc w odpluskwianiu, opinię.

ps. sam CMS, który na potrzeby strony wykonałem - jest chyba nie najgorzej napisany. Zachęcam do zapoznania się z nim. Jeśli komuś się przyda, jest free.

Strona główna

a potem kolejna wybrana losowo

deklaracja strony

do tego znaczniki marquee, center, font, table ahh
jak na taki prosty layout to sporo tego php nie oceniam bo sam się dopiero uczę więc niech lepsi sieją zniszczenie ;p

A co jest nie tak z table? Rozumiem ze w X-ach to grzech... sam zauważyłes, że deklaracja html( wlaściwie jej brak)... więc table tag jak kazdy inny.
Popracuje nad tagami, zerknij w php, to jest glówny punkt programu.

Jaki jest sens tworzenia 5 instancji Smarty? Szczególnie, że nie zmieniasz nic w konfiguracji żadnego.
Do katalogu templates/ (od szablonów) wpychasz jakieś klasy PHP?
Klasa strona - jaki jest jej zadanie? Bo chyba robi "wszystki"
Z tego co widzę utworzyłeś masę klas, w których są tylko komunikaty? Już lepiej zrobić jedną tablicę wielowymiarową z tymi komunikatami...
Kod jest źle sformatowany - ciężko go czytać

Jakbym sprawdzał każdy plik to pewnie by się ta lista nieco wydłużyła

Kolejna strona, nie odbiegająca od standardów w tym dziale. Okodowanie jest tragiczne, gdybym chciał wymieniać wszystko co jest nie tak, musiałbym się rozpisać na dwie strony - polecam kurs (x)html + css.

Graficznie i wizualnie jak strona sprzed 20 lat. Po co ta migająca data w JS? Wysyłasz użytkownikowi stronę kodu, a wystarczyłaby jedna linijka w php i kilka bajtów w html.

?
Dużo błędów, literówek, całość nieczytelna, po o tyle banerów na stronie głównej? Bardzo słaba strona.

Widze sami mądrale!
Pilsener w 1988 roku miałes internet? Ciekawe! Może graficznie nie błyszczy, kursu chyba już nie potrzebuje, zerknij przycisk na dole strony. Popatrzyles na stronę przez minute i nadymasz klate. Powiedziałeś wszystko, czyli nic! Pffff :/

Crozin:
Podzielnoje smatry to naprawde taki blad? Ty nie lubisz tak, ja lubie, dla mnie wszystko jest przejrzyste. Czy jesli nie preferuje twojej metody, to zaraz zle? Nie może być all w 1 smarcie bo srodek się zmienia(z 2 kolum na 1).
Klasa "strona" zawiera wszystkie funkcje, komunikaty.

Przykład : tablica[3][15](będziesz pamietał do czego to?... nie pisz mi tablica['glowna']['blad1'] jest krótsze )
$klasy->glowna->art_add->$blad1 wygląda bardzo przejrzyście. Może w przyszłości zechcę wciskać funkcje w galeziach klasy "strona". Użyłem klas by trochę poćwiczyć ich wykorzystanie, oswoić się z nimi.

Sam plik z komunikatami i klasami jest do podziału na 2 czesci, komunikaty zostana w katalogu templates, klasy i funkcje polecą do incudów.

Z tymi ortami, to już nie wkur... ! Bez jaj, gdzie Wy tam widzicie tą masę ortów? Ok, kilka literówek się trafi, są wyszukiwane i poprawiane!

Chłopie spokojnie, oburzasz się niepotrzebnie, a do tego nie masz racji. Jak dajesz coś do oceny to nie oczekuj, że wszyscy będą piać z zachwytu, szczególnie wrzucając takie coś.

To, że strona przechodzi walidację to nie znaczy, że jest już ok. Używasz np. div'ów do tworzenia marginesów a tak się nie robi. CSS inline - tak się nie robi. Lay na tabelach... Jak chcesz tworzyć strony tak jak kilka lat temu to ok, nikt Ci nie broni.

Mówisz, że masz fajne PHP a ja tam widzę np. nieoptymalne i nie do końca prawidłowe zapytania SQL, które w dodatku są dziurawe (chyba, że w jakimś innym miejscu filtrujesz dane, którego nie znalazłem)

Także odejdź od komputera, pobiegaj trochę , ochłoń i wróć do nauki.

Dobry pomysł, tak właśnie zrobię.. spacer:P

Spokojnie, spokojnie skoro coś zapodajesz to powinienieś liczyć się z krytyką (z pokorą ją przyjąć i zastanowić się nad tym co można poprawić) . Mi głównie nie podoba się to, że masz od pytki zagnieżdżonych tabel, ponieważ tak się już nie pisze. Ja sam do niedawna robiłem tego typu stworki a teraz staram się przejść na nowsze standardy pisania stron. I w takich uwagach nie ma zawiści czy złośliwości - po prostu jesteś traktowany jak ktoś kto się zna na rzeczy a ktoś kto się zna nie robi czegoś takiego (mówię tutaj o samym html jak coś ). Dostałeś listę tego co jest źle i teraz pracuj nad tym, żeby to poprawić, bo najważniejsze to wyciągnąć wnioski z błędów . Jeżeli dałbyś tą samą stronę, ale bez tabel tylko na divach to z pewnością nie spotkałoby się to z aż tak ostrym odzewem, ponieważ widać by było, że wiesc 'ocb', ale pisałeś stronę np. o 3:30 i o prau rzeczach zapomniałeś . Powodzenia w nauce .

Tworzenie nowych instancji klasy to dodatkowe obciążenie. W dodatku, gdy w ogóle nie korzysta się z 'możliwości' jakie dają dwie lub więcej instacji jest to zbyteczne obciążenie.Ale tutaj nie chodzi o "lubienie/nie lubienie" tylko o to jakie korzyści się otrzymuje z danego rozwiązania. Tu nie ma żadnych... Nawet nie znasz 'mojej metody' (ona zresztą w ogóle nie opiera się o Smarty) więc proszę bez takich komentarzy. Zresztą czy ja gdzieś napisałem, że ma być "po mojemu"?Ale jedną instancją klasy możesz wyświetlić kilka szablonów - nie widzę problemu z tym.I to jest podstawowy błąd... klasy nie mają służyć jako "paczki" dla funkcji...Powiedzmy, że jeżeli chodzi o przejrzystość to jest to pojęcie względne. Ale klasa przechowywująca jedynie komunikaty? To się mija z ideą OOP.Zdecydowanie lepiej się z nimi oswoisz czytając kody innych programistów - którzy mają nieco większe pojęcie n/t OOP O kilka literówek za dużo

PS. Walidator jest tylko w stanie sprawdzić poprawność składniową kodu - ale o semantyce czy dostępności nie ma mowy...
PS2. Żeby nie było - w żadnej mojej wypowiedzi nie ma złośliwości...

Ok, już przestaję zadzierać nosa, w końcu po to sie tu zarejestrowałem by podkradać dobre nawyki, uczyć się nowego.
Chętnie poszerzę swoją wiedzę nt. klas, ich przeznaczenia, właściwego wykorzystania...
O samych smartach też wiem nie dużo, zaczynam z nimi zabawę a podobają mi się bardzo... inny wymiar pisania.

Mam pytanie. Czy można wcisnąc smarta w smarta. Co przez to mam na myśli? Wczytujemy jednego smarta, skrypcik go mieli, następnie wyniku nie wywalamy do klienta ale używamy go jako element w innym smarcie.
W ten sposób tworzymy np jakies menu i je wciskamy w szkielet strony. Na koncu całość wysyłamy do klienta.
Mam nadzieje, że nie zakręciłem bardzo i ktoś mnie nauczy czegoś nowego.

ps. Crozin podaj, jeśli możesz, link do stron z kwiatkami, dzięki.

W smarty mozesz tworzysz pluginy, portlety i je wkladac do szablonu smarty. To co zaproponowales mija sie z celem.
Co to sa portlety? Nic innego jak szablon smarty, ktory poprostu includujesz w innym szablonie.
Co to jest plugin - zajrzyj do dokumentacji smarty

Dzięki za nakierowanie ... Wygoglałem fajny manual do smarty.
Pewnie znacię to, jeśli jednak się trafi co nie zna, polecam.
Pozdrawiam i oczekuje komentarzy dot. CMSa.

Po co Ci nagłówki h2, skoro nie ma h1? ;p
Jak bedziesz wszystkie style umieszczał bezposrednio w kodzie html style="..." ciężko będzie zmienić stylik na inny. A to jest przecież magia CSS.
Wygląd dobrze napisanej strony łatwo zmieniać zmieniając arkusz CSS z niewielkimi oczywiście poprawkami. Wpisz se w goglach CSS ZEN GARDEN, może się przyda.
Po co dajesz ikone W3C dot. walidacji XHTML, skoro CSS się nie waliduje?
Albo robisz walidacje i tego i tego albo wywal ten fajny niebieski przycisk na dole stronki ;oO Powodzenia!

AJAX:
w3c skasowalem ale to nie byla walidacja xhtml lecz html4.1....

Fakt stronka moze nie blyszczy, mialem nadzieje, ze ludzi zaciekawi CMS, ktos go odpali, wypowie sie, poradzi... chyba nikomu sie nie chcialo!
Pozdrawiam

Ja byłem jednym z tych co to ściągneli i zerknęli w kod - i powiem tyle:
kod HTML jest na podobnym poziomie co kod PHP, czyli niskim.

Niskim, czyli jakim?
Co chcesz mi zarzucić,że skrypt niechlujnie napisany, pełen złych nawyków, dziur? Funkcjonalność skryptu jest zerowa? Czy wg. Ciebie prosty kod to błąd? Czy wg. Ciebie "Hello world!" powinienem rozbudowywać do rozmiarów wielkiej aplikacji? Ja uważam, ze prosty zrozumiały, kod to wielki PLUS!
Zauważ, że dzięki skórkom smarta każdy może sobie zmienić wygląd panelu ADM w 90%! Tabele w MySQL są ładnie powiązane po generowanych kluczach md5. Wysyłane dane z formularza są filtrowane zanim zostaną dopuszczone do zapytania mysql. Odnaleziony błąd w danych z formularza powoduje wyświetlenie informacji o błędzie, proces dodawania danych w mysql jest zatrzymywany.
Pozdrawiam

To co napisałeś, to podstawa. Fundamenty tworzenia aplikacji internetowych. Wróć do swojego przejrzystego, zrozumiałego kodu za kilka lat, gdy przyjdzie Ci modyfikować stronę i wyciągnij wnioski, które masz podane na tacy od panów wyżej.