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

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

+138

1

txtCollimator.parentNode.parentNode.parentNode.parentNode.parentNode.style.display = "none";

Прапрапрадедушка можно уже не показывать.
Дикая вложенность UserContol в ASP.Net дает о себе знать.

vaceknt, 11 Августа 2009

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

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

+158

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
...

$date1['year'] = ($_POST['year1']>=2006 && $_POST['year1']<=2037) ? $_POST['year1'] : (($_GET['year1']>=2006 && $_GET['year1']<=2037) ? $_GET['year1'] : date('Y'));
$date1['month'] = ($_POST['month1']>=1 && $_POST['month1']<=12) ? $_POST['month1'] : (($_GET['month1']>=1 && $_GET['month1']<=12) ? $_GET['month1'] : date('n'));
$date1['day'] = $type>2 ? 1 : (($_POST['day1']>=1 && $_POST['day1']<=31) ? $_POST['day1'] : (($_GET['day1']>=1 && $_GET['day1']<=31) ? $_GET['day1'] : date('j')));
$date1['hour'] = $type>1 ? 0 : ((isset($_POST['hour1']) && $_POST['hour1']>=0 && $_POST['hour1']<=23) ? $_POST['hour1'] : ((isset($_GET['hour1']) && $_GET['hour1']>=0 && $_GET['hour1']<=23) ? $_GET['hour1'] : 0));
$date1['min'] = $type ? 0 : ((isset($_POST['min1']) && $_POST['min1']>=0 && $_POST['min1']<=59) ? $_POST['min1'] : ((isset($_GET['min1']) && $_GET['min1']>=0 && $_GET['min1']<=59) ? $_GET['min1'] : 0));

$date2['year'] = ($_POST['year2']>=2006 && $_POST['year2']<=2037) ? $_POST['year2'] : (($_GET['year2']>=2006 && $_GET['year2']<=2037) ? $_GET['year2'] : date('Y'));
$date2['month'] = ($_POST['month2']>=1 && $_POST['month2']<=12) ? $_POST['month2'] : (($_GET['month2']>=1 && $_GET['month2']<=12) ? $_GET['month2'] : date('n'));
$date2['day'] = $type>2 ? cal_days_in_month(CAL_GREGORIAN, $date2['month'], $date2['year']) : (($_POST['day2']>=1 && $_POST['day2']<=31) ? $_POST['day2'] : (($_GET['day2']>=1 && $_GET['day2']<=31) ? $_GET['day2'] : date('j')));
$date2['hour'] = $type>1 ? 23 : ((isset($_POST['hour2']) && $_POST['hour2']>=0 && $_POST['hour2']<=23) ? $_POST['hour2'] : ((isset($_GET['hour2']) && $_GET['hour2']>=0 && $_GET['hour2']<=23) ? $_GET['hour2'] : 23));
$date2['min'] = $type ? 59 : ((isset($_POST['min2']) && $_POST['min2']>=0 && $_POST['min2']<=59) ? $_POST['min2'] : ((isset($_GET['min2']) && $_GET['min2']>=0 && $_GET['min2']<=59) ? $_GET['min2'] : 59));

$date1['day'] = cal_days_in_month(CAL_GREGORIAN, $date1['month'], $date1['year'])<$date1['day'] ? cal_days_in_month(CAL_GREGORIAN, $date1['month'], $date1['year']) : $date1['day'];
$date2['day'] = cal_days_in_month(CAL_GREGORIAN, $date2['month'], $date2['year'])<$date2['day'] ? cal_days_in_month(CAL_GREGORIAN, $date2['month'], $date2['year']) : $date2['day'];

$time_from = mktime($date1['hour'], $date1['min'], 1, $date1['month'], $date1['day'], $date1['year']);
$time_to = mktime($date2['hour'], $date2['min'], 59, $date2['month'], $date2['day'], $date2['year']);

