Лучший говнокод за всё время

Лучший говнокод

В номинации:

Лучший рейтинг
Самый комментируемый

За время:

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

0

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
61
62
63
64
65
66
67
68
69
70
71
72
73

<snippet>
	<content><![CDATA[
/**
 * стандартный класс модуля
 */
Class adeveloper_main extends CModule
{
	const MODULE_ID = 'adeveloper.main';
	var \$MODULE_ID = 'adeveloper.main'; 
	var \$MODULE_VERSION;
	var \$MODULE_VERSION_DATE;
	var \$MODULE_NAME;
	var \$MODULE_DESCRIPTION;
	var \$MODULE_CSS;
	var \$strError = '';
	function __construct()
	{
		\$arModuleVersion = array();
		include(dirname(__FILE__)."/version.php");
		\$this->MODULE_VERSION = \$arModuleVersion["VERSION"];
		\$this->MODULE_VERSION_DATE = \$arModuleVersion["VERSION_DATE"];
		\$this->MODULE_NAME = "Module Name";
		\$this->MODULE_DESCRIPTION = "Module desc";
		\$this->PARTNER_NAME = "Adeveloper";
		\$this->PARTNER_URI = "http://adeveloper.ru";
	}
	function InstallDB(\$arParams = array())
	{
		//RegisterModuleDependences('main', 'OnBuildGlobalMenu', self::MODULE_ID, 'CAdeveloperBp', 'OnBuildGlobalMenu');
		return true;
	}
	function UnInstallDB(\$arParams = array())
	{
		//UnRegisterModuleDependences('main', 'OnBuildGlobalMenu', self::MODULE_ID, 'CAdeveloperBp', 'OnBuildGlobalMenu');
		return true;
	}
	function InstallEvents()
	{
		return true;
	}
	function UnInstallEvents()
	{
		return true;
	}
	function InstallFiles(\$arParams = array())
	{
		return true;
	}
	function UnInstallFiles()
	{
		return true;
	}
	function DoInstall()
	{
//		global \$APPLICATION;
	
		RegisterModule(self::MODULE_ID);
	}
	/**
	 *  @todo добавить блок удаления инфоблока, пока не делаю, чтобы лишнего не удалить
	 */
	function DoUninstall()
	{
//		global \$APPLICATION;
		UnRegisterModule(self::MODULE_ID);
	}
};
]]></content>
	<!-- Optional: Set a tabTrigger to define how to trigger the snippet -->
	<tabTrigger>bx-module-class</tabTrigger>
	<!-- Optional: Set a scope to limit where the snippet will trigger -->
	<scope>source.php</scope>
</snippet>

Шел 2016 год, а var и globals все никак не могу покинуть адептов битрикса.

https://github.com/adevelopers/bitrix/blob/master/BitrixSnippets/modules/bx-module-class.sublime-snippet

Keeper, 03 Мая 2016

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

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

0

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

public function getWinnerEmail($id)
{
        if (!is_numeric($id)) {
            throw new BadRequestHttpException('Ошибка:' . __FUNCTION__ . ' ' . __LINE___);
        }

        //$project = Projects::findOne(['id' => $id]);
        $project = (new \yii\db\Query())
            ->select(['id', 'win_user_id'])
            ->from(self::tableName())
            ->where(['id' => $id])
            ->one();
//        if ($project['win_user_id'] == null) {
//            return false;
//        }

        $result = User::findOne(['id' => $project['win_user_id']]);
        if ($result != null) {
            return ['id' => $result->id, 'email' => $result->email];
        }
}

Метод модели "Project"
PHP-Framework yii2

NotFake, 22 Марта 2016

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

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

−1

1

http://freelansim.ru/tasks/139911?utm_source=toster&utm_medium=referral&utm_campaign=task&utm_content=block_freelansim

Ну просто поржать.
А так вообще печально.

ЗЫ. Не давно на хабре читал про бакдор в wordpress е залившейся через обновление плагина.

Vasiliy, 18 Марта 2016

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

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

+2

01
02
03
04
05
06
07
08
09
10

// Статическая типизация для PHP < 7.0

set_error_handler(function(){});

function foo(int $bar)
{
	var_dump($bar);
}

foo(1); // Работает и выводит int(1)

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

KiTE, 16 Марта 2016

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

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

+6

1
2
3
4
5

// найдем максимальное значение идентификатора
$SQL = "SELECT MAX( id ) AS id FROM  `filter_lists`  WHERE project=1";
$id = query($SQL);
//новый идентификатор
$id = $id['0']['id'] + 1;

Auto increment PHP way

хуита, 09 Марта 2016

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

C# / Говнокод #19419

−1

01
02
03
04
05
06
07
08
09
10
11

public void SaveModels(IEnumerable<Activity> models)
 {
            if (models == null && models.Count() == 0) return;
            // step 1/3: remove empty models
            var empty = models.Where(m => !m.ForecastedValue.HasValue && !m.ActualValue.HasValue).ToList();
            if (empty != null)
            {
                models = models.Except(empty);
            }
           .....
}

Зачем такая конструкция, если можно просто
models = models.Where(m => m.ForecastedValue.HasValue && m.ActualValue.HasValue).ToList()
К тому же проверка на null бесполезна - ни Where, ни ToList не могут вернуть null. Даже если в коллекции ничего не останется.

sans, 08 Февраля 2016

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

Говнокод: по колено в коде.