Oddaje pod ocenę mój skrypt komentarzy PGbook jak i stronę www.pgbook.pl i czekam na wasze opinie.

www.PGbook.pl

krótko: "layout profesjonalny i przejzysty, strona z przyszloscia"

XSS http://www.plikigrafiki.com/pg/product.php...=8&p_id=919

Bardzo kiepski PHP połączony z html i css, koncepcja od początku błędna, choćby:

[PHP] pobierz, plaintext 
$pg_lang['again'] = "Oddałeś niedawno komentarz.";
$pg_lang['clone'] = "Pola nie mogą składać się z samych spacji.";
$pg_lang['fill'] = "Pola muszą być uzupełnione.";
$pg_lang['much'] = "Za duzo znaków w jednym z pól !!!"; 
[PHP] pobierz, plaintext 

- pliki językowe. Jak sobie wyobrażasz dać to do tłumaczenia czy pracować na tym? Trzeba znać nazwy poszczególnych zmiennych by z tego korzystać czy coś dodać, nie warto wynajdywać koła na nowo - lepiej używać standardów.

[PHP] pobierz, plaintext 
      if($vote==1){
 
       mysql_query("SET NAMES 'utf8'");
       $pg_sql="insert INTO pg_vote VALUES('$to', '+1', '$ip')";
       $pg_sql_E = mysql_query($pg_sql);
 
       echo '<span style="color:lime;">+1</span>';        
 
      }else{
 
       mysql_query("SET NAMES 'utf8'");
       $pg_sql="insert INTO pg_vote VALUES('$to', '-1', '$ip')";
       $pg_sql_E = mysql_query($pg_sql);
 
       echo '<span style="color:red;\">-1</span>';     
 
      }
 
    } 
 
}else{
 
  echo "Error";
}  
[PHP] pobierz, plaintext 

- po co ciągle set names? I po co co chwilę "echować" coś na ekran? Po to są zmienne, by na nich pracować, odbieramy dane, generujemy zmienne, łączymy je z html i wysyłamy do przeglądarki - taka jest logika aplikacji www, jak mam użyć takiego skryptu w swoim projekcie jak on mi coś echuje ciągle? I ten "Error" już nie tłumaczysz?

Lepiej od razu nabrać dobrych nawyków i na początek choć oddzielić PHP od HTML (system szablonów polecam, prosty lub popularny) oraz przestawić się na myślenie algorytmiczne a nie skryptowe i podzielić kod funkcjonalnie.

Zadziwiasz mnie


Jeśli chodzi o grafikę - mi się nie podoba, może i jest przejrzysta, ale czegoś mi w niej brakuje. Jednak głównie się skupiłem na samym skrypcie prezentowanym przez stronę i szczerze mówiąc doznałem szoku, gdy tylko zobaczyłem "<?", w plikach php. To od razu zniechęciło mnie do dalszego analizowania strony.

Wytłumacz mi to Ramzaa jeśli możesz.


Ty ..Ty...Ty....
Dzięki SHiP

nowe wersje php "nie obsługują" już tego znacznika, a bynajmniej nie odczytują tego jako skrypt php, choć mogę się mylić, ale samo używanie <? zamiast <?php bez żadnego zastosowania (o ile się nie mylę) jest mało... profesjonalne? i wygląda nie najlepiej. Poza tym u mnie na xamppie, z najnowszą wersją php, Twoja aplikacja by nie zadziałała, bo zamiast zparsowanego skryptu, otrzymał bym kod php w html.

@Ramzaa: w php.ini jest opcja którą włącza/wyłącza short_tags dlatego powinno się tego unikać. Zawsze może się znaleźć serwer, który ma to wyłączone.