PHP — Говнокод.ру

PHP / Говнокод #7960

+162

01
02
03
04
05
06
07
08
09
10

# Если ошибка при авторизации
begin_page("Вход");

echo("<font color='red'>Ошибка при наборе логина или пароля</font>\n");

}

# Форма авторизации
if (!headers_sent())
begin_page("Вход");

Поручили мне отрефакторить систему "Компьютерный класс" и баги в ней пофиксить.
В login.php сразу же гениальнейшая конструкция.

7ion, 23 Сентября 2011

Комментарии (11)

PHP / Говнокод #7959

+161

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60

if (!GetCurrentCurrency()) { print $_lang[ErrorGetCurrentCurrency]; mclose(); exit; }

@set_time_limit(0);

$company_name=GetSetting('company_name');
$company_url=GetSetting('company_url');
$billing_url=GetSetting('billing_url');
$support_url=GetSetting('support_url');
$manager_email=GetSetting('manager_email');

$cronAutoSuspend = GetSetting('cronAutoSuspend');
$cronSendAdminEndOrder = GetSetting('cronSendAdminEndOrder');
$cronSendClientBillRemind = GetSetting('cronSendClientBillRemind');
$cronSendClientBillRemindDays = GetSetting('cronSendClientBillRemindDays');
$cronSendClientEndOrderRemind = GetSetting('cronSendClientEndOrderRemind');
$cronSendClientEndOrderRemindDays = GetSetting('cronSendClientEndOrderRemindDays');
$cronSendClientEndOrderDomainRemind = GetSetting('cronSendClientEndOrderDomainRemind');
$cronSendClientEndOrderDomainRemindDays = GetSetting('cronSendClientEndOrderDomainRemindDays');
$cronDeleteBill = GetSetting('cronDeleteBill');
$cronDeleteBillDay = GetSetting('cronDeleteBillDay');
$cronDeleteNeobrabOrdersWithoutBills = GetSetting('cronDeleteNeobrabOrdersWithoutBills');
$cronDeleteNeobrabOrdersDomainsWithoutBills = GetSetting('cronDeleteNeobrabOrdersDomainsWithoutBills');
$cronDeleteOrder = GetSetting('cronDeleteOrder');
$cronDeleteOrderDay = GetSetting('cronDeleteOrderDay');
$cronAutoDelete = GetSetting('cronAutoDelete');
$cronDeleteOrderDomain = GetSetting('cronDeleteOrderDomain');
$cronDeleteOrderDomainDay = GetSetting('cronDeleteOrderDomainDay');
$cronCloseTickets = GetSetting('cronCloseTickets');
$cronCloseTicketsDay = GetSetting('cronCloseTicketsDay');
$cronDeleteTickets = GetSetting('cronDeleteTickets');
$cronDeleteTicketsDay = GetSetting('cronDeleteTicketsDay');
$cronDeleteUserLogs = GetSetting('cronDeleteUserLogs');
$cronDeleteUserLogsDay = GetSetting('cronDeleteUserLogsDay');
$cronDeleteAdminLogs = GetSetting('cronDeleteAdminLogs');
$cronDeleteAdminLogsDay = GetSetting('cronDeleteAdminLogsDay');
$cronSendClientEndOrderShopRemind = GetSetting('cronSendClientEndOrderShopRemind');
$cronSendClientEndOrderShopRemindDays = GetSetting('cronSendClientEndOrderShopRemindDays');
$cronDeleteNeobrabOrdersShopWithoutBills = GetSetting('cronDeleteNeobrabOrdersShopWithoutBills');
$cronDeleteOrderShop = GetSetting('cronDeleteOrderShop');
$cronDeleteOrderShopDay = GetSetting('cronDeleteOrderShopDay');
$cronSendClientEndTestRemind = GetSetting('cronSendClientEndTestRemind');
$cronSendClientEndTestRemindDays = GetSetting('cronSendClientEndTestRemindDays');

$autoRenewEnable=GetSetting("autoRenewEnable");
$autoRenewClientChange=GetSetting("autoRenewClientChange");
$autoRenewDay=GetSetting("autoRenewDay");
$autoRenewDomainDay=GetSetting("autoRenewDomainDay");
$autoRenewShopDay=GetSetting("autoRenewShopDay");

$attachPDFtoBill = GetSetting("attachPDFtoBill");

$cronSendClientEndTestRemind = GetSetting('cronSendClientEndTestRemind');
$cronSendClientEndTestRemindDays = GetSetting('cronSendClientEndTestRemindDays');
$cronAutoDeleteTest = GetSetting('cronAutoDeleteTest');



