Ответ

4ept 11.08.2008 18:18

- слишком много tmp и temp. неужели нельзя придумать нормальные названия этмм переменным, которые отражают суть? (+ объявить их локальными);

Имена и отображают то, что нужно:
$temp_file - ссылка на временный файл, который хранится на сервере, пока не удалится или не ресайзнется.
${IMGTEMP} - путь к папке, в которой хранятся временные файлы (в оригинале прописано $CONST:IMGTEMP) и в классе $CONST хранятся все глобальные переменные.

- $result[0] в Iinit: обычно statis==0 означает, что все хорошо, а если статус !=0 -- то он сообщяет об ошибке (ошибок может быть много разных). в своём коде также стоит придерживаться этого общепринятого правила;

^Iinit[] возвращает истину, если прошло все хорошо, т.е. подтверждает, что инициализация изображения на сервере прошла успешно: формат принят, ресайз сделан и т.д.. В случае, если не так, то прописывается код ошибки $error_code. Который и возвращается уже в конечном итоге, когда изображение обработано.
Если $error_code==0 - радостно улыбаемся, если равен 1 - злобно ругаемся, что формат не тот, 2 - место на диске закончилось и т.д.
Я же здесь не весь код привел, а только ключевой момент:)

---