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

    +169

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    <?php 
function findAttribute($object, $attribute) { 
  foreach($object->attributes() as $a => $b) { 
    if ($a == $attribute) { 
      $return = $b; 
    } 
  } 
  if($return) { 
    return $return; 
  } 
} 
?>

    так человек предлагает возвращать нужный аттрибут из xml ноды.
    http://ru.php.net/manual/en/simplexmlelement.attributes.php#42692

    Запостил: striker, 29 Марта 2010

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

    • ava guest 29.03.2010 07:06 # +1

      Очень радует перебор ассоциативного массива и return только в случае если аттрибут найден. К С++ такого бы не пустил даже, да и к PHP тоже.
      Ответить

      • ava guest 31.03.2010 20:35 # 0

        Перебор ассоциативного массива медленный?
        А почему?
        Ответить

    • ava azzz 29.03.2010 10:05 # +5

      Вообще странно, нафига эти долбоёбы постят примеры кода на php.net? Потом чайники идут и читают маны и думают, что это - правильные примеры. А тем временем, на php.net over 99% такого говна. Расстрелять
      Ответить

      • ava larrygingras2 09.04.2010 20:32 # 0

        Да вот да, там следовало бы сделать команду модераторов, чтобы вычищать такой говнокод...
        Ответить

    • ava HyperGeek 29.03.2010 10:13 # 0

      Это потеря потерь...
      Ответить

    • ava guest 29.03.2010 13:30 # 0

      XPath уже не в моде?
      Ответить

      • ava Suor 29.03.2010 17:52 # 0

        XPath медленнее даже такого перебора
        Ответить

        • ava guest 29.03.2010 20:10 # 0

          не обязательно, если обращаться через SimpleXML, то может и быстрее...
          Ответить

      • ava striker 30.03.2010 01:10 # +0.2 

        $object[$attribute];

        к аттрибутам в simplexml можно обращаться как к элементам массива.

        http://ru.php.net/manual/en/simplexml.examples-basic.php
        SimpleXML can also access element attributes. Access attributes of an element just as you would elements of an array .
        Ответить

    • ava striker 01.04.2010 01:04 # +1.2

      хотя бы break надо поставить, а не перебирать все элементы даже если нужный найден
      Ответить

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