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

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

+166

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

<?php
# bla bla bla...
public function __construct ($dbname, $host = null, $user = null, $pass = null) {

	$numargs = func_num_args ();
	if ($numargs == 1)
		parent::__construct ($dbname);
	else
		parent::__construct ($dbname, $host = null, $user = null, $pass = null);
        $this->begin = time ();
  }
# bla bla bla...

"гениальный" вызов конструктора!

dwinner, 12 Ноября 2010

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

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

+157

01
02
03
04
05
06
07
08
09
10
11

void Scene3D::DrawTriangle(const Point3D &A, const Point3D &B, const Point3D &C, const Color& color)
{
        // ---------------------------------------------------------------------------------------
        // вспомогательные вычисления
        // нормаль
        const Vector3D& n = (B - A) ^ (C - A);

        // ...

        // центр треугольника
        const Point3D& medium = (A + B + C) / 3.0;

Руки как-то привыкли const TypeName& variableName набирать в определении параметров методов.
И случайно набралось такое (строки 6, 11)

Операторы (+, -, ^, /) над векторами возвращают Vector3D, не const Vector3D&.

Заметил только через полгода, и всё это время оно почему-то работало, и даже ворнингов не было.

Но такое ведь не должно работать!
Результат вычисления в правой части присваивания структура, то есть она возвращается в стеке. Если бы я присвоил её какой-то локальной переменной, для которой выделена память в стеке текущей функции, то она бы перед удалением скопировалась в локальную переменную. А так получается что ссылка (n, medium) указывает куда-то на стек, где временно хранится возвращенное оператором значение. И при следующем вызове любой функции эта область стека должна перезаписаться.

burdakovd, 12 Ноября 2010

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

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

+75

1
2
3

for ( Cluster c : Clusters.getInstance( ).listValues( ) ) {
  stopNet.newInstance( ).dispatch( cluster );
}

Суровые калифорнийцы как всегда суровы...

raorn, 12 Ноября 2010

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

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

+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
28
29
30
31
32
33
34
35
36
37

index.php:
<?php

	define("TEMPLATE_FOLDER", "./templates/");

	class Template
	{
		
		public function view($template, $data)
		{
			foreach($data as $key => $variable)
				$$key = $variable;
			
			require(TEMPLATE_FOLDER . $template . '.php');
			
			foreach($data as $key => $variable)
				unset($$key);
		}
		
	}
	
	$template = new Template();
	
	$data['test'] = array(1, 2, 3);
	
	$template->view('index', $data);
	
?>

templates/index.php:
<html>
	<body>
		<?php foreach($test as $row): ?>
			<?php echo $row ?><br/><br/>
		<?php endforeach; ?>
	</body>
</html>

Говно ли?

TapTap, 12 Ноября 2010

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

Куча / Говнокод #4599

+131

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

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"
"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
  <head>
    <title>Билайн анкетирование</title>
    <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
    <meta name="description" content="Билайн анкетирование">
    <meta name="keywords" content="Билайн анкетирование">
    <link href="/stylesheets/main_stl.css"  type=text/css rel=stylesheet>
    

  <script src="/javascripts/jquery-1.4.2.min.js" language="javascript"></script>

  <script src="/javascripts/jquery.maskedinput-1.2.2.min.js" language="javascript"></script>
  <script src="/javascripts/custom/masked.js" language="javascript"></script>


  </head>

  <body>

    <table id=main  cellspacing=0 cellpadding=0 border=0>

      <!-- HEAD -->
      <tr>
        <td style="height:128px; text-align:center;" align=center>
          <table style="height:128px; width:798px; margin:0px auto;" cellspacing=0 cellpadding=0 border=0><tr>
              <td style="padding-top:70px; padding-left:32px;  vertical-align:top; width:681px;"><img src="/images/beetitle.png"></td>
              <td style="padding-top:33px; vertical-align:top; width:117px;"><img src="/images/beelogo.png"></td>
          </tr></table>
        </td>
      </tr>

      <!-- BODY -->
      <tr>
        <td style="text-align:center;" valign=top align=center>
          
...

Это от сюда: http://opros-beeline.ru/login
Интересный у них там XHTML

Arigato, 12 Ноября 2010

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

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

+144

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

#include <stdio.h>

// Говнокод здесь
#define ABS(x) ((x) ? (x) : (-(x)))

int main () {
	char ch = -128;
	short sm = -32768;
	int i = -2147483647; i--;                                    // здесь
	long long l = -9223372036854775807LL; l--; // и здесь декременты нужны
	                                                                             // для подавления варнингов
	
	printf("char:\t\tABS(%hhd) = %hhd\n", ch, ABS(ch));
	printf("short:\t\tABS(%hd) = %hd\n", sh, ABS(sh));
	printf("int:\t\tABS(%d) = %d\n", i, ABS(i));
	printf("long long:\tABS(%lld) = %lld\n", l, ABS(l));

	return 0;
}

Может быть, баян, но...
Всем знакомый макрос ABS способен сделать большую гадость.
Вот вывод программы:
char: ABS(-128) = -128
short: ABS(-32768) = -32768
int: ABS(-2147483648) = -2147483648
long long: ABS(-9223372036854775808) = -9223372036854775808

whiskey, 12 Ноября 2010

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

Objective C / Говнокод #4597

−344

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

NSInteger lengthString=[string length]-100;
	for (NSInteger i=13000;i<lengthString;i++){ //ограничение поиска с начала, ищем пары ссылка - заголовок
		if ([string characterAtIndex:i]=='<' &&
			[string characterAtIndex:i+1]=='a' &&
			[string characterAtIndex:i+2]==' ' && 
			[string characterAtIndex:i+3]=='h' &&
			[string characterAtIndex:i+4]=='r' && 
			[string characterAtIndex:i+5]=='e' &&
			[string characterAtIndex:i+6]=='f' && 
			[string characterAtIndex:i+7]=='=') {
			indexForUrl=i+16;
		}

парсинг ответа на поисковый запрос к Google...

ниже еще пару сотен веток if

realsugar, 12 Ноября 2010

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

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

+144

1
2
3
4

if (!bNextPageVisited)
this->button(QWizard::NextButton)->setEnabled(false);
if (bNextPageVisited)
this->button(QWizard::NextButton)->setEnabled(true);

Упячка мозга?..

des-1008d, 12 Ноября 2010

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

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

+68

1
2
3

if (url == null) {
 throw new RuntimeException("Error reading resource " + url.getFile());
}

ха.
туториалы jogl

Lure Of Chaos, 11 Ноября 2010

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

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

−121

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

sub captcha {
	my $msg = $_[0];
	## make colors for validation image into hex again ##
	$rgb_foreground =~ s/\#//g;
	$rgb_shade =~ s/\#//g;
	$rgb_background =~ s/\#//g;
	$r_f = substr($rgb_foreground,0,2);
	$g_f = substr($rgb_foreground,2,2);
	$b_f = substr($rgb_foreground,4,2);
	$r_s = substr($rgb_shade,0,2);
	$g_s = substr($rgb_shade,2,2);
	$b_s = substr($rgb_shade,4,2);
	$r_b = substr($rgb_background,0,2);
	$g_b = substr($rgb_background,2,2);
	$b_b = substr($rgb_background,4,2);
// Тут я вырезал кусок никчемного кода
$ci{' '} = qq~
.......
.......
.......
.......
.......
.......
.......
.......
.......
.......
~;
$ci{'!'} = qq~
.......
...X...
...X...
...X...
...X...
...X...
.......
...X...
.......
.......
~;
$ci{'"'} = qq~
.......
..X.X..
..X.X..
..X.X..
.......
.......
.......
.......
.......
.......
~;
$ci{'#'} = qq~
.......
..X.X..
..X.X..
.XXXXX.
..X.X..
.XXXXX.
..X.X..
..X.X..
.......
.......
~;
$ci{'$'} = qq~
.......
...X...
..XXXX.
.X.X...
..XXX..
...X.X.
.XXXX..
...X...
.......
.......
~;
// Дальше идет вся таблица ASCII

Вот таким вот образом выводили каптчу в далеком 2009 году. Очень надежный метод :)

Arigato, 11 Ноября 2010

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

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