$types = array(0,1,2,3);
switch ($type) {
	case 0:
		$sql_cols = "minute(from_unixtime(data.log_time)) as \"i\", hour(from_unixtime(data.log_time)) as \"G\", day(from_unixtime(data.log_time)) as \"j\", month(from_unixtime(data.log_time)) as \"n\", year(from_unixtime(data.log_time)) as \"Y\"";
		$sql_group_by = 'group by "i", "G", "j", "n", "Y"';
		break;
	case 1:
		$sql_cols = "'0' as \"i\", hour(from_unixtime(data.log_time)) as \"G\", day(from_unixtime(data.log_time)) as \"j\", month(from_unixtime(data.log_time)) as \"n\", year(from_unixtime(data.log_time)) as \"Y\"";
		$sql_group_by = 'group by "G", "j", "n", "Y"';
		break;
	case 2:
		$sql_cols = "'0' as \"i\", '0' as \"G\", day(from_unixtime(data.log_time)) as \"j\", month(from_unixtime(data.log_time)) as \"n\", year(from_unixtime(data.log_time)) as \"Y\"";
		$sql_group_by = 'group by "j", "n", "Y"';
		break;
	case 3:
		$sql_cols = "'0' as \"i\", '0' as \"G\", '0' as \"j\", month(from_unixtime(data.log_time)) as \"n\", year(from_unixtime(data.log_time)) as \"Y\"";
		$sql_group_by = 'group by "n", "Y"';
		break;
	default:
		$sql_cols = "'0' as \"i\", '0' as \"G\", '0' as \"j\", '0' as \"n\", year(from_unixtime(data.log_time)) as \"Y\"";
		$sql_group_by = 'group by "Y"';
		$type = 2;
}

...

Писал "биллинг" попутно изучая пхп) скрипт вывода статистики

PycmaM, 11 Августа 2009

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

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

+144.9

1

/a-z{3,}/.test(str);

Проверка содержит ли функция 3 символа алфавита или больше.
Правильно так: /[a-z]{3,}/

Oleg_quadro, 11 Августа 2009

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

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

+163.2

1
2
3
4
5

$isLoginned = true;

if (strlen(strval($isLoginned)) == 4) {
} else {
}

В одном из опенсурсных форумов в исходниках видел

chipfind, 11 Августа 2009

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

Pascal / Говнокод #1531

+91.8

01
02
03
04
05
06
07
08
09
10

...
      if DM.LastCOID <> -1 then
      begin
        if Pos(IntToStr(DM.LastCOID), Label1.Caption) <= 0 then
        begin
          Label1.Caption := 'Телефон: ' + '(' + IntToStr(DM.LastCOID) + ')';
          DataSet['CHANNELOUTERID'] := IntToStr(DM.LastCOID);
        end;
      end;
...

LastCOID - номер внешней линии, который отображается в лейбле Label1 и заносится в поле базы. Как и чем думал автор вставляя второе условие? Самое главное, для чего? В итоге, софтина иногда пропускала сохранение LastCOID в БД, тем самым портя статистику и настроение клиента.

ybinzu, 11 Августа 2009

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

Perl / Говнокод #1530

−117.2

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

my $cache_sql="SELECT `content`, `insertTime` FROM _cache WHERE `confid`='$$conf{confid}'";
...
    $cache_sql.=" AND `location`=".$self->{dbh}->quote($url);
...
    $cache_sql.=" AND `cgiparams`=".$self->{dbh}->quote($cgi_hash);
...
    $cache_sql.=" AND `callparams`=".$self->{dbh}->quote($callp_hash);
...
    $cache_sql.=" AND `userid`=".int($$params{userid});
...
    $cache_sql.=" AND `groupid`=".int($$params{groupid});

   my $replace_sql=$cache_sql;
   $replace_sql=~s/SELECT.*?FROM\s+_cache\s+WHERE/REPLACE INTO _cache SET/s;

perl+sql

poltora, 11 Августа 2009

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

Perl / Говнокод #1529

−101

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

