Оцените код

[zendushka]

Здравствуйте!
Оцените, пожалуйста, мой код.
Укажите недочеты, ошибки и т.д.
Что можно сделать лучше?

ООП, MVC, без использования фреймворков
Архив можно скачать здесь - http://bit.ly/hK04vQ
Заранее спасибо.

 

[Ragazzo]

Ну так норм(если учесть сколько там вообще есть)...чем-то CI навеяло, правда класс аплоадера у тебя как-то слишком велик по сравнению с другими...)

 

[AmdY]

смемени нафик этот грёбаный .htacces с реврайтом, который будет убивать приложение при отсутствии любого файла, типа картинок, js и даже favicon.ico
отформатируй код, это делает простым сочетанием трёх клавишь в ide.
самое существенное - используй модели, не должно быть sql в контроллере.
там ещё куча траблов, но это главные изменения на сейчас.

 

[Духовность™]

правда класс аплоадера у тебя как-то слишком велик по сравнению с другими...)

потому,что не его. и класс мега-дебильный - сам себе жнец и на дуде игрец. по моему, он только разве кофе не умеет варить =)

Оцените, пожалуйста, мой код. ООП, MVC, без использования фреймворков

Тут нет ООП и MVC. Тут структурный говнокод, обернутый в классы, именуемые контроллерами. Кроме того, нет даже отделения верстки от PHP:

// check image
			if((!isset($_FILES['image']['name']) || empty($_FILES['image']['name'])) && $_POST['button'] == 'Upload') $errors[] = '<div style="color: red;">Select image!</div>';
			elseif(!check_ext($_FILES['image']['name']) && $_POST['button'] == 'Upload') $errors[] = '<div style="color: red;">Allowed extensions: jpg,jpeg,gif,png!</div>';
			else{
				
				// upload and resize image
				if($description) $errors = upload_resize_save_img($_FILES['image'],isset($_POST['edit']) ? $_POST['edit'] : 0,$description);
				
			}

Так что для начала прочти это http://phpfaq.ru/tpl

потом, твой способ шаблонизации

<tr>
	<td width="24"><a href="edit/?id={ID}"><img src="front/img/template/edit.gif" border="0" /></a></td>
	<td width="39"><a href="?del={ID}"><img src="front/img/template/delete.gif" border="0" /></a>&nbsp;</td>
	<td width="408" align="center"><img src="front/img/uploaded/resized/{IMG}" /></td>
	<td width="201">{DESCR}</td>
</tr>

- это ерунда. Используй обычный PHP в качестве шаблонизатора:

<tr>
	<td width="24"><a href="edit/?id=<?=$id?>"><img src="front/img/template/edit.gif" border="0" /></a></td>
	<td width="39"><a href="?del=<?=$id?>"><img src="front/img/template/delete.gif" border="0" /></a>&nbsp;</td>
	<td width="408" align="center"><img src="front/img/uploaded/resized/<?=$img?>" /></td>
	<td width="201"><?=$descr?></td>
</tr>

 

[Ragazzo]

triumvirat
там же 5-6 файликов...ясен пень что там не будет ZF или Yii, но впрочем AmdY уже все корректно отметил...