 [www]Stronka PPP pod ocennę, Strona Poradni Psych.-Ped. |
  |
| [www]Stronka PPP pod ocennę, Strona Poradni Psych.-Ped. |
  30.03.2008, 22:40:27
Post
#1
|
|
|
Grupa: Zarejestrowani Postów: 8 Pomógł: 0 Dołączył: 30.03.2008 Ostrzeżenie: (0%) 
 |
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. Ten post edytował cone 1.04.2008, 21:23:31 |
 |
 
|
|
30.03.2008, 23:12:27
Post
#2
|
|
|
Grupa: Zarejestrowani Postów: 147 Pomógł: 3 Dołączył: 28.12.2007 Skąd: toronto Ostrzeżenie: (0%)
|
Strona główna
Cytat Failed validation, 95 Errors  a potem kolejna wybrana losowo Cytat Failed validation, 139 Errors deklaracja strony  Kod <html> 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
|
|
|
|
|
30.03.2008, 23:27:56
Post
#3
|
|
|
Grupa: Zarejestrowani Postów: 8 Pomógł: 0 Dołączył: 30.03.2008 Ostrzeżenie: (0%)
|
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. |
|
|
|
|
31.03.2008, 07:05:13
Post
#4
|
|
 Grupa: Zarejestrowani Postów: 6 476 Pomógł: 1306 Dołączył: 6.08.2006 Skąd: Kraków Ostrzeżenie: (0%)
|
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 
|
|
|
|
|
31.03.2008, 09:01:56
Post
#5
|
|
 Grupa: Zarejestrowani Postów: 1 590 Pomógł: 185 Dołączył: 19.04.2006 Skąd: Gdańsk Ostrzeżenie: (0%)
|
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. Cytat dzieći ?Dużo błędów, literówek, całość nieczytelna, po o tyle banerów na stronie głównej? Bardzo słaba strona. |
|
|
|
|
31.03.2008, 10:07:39
Post
#6
|
|
|
Grupa: Zarejestrowani Postów: 8 Pomógł: 0 Dołączył: 30.03.2008 Ostrzeżenie: (0%)
|
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! |
|
|
|
|
31.03.2008, 10:40:38
Post
#7
|
|
 Grupa: Zarejestrowani Postów: 240 Pomógł: 13 Dołączył: 1.06.2007 Skąd: Wrocław Ostrzeżenie: (0%)
|
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.
-------------------- Good programming is 99% sweat and 1% coffee.
Make it idiot proof and someone will make a better idiot... |
|
|
|
|
31.03.2008, 11:00:57
Post
#8
|
|
|
Grupa: Zarejestrowani Postów: 8 Pomógł: 0 Dołączył: 30.03.2008 Ostrzeżenie: (0%)
|
Dobry pomysł, tak właśnie zrobię.. spacer:P
|
|
|
|
|
31.03.2008, 16:29:18
Post
#9
|
|
|
Grupa: Zarejestrowani Postów: 82 Pomógł: 6 Dołączył: 23.03.2008 Skąd: Szczecin Ostrzeżenie: (0%)
|
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 .
-------------------- Pracuję na: Apache 2.2.4, MySQL 5.0.20a, PHP 5.3.0
|
|
|
|
|
31.03.2008, 18:47:04
Post
#10
|
|
|
Grupa: Zarejestrowani Postów: 6 476 Pomógł: 1306 Dołączył: 6.08.2006 Skąd: Kraków Ostrzeżenie: (0%)
|
Cytat Podzielnoje smatry to naprawde taki blad? 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.Cytat Ty nie lubisz tak, ja lubie, dla mnie wszystko jest przejrzyste. Ale tutaj nie chodzi o "lubienie/nie lubienie" tylko o to jakie korzyści się otrzymuje z danego rozwiązania. Tu nie ma żadnych... Cytat Czy jesli nie preferuje twojej metody, to zaraz zle? 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"?Cytat Nie może być all w 1 smarcie bo srodek się zmienia(z 2 kolum na 1). Ale jedną instancją klasy możesz wyświetlić kilka szablonów - nie widzę problemu z tym.Cytat Klasa "strona" zawiera wszystkie funkcje, komunikaty. I to jest podstawowy błąd... klasy nie mają służyć jako "paczki" dla funkcji...Cytat Przykład : tablica[3][15](będziesz pamietał do czego to?... nie pisz mi tablica['glowna']['blad1'] jest krótsze ) 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.$klasy->glowna->art_add->$blad1 wygląda bardzo przejrzyście. Cytat Użyłem klas by trochę poćwiczyć ich wykorzystanie, oswoić się z nimi. Zdecydowanie lepiej się z nimi oswoisz czytając kody innych programistów - którzy mają nieco większe pojęcie n/t OOP Cytat 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! O kilka literówek za dużoPS. 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... |
|
|
|
|
31.03.2008, 20:29:11
Post
#11
|
|
|
Grupa: Zarejestrowani Postów: 8 Pomógł: 0 Dołączył: 30.03.2008 Ostrzeżenie: (0%)
|
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. Ten post edytował cone 31.03.2008, 20:30:32 |
|
|
|
|
31.03.2008, 20:33:34
Post
#12
|
|
 Grupa: Moderatorzy Postów: 36 557 Pomógł: 6315 Dołączył: 27.12.2004 |
Cytat 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 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
-------------------- "Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista "Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer |
|
|
|
|
2.04.2008, 15:18:37
Post
#13
|
|
|
Grupa: Zarejestrowani Postów: 8 Pomógł: 0 Dołączył: 30.03.2008 Ostrzeżenie: (0%)
|
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. |
|
|
|
|
3.04.2008, 08:21:36
Post
#14
|
|
 Grupa: Zarejestrowani Postów: 172 Pomógł: 9 Dołączył: 22.03.2008 Skąd: Wrocław Ostrzeżenie: (0%)
|
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! |
|
|
|
|
9.04.2008, 15:16:37
Post
#15
|
|
|
Grupa: Zarejestrowani Postów: 8 Pomógł: 0 Dołączył: 30.03.2008 Ostrzeżenie: (0%)
|
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 |
|
|
|
|
9.04.2008, 15:32:17
Post
#16
|
|
|
Grupa: Zarejestrowani Postów: 6 476 Pomógł: 1306 Dołączył: 6.08.2006 Skąd: Kraków Ostrzeżenie: (0%)
|
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. |
|
|
|
|
23.04.2008, 21:52:02
Post
#17
|
|
|
Grupa: Zarejestrowani Postów: 8 Pomógł: 0 Dołączył: 30.03.2008 Ostrzeżenie: (0%)
|
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 Ten post edytował cone 23.04.2008, 21:55:41 |
|
|
|
|
24.04.2008, 10:07:53
Post
#18
|
|
|
Grupa: Zarejestrowani Postów: 288 Pomógł: 12 Dołączył: 2.12.2005 Ostrzeżenie: (0%)
|
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.
|
|
|
|
|
|
Wersja Lo-Fi | Aktualny czas: 14.08.2025 - 06:36 |
