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

    +165

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    if ($row = mysql_fetch_array($result))
    {
    	$obegin = $row[0];
    	$vbegin = $row[1];
    	$lbegin = $row[2];
    	$oclose = $row[3];
    	$vclose = $row[4];
    	$opodk = $row[5];
    	$vpodk = $row[6];
    	$lpodk = $row[7];
    	$oend = $row[8];
    	$vend = $row[9];
    	$lend = $row[10];
    	$ozatr = $row[11];
    	$vzatr = $row[12];
    	$lzatr = $row[13];
    	$ooplat = $row[14];
    	$voplat = $row[15];
    	$loplat = $row[16];
    	$opay = $row[17];
    	$vpay = $row[18];
    	$lpay = $row[19];
    	$otraf = $row[20];
    	$vtraf = $row[21];
    	$ltraf = $row[22];
    }

    Даже не смешно

    Запостил: Werdn, 30 Июля 2010

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

    • та да...
      Ответить
    • mysql_fetch_assoc помогло бы

      а так хоть лучше переменные, чем числовой индекс
      Ответить
      • foreach($row as $field) {
        $$field = $field;
        }
        Ответить
        • возможно
          Ответить
          • невозможно!
            только если так:
            // чуть выше mysql_fetch_assoc
            foreach($row as $key => $field) {
              $$key= $field;
            }

            еще некоторые extract юзают
            Ответить
      • mysql_fetch_array() возвращает ассоциативный массив, переменные используются только один раз
        Доверяя хеш-функции PHP можно было бы написать вообще так:
        $row['чего там надо']

        Пните меня, если это плохо
        Ответить
        • по умолчанию вернет и то и то, а зачем нам это надо?
          Ответить
          • я вообще не знаю кому оно и зачем надо, там много такого
            (сам страшными тайнами php не владею, но на это смотреть не могу)
            Ответить
    • В CMS NetCat примерно так же выглядит раскидывание по переменным результатов SQL-запроса, вытаскивающего данные компонента. А перед этим ребята долго решают, из каких индексов что вытаскивать в зависимости от того, как составляется запрос (там несколько вариантов). Показывать не буду, больно много строк.
      Ответить
    • а еще бывает fetch_object. Вот вам и ООП и кусок ORMа на коленке. Почему не юзать?
      Ответить
      • если бы еще и связи обьектов восстанавливались )))
        Ответить
        • если бы во рту грибы выросли, то вся система бы стала простой и понятной, а так даже IDE распарсить код местами не может, не говоря о многочисленных копиях inc.php с незначительными правками
          Ответить
          • ну это же идиотично-динамичный пхп, где переменная сама не знает, какого она типа, и как над ней издевались в прошлых инклудах и будут издеваться в будущих
            Ответить
        • ага, и невинная строчка типа "echo $user->getBoss()->getOffice()->getStage()" приводила бы к сорока двум запросам)
          Ответить
    • Судя по именам переменных - кустарный биллинг, а в нём вся тарификация решительно свалена в одну навозную кучу таблицу. Блжад, эксель же есть для этого
      Ответить
      • Накажи автора своими методами, может поймёт, что так делать плохо.
        Ответить

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