Advertisement
Guest User

Untitled

a guest
May 28th, 2015
248
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
PHP 1.36 KB | None | 0 0
  1. 1. Делайте функции более универсальными, например allFiles можно сделать таким образом, чтобы она возвращала массив файлов, а уже потом его можно было обработать по своему усмотрению в других частях сайта.
  2. 2.          $dirToUpload = 'galery';
  3.             if(isset($dirToUpload)) {
  4.                 mkdir($dirToUpload);
  5.             }
  6. Код будет отрабатывать всегда, т.к. переменная $dirToUpload проинициализирована. В результате, при попытке загрузить изображение, получаем предупреждение, header не срабатывает.
  7. 3. После header(location) необходимо прерывать работу скрипта с помощью exit.
  8. 4. $content = 'content/'.$_GET['page'].'.html'; чтобы потом сделать file_get_contents() - плохо. У вас же есть шаблонизатор, который, без доп. параметров, просто выведет содержимое шаблона. По логике вещей, в шаблонизатор могут подставляться какие-то глобальные переменные, например.
Advertisement
Add Comment
Please, Sign In to add comment
Advertisement