Skrypt do oceny ;p

[GeoAce 67]

W celu poćwiczenia php, zrobiłem skrypt na systemie newsów ^^

Demo: http://progress.cba.pl/

Download:

 

Screeny z panelu admina:

 

Zapraszam do oceny ;p

Edytowano Lipiec 29, 2012 przez GeoAce (zobacz historię edycji)

[HaPe 242]

Po jakiego grzyba moduł licencji jeżeli ćwiczysz php?

Pokaż lepiej source, po nim można stwierdzić Twój poziom języka

[GeoAce 67]

Link znikł Plose; http://www.speedyshare.com/r4Nmu/TekstEngine.rar

 

Dodam że jestem początkujący

[HaPe 242]

Buforowanie jest złym lekiem na błędy

[ednet 136]

jesteś początkujący, ale nie przejmuj się, 90% osób "robiących strony www" nie zna php ma takim poziomie.

 

kilka uwag:

1. przejdź na prosty framework (np. Codeigniter) - będzie łatwiej, przyjemniej, bezpieczniej....

 

Jeśli nie chcesz używac frameworka to poniżej są uwagi do Twojego kodu:

 

2. czytelniej będzie jak wszystkie funkcje wrzucisz do zewnętrznego pliku i będziesz go includował

3. używaj prostej abstrakcji do bazy danych

4. KONIECZNIE używaj szablonów, np smarty

5. wiele plików ma powtarzający się kod np między <!DOCTYPE ...........<div id="wrapper">. Wywalić go do zewnetrznego pliku i includowac. To samo tyczy sie stopki.

6. mysql_close(); na koncu pliku nie jest konieczne. Połączenie wywołujesz funkcją mysql_connect i jest ono automatycznie kończone po zakończeniu działania skryptu.

 

Ed

 

Po jakiego grzyba moduł licencji jeżeli ćwiczysz php?

 

i dobrze!

niech się uczy prawidłowych nawyków, trzeba szanować swoją pracę i wiedzę

[Kolopik 222]

Noo nawet spoko ten skrypt. Zrób jeszcze wersję z obrazkami i sprzedawaj na Allegro

[Miłosz 2311]

Plose;

oraz ;p w nazwie tematu..

 

Dodam że jestem początkujący

Jak na podstawówke to całkiem nieźle.

 

Nie mieszaj nazw polskich i angielskich w zmiennych. Po co wildcard do wyciągnęcia trzech pól z rekordu? Nie wydaje mi się aby ktoś miał nick na 64kb wielkości

 

trzeba szanować swoją pracę i wiedzę

Oraz użytkowników na forum..

[GeoAce 67]

Nie chodzę do podstawówki ;p kilka lat temu skończyłem

Funkcje dodałem do osobnego pliku.

Tylko męczę się z szablonem

require_once "themes/<?php echo $theme ?>/css/reset.css"

Nie chce mi to działać .I Właściwie nie może, ktoś ma pomysł jak to zrobić?

 

 

@kolopik

Z obrazkami nie dam rady, nie mam pojęcia jak się za to zabrać. Ten pomysł na skrypt jest w miarę oryginalny. Licencja po to, że jak ktoś chce usunąć stopkę, to musi kupić premium. Na allegro i tego wystawie, jak dopracuję.

Edytowano Lipiec 30, 2012 przez GeoAce (zobacz historię edycji)

[regdos 1848]

require_once "themes/<?php echo $theme ?>/css/reset.css"

 

np.

include  'themes/'.$theme.'/css/reset.css';

[xorg 693]

Zerknalem tylko w jeden plik, wiecej czasu nie mam, i tak:

 

- nie uzywaj mysql w php, uzyj rozszerzenia mysqli badz PDO

- nie mieszaj HTML5 z HTML4 (Strict, sic!)

[SaVaGe123 21]

require_once "themes/<?php echo $theme ?>/css/reset.css"

<?php
require <?php echo  $theme ?>;
?>

Wsadziłeś 2 znaczniki

Tak jak regdos napisał, używaj include jak się uparłeś może być i once.

Kiedyś nie mogłem ogarnąć operatora łączenia ciągów, ale masz już wyżej gotowy.

Przyczepie się jedynie do tego że wyróżnienie po najechaniu myszką zrobiłeś całkiem na odwrót! Tekst powinien się robić grubszy po najechaniu myszką. A ten cień pod panelami powinien być szary, a po najechaniu czarny.

 

EDIT

 

+ do tego jakieś fajne hovery dla menu

Edytowano Lipiec 30, 2012 przez SaVaGe123 (zobacz historię edycji)

[GeoAce 67]

Hover zrobiony.

 

Ale tu mam problem:

