не работает класс

скобок не хватает. дальше не смотрел.
да и польза от такого "класса" весьма сомнительна.

 

класс писал либо новичок, либо садист-извращенец.

 

Привет, кеп.

 

PHP:

1.  
2. <?php
3. public function file_upload_data()
4.      {
5.          $this->file_size = $file_size;
6.          $this->file_info = $file_info;
7.          $this->file_extended = $file_extended;
8.          $this->file_server_patch = $file_server_patch;
9.          $this->s = $s;
10. ...
11. }
12.  

Это что за бред? Да и вообще установка всех параметров должна идти через методы, а не через паблик переменные класса

 

опять пост ТС не до конца дочитал
лучше займитесь процедурным программированием, а то с вашим классом вас тут палками забьют ;d

 

Все оописты невъебенные блин...

 

alexey_baranov
да ладно?)

 

В атрибуте action тега form нужно писать не upload.class.php, а название того скрипта, который принимает и обрабатывает файл. То есть того скрипта, в котором написано вот это:

PHP:

1. <?php
2. if(isset($_FILES['file'])) {
3.     include "upload.class.php";
4. // ...
5. ?>

 

А что тоже самое?

 

Ты забыл закрыть не только класс, но также и метод file_upload_data() - т.е. не хватает двух закрывающих фигурных скобок.

Вот этот блок в начале метода file_upload_data() действительно не нужен:

PHP:

1. <?php
2. // ...
3.     $this->file_size = $file_size;
4.     $this->file_info = $file_info;
5.     $this->file_extended = $file_extended;
6.     $this->file_server_patch = $file_server_patch;
7.     $this->s = $s;
8. // ...

Он обнулит все перечисленные в нём внутренние переменные класса, поскольку переменные $file_size, $file_info и т.д. внутри метода не заданы.
И необходимости в этом блоке кода нет, так как переменные у тебя public и ты задаёшь им значения явно в скрипте, который использует класс. Например, вот одна из строк, задающих значения:

PHP:

1. $upload_class->file_server_patch="uploads";

Чтобы иметь возможность задать параметр $s при вызове метода - упомяни тогда в объявлении метода об этой переменной:

PHP:

1. <?php
2. // ...
3. public function file_upload_data($s = 0) {
4.     // ...
5. }
6. // ...
7. ?>

Из-за недостающих фигурных скобок, PHP не может разобрать файл class.upload.php , т.е. тут синтаксическая ошибка. Файлы, в которых есть синтаксическая ошибка PHP не выполняет вообще, в том числе и всё то, что выше того места, где ошибка. Т.е. инструкции:

PHP:

1. <?php
2. // ...
3. ini_set('display_errors',1);
4. error_reporting(E_ALL);

не выполняются.
Помести эти инструкции в основной файл, который инклудит файл с классом. Где-то в начале, на следующей строчке после первого <?php

Вот ещё ошибочка:
$upload_class->file_server_patch="uploads";
$upload_class->file_new_name = dirname (__FILE__).$upload_class->file_server_patch;
функция dirname() убирает не только название файла на конце, но и последний слэш. Т.е. писать надо как-то так:

$upload_class->file_new_name = dirname (__FILE__) . DIRECTORY_SEPARATOR . $upload_class->file_server_patch;

 

Слушай, я думал попробовать тебе помочь, скопировал твой код, запустил, начал править/разбираться... но тут косяк на косяке.

Для начала в форме у тебя

HTML:

1. <input type="file" name="[b]filename[/b]" size="45">

а в обработчике

PHP:

1. if(isset($_FILES['[b]file[/b]'])){

Нужно, чтобы в форме и в обработчике названия совпадали.
size="45" - там нафиг не нужен.

По поводу путей и названия файла. В примере выше я просто написал, как это может быть. Вообще если подумать, то нужно сделать так, чтобы путь к каталогу для сохранения файлов был правильным абсолютным путём. Например, так:

PHP:

1. $upload_class->file_server_patch = dirname(__FILE__) . DIRECTORY_SEPARATOR . "uploads";

Название файла - можешь использовать исходное, если хочешь (то, которое было у пользователя, который загрузил файл). Однако тут нужно подумать, что делать, если кто-то загрузил файл с таким же названием файла, как у какого-то другого пользователя, который загрузил файл раньше. Можно переименовывать, можно создавать новую папку или выводить сообщение об ошибке и просить пользователя дать новое название файлу (последнее - бред, конечно).
Так можно использовать просто исходное название файла, которое было у пользователя, загружающего файл:

PHP:

1. $upload_class->file_new_name = $_FILES['filename']['name'];

Соответственно, в самом классе (когда копируешь файл) соединяешь путь к файлу с названием файла, но между ними нужно вставить разделитель путей, используемый в той или иной ОС:

PHP:

1. <?php
2. // ...
3. if(move_uploaded_file($this->file_info['tmp_name'], $this->file_server_patch . DIRECTORY_SEPARATOR . $this->file_new_name))
4. // ...

Важный момент, который ты не учитываешь. Или не знаешь о нём. Или не понимаешь, что значат эти слова.
В общем у классов есть методы-конструкторы. Допустим, ты создаёшь класс так:

PHP:

1. <?php
2. class className {
3.     private $someVar;
4.     public function functionName() {
5.         // some code here
6.     }
7. }
8. ?>

Так вот, если у тебя название класса className и название метода functionName одинаковы, то считается, что данный метод является методом-конструктором. Это старый синтаксис, сейчас принято методы-конструкторы называть словом __construct() , но, тем не менее, для совместимости новых версий php со старыми скриптами этот синтаксис по-прежнему поддерживается.
В твоём коде класс имеет название file_upload и метод, который загружает файл имеет такое же название:

PHP:

1. private function file_upload()

Т.е. получается, что ты объявляешь его методом-конструктором. Но сразу возникает ряд косяков. Во-первых, конструктор не может быть private, поскольку он ВСЕГДА вызывается из внешнего кода, а не из самого класса. Только public. Ну а во-вторых... Конструктор всегда запускается в момент создания экземпляра класса (объекта). Т.е. вот ты написал в коде:

PHP:

1. $upload_class= new file_upload();

Всё! В этот момент конструктор уже выполнился. Он выполняется сразу, как только ты создаёшь объект из класса.
А теперь смотри, что происходит. У тебя конструктор копирует загруженный файл в какую-то новую папку. Но он ещё НЕ знает, какое ты хочешь название файла, в какую папку ты хочешь скопировать, какие допустимы разрешения, какой должен быть размер файла и пр.
Ведь сперва же создаётся класс, а потом уже указываются все эти значения - пути, названия и пр. Но указывается всё это, выходит, уже после того, как метод-конструктор, который согласно твоему коду должен быть скопировать файл, отработал. Это понятно?

Но это ещё далеко не всё. Смотри, в первых строках метода file_upload() (который является конструктором класса file_upload) ты пишешь:

PHP:

1. $this->file_new_name = $file_new_name;
2. $this->upload_link = $upload_link;

Но при этом данный метод НЕ принимает никаких входящих параметров. Слова global (для того, чтобы получить доступ к "внешним" для метода переменным - тоже нет. Соответственно, внутри этого метода переменных $file_new_name и $upload_link - не существует. И когда выполняются эти 2 строчки кода, то значения $this->file_new_name и $this->upload_link становятся равными значению null (которое может интерпретироваться по-разному в зависимости от контекста: обычно это либо false, либо пустая строка, либо число 0). А на следующей строчке ты пишешь:

PHP:

1. <?php
2. // ...
3. move_uploaded_file($this->file_info['tmp_name'], $this->file_server_patch.$this->file_new_name)

Поскольку данный метод является методом-конструктором, и переменные класса ещё имеют значения по-умолчанию (которые ты сам прописываешь в первых строках класса), то они и будут использованы. Ведь конструктор выполняется ДО всех обращений к объекту. То есть ДО вот этого:

PHP:

1. <?php
2. // ...
3. $upload_class->file_server_patch = "uploads";
4. $upload_class->file_extended = array(".jpg", ".gif", ".png");
5. // и т.д...

Таким образом, $this->file_info равна просто пустому массиву (поскольку ты в начале класса пишешь public $file_info = array();), $this->file_server_patch будет равна слэшу (в начале ты пишешь: public $file_server_patch = "/";) ну и $this->file_new_name будет равна значению null, поскольку ты в первой строке метода file_upload() сам это значение, по сути получается, сбрасываешь, даже если бы оно и было. Итого мы получаем такую конструкцию:

PHP:

1. move_uploaded_file(null, "/");

Естественно, это ничего никуда не скопирует.

Но и сейчас по-прежнему не всё. Посмотри на то, как ты генерируешь ссылки при возникновении ошибок:

PHP:

1. <?php
2. // ...
3. echo"<a href="index.php?microtime();">Загрузить еще</a>";

Тут сразу как минимум два косяка в одной строке кода:
Во первых ты используешь одинаковые кавычки. В начале первая двойная кавычка после слова echo начала текстовую строку. Но уже через несколько букв у тебя снова появляется двойная кавычка, после символа =, который следует после атрибута href. В этом месте - всё, строка закрылась. То, что идёт дальше - уже не является строкой текста, а расценивается, как часть кода программы. Для того, чтобы написать символ двойной кавычки внутри строки, объявленной с помощью двойных кавычек - нужно экранировать все "внутристроковые" двойныек кавычки. Например так:

PHP:

1. <?php
2. // ...
3. echo "<a href=\"index.php?microtime();\">Загрузить еще</a>";

Также можно обрамить строку одинарными кавычками. Тогда двойные кавычки внутри строки будет не обязательно экранировать:

PHP:

1. <?php
2. // ...
3. echo '<a href="index.php?microtime();">Загрузить еще</a>';

Ну а второй косяк в том, что ты пытаешься вписать функцию microtime() прямо в строку символов. Не выйдет. Чтобы встроить в строку значение, которое возвращает функция, нужно выполнить конкатенацию строк (склеить строки):

PHP:

1. <?php
2. // ...
3. echo "<a href=\"index.php?" . microtime() . "\">Загрузить еще</a>";

Ну и ещё косячёк в начале класса при объявлении переменных:

PHP:

1. public $d = @opendir($file_server_patch);

Вообще тут я не знаю точно, но у меня это вызывало ошибку. При объявлении переменной $d можешь вообще никакого значения ей не присваивать, а директорию открывай уже внутри какого-нибудь метода.

Это так, то что я нашёл за то время, которое попробовал разобраться с твоим кодом. Естественно, не факт, что это все ошибки, которые там есть. Это лишь те, которые я заметил.

И напоследок: внимательно, от начала и до конца прочитай главу "Введение в объектно-ориентированное программирование" какой-нибудь книги по PHP 5, а также главу, в которой говорится об области видимости переменных. Это самый минимум, который нужен тебе для понимания того, что ты сам сейчас накодил.

И да, перенеси, наконец,

PHP:

1. ini_set('display_errors',1);
2. error_reporting(E_ALL);

в основной файл, который вызывается, когда отправляешь заполненную форму (я так понимаю, что это form.php). Как я и писал выше, вставь это после первого <?php

 

за что я уважаю sobachnik'а, так это за то, что он разберется в самом беспорядочном хламе