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

SQL / Говнокод #9386

−121

1
2
3
4
5

CASE st.testmode_order_active
WHEN NULL THEN '1'
WHEN '1' THEN '0'
WHEN '0' THEN '1'
as testmode_product_active

кусок sql запроса с реального проекта. логика в БД одна, в приложении - обратная.

codename, 10 Февраля 2012

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

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

+997

1
2
3
4
5
6
7
8
9

for (int z=NewArray[y-1];z<(NewArray[y]-1);z++)
                {
                  AnsiString ts = IntToStr(z+1);
                  int inum = -1;
                  try{
                    inum = StrToInt(ts);
                  } catch(...) {}
..... //дальше inum не используется
                }

Завис надолго. Так и не понял, зачем автор преобразовывает ts обратно

dm-ua, 10 Февраля 2012

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

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

+154

01
02
03
04
05
06
07
08
09
10

foreach($files as $k => $obj){
	foreach($obj as $key => $val){
		$temp[$val['file_sort']]=$key;	
	}
	ksort($temp);
	foreach($temp as $key => $val){
		$temp2[$val]=$obj[$val];
	}
	$files[$k]=$temp2;
}

не осилил usort(), удаляет ключи... =((

Zyets, 10 Февраля 2012

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

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

+141

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

private void tlistObject_SelectedIndexChanged(object sender, EventArgs e)
{
	IMapObject oTarget = this.tlistObjects.SelectedItem as IMapObject;

	if (oTarget != null)
	{
		if (this.SelectedObject != null)
		{
			this.SelectedObject.ObjectMode = ObjectModeElements.None;
		}
		this.SelectedObject = this.tlistObjects.SelectedItem as IMapObject;
		this.SelectedObject.ObjectMode = ObjectModeElements.Selected;
	}
}

(Is + Explicit приведение) или (As + проверка на Null)?
Ход конём.
ОБЯЗАТЕЛЬНОЕ использование this при обращении к полям, свойствам и методам текущего класса - отдельная тема. Это одна из особенностей неповторимого стиля, присущего аффтару. Весь код забит thisами под завязку.

ICELedyanoj, 10 Февраля 2012

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

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

+156

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

function mydebug($arr1=array(), $narr1='', $arr2=array(), $narr2='', $arr3=array(),$narr3='',$arr4=array(),$narr4='',$arr5=array(),$narr5='',$arr6=array(),$narr6='',$arr7=array(),$narr7='' ) {
	$var = 'arr';
	$nvar =  'narr';
	$quit = false;
	for($i=1; $i<8; $i++) {
		if(${$var.$i} == 'exit' ) {
				$quit = true;
				continue;
			}
			 
			elseif( !is_array(${$var.$i}) || is_array(${$nvar.$i})) {
				${$var.$i} = false;
				${$nvar.$i} = false;
			}
			 
			elseif(!empty(${$var.$i})) {
				echo ${$nvar.$i}.'<br>';
				var_dump(${$var.$i});
				echo "<hr>";
			}
			 
			else continue;
	}
	if($quit == true) {
		exit;
	}
}

breathe, 10 Февраля 2012

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

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

+162

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

<?php
//Класс для получения и сохранения данных пользователя.
class User{    
	const TABLE = 'users';	
	//Создаём к каждому полю в таблице users переменную (всего столбцов в базе >45)
	var $id = 0;
	var $user_type = 'private';
	var $name = '';
	var $surname = '';
	var $pk = '';
	//.... Тут ещё около 40 переменных	
	var $contactMail = '';
	var $contactPhone = '';
		
    //....Конструктор, получение данных, сохранение данных
    
	//Функции для получения данных пользователя
    function getId() {return $this->id;}
    function getUser_type() {return $this->user_type;}
    function getName() {return $this->name;}
    function getSurname() {return $this->surname;}
    //.... Тут ещё около 40 аналогичных функций
    function getContactMail() {return $this->contactMail;}
    function getContactPhone() {return $this->contactPhone;}
    
    //Функции для изменения данных пользователя (в конце нужно вызвать функцию для сохранения)
    function setId($data) {$this->id = $data;}
    function setUser_type($data) {$this->user_type = $data;}
    function setName($data) {$this->name = $data;}
    //.... Тут ещё около 40 аналогичных функций
    function setContactPhone($data) {$this->contactPhone = $data;}
    function setContact_type($data) {$this->contact_type = $data;}
   
	//функция для сохранения данных
    function save(){    	        
    	$fields = Array();
    	$fields['user_type'] = $this->user_type;
	    $fields['name'] = $this->name;
	    //.... Тут ещё около 40 переменных	
	    $fields['contactPhone'] = $this->contactPhone;
	    $fields['contact_type'] = $this->contact_type;
	       	
    	$this->id = Db::make(self::TABLE, $this->id, $fields);
    	return 1;
    }
}

Класс для получения и сохранения данных пользователя.

Присутсвуют и аналогичные классы для других таблиц в которых количество переменных не меньше.

Edd, 10 Февраля 2012

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

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

+153

1
2
3

do_site_map($result, $url_info);
do_robots();
htaccess_do($result, $url_info);

jonnywalker, 10 Февраля 2012

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

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

+171

1
2
3
4

//получаем ip-аддрес
ob_start();
$ip=$_SERVER['REMOTE_ADDR']; echo "$ip ";
$ipusers = ob_get_clean();

http://alltheragefaces.com/img/faces/large/misc-jackie-chan-l.png

TBoolean, 10 Февраля 2012

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

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

−143

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

Функция ПолучитьМаксимальныйДокументРеализации(ТЗРеализаций)
	МаксДок = ПолучитьПустоеЗначение("Документ");
	ТЗРеализаций.ВыбратьСтроки();
	Пока ТЗРеализаций.ПолучитьСтроку()=1 Цикл
		Если МаксДок = ПолучитьПустоеЗначение("Документ") Тогда
			МаксДок =  ТЗРеализаций.ДокументРеализации;
		ИначеЕсли МаксДок.ДатаДок < ТЗРеализаций.ДокументРеализации.ДатаДок Тогда
		    МаксДок =  ТЗРеализаций.ДокументРеализации;
		КонецЕсли;
		
	КонецЦикла;
	Возврат МаксДок;
КонецФункции

Во как надо получать последний документ из таблицы!

jerokee, 10 Февраля 2012