Screamer
Класс для загрузки файлов
07 Окт 2012, 17:35tipsun
Это конечно не скрипт, но.. велосипед 
Посоветуйте по коду что-нибудь, пожалуйста.
Здесь был говнокод
Flyd
tipsun пишет:
"Это конечно не скрипт, но.. велосипед
Посоветуйте по коду что-нибудь, пожалуйста.
[php:1:071c29731a]
[/php:1:071c29731a]"
Я бы просто удалил этот код и было бы идеально
Okula
Мда, код ужасен, и местами несёт в себе полный бред...
добавлено спустя 2 минуты:
Например:
[php:1:942e141378]$_COOKIE[$this->_name] = $value;[/php:1:942e141378]
Кто учил тебе так записывать значения в куки?
Про констранты я вообще молчу.
Okula
Смотри что ты делаешь:
Ты проверяешь была ли задана константа (возьмём любую) и если она небыла задана задаёшь её значение NULL
[php:1:82906ff9da]if (! defined('COOKIE_NAME_DEFAULT')) {
define('COOKIE_NAME_DEFAULT', null);
}[/php:1:82906ff9da]
Потом ты присваешь свойству класса значение константы (которое NULL).
[php:1:82906ff9da]private $_name = COOKIE_NAME_DEFAULT;[/php:1:82906ff9da]
Неопределённое свойство класса итак будет иметь значение NULL
[php:1:82906ff9da]private $_name;[/php:1:82906ff9da]
Будет иметь значение NULL.
Вопрос: к чему тут все твои манипуляции?
P.S. он риторический, можешь не отвечать
tipsun
Okula, ну а если константа будет задана?
если не будет задана, то в метод присвоится, скорее всего (хотя я не проверял), имя константы.
Насчет присваивания $_COOKIE, я подумал, что наверное почему-то неплохо было бы, чтоб значение куки уже было в массиве, без перезагрузки страницы.
А так я забыл, что:
[php:1:79911771d5]
<?php
setcookie('cookieName[key1]', $value); // если не ошибаюсь.
[/php:1:79911771d5]
Okula
Используя внешние константы ты жёстко привязываешь класс к внешнему коду, чего делать нельзя. Это противоречит принципам ООП.
09 Окт 2012, 22:50Flyd
tipsun пишет:
"Flyd, не флуди ;D"
Я вообще-то серьезно. Не пойму зачем одну единственную встроенную функцию заворачивать в целый класс