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

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

+120

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

// See where we currently are in a calendar quarter.
// In Java, JANUARY == 0.
// Yes, I could just write (3 - currentMonth % 3),
// but this is clearer
switch (currentMonth % 3) {
case 0: // January, April, July, October
	monthsToInclude = 3; // include whole last quarter
	break;
case 1: // February, May, August, November
	monthsToInclude = 2; // include first two months of this quarter
	break;
default: // March, June, September, December
	monthsToInclude = 1; // include first month of this quarter
	break;
}

Или всё-таки наплевать на читаемость и заменить короткой версией?

someone, 23 Августа 2012

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

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

+140

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

// http://stackoverflow.com/questions/12033790/locknew-object-cargo-cult-or-some-crazy-language-special-case
private void foo()
{
    if (InvokeRequired)
    {
        lock (new object())
        {
            if (m_bar!= null)
                Invoke(new fooDelegate(foo), new object[] { });
        }
    }
    else
    {
        if(OnBazChanged != null)
            OnBazChanged();
    }
}

По одной из версий, этот бессмысленный lock - это "оптимизация", раньше было поле private readonly object lockObj = new object();
P.S. Поставьте таки autocomplete="off" в поле ввода капчи <input class="captcha" id="formElm_captcha" name="captcha" type="text" value=""> - задолбала она старые капчи предлагать.

victor-homyakov, 23 Августа 2012

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

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

+67

1

$msg = (($this->data[0] == '1' || $this->auth->demo == 1) ? "orderCheck" : "orderInsert")." ".(($this->data[4] == '1') ? "1" : "0")."*|*".$a[$ww][$objs[0][0]][0]['o']."%|%".$a[$ww][$objs[0][0]][0]['h']."%|%%|%".$a[$ww][$objs[0][0]][0]['e']."%|%".$this->data[9].$hord."*|*".$this->regiond['phone_code'].(($this->auth->demo==1) ? $this->regiond['demo_phone'] : $this->auth->phone)."*|*".$nord."*|*".$this->data[7]."*|*".$this->regiond['phone_code'].(($this->auth->demo==1) ? $this->regiond['demo_phone'] : $this->data[6])."*|*".$this->data[5]/*(($this->data[5] == '2' && $this->regiond['lux'] == '1') ? "deluxe" : "")*/."*|*".$this->java."*|*".$this->data[13];

Картина Репина. PHP притворяется регэкспом.

blessed, 23 Августа 2012

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

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

−99

1
2
3
4
5
6
7

// WTF is 8.5? Oo I will never, ever hardcode some random values.
	linkStatusImage.frame = CGRectMake (5, height / 2 - 8.5, 17, 17);
	descriptionLabel.frame = CGRectMake (27, 5, width - 32 - rightMargin, 20);
	authorLabel.frame = CGRectMake (27, 30, width - 32 - rightMargin, 20);
	addressLabel.frame = CGRectMake (27, 55, width - 32 - rightMargin, height - 60);
	separatorImage.frame = CGRectMake (0, height - 2, separatorImage.image.size.width, 2);
	favButton.frame = CGRectMake (width - 22, height / 2 - 8.5, 17, 16);

byss, 23 Августа 2012

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

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

+115

1
2
3
4

-- Настало время запостить самое главное гавно Хаскеля:
-- http://www.johndcook.com/blog/2010/05/18/pure-functions-have-side-effects/
-- У чистых функций всегда есть side-effects, которые выражаются в пожирании памяти и CPU. 
-- Очевидно конечно, но почему об этом не орут на первой странице каждой книги по функциональному программированию?

Я проверил, код компилируется:
http://ideone.com/sozuV

HaskellGovno, 23 Августа 2012

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

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

+48

1
2
3
4
5
6
7

//Функция проверки закачиваемого файла
function fileNameCheck($file) {
	if(preg_match("/[^a-z\d\-\._]/i", $file)) {
		return true;
	}
	return false;
}

Приятно удивляет соответствие названия функции с её возвращаемым значением.

Edward, 22 Августа 2012

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

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

+53

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