<?
include "ustawienia.php";

require_once "themes/<?php echo $theme ?>/header.php"
?>

Jak zamknę php wcześniej, to będzie błąd bo require nie będzie w php

Edytowano Lipiec 30, 2012 przez GeoAce (zobacz historię edycji)

[elcct 159]

4. KONIECZNIE używaj szablonów, np smarty

 

Po co używać systemu szablonów, jeśli php sam w sobie jest takim systemem? To jest złamanie zasady DRY - Don't Repeat Yourself.

Widoki można trzymać w osobnych plikach php - tak jest na przykład zorganizowany framework Kohana.

[xorg 693]

Moze dlatego zeby skaownosc takiej aplikacji byla wieksza i kazdy mogl dostosowac pod siebie szablony ?

[Macsch 122]

4. KONIECZNIE używaj szablonów, np smarty

 

Twig.

[GeoAce 67]

A odpowie ktoś

Hover zrobiony.

 

Ale tu mam problem:

 

<?

include "ustawienia.php";

 

require_once "themes/<?php echo $theme ?>/header.php"

?>

Jak zamknę php wcześniej, to będzie błąd bo require nie będzie w php

[xorg 693]

<?
include_once "ustawienia.php";	 
include_once "themes/$theme/header.php";
?>

[GeoAce 67]

Działa dzięki. A jak Wam się podoba hover ? najedzcie myszką na coś

Poprawiony skrypt: http://www.mediafire.com/?n4zr4srngbdlntl

Edytowano Lipiec 30, 2012 przez GeoAce (zobacz historię edycji)

[Macsch 122]

Początkowo myślałem że ten podany kod:

 

<?

include "ustawienia.php";

 

require_once "themes/<?php echo $theme ?>/header.php"

?>

 

To niecny żart autora. Jednak nie.

[elcct 159]

Moze dlatego zeby skaownosc takiej aplikacji byla wieksza i kazdy mogl dostosowac pod siebie szablony ?

 

Co ma szablon do skalowalności? Szablon w php można tak samo łatwo dostosowywać (a nawet łatwiej, bo nie trzeba się uczyć dodatkowej składni).

[SaVaGe123 21]

Działa dzięki. A jak Wam się podoba hover ? najedzcie myszką na coś

Poprawiony skrypt: http://www.mediafire.com/?n4zr4srngbdlntl

Hovery na menu nie działają, a to przesuwanie panelu do góry po najechaniu myszką jest okropne!

Gdy najeżdżasz i zaczynasz czytać tekst ucieka ci z oczu. Usuń to a reszta okey.

 

EDIT

 

Jeszcze zginął ci ten szary cień ten jest zbyt czarny.

Edytowano Lipiec 30, 2012 przez SaVaGe123 (zobacz historię edycji)

[GeoAce 67]

Mi tekst nie ucieka. Na menu działa, tylko jest inny.

[ednet 136]

 

Po co używać systemu szablonów, jeśli php sam w sobie jest takim systemem? To jest złamanie zasady DRY - Don't Repeat Yourself.

Widoki można trzymać w osobnych plikach php - tak jest na przykład zorganizowany framework Kohana.

 

mozna uzywac czystego php jako szablonów, sam wiele lat tak robilem.

Teraz jednak chwale sobie smarty bo robi to co powinno. Poza tym że nie trzeba się uczyć składni, to nie ma żadnych zalet. Nawet kod strukturalny ze smartym jest bardziej uporządkowany.

 

Niech się uczy się "standardu", łatwiej bedzie mu się wdrożyć inne gotowe systemy.

 

Osobna wada php jako szablonu w moim wypadku to że chcialem go rozbudowywać o dodatkowe funkcje np. regexp itp i po jakimś czasie z prostego systemu stał się bardziej rozbudowany - prawie jak gotowe systemu.

[elcct 159]

Osobna wada php jako szablonu w moim wypadku to że chcialem go rozbudowywać o dodatkowe funkcje np. regexp itp i po jakimś czasie z prostego systemu stał się bardziej rozbudowany - prawie jak gotowe systemu.

 

Być może coś robiłeś źle. Szablon służy do prezentacji danych, więc do czego był ci tam potrzebny regexp?

Jedyna logika jaka powinna znajdować się w szablonie, to taka na potrzeby prezentacji danych.

[ednet 136]

 

Być może coś robiłeś źle. Szablon służy do prezentacji danych, więc do czego był ci tam potrzebny regexp?

Jedyna logika jaka powinna znajdować się w szablonie, to taka na potrzeby prezentacji danych.

mialem na mysli tworzenie różnych funkcji ułatwiających prezentację danych od prostych formatyjących tekst po grida prezentującego dane.