1. PHP / Говнокод #2165

    +160.4

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    <?php 
// ...
$vars = get_class_vars($class_name);
		if(is_array($vars)) {
			foreach($vars as $var => $val) {
				eval('$this->' . $var . ' = \'' . $val . '\';');
			}
		}

    некакая UNI-CMS
    это не самое худшее. что я там нашел ... в общем мрак... (

    Запостил: nicolay-punin, 22 Ноября 2009

    Комментарии (11) RSS

    • ava corristo 23.11.2009 02:49 # +1

      а нафига?
      Ответить

    • ava nechin 23.11.2009 11:14 # 0

      Что-то маразмом попахивает.
      Ответить

    • ava paranoid 23.11.2009 13:32 # 0

      вот такое взаимодействие между классами как-то на стрём подсаживает $vars = get_class_vars($class_name);

      if(is_array($vars)) foreach($vars as $var => $val) - проверили - и гуд

      а вот eval здесь явно лишний.
      Ответить

    • ava guest 24.11.2009 04:44 # +1

      Проверка на массивоность не понятна. get_class_vars - выдает всегда массив.

      А вот что понапихано в свойства непонятно, может там какие формулы или структурки языковые?
      Ответить

    • ava guest 24.11.2009 12:53 # 0

      Судя по логике, этот кусок кода просто копирует все поля переданного объекта ($class_name) в текущий.
      В принципе, ничего быдлокодового не вижу, хотя проверка is_array() явно избыточна.
      Но минусовать не буду.
      Ответить

      • ava nicolay-punin 24.11.2009 13:01 # 0

        ну вы даете...

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

        про безопасность при использовании eval я вообще молчу... оно открывает новые просторы для разны php-иньекций...

        про отладку таких вещей я вообще молчу...
        Ответить

        • ava guest 24.11.2009 17:24 # 0

          Использование eval() - не признак говнокода. Это признак неправильного дизайна системы
          Ответить

      • ava interested 24.11.2009 19:06 # 0

        Тут, вроде бы всё хитрее, обстоит... Функция отдаёт все предустановленные поля класса. То есть, с конкретным объектом эта функция не связана.
        Это скорее похоже на попытку реализовать некое отражение и построить тип во время исполнения. Для php4, где, кажется, рефлекций нет, это некоторый "выход" для тех, кто активно отражениями ранее пользовался в других языках.

        Конечно, по такому короткому куску невозможно судить о "намерении"...
        Ответить

      • ava guest 01.12.2009 19:56 # +1

        добавлю $this->{$var} = $val; заместо eval()
        Ответить

    • ava guest 05.12.2009 00:22 # 0

      табуляции пробелами не учили?
      Ответить

    Добавить комментарий