Ответ

Misha v.3 11.08.2008 17:32

- проверять ^if($photo) нехорошо. ведь в принципе может быть передана строка, которая в числовом контексте выдаст ошибку. лучше оставить просто ^if($photo is file");

- у вас комментарии не комментарии на самом деле. комментарием является строка только в том случае, если символ '#' находится в первой её позиции. кроме того часть комментарием избыточны, а ещё часть уже просто потеряла актуальность. избыточное комментирование тоже является злом;

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

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

- ^case[jpeg]{...} + ^case[jpg]{...} => ^case[jpg;jpeg]{...} (а ещё может быть JPG т.д., т.е. в switch лучше написать .lower[]).

---