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

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

+163

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

std::vector<double> WBuffer;
std::vector<double> CleanWBuffer;
std::vector<Color> PixelBuffer;
std::vector<Color> CleanPixelBuffer;

void Scene3D::ClearBuffers()
{
        const size_t n = static_cast<size_t>(ScreenSize[0] * ScreenSize[1]);

        memcpy(&*(WBuffer.begin()), &*(CleanWBuffer.begin()), n * sizeof(*(WBuffer.begin())));
        memcpy(&*(PixelBuffer.begin()), &*(CleanPixelBuffer.begin()), n * sizeof(*(PixelBuffer.begin())));
}

Быстрая очистка буферов.
CleanWBuffer предварительно заполнен 0.0, CleanPixelBuffer предварительно заполнен нужным цветом.

Можно было воспользоваться std::fill, но оно работает в несколько раз дольше.

Пришлось так вот лезть в потроха std::vector. Доставляют подряд идущие & и *.

burdakovd, 12 Ноября 2010

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

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)

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

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

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