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

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

+145.5

1
2
3
4
5

if($record->getTitle()) {
  $request->setAttribute('title', $record->getTitle());
}else{
  $request->setAttribute('title', $record->getName());
}

определение существования метода у объекта $record

seroy, 18 Сентября 2009

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

1C / Говнокод #1867

−97.6

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

Запрос1 = Новый Запрос;
	Запрос1.Текст = "ВЫБРАТЬ
	|	Номенклатура.ЕдиницаИзмерения
	|ИЗ
	|	Справочник.Номенклатура КАК Номенклатура
	|ГДЕ
	|	Номенклатура.Ссылка = &ном";
	
	
	Запрос1.УстановитьПараметр("Ном", ЭлементыФормы.Товары.ТекущаяСтрока.Номенклатура);
	
	
	
	Результат1 = Запрос1.Выполнить();
	Выборка1 = Результат1.Выбрать();
	
	Пока Выборка1.Следующий() Цикл
		ед = выборка1.единицаизмерения;
	КонецЦикла;			
	
	ЭлементыФормы.Товары.ТекущаяСтрока.ЕдИзм= ед;

"Простой" способ подставить единицу измерения в табличную часть))

vedvd, 18 Сентября 2009

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

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

+156.1

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
86
87
88
89
90
91
92
93
94
95

function CompletesGetTableOld($clist, $mid =0)
{
  if(strlen($clist) or $mid > 0)
  {
    $com = CatalogComplsCostsGet(0, 0, $clist, $mid);
    $csz = sizeof($com['id']);
    if($csz > 0)
    {
        $param = CatalogComplParamsGet();
        $psz = sizeof($param);
        $tds = '';
        
        $cVals = CatalogComplParamValsGet(implode(",", $com['cid']));
        
        echo '
		<table cellpadding="4" cellspacing="0" border="0" width="700" class="table-comp">
		 <tr><td width="50%" class="first"></td>
       	';
        for($i = 0; $i < $csz; $i++)
 	    {
 	        echo '<td align="center">'.$com['name'][$i].'</td>';
 	        $tds .= '<td> &nbsp; </td>';
 	    }
        echo '</tr>';
        
        $prgrnm = '';
        for($i = 0; $i < $psz; $i++)
 	    {
 	        $data = false;
 	        $html = '';
 	        if($i==0||$param[$i]['gr_name']!=$prgrnm)
 	        {
				$html='
				<tr>
					<td class="first"><br><b>'.$param[$i]['gr_name'].'</b></td>';for($ii = 0; $ii < $csz; $ii++)$html.='<td class="second" > &nbsp; </td>';
					$html.= $tds.'
				</tr>';
 	        }
 	        $html.= '
 	        <tr>
 	          <td class="first">'.$param[$i]['name'].'</td>';
 	        
 	        for($ii = 0; $ii < $csz; $ii++)
     	    {
     	        @$val = $cVals[$com['cid'][$ii]][$param[$i]['id']];
     	        if(isset($val) and strlen($val))
     	        {
     	            if($param[$i]['type'] == 'CH') 
     	            {
     	                if($val == 1)
     	                {
     	                    $html.= '<td class="second" align="center"><img src="/images/tochka.gif" ></td>'; 
     	                    $data = true;
     	                }
     	                else $html.= '<td class="second"> &nbsp; </td>';
     	                
     	            }
     	            else 
     	            {
     	                $html.= '<td class="second" align="center">'.$val.'</td>';
     	                $data = true;
     	            }
     	        }
     	        else $html.= '<td class="second"> &nbsp; </td>';
     	    }
     	    $html.= '</tr>';
     	    
     	    if($data) 
     	    {
     	        $prgrnm = $param[$i]['gr_name'];
     	        echo $html;
     	    }
 	    }
 	    
 	    echo '
 	     <tr><td width="50%" class="first"><b>????</b></td>';
 	    for($i = 0; $i < $csz; $i++)
 	    {
 	    	$cost = intval($com['cost'][$i]);
 	        if($cost > 1000000) $cost = '<nobr>'.substr($cost, 0, -6)." ".substr($cost, 1, -3).". ".substr($cost, 4).'</nobr> ???';
 	        elseif($cost > 100) $cost = '<nobr>'.substr($cost, 0, -3)." ".substr($cost, strlen($cost)-3).'</nobr> ???';
 	        else $cost = '';
     	        
 	        echo '<td class="second" align="center" height="30">'.$cost.'</td>';
 	    }
               
        echo '</tr>
        </table>';
         
        return true;
    }
  }
   
  return false;
}

обратите внимание на переменную $data. долго и упорно собираем из кусочков $html, и только если $data истинно тогда выводим $html (примерно 5-10% случаев)

gorky, 18 Сентября 2009

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

C++ / Говнокод #1865

+64.2

01
02
03
04
05
06
07
08
09
10