class MYSQL_ACTIONS
{      
		public $arguments = array();
		public $values;
		public $rows;
		public $check = true;
		public $list_array = array();
		function __construct ()
		{
			$this->arguments = func_get_args();
			$z = '';
			foreach ($this->arguments as $value)
			{
				$y ="$value,";
				$z .= $y;
			}
			$this->rows = substr("$z",0,-1);
		}
		function check_isset()
		{
			$argument = func_get_args();
			for ($i = 0; $i < count($argument); $i++)
			{
				foreach ($argument as $value)
				{      
					${$value} = $this->list_array[$i++];
				}
			 
			}
		 
			for($i = 0; $i < count($argument); $i++)
			{
				$x = ${$argument[$i]};
				if (empty($x))
				{
					$this->check = false;
					break;
				}
			}
		}
		function mysql_action($table)
		{
			$processing = new GET_(processing);
			$date = date("Y-m-d H:i:s");
		 
			for ($i = 0; $i < count($this->arguments); $i++)
			{
				foreach ($this->arguments as $value)
				{      
					${$value} = $this->list_array[$i++];
				}
						 
			}
		 
			$x ='';
			foreach ($this->arguments as $value)
			{
				$y =" '${$value}',";
				$x .= $y;
			}
			$this->values = substr("$x",0,-1);
		 
			if ($processing->variable == "add")
			{              
				if ($this->check == true)
				{
					$result = mysql_query("INSERT INTO {$table} ({$this->rows}) VALUES ({$this->values})");
					if ($result == true)
					{
						echo "";
					}
					else
					{
						echo "";
					}
				}
				else
				{
					echo "";
				}
			}
		}
}

Очередное говно с просторов интернета

BioMan, 22 Августа 2012

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

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

+116

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

delegate β ƒ<α, β>(α x);
delegate α γ<α>(γ<α> f);
 
static ƒ<α, β> Y<α, β>(ƒ<ƒ<α, β>, ƒ<α, β>> f) {
  return new γ<ƒ<α, β>>(h => F => f(h(h))(F))(h => F => f(h(h))(F));
}

static ƒ<int, int> Fact(ƒ<int, int> fact) {
  return n => (n == 0) ? 1 : n * fact(n - 1);
}
...
var fact = Y<int, int>(Fact);
Console.WriteLine("fact(6) = {0}", fact(6));

HaskellGovno, 22 Августа 2012

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

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

+141

1
2
3
4
5

<select>
	<option class="a2" value="0" selected="selected">любой возраст</option>
	<option class="a2" value="1">0 мес - 6 мес</option>
	<option class="a2" value="2">0 мес - 12 мес</option>
</select>

selected="selected" для элемента, выбираемого по дефолту - зачем?
ps. Код - кусок статичного html.

domaster, 22 Августа 2012

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

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

+106

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

function ReplaceUrl(txt: WideString): WideString;
var
  i, j: integer;
  tmp, Url: WideString;
begin
  Result := '';
  I := 1;
  while I <= Length(txt) do
  begin
    tmp := '';
    if WideSameText(tmp + txt[i]+txt[i+1]+txt[i+2]+txt[i+3]+txt[i+4]+txt[i+5]+txt[i+6], 'http://') or
       WideSameText(tmp + txt[i]+txt[i+1]+txt[i+2]+txt[i+3]+txt[i+4]+txt[i+5]+txt[i+6], 'ed2k://') or
       WideSameText(tmp + txt[i]+txt[i+1]+txt[i+2]+txt[i+3]+txt[i+4]+txt[i+5],          'ftp://') or
       WideSameText(tmp + txt[i]+txt[i+1]+txt[i+2]+txt[i+3],                            'www.') or
       WideSameText(tmp + txt[i]+txt[i+1]+txt[i+2]+txt[i+3]+txt[i+4]+txt[i+5]+txt[i+6]+txt[i+7], 'https://') then
    begin
      Url := '';
      for j := I to Length(txt) do
      begin
        if (txt[j] <> ' ') and (ord(txt[j]) < 255) then
          Url := Url + txt[J]
        else
          Break;
      end;
 
      Result := Result + Format(C_HTML_URL, [Url, Url]);
 
      I := J;
    end else
    begin
      Result := Result + txt[I];
 
      Inc(I);
    end;
  end;
 
end;

http://www.bvbcode.com/code/vhk2e8rb-1638045

63F45EF45RB65R6VR, 21 Августа 2012

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

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