Prosze o ocenę http://frogss.com

1. Pomieszany kod logiki z prezentacyjnym.
2. http://img110.imageshack.us/my.php?image=cmsik1kd1.png
3. Zaraz po instalacji: http://img136.imageshack.us/my.php?image=cmsik2gq6.png

Dalej nawet nie sprawdzam. Dlaczego? Dlatego, że przeciętny użytkownik nie chce grzebać w kodzie i ściągnie joomle.

Ogólnie kod oceniłbym na jakieś 2/10
Proponuję, żebyć popracowął nad tym wszystkim i przemyślał raz jeszcze całość.
Mimo, ze projekt mi sie nię podoba, to życzę jak najlepiej i pozdrawiam.

po pierwsze sciagles stara wersje aktualna to 0.6 wersja a ty scaigles 0.5 chyba na to wyglada..

index.php

[PHP] pobierz, plaintext 
<?php 
//Engine strony
include ("config.php");
include ("module/news.php");
//Koniec Engine strony
 
function pokaz() {
global $cmd,$id,$news_limit,$mode,$act,$lan;
if(isset($id)) {
 
  czytaj('');
}
elseif ($mode == ''){
	news($news_limit);
} elseif ($mode == 'add_news'){
news_t_open();
echo '<table style="font-size: 10px; color: black;" cellspacing="0" cellpadding="0" width="100%">
<tr>
<td align="left" style="padding-left: 3px;">
<b>Administracja Newsami</b> :: <font color="silver"><<a href="#">'.$lan['news_add_news'].'</a>></font>
</td>
</tr>
</table>';
news_t_close();
news_tresc_open();
   dodaj_newsa();
news_tresc_close();
}
}
 
 
//Ładowanie Skóry
include ("theme/$theme/index.php");
?>
[PHP] pobierz, plaintext 

1. Gdzie jest system szablonów?!
2. Optymalizacji więcej " używa się tylko w wyjątkowych okolicznościach

module/artykuly.php

[PHP] pobierz, plaintext 
<?php
function dzialy() {
global $PHP_SELF,$top,$lan;
$query = "SELECT * FROM artykuly_dzialy ORDER BY tytul1";
$result = mysql_query ($query) or die(mysql_error());
while($rowek = mysql_fetch_array($result)) {
 
$id = $rowek['id'];
$tytul1 = $rowek['tytul1'];
 
$tytul1 = stripslashes($tytul1);
echo '<table width="98%" align="center">
<tr>
					<td><table width="100%" border="0" cellspacing="0" cellpadding="0">
<tr><td height="15"><table border="0" cellspacing="0" cellpadding="0"><tr align="left"><td width="20"></td><td>';
$ile = mysql_num_rows(mysql_query("SELECT * FROM artykuly WHERE dzial='".$tytul1."' AND status=1"));
if($ile<>0) {
lista($dzial);
}
echo '<a href="artykuly.php?dzial='.$tytul1.'"><b>&rsaquo;</b> '.$tytul1.'</a> '.$lan['article_art'].':
<b>('.$ile.')</b> </td></tr></table></td></tr>					
</table></td>
				  </table>';
}
echo '<br /><center><a href="artykuly.php?mode=add_article">'.$lan['article_add'].'</a></center>';
}
//...
?>
[PHP] pobierz, plaintext 

3. Gdzie jest cache zapytań do SQL'a.. przydałaby się również jakaś dobra klasa do obsługi bazy danych..
4. global'ów juz się chyba nie używa..

Dużo do zmiany.. Ale może być dobrze

jak poprzednicy globali juz sie nie stosuje. co do klasy do bazy danych polecam DB_DataObject super sprawa.

kod wyglada jak śmietnik. Nie ma smartów czy innego systemu szablonów. Każda zmiana w kodzie może byc męczoca i czasochlonna.

-3/10

Po przeczytaniu:

Odechciało mi się ściągać tego "Najlepszego systemu CMS".

Szablony z jp. Normalnie wychował się na JP :|