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

    • ava cartman 30.07.2010 17:53 # 0

      та да...
      Ответить

    • ava Lure Of Chaos 30.07.2010 19:19 # 0

      mysql_fetch_assoc помогло бы

      а так хоть лучше переменные, чем числовой индекс
      Ответить

      • ava cartman 30.07.2010 19:30 # +5

        foreach($row as $field) {
        $$field = $field;
        }
        Ответить

        • ava Lure Of Chaos 30.07.2010 19:32 # 0

          возможно
          Ответить

          • ava asics 02.08.2010 14:53 # 0

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

            еще некоторые extract юзают
            Ответить

      • ava Werdn 30.07.2010 19:48 # 0

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

        Пните меня, если это плохо
        Ответить

        • ava Lure Of Chaos 30.07.2010 19:49 # 0

          по умолчанию вернет и то и то, а зачем нам это надо?
          Ответить

          • ava Werdn 30.07.2010 19:55 # 0

            я вообще не знаю кому оно и зачем надо, там много такого
            (сам страшными тайнами php не владею, но на это смотреть не могу)
            Ответить

    • ava telnet 30.07.2010 20:22 # 0

      В CMS NetCat примерно так же выглядит раскидывание по переменным результатов SQL-запроса, вытаскивающего данные компонента. А перед этим ребята долго решают, из каких индексов что вытаскивать в зависимости от того, как составляется запрос (там несколько вариантов). Показывать не буду, больно много строк.
      Ответить

    • ava Анонимус 31.07.2010 18:55 # 0

      а еще бывает fetch_object. Вот вам и ООП и кусок ORMа на коленке. Почему не юзать?
      Ответить

      • ava Lure Of Chaos 31.07.2010 19:07 # 0

        если бы еще и связи обьектов восстанавливались )))
        Ответить

        • ava Werdn 31.07.2010 19:10 # 0

          если бы во рту грибы выросли, то вся система бы стала простой и понятной, а так даже IDE распарсить код местами не может, не говоря о многочисленных копиях inc.php с незначительными правками
          Ответить

          • ava Lure Of Chaos 31.07.2010 19:14 # 0

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

        • ava Анонимус 31.07.2010 19:11 # 0

          ага, и невинная строчка типа "echo $user->getBoss()->getOffice()->getStage()" приводила бы к сорока двум запросам)
          Ответить

    • ava Черный Властелин 31.07.2010 20:49 # 0

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

      • ava This is obvious 31.07.2010 21:44 # +1

        Накажи автора своими методами, может поймёт, что так делать плохо.
        Ответить

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