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

    +161

    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
    function getGeneralOfficesStats(&$stats_year, &$stats_office_logo, &$stats_quant_client, &$stats_summ_client, &$stats_summ_supplier, &$stats_summ_delivery, &$stats_without_complex_price, &$quant_position, &$top_quant_client, &$top_summ_client, &$top_summ_supplier, &$top_summ_delivery, &$top_stats_without_complex_price, &$top_quant_position, $date_start_time=0, $date_end_time=0, $office_logo="")
       {
         $sql="SELECT t1.month, t1.year, t1.office_logo, t1.quant_client, t1.summ_client, t1.summ_supplier, 
    	       t1.summ_delivery, t1.stats_without_complex_price,
               t1.quant_position, t1.top_quant_client, t1.top_summ_client, 
               t1.top_summ_supplier, t1.top_summ_delivery, t1.top_stats_without_complex_price, t1.top_quant_position
    	       FROM stats_offices AS t1
    	       INNER JOIN offices AS t2 ON t1.office_logo=t2.name AND t1.currency_logo=t2.currency
    	       WHERE 1=1
    	       ".(($date_start_time>0 && $date_end_time>0)?" AND STR_TO_DATE(CONCAT(t1.year,'-',IF(LENGTH(t1.month)=1,'0',''),t1.month,'-01 01:01:01'), '%Y-%m-%d %H:%i:%s') BETWEEN STR_TO_DATE('".date("Y-m-01 00:00:00",$date_start_time)."', '%Y-%m-%d %H:%i:%s') AND STR_TO_DATE('".date("Y-m-d 23:59:59",$date_end_time)."', '%Y-%m-%d %H:%i:%s') ":"")."
    	       ".((!empty($office_logo))?" AND t1.office_logo='".mysql_escape_string($office_logo)."' ":"")."
    	       ORDER BY t1.office_logo, t1.year DESC, t1.month DESC";	
    	 list($stats_month, $stats_year, $stats_office_logo, $stats_quant_client, $stats_summ_client, $stats_summ_supplier, $stats_summ_delivery, $stats_without_complex_price, $quant_position, $top_quant_client, $top_summ_client, $top_summ_supplier, $top_summ_delivery, $top_stats_without_complex_price, $top_quant_position)=$this->query($sql);
    	 return $stats_month;   
       }

    Сменил работу. Стал знакомится с кодом. Тихо охуеваю.

    Запостил: Vasiliy, 24 Ноября 2014

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

    • там все такое.
      Ответить
    • Тут был комментарий, оскорбляющий язык PHP
      Ответить
      • Модерация?
        Ответить
        • Внутренний цензор)
          Ответить
          • Вы просто видимо пропустили эту тему, когда на говнокоде подняла голову кровавая модерация.
            Ответить
            • Видимо да. Давно это было?

              Модераторы ставили плюсики чтоб SU.GOVNOKOD не сняли с боны;)
              Ответить
              • http://govnokod.ru/11495Одно из первых сообщений (если не первое) написал я, когда уходил в отпуск. Потом Тарас, когда уезжал к бабушке, а потом началась эпидемия.
                Ответить
                • Уважаемые члены сообщества! Просим вас регистрироваться в качестве модераторов. Стоимость регистрации $50 с учетом НДС. Переведите сумму на наш кошелек в системе Яндекс.Деньги удалено модератором и сообщите нам номер транзакции. Торопитесь: количество модеративных аккаунтов ограничено.
                  Ответить
            • Комментарий удалён модератором.
              Ответить
    • типичный пхп легаси, было сто раз
      Ответить
    • Как вообще авторы живут со столь длинными строками? Даже с автоматическим переносом это выглядит жестоко.
      Ответить
      • write-only же. Накопипастил и забыл.
        Ответить
      • Скорее всего эта "функция" просто переписывается целиком, и все.
        Ответить
        • Совершено верно. Метод на задачу. Нужно что сделать, добавляем функцию ебошим как хочется передаем её все что надо. Ндо вренуть более 1 значения хуйня есть же & возвращаем далее все это скармиливаем смарти. Да да по коду нас ожидают увлекательные портянки $smarty->assign и да еще совсем забыл в конце скрипта обязательно
          портянка из unset() ибо не кто не знает куда этот файл может быть заинклуден. глобальный скоп хуле.
          Ответить
          • > Ндо вренуть более 1 значения хуйня есть же & возвращаем далее все это скармиливаем смарти.
            Что-то очень знакомое, где-то я видел что-то подобное... вореции
            Ответить
            • Си?
              Ответить
              • причем все три вида си: и плейн и кресты и шарп)
                Ответить
              • Кстати, я такое видал в жаба проектах посредством Ref:
                public class Ref<T> {
                  private T myValue;
                /*Ну тут понятно что, да?*/
                }
                Ответить
                • И массивами еще делают:
                  public void foo(int[] a) {
                      a[0] = 5;
                  }
                  int[] a = new int[1];
                  foo(a);
                  Ответить
                  • я такое первый раз в авто-рефакторинге Эклипса. Спросил у коллеги, что за питушня - он сказал, что ему ide сгенерила при выделении метода, типа всё норм.

                    Иногда ещё юзают AtomicXxx.
                    Ответить
                    • >Иногда ещё юзают AtomicXxx.
                      Уже где-то обсуждали на гк как лучше получать данные из колобков-замыканий, и вроде пришли к выводу что атомики в большинстве случаев переголова из-за волатилов и неудобных длинных методов.
                      Ответить
                    • >>Иногда ещё юзают AtomicXxx.
                      :))))))
                      Микроскопом так удобно угнетать квашеную капусту!
                      Ответить
                  • фу, бля! А почему массив? Давайте сразу ConcurrentSkipListMap, чтоб смешнее
                    Ответить
              • Я про знакомый стиль изложения говорил. И даже в спойлере написал.
                Ответить
            • "передай-ка мне указатель на память, а я её заполню":)
              Ответить

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