Просто, для того чтобы  что то делать, нужно четко себе представлять, что делает та или иная операция. При отсутствии  того, люди пишут курьезный код. 


unlink - удаляет файл и возвращает TRUE в случае успешного завершения. 

т.е. в данном случае, программист пытается удалить файл повторно. 

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

сначало проверка возможно ли это, а потом уже действие 

Это сообщение отредактировал(а) Rock - 28.8.2006, 20:32

Rock, 

Не прав ты, камрад. Это функция библиотечная и элементарная как 2х2
Любой новичок/начинающий/незнающий/прочий должен СНАЧАЛА ПРОЧТИТАТЬ МАНУАЛ, особенно после ознакомления с этой темой. RTFM, как говорится!
А если он будет продолжать делать точно так-же, только втихую, - то он просто ламер - т.е. человек, нежелающий учиться.


Это сообщение отредактировал(а) smartov - 29.8.2006, 11:30

smartov, эту функцию я просто как пример привел, смысл то более глобальный, чем объяснение библиотечных функций

Rock, ну значит пример был неудачный. А обьяснять... кому. Друг-другу чтоль? Если спросят - обьясним.

из библиотеки самых бесполезных кусков кода топик превращается в библиотеку самого бесполезного флуда =)

Модератор: прекращаем пустую болтовню!

Товарищи! Если выкладывате какой-то код, хотя бы слегка комментируйте в нем происходящее! Так действительно будет лучше.

Кто-то спрашивал на счет шорт тегов: не используйте это безобразие! Я час копался - почему у меня не видно файла, который сгенерирован для экспорта (расширением html) и, с XML контентом. 
Угадайте почему? Потому что кто-то поставил хандлер РНР на html файлы + на сервере в конфиге были разрешены шорт теги! В итоге РНР матерится трехэтажным матом на XML и мы получаем пустую страницу, ибо (для тех кто не в курсе) XML файл начинается с объявления <?xml!

Добавлено @ 16:48 


Наверное человек просто хотел проверить существование файла. Из этого делаем вывод что нужно было делать так:



Добавлено @ 16:51 
Замечательно, нашел тему просто переполненную безобразием называемым "иф адом")) http://forum.vingrad.ru/index.php?showtopic=109772

IZ@TOP, не так страшен иф ад как текст с картинкой до Тега html......

Привет всем
на форуме видел

верно говорят трудно разбираться.
А чем можно заменить

разбить на функции, к примеру

Это называется рефакторинг кода.

Всемогущий, 
тут дейтсивтельно подходов много.
Можно как подсказал товарищ, разбить на фукнции.
Первые триста строк - одна функция. Вторые (по else) - вторая.
Это самый простой и в общем-то неправильный вариант. 
Потому-что я считаю и мой опыт это подтверждает, что можно всегда избежать ситуаций в которых иф 300 строк элс - 300 строк.
Наверняка можно построить логику по другому. Многое наверняка общее. Многое наверняка можно написать по-другому.

необязательно так страшно. интуиция мне подсказывает, что почти наверняка 50% кода в обоих ветвях могут быть объединены в одну функцию. возможно - с разными параметрами. хотя, конечно, вариант

правильнее.

Мне тоже, но реально кода в глаза я не видел...