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

    +167

    1. 1
    2. 2
    3. 3
    $query1234 = "select ....";
$result1234 = mysql_query($query1234);
$rs1234 = mysql_fetch_assoc($result1234);

    Это до такой степени не было фантазии придумать осмысленное название переменным. И бедные боялись, что перепишет где-то другие $query, $result, $rs. Не говоря уже о том, что никакой модели, сплошные фетчи, вместо того, чтобы вытянуть всё сразу

    Запостил: Sash, 03 Ноября 2011

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

    • ava dev6alexander 03.11.2011 09:25 # 0

      Видимо чел спешил и не хотел особо оптимизировать код :)
      Ответить

      • ava Nemoden 03.11.2011 16:08 # +3

        С каких пор давать нормальные названия переменным является оптимизацией?
        Ответить

        • ava roman-kashitsyn 03.11.2011 16:13 # 0

          возможно фраза относилась к
          > сплошные фетчи, вместо того, чтобы вытянуть всё сразу
          Ответить

          • ava Nemoden 03.11.2011 16:27 # 0

            О, скорее всего да, прошу прощения у dev6alexander в таком случае.
            Ответить

    • ava Byte 03.11.2011 18:50 # 0

      Может это сгенерированный код ))
      Правда у пхп, на мой взгляд, не самая лучшая система областей видимости переменных, что впрочем не оправдывает такого варварского отношения к названиям переменных.
      Ответить

      • ava Lure Of Chaos 03.11.2011 19:03 # 0

        а что не так с областью видимости? вполне хватает для практических целей
        ну кроме того, что неплохо было бы ограничить переменную блоком, где она обьявлена
        Ответить

        • ava Vasiliy 04.11.2011 08:09 # 0

          эти блоки называются функции.
          Ответить

          • ava Lure Of Chaos 04.11.2011 11:51 # 0

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

    • ava guest 04.11.2011 15:44 # −1

      Постить автосгененый код - плохо
      Ответить

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