По колено в коде — Говнокод.ру

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)

Python / Говнокод #7956

−181

01
02
03
04
05
06
07
08
09
10
11

def main():
    iface = "eth0:"
    f = open("/proc/net/dev","r")
    lines = f.readlines()
    f.close()
    x = 0
    r = '\s+'+iface+'\s+\d+\s+\d+\s+\d+\s+\d+\s+\d+\s+\d+\s+\d+\s+\d+\s+\d+\s+\d+\s+\d+\s+\d+\s+\d+\s+\d+\s+\d+\s+\d+'
    while x < len(lines):
        result = re.match(r, lines[x])
        print result
        x +=1

Мсье знает толк в регулярках

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

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

Java / Говнокод #7955

+148

1
2
3
4
5
6

public List<OrderEntity> getOrders() {
    if (orders == null) {
        orders = new ArrayList<OrderEntity>();
    }
    return orders;
}

Потокобезопасность? Не, не слышал.

roman-kashitsyn, 23 Сентября 2011

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

Java / Говнокод #7954

+85

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
74
75
76
77
78
79
80
81
82
83
84
85

nLen = m_aRealTexts[index].m_txtString.length();
for (int iyhx = 0; iyhx < nLen; iyhx++) {
	cLetter = m_aRealTexts[index].m_txtString.charAt(iyhx);
	CTxtCharStyle pCharNode = new CTxtCharStyle(cLetter);
	pCharNode.SetTxtFontFace(m_nFontFace);
	m_aRealTexts[index].m_txtChars.add(pCharNode);
}

...


public void SetTxtFontFace(byte nType)
{
	String sLine, sTemp;
	int nMark;
	float xValue, yValue;		//笔画点位的临时坐标
	byte bIsBigChar = 0;		//1表示ANSI字体UserArial_ansi.uft，2表示Unicode字体UserArial_unicode.uft，3表示宋体hztxt1.shx
	m_cStroke.clear();			//清空笔画路径链表
	
	if(nType == 1)
	{
		if((int)m_cChar<0x7F || ((int)m_cChar>=0x2160&&(int)m_cChar<=0x2169))
		{
			//按Unicode编码顺序处理字符查找指定字库文件，避免读取不必要的字库文件
			InputStream fInFile = this.getClass().getResourceAsStream("/TagResources/UserArial_ansi.uft");
			
			...
			
			fInFile.close();
		}
		else if((int)m_cChar>=0x007F && (int)m_cChar<0x2FFF)
		{
			//字符在UserArial_ansi.uft中未找到，看是否在UserArial_unicode.uft中
			int nGetByte;
			sLine = new String("");
			sTemp = new String("");
			//float xValue, yValue;
			byte[] CharUnicode = new byte[2];			//一次读取两个字节，为一个字符
			byte[] cBytes = new byte[2];
			InputStream fInFile = this.getClass().getResourceAsStream("/TagResources/UserArial_unicode.uft");
			while((nGetByte = fInFile.read(cBytes, 0, 2)) > 0)
			{
				...
			}
			fInFile.close();
		}
		else if((int)m_cChar>=0x3000)
		{
			//字符在西文字库UserArial.uft中未找到，是大字体，则在宋体文件hztxt1.shx中查找
			m_cWidth = 112.0f;	//对于宋体，左下角点为坐标原点，包围盒长宽皆为127，调整量为(127-112)/2=5
			m_cHeight = 112.0f;
			byte nMoveDown = 5;
			InputStream fInFile = this.getClass().getResourceAsStream("/TagResources/hztxt1.shx");
			
			...
			
			fInFile.close();
		}
		else if(bIsBigChar == 0)
		{
			//仍然没有找到字符字体，则按空格处理
			m_cWidth = 12.0f;
			m_cHeight = 16.0f;
			m_cStroke.clear();
		}
	}
	else if(nType == 2)
	{
		try
		{
			
			InputStream fInFile = this.getClass().getResourceAsStream("/TagResources/hztxt1.shx");
			
			...
			
			fInFile.close();
			
		}
		catch(IOException e)
		{
			//异常处理
			e.printStackTrace();
		}
	}
}

Ещё один отжиг господ китайцев.

Магические числа вместо энумов и закрытие файлов вне finally - это ещё полбеды. Больше всего умиляет, что при каждой отрисовке отдельно для каждой буквы файл шрифта открывается и сканируется по новой. Неудивительно, что отрисовка так тормозит. Буду переписывать это дело - загружать глифы из файла один раз и затем дёргать их из кэша.

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

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

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)

Java / Говнокод #7950

+81

1

item.getName().equals("")==false

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

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

JavaScript / Говнокод #7949

+167

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

<!--//
function click() {
if (event.button==2) {
// Чтобы отключить левую кнопку поставьте цифру 1
// Чтобы отключить третию кнопку поставьте цифру 3
alert('Кнопка мышки отключена'); 
} }
// Здесь введите свою надпись
function press(){
alert('Клавиатура отключина');
}
// Здесь введите свою надпись
document.onmousedown=click;
document.onkeydown=press;
// -->

для полноты ощущений рекомендуется посетить источник http://www.taxi-svt.ru/

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

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

Java / Говнокод #7948

+78

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

PreparedStatement statement = createStatement();

for (Value value: values) {

	if (statement == null) {
		statement = createStatement();
	}

	try {
		if (statement.isClosed()) {
			statement = createStatement();
		}
	} catch (Exception e) {
		logger.error("...", e);
	}

	// ...
}

"бог любит троицу"

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

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

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