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

    • Это как раз тот случай, когда значения аргументов по умолчанию совершенно бесполезны и только усложняют чтение кода. Не говоря уже о количестве аргументов.
      Ответить
    • массив, не не слышал !?
      Ответить
      • Лучше объект. В массиве легко опечататься и не заметить. К тому-ж можно сразу в объекте и дефолты прописать. :)
        Ответить
        • пыхопроблемы, питоний dict вполне подошел бы
          Ответить
      • Это как раз тот случай, когда человек после этого должен встать из-за рабочего стола, и больше не заниматься своим делом.
        Ответить
    • Typical PHP
      Ответить
      • *Тупикалл
        upfix

        PS: Похоже я сильно поддаюсь влиянию, тк на эту мысль натолкнули в соседнем треде...
        Ответить
    • это потрясающе! количество аргументов! $po_numb специально 2 раза объявлена? на всякий случай)
      Ответить
      • Тебе чё, не в лом смотреть было на имена?
        Ответить
    • // Very usefull function by HaCKeRk0lYa666
      Ответить
    • Москвастек не резиноваяый!
      Ответить
    • [мечтательно]
      Может там юзают хитрый IoC контейнер. И оно само инжектится.
      Ответить
    • Надо было мне еще запостить, как это используется.... К примеру, выбрать заказы для определенного $sales_id (27 в моем примере):

      $orders = getOrderStatus( '', '', '', 0, 0, '', '', 0, 0, '', '', '', '', '', 0, '', 27);
      Ответить
      • Да так половина мелкософтовских апи выглядит
        Ответить
        • Так выглядит, небось, половина всего структурного API.
          Сложный контракт простыми словами...
          Ответить
        • В мелкософтовском АПИ ещё есть первый параметр, который определяет, какие из остальных 10 параметров (указателей на структуры из 10 полей, первое из которых - размер структуры) нужны, а какие игнорировать.
          Ответить
    • Почти Универсальная Функция Всего и Всего Такого.
      Ответить
    • God function?
      Ответить
      • Слышал что у нас в прикладном отделе есть методы в 4к строк)
        Ответить
        • Гораздо веселее смотрелись бы методы в 4к столбцов ;)
          Ответить
          • причем
            ...
            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
            ...
            Ответить

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