my $iref=$self->{ditems}->_get_item({itemid=>$weather_itemid, set=>'Weather', fields=>['MinT', 'MaxT', 'Precip', 'Cloud', 'twater'], prefix=>$$p{prefix}});
    if ($$iref{$$p{prefix}.'MinT'}<0 && $$iref{$$p{prefix}.'MaxT'}<=0) {
        my $tmp=$$iref{$$p{prefix}.'MaxT'};
        $$iref{$$p{prefix}.'MaxT'}=$$iref{$$p{prefix}.'MinT'};
        $$iref{$$p{prefix}.'MaxT'}=$tmp;
    };
    #warn $$iref{$$p{prefix}.'MinT'} if !$$iref{$$p{prefix}.'MinT'}>0;
    $$iref{$$p{prefix}.'T'}=int( ($$iref{$$p{prefix}.'MinT'}+$$iref{$$p{prefix}.'MaxT'})/2 );
    $$iref{$$p{prefix}.'T'}='+'.$$iref{$$p{prefix}.'T'} if $$iref{$$p{prefix}.'T'}>0;
    #$$iref{$$p{prefix}.'T'}=~s/\-/\&\#150\;/gs;
    $$iref{$$p{prefix}.'MinT'}='+'.$$iref{$$p{prefix}.'MinT'} if ($$iref{$$p{prefix}.'MinT'}>0);
    $$iref{$$p{prefix}.'MaxT'}='+'.$$iref{$$p{prefix}.'MaxT'} if ($$iref{$$p{prefix}.'MaxT'}>0);
    $$iref{$$p{prefix}.'twater'}='+'.$$iref{$$p{prefix}.'twater'} if ($$iref{$$p{prefix}.'twater'}>0);
    
    $$iref{$$p{prefix}.'time'}=$$p{daystart}+$$p{hour}*3600;

poltora, 11 Августа 2009

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

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

+161

1
2

if ($_GET['prot'] && !ereg("^[_a-zA-Z0-9]+$",$_GET['prot'])) 	$_GET['prot'] = "";
if ($_GET['id']   && !ereg("^[_a-zA-Z0-9]+$",$_GET['id'])) 	$_GET['id'] = "";

разбирал один скрипт и нашел там такую проверку
это особенно весело смотрится, если учесть, что в этих переменных должно быть число

dead_star, 11 Августа 2009

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

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

+155

1
2
3
4
5
6
7
8
9

public function throwExceptions($flag = null)
    {
        if ($flag !== null) {
            $this->_throwExceptions = (bool) $flag;
            return $this;
        }

        return $this->_throwExceptions;
    }

Зенд

getme, 11 Августа 2009

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

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

+34

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

void dump_indexes(map<int,stride_slices> picks, ofstream &f);
{
  f<<"# full size: "<<picks.size()<<endl;
  for(map<int, stride_slices>::iterator
	p=picks.begin(); p!=picks.end(); ++p)
    {
      f<<"## "
       <<p->first<<"'st stride slices count: "
       <<p->second.size()<<endl;
      
      for(stride_slices::iterator
	    pp=p->second.begin(); pp!=p->second.end(); ++pp)
	{
	  f<<"### "
	   << p->first<<"'st stride, "
	   <<pp->first<<"'st slice size: "
	   <<pp->second.size()<<endl;
	  for(max_slice::iterator
		ppp=pp->second.begin(); ppp!=pp->second.end(); ++ppp)
	    {
	      f<<"#### "
	       <<p->first<<"'st stride, "
	       <<pp->first<<"'st slice, "
	       <<ppp->first<<"'st chunk size:"
	       <<ppp->second.size()<<endl;
	      
	      for(max_chunk::iterator
		    pppp=ppp->second.begin(); pppp!=ppp->second.end(); ++pppp)
		f<<"       "
		 <<p->first<<" "
		 <<pp->first<<" "
		 <<ppp->first<<" "
		 <<pppp->first<<" : "
		 <<pppp->second<<endl;	
	    }
	}
    }
}

а как нормально просмотреть кучу вложенных контейнеров?

Biburath, 11 Августа 2009

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

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