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

    −50

    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
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    <?php
    //ini_set('display_errors', 1);
    //ini_set('display_startup_errors', 1);
    //error_reporting(E_ALL);
    
            
    	include('config.php');
    require_once dirname(__FILE__) . '/../../../../system/init.php';
            $user = new user();
            $code = NULL;
            $userid = $user::getId();
    
    	$hash = md5($_POST['action'].';'.$_POST['orderSumAmount'].';'.$_POST['orderSumCurrencyPaycash'].';'.$_POST['orderSumBankPaycash'].';'.$configs['shopId'].';'.$_POST['invoiceId'].';'.$_POST['customerNumber'].';'.$configs['ShopPassword']);		
    	if (strtolower($hash) != strtolower($_POST['md5']))
    //if (0 == 1)
                   { 
    		$code = 1;
    //echo 'Ошибка ' . $code;
    	}
    	else {
    
    
    		$code = 0;
    	}
    
    
    		print '<?xml version="1.0" encoding="UTF-8"?>';
    		print '<paymentAvisoResponse performedDatetime="'. $configs['shopId']  .'" code="'. $code .'" invoiceId="'. $configs['shopId']  .'" shopId="'. $configs['shopId'] .'"/>';
    
    //echo 'В конце  ' . $code;
    
    //$code = ($code === 0) ? $user::setPayment($user::getId(), date ("Y-m-d")) :
    //var_dump($code);
    //var_dump($user::getId());
    
    unset($user);
    
    ?>

    Без комментариев. Самый самый говнокод, созданный моим теперь уже бывшим коллегой.
    Хуже этого "шедевра" я так полагаю будет трудно найти. Хотя, это ж PHP, кот. кто попало типа как "осваивает", а потом появляется ЭТО.
    Угадайте, что делает этот код или по крайней мере должен был делать.

    P.S. Оригинальное форматирование, точнее его отсутствие, бережно сохранено.

    Запостил: skigf, 23 Сентября 2016

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

    • показать все, что скрытоНормальный код. Если ты распиздатый критик, напиши свою версию. Восхитимся, побагорим.
      Я, кстати, тоже несколько раз реализовывал интеграцию с "Яндекс.Кассой".
      Ответить
      • Рукожопых "пхппрогеров" так понимаю дохера и более нынче.
        Включи мозг и глянь почему последние строчки не будут работать.
        Ответить
    • А почему в performedDatetime айдишка магазина?
      Ответить
      • Спросите это у того, кто ЭТО "накодил".
        Ответить
    • if (strtolower($hash) != strtolower($_POST['md5']))


      Вот будет здорово, если $hash начинается с цифр.
      Ответить
    • плюс видны уши "классического дебага"
      Ответить
    • показать все, что скрытокакой багор 5)5)5)5)
      Ответить
    • Возмущению Ылшпа не было предела, когда он узрел в манускрипте сем наготу идентификатора магазина. А уж про бесовски сыромятые входящие данные и речи быть не может. Ылшпа не раздумывая более ни о чём сотворил хрестное знамение и наслал проклятия на того летописца, который манускрипт сей сотворил. После, правда, дюже передумал и помолился богу своему Робокассовскому за упокой души его заблудшей.
      Ответить

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