Говнокод

<?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.09.2016 (Updated 31.03.2018)

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

CrashTesterAnusov 10 лет ago # 0

Нормальный код. Если ты распиздатый критик, напиши свою версию. Восхитимся, побагорим.
Я, кстати, тоже несколько раз реализовывал интеграцию с "Яндекс.Кассой".
Ответить
- skigf 10 лет ago # 0 ⇈
  
  Рукожопых "пхппрогеров" так понимаю дохера и более нынче.
  Включи мозг и глянь почему последние строчки не будут работать.
  Ответить
bormand 10 лет ago # 0

А почему в performedDatetime айдишка магазина?
Ответить
- skigf 10 лет ago # 0 ⇈
  
  Спросите это у того, кто ЭТО "накодил".
  Ответить
inkanus-gray 10 лет ago # 0
```
if (strtolower($hash) != strtolower($_POST['md5']))
```
Вот будет здорово, если $hash начинается с цифр.
Ответить
- bormand 10 лет ago # 0 ⇈
  
  Или с 0e
  Ответить
Lure Of Chaos 10 лет ago # 0

плюс видны уши "классического дебага"
Ответить
- barop 10 лет ago # 0 ⇈
  
  >>//var_dump($user::getId());
  php)
  Ответить
bagor 10 лет ago # 0

какой багор 5)5)5)5)
Ответить
bot 10 лет ago # 0

Возмущению Ылшпа не было предела, когда он узрел в манускрипте сем наготу идентификатора магазина. А уж про бесовски сыромятые входящие данные и речи быть не может. Ылшпа не раздумывая более ни о чём сотворил хрестное знамение и наслал проклятия на того летописца, который манускрипт сей сотворил. После, правда, дюже передумал и помолился богу своему Робокассовскому за упокой души его заблудшей.
Ответить
- CrashTesterAnusov 10 лет ago # 0 ⇈
  
  Узрел тебе за щеку, проверяй.
  Ответить
  - XEP 10 лет ago # 0 ⇈
    
    Съебался отсюда быстро!!!
    Ответить