[SKRYPT] PGbook Opcje

[Lion_87]

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

www.PGbook.pl

[Pilsener]

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.