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

    +72

    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
    /*
 * Called from a large number of places
 * By Ofer
 */
static function getOrderStatus( $po_numb='', $supplier='', $shipping='', $id=0,$customer_id=0,$order='',$order_asc='',$item_status_id=0,$order_status=0, $start_date='',$end_date='',
		$name_begins_with = '', $phone = '', $email = '', $list_mgr_id = 0, $is_corporate = '', $sales_id=0,
		$first_name = '', $last_name='', $city='', $state='', 
		$zip='', $store_numb=0, $po_numb='', $release_date='',$supplier=0, 
		$tracking_numb='', $payment_method='', $shipped_balance='', $avs='', $last4='', 
		$brand_id=0, $model_numb='',$damage='', $delivery_issue='', $past_damage='', 
		$coming_back='', $file_claim_ups='', $file_claim_frt='', $rewview='', $balance='', $shipper=0,
		$has_balance = 0, $sales_account_id=0, $reference_number="", $third_party_order='', $ebay_id='', $trucker_id='',$get_total=false, 
		$limit=0, $start=0, $use_dates=0) {

    Просто очаровательный комментарий!

    Запостил: paulrudy, 05 Сентября 2012

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

    • ava scriptin 06.09.2012 01:04 # 0

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

    • ava nepster 06.09.2012 02:19 # −4

      массив, не не слышал !?
      Ответить

      • ava Lowezar 06.09.2012 08:41 # +6

        Лучше объект. В массиве легко опечататься и не заметить. К тому-ж можно сразу в объекте и дефолты прописать. :)
        Ответить

        • ava guest 06.09.2012 23:37 # +2

          пыхопроблемы, питоний dict вполне подошел бы
          Ответить

      • ava Lure_Of_Chaos2 06.09.2012 18:15 # +1

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

    • ava Miketsukami 06.09.2012 08:56 # +3

      Typical PHP
      Ответить

      • ava LispGovno 06.09.2012 19:15 # −1

        *Тупикалл
        upfix

        PS: Похоже я сильно поддаюсь влиянию, тк на эту мысль натолкнули в соседнем треде...
        Ответить

    • ava ghetto 06.09.2012 08:57 # +5

      это потрясающе! количество аргументов! $po_numb специально 2 раза объявлена? на всякий случай)
      Ответить

      • ava quest 06.09.2012 21:21 # +7

        Тебе чё, не в лом смотреть было на имена?
        Ответить

    • ava Steve_Brown 06.09.2012 10:25 # +4

      // Very usefull function by HaCKeRk0lYa666
      Ответить

    • ava psycho-coder 06.09.2012 12:29 # −2

      Москвастек не резиноваяый!
      Ответить

    • ava 3.14159265 06.09.2012 14:08 # +1

      [мечтательно]
      Может там юзают хитрый IoC контейнер. И оно само инжектится.
      Ответить

    • ava paulrudy 06.09.2012 16:33 # +9

      Надо было мне еще запостить, как это используется.... К примеру, выбрать заказы для определенного $sales_id (27 в моем примере):

      $orders = getOrderStatus( '', '', '', 0, 0, '', '', 0, 0, '', '', '', '', '', 0, '', 27);
      Ответить

      • ava roman-kashitsyn 06.09.2012 20:15 # +13

        Да так половина мелкософтовских апи выглядит
        Ответить

        • ava interested 06.09.2012 20:19 # −1

          Так выглядит, небось, половина всего структурного API.
          Сложный контракт простыми словами...
          Ответить

        • ava quest 06.09.2012 21:23 # +4

          В мелкософтовском АПИ ещё есть первый параметр, который определяет, какие из остальных 10 параметров (указателей на структуры из 10 полей, первое из которых - размер структуры) нужны, а какие игнорировать.
          Ответить

    • ava eth0 06.09.2012 19:05 # +1

      Почти Универсальная Функция Всего и Всего Такого.
      Ответить

    • ava Vindicar 07.09.2012 08:59 # +4

      God function?
      Ответить

      • ava nur 07.09.2012 10:09 # +1

        Слышал что у нас в прикладном отделе есть методы в 4к строк)
        Ответить

        • ava bormand 07.09.2012 10:53 # +2

          Гораздо веселее смотрелись бы методы в 4к столбцов ;)
          Ответить

          • ava Lure Of Chaos 07.09.2012 13:46 # +3

            причем 
            ...
01001 00001 10101 01010 10010 10111 00010 10101 01010
01010 11101 01000 01010 10111 10101 01010 11110 10101
01011 01011 01010 10101 10100 10010 10101 01010 10010
...
            Ответить

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