if ($cronCloseTickets and intval($cronCloseTicketsDay) > 0) {

и так дальше с каждой переменной ~ 1500 строк кода

Равшан,где сериализация?
Где сериализация кодя я тебя спашиваю?!

- Где сериализация, насяльника, где?

tertychniy, 23 Сентября 2011

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

PHP / Говнокод #7958

+159

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21

<?
session_set_cookie_params(86400);
session_start();

$full_home_path = dirname(__FILE__);
require_once($full_home_path.'/_rootload.php');

if ($do == "logout" and !$pass) {
        userLogOut();
        Header("Location: billing.php");
        exit;
}

if ($dh = opendir($full_home_path)) {
	while (($my->xcsd31s = readdir($dh)) !== false) {
		if (preg_match("/_rootconfig/ui",$my->xcsd31s) and $my->xcsd31s != "_rootconfig.php") {
			$my->azazqwe2a = $my->xcsd31s;
			break;
		}
	}
	closedir($dh);

Индусский быдлокод по русски.
Таким мастерски написаным говнокодом пестрит все 8 мегабайт уйобища с гордым названием биллинг.

tertychniy, 23 Сентября 2011

Комментарии (10)

PHP / Говнокод #7957

+159

001
002
003
004
005
006
007
008
009
010
011
012
013
014
015
016
017
018
019
020
021
022
023
024
025
026
027
028
029
030
031
032
033
034
035
036
037
038
039
040
041
042
043
044
045
046
047
048
049
050
051
052
053
054
055
056
057
058
059
060
061
062
063
064
065
066
067
068
069
070
071
072
073
074
075
076
077
078
079
080
081
082
083
084
085
086
087
088
089
090
091
092
093
094
095
096
097
098
099
100

AddEventHandler("iblock", "OnAfterIBlockElementUpdate", Array("MyClass", "OnAfterIBlockElementUpdateHandler"));
class MyClass
{
    // создаем обработчик события "OnAfterIBlockElementUpdate"
    function OnAfterIBlockElementUpdateHandler(&$arFields)
    {
//привязка элементов (формирование комплектов)
if(CModule::IncludeModule('iblock')){
$id=$arFields['ID'];
$i=0;
$komplekt=array();
$elements=array();
$db_props = CIBlockElement::GetProperty(24, $id, "sort", "asc", Array("CODE"=>"KOMPLEKT"));
while($ar_props = $db_props->Fetch()){
$i++;
$komplekt[$i]=$ar_props["VALUE"];
}
for($j=1; $j<=$i; $j++){
$elements[$komplekt[$j]][0]=$id;
for($k=1; $k<=$i; $k++){
if($komplekt[$j]!=$komplekt[$k]){
$elements[$komplekt[$j]][$k]=$komplekt[$k];
}
}
}
for($z=1; $z<=$i; $z++){
CIBlockElement::SetPropertyValueCode($komplekt[$z], "KOMPLEKT", $elements[$komplekt[$z]]);
}
}
//автоматически добавляем скидку в торговом каталоге
if(CModule::IncludeModule('iblock')&& CModule::IncludeModule('catalog')){
$discount10=array();
$discount15=array();
$discount20=array();
$dbProductDiscounts = CCatalogDiscount::GetDiscountProductsList(
array(),
array(),
false,
false,
array()
);
while ($arProductDiscounts = $dbProductDiscounts->Fetch())
{
if($arProductDiscounts['DISCOUNT_ID']==5){
$discount10[]=$arProductDiscounts['PRODUCT_ID'];
}
if($arProductDiscounts['DISCOUNT_ID']==6){
$discount15[]=$arProductDiscounts['PRODUCT_ID'];
}
if($arProductDiscounts['DISCOUNT_ID']==7){
$discount20[]=$arProductDiscounts['PRODUCT_ID'];
}
}
$db_props2 = CIBlockElement::GetProperty(24, $id, "sort", "asc", Array("CODE"=>"SALE"));
while($ar_props2 = $db_props2->Fetch()){

if(!in_array($id, $discount10) && $ar_props2['VALUE']==66){
$discount10[]=$id;
}
if(in_array($id, $discount10) && $ar_props2['VALUE']==""){
$num=count($discount10);
for($i=0; $i<$num; $i++){
if($discount10[$i]==$id){
unset($discount10[$i]);
}
}
}
if(!in_array($id, $discount15) && $ar_props2['VALUE']==67){
$discount15[]=$id;
}
if(in_array($id, $discount15) && $ar_props2['VALUE']==""){
$num=count($discount15);
for($i=0; $i<$num; $i++){
if($discount15[$i]==$id){
unset($discount15[$i]);
}
}
}
if(!in_array($id, $discount20) && $ar_props2['VALUE']==68){
$discount20[]=$id;
}
if(in_array($id, $discount20) && $ar_props2['VALUE']==""){
$num=count($discount20);
for($i=0; $i<$num; $i++){
if($discount20[$i]==$id){
unset($discount20[$i]);
}
}
}
}
if($discount10){
CCatalogDiscount::Update(5, array("ACTIVE"=>"Y", "PRODUCT_IDS"=>$discount10));
}
else{
CCatalogDiscount::Update(5, array("ACTIVE"=>"N"));
}
if($discount15){
CCatalogDiscount::Update(6, array("ACTIVE"=>"Y", "PRODUCT_IDS"=>$discount15));
}
else{

Оригинальное форматирование кода сохранено. Код до 7-ой строчки - копипаста из документации. Жаль все не влезло, шедевр

inside, 23 Сентября 2011

Комментарии (15)

PHP / Говнокод #7953

+164

1
2
3
4
5
6
7
8
9

$arr = array(1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1);

function EtpGetEtpRoleCheckboxes($checked = array(1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1))
{
	$roles = EtpGetEtpRoleArray();
	foreach($roles as $key => $role)
	  $checkboxes .= "<input type=checkbox value='1' name='etp_roles[".$key."]' ".($checked[($key)] > 0 ? 'checked' : '')."> ".$role."<br />";
  return substr($checkboxes,0,-6);
}

с array_fill программист был не знаком. И так по всему скрипту. Это к первой строчке.

В функции меня удивило задание имен <input>'ам и value="1".

windrun, 23 Сентября 2011

Комментарии (9)

PHP / Говнокод #7952

+165

1
2
3
4
5

public function record_db_tool($string) {
	$string = (string)$string;
	$string = intval($string);
	return $string;
}

Видимо это проще, нежели скажем просто «intval(...);»

nethak, 23 Сентября 2011

Комментарии (5)

PHP / Говнокод #7951

+166

1
2
3
4
5
6

# Защищаемся от XSS (Для начала хоть как то...)
public function cleener_xss($string) {
	$string = @preg_replace('#(<[^>]+?)style[\x00-\x20]*=[\x00-\x20]*[`\'"]*.*?expression[\x00-\x20]*\([^>]*+>#i', '$1>', $string);
	$string = @preg_replace('#(<[^>]+?)style[\x00-\x20]*=[\x00-\x20]*[`\'"]*.*?behaviour[\x00-\x20]*\([^>]*+>#i', '$1>', $string);
	return $string;
}

nethak, 23 Сентября 2011

Комментарии (18)

PHP / Говнокод #7947

+166

1
2
3
4

$params['BannerIDS'] = null;
$params['BannerIDS'] = $banners;
 $t_banners = null;
unset($t_banners);

Ну так чтоб наверняка

govnoacc, 23 Сентября 2011

Комментарии (7)

PHP / Говнокод #7946

+147

01
02
03
04
05
06
07
08
09
10
11
12
13

PHP язык несомненно порочный. 
1. Взять, например, правило именования переменных - перед именем оных обязательно ставить знак доллара. 
Т.е. в PHP закреплен (подсознательно или сознательно) посыл к тому что во главе любой вещи стоит доллар, 
деньги, американские деньги, раздутая финансовая
пирамида современности как нечто обязательное к употреблению,
первостепенное, стоящее во главе всех вещей. 
2. Заметим так же, что перед названиями функций, т.е. участков кода занимающихся программной работой, 
подобного требования в PHP не существует. 
Тем самым PHP внедряет в подсознание идеи обесценивания работы и труда. 
3. Нетрудно связать второе с первым и понять что PH
психо-идеологически стимулирует склонность к рабству перед
вещественным, отрицает справедливого вознаграждения за труд, но превозносит власть денег, золота, роскоши и
вообще всего материального. Инспирирует рабовладельчество вещи над человеком, а не человека над вещью.

http://www.gamedev.ru/flame/forum/?id=152692#m10

C_Plus_Plus_Govno, 23 Сентября 2011

Комментарии (79)

PHP / Говнокод #7941

+163

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28

/// Здесь настройки подключения к БД
$bd_host = "openserver";

$bd_user     = "root"; // - логин
$bd_password = ""; // - пароль
$bd_base     = "test"; // - имя базы


mysql_connect($bd_host, $bd_user, $bd_password) or die(mysql_error());
mysql_select_db($bd_base);


// параметры
$conf = mysql_query("SELECT * FROM Settings where s='1' limit 1");	

$cr=mysql_fetch_assoc($conf);


$SITENAME  = $cr['sitename'];
$SITENAME  = str_replace('+',"<br>",$SITENAME);

$SITED     = $cr['domen'];
$SITEDESC  = $cr['description'];
$SITEKEYW  = $cr['keywords'];
$SEC	   = $cr['seconds'];

// другие конфигурации 
$sys_testmode = "false"	//- режим тестирования (сайты не показываются) true-вкл | false-выкл

Гордый файл config.php

Мартин, 22 Сентября 2011