ReturnType SomeFunction(...)
{
  ...
  {
    Mutex mutex;
    Mutex::ScopedLock lock(mutex);
    ...
  }
  ...
}

убер синхронизация

g26g, 18 Сентября 2009

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

Си / Говнокод #1864

+98

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

BOOL NewStartUsed()
{
  if ((BOOL) -1 == bNewStartUsed) {
    bNewStartUsed = FALSE;
    bNewStartUsed = TRUE;
    int iVal = bNewStartUsed;

    if (GetConfigInt(&iVal, "NewStart", "Common", ConfigFile)) {
      bNewStartUsed = (0 != iVal);
    }
  }
  return bNewStartUsed;
}

просто но комментс...

kubrinsky, 18 Сентября 2009

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

1C / Говнокод #1863

−108.2

1
2

Движение.Состояние = Перечисления.СостоянияРаботникаОрганизации[Причина.Метаданные().ЗначенияПеречисления[Перечисления[Причина.Метаданные().Имя].Индекс(Причина)].Имя];
.

Видимо, защита от шпиона: вот так определяется значение перечисления. (Причина - это тоже перечисление)

Kopchuga, 18 Сентября 2009

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

C++ / Говнокод #1862

+51.6

1
2
3
4
5
6
7
8

private: System::Void button2_MouseHover(System::Object^  sender, System::EventArgs^  e) {
				 int x = rand() % 428-75; //минусуем ширину кнопки, посчитать впадлу
				 int y = rand() % 301-23; // - высоту
				 this->button2->Location::set(System::Drawing::Point(x,y));
			 }
	private: System::Void button1_Click(System::Object^  sender, System::EventArgs^  e) {						
				 MessageBox::Show("Мы так и считали", "=)", MessageBoxButtons::OK, MessageBoxIcon::Asterisk);
			 }

лень

ChakuZa, 17 Сентября 2009

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

ActionScript / Говнокод #1861

−90.9

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

function getActiveRoot():int
{
	for(var c:int=0;c<i;c++) if(r_arr[c].isActive) return c;
	return -1;
}





public function addEntry():void
{
	if(getActiveRoot()<0)
	{
		r_arr.push(new Roots(i,in_txt.text,""));
		r_arr[i].addEventListener(MouseEvent.MOUSE_DOWN, downHandler);
		r_arr[i].addEventListener(MouseEvent.MOUSE_UP, upHandler);
				
		addChild(r_arr[i++]);
	}
	else
	{
	        r_arr[getActiveRoot()].w_arr.push(new Words(getActiveRoot(),r_arr[getActiveRoot()].wi,in_txt.text,""));
		r_arr[getActiveRoot()].w_arr[r_arr[getActiveRoot()].wi].addEventListener(MouseEvent.MOUSE_DOWN, WdownHandler);
		r_arr[getActiveRoot()].w_arr[r_arr[getActiveRoot()].wi].addEventListener(MouseEvent.MOUSE_UP, WupHandler);
				
		var ls:Shape=new Shape();
		ls.graphics.lineStyle(1, 0xFF0000,0.2);
		ls.graphics.moveTo(r_arr[getActiveRoot()].x,r_arr[getActiveRoot()].y);
	        ls.graphics.lineTo(r_arr[getActiveRoot()].w_arr[r_arr[getActiveRoot()].wi].x,r_arr[getActiveRoot()].w_arr[r_arr[getActiveRoot()].wi].y);
		addChildAt(ls,++wd);
		addChildAt(r_arr[getActiveRoot()].w_arr[r_arr[getActiveRoot()].wi++],++wd);
	}
}

Ой. А давайте проц погоняем! Чтоб не скучал. Отовсюду будем запускать getActiveRoot() чтобы узнать индекс! Ха-ха-ха..

t8apb, 17 Сентября 2009

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

ActionScript / Говнокод #1860

−91.2

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

function upHandler(event:MouseEvent):void 
{
        //r_arr[event.currentTarget.index].stopDrag();
				
	event.currentTarget.stopDrag();
				
        //r_arr[event.currentTarget.index].removeChildAt(2);

	if((r_arr[event.currentTarget.index].x>550)||(r_arr[event.currentTarget.index].x<160)||(r_arr[event.currentTarget.index].y>400)||(r_arr[event.currentTarget.index].y<0))
	{
	        r_arr[event.currentTarget.index].x=546;
		r_arr[event.currentTarget.index].y=(r_arr[event.currentTarget.index].index+1)*30;
	}
}

Сама семантика малопримечательна. Вся соль в способе адресации к объекту массива r_arr при совершении события мыши прямо над этим же объектом. Методы и свойства объекта достаются через ***у по индексу из-за пазухи: r_arr[event.currentTarget.index].removeChildAt(2)
вместо достаточного: event.currentTarget.removeChildAt(2)