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

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

+162.3

1
2
3
4
5
6

$comments = (isset($_POST['lici-'.$login->id.'-comments']))?"yes":"no";

if($comments === "yes")
{
$xml .= "<nocomment>1</nocomment>\r\n";
}

Код из плагина кросспостера для wordpress.
Автор, видимо, не знает про булев тип. Все бы было хорошо, если бы плагин не закрывал комментарии, когда их нужно было открывать.

gunya, 31 Декабря 2009

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

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

+131.4

01
02
03
04
05
06
07
08
09
10
11

internal static System.Drawing.Font ToWindowsFont(Microsoft.Office.Interop.PowerPoint.Font ppFont)
{
	FontStyle style = ToGdiFontStyle(ppFont.Style);
	System.Drawing.Font gdiFont = new System.Drawing.Font(ppFont.Name, ppFont.Size, style);
	// из-за следующей строчки проект не скомпилируется под .NET 2.0, требуя ещё и один из более новых SP
	if ( windowsFont.Name != windowsFont.OriginalFontName )
	{
		windowsFont = new System.Drawing.Font(TextConverter.DefaultUnicodeSubstituteFont, ppFont.Size, style);
	}
	return windowsFont;
}

Баг был случайно найден на виртуалке с чистой Windows XP и MS Visual Studio 2005, поскольку мы по собственной глупости упустили тот факт, что на хост-машинах давно стоит .NET 3.5. Строчка "if ( windowsFont.Name != ppFont.Name )" решает проблему совместимости. Вот как иногда из-за маленькой, не бросающейся в глаза, ошибки можно завалить весь проект. Каюсь, этот код - мой. ))

cyba, 31 Декабря 2009

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

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

+184.4

1

$happy = new Year();

C наступающим ;)

miami, 31 Декабря 2009

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

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

+115.7

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

(defun create (a) ;; список из составляющих атомов списка A
	(cond
		((null a) nil)
		((atom a) (list a))
		(T (append (create (car a)) (create (cdr a))))
	)
)

(defun uniq (a) ;; оставить только уникальные атомы
	(cond 
		((null a) nil)
		((member (car a) (cdr a)) (uniq (cdr a)))
		(t (cons (car a) (uniq (cdr a))))
	)
)

(defun compare (a b) ;; сравнение состава списков
	(if (= (length a) (length b)) 
		(diff a b) 
		nil
	)
)

(defun diff (a b) ;; проверка состава 2x списков
	(cond
		((null a) T)
		(T (if (member (car a) b) 
			(diff (cdr a) b) 
			nil )
		)
	)
)

(defun main (a b)
	(compare
		(uniq (create a))
		(uniq (create b))
	)
)

а нужно было всего то проверить состоят ли 2 списка из одних и тех же атомов

xXx_totalwar, 31 Декабря 2009

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

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

+67.4

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

// die elemente aufbauen
// wasserstoff (gruppe 1, Nichtmetalle)
Element* wasserstoff = new Element(this);
wasserstoff->setObjectName("wasserstoff");
wasserstoff->setze_element_namen(tr("Hydrogen"));
wasserstoff->setze_atomzahl(1);
wasserstoff->setze_symbol("H");
wasserstoff->setze_masse("1,00794 u");
wasserstoff->setze_exakte_masse("1.007825032");
wasserstoff->setze_ionisation("13,5984 kJ/mol");
wasserstoff->setze_elektronenaffinitaet("1.007825032");
wasserstoff->setze_elektronen_negativitaet("2,2");
wasserstoff->setze_kovalenter("37 pm");
wasserstoff->setze_van_der_waals_radius("120 pm");
wasserstoff->setze_schmelzpunkt("13.81");
wasserstoff->setze_siedepunkt("20.28");
wasserstoff->setze_familie(tr("nonmetal"));
wasserstoff->setze_gruppenfarbe(gruppenfarben.at(1));
wasserstoff->setze_entdeckungsjahr(1766);
wasserstoff->setze_position(0, 1);
elementliste.insert(wasserstoff->atomzahl(), wasserstoff);
wasserstoff->registriere_agrgatzustandsfarben(agregatzustandsfarben);
wasserstoff->fuege_elektronen_hinzu(1, 1);

// helium (gruppe 18, Edelgase)
Element* helium = new Element(this);
helium->setObjectName("helium");
helium->setze_element_namen(tr("Helium"));
helium->setze_atomzahl(2);
helium->setze_symbol("He");
helium->setze_masse("4,0026 u");
helium->setze_exakte_masse("4.002603254");
helium->setze_ionisation("24,5874 kJ/mol");
helium->setze_elektronenaffinitaet("4.002603254");
helium->setze_elektronen_negativitaet("-1");
helium->setze_kovalenter("32 pm");
helium->setze_van_der_waals_radius("140 pm");
helium->setze_schmelzpunkt("0.95");
helium->setze_siedepunkt("4.216");
helium->setze_familie(tr("noble gases"));
helium->setze_gruppenfarbe(gruppenfarben.at(18));
helium->setze_entdeckungsjahr(1868);
helium->setze_position(17, 1);
elementliste.insert(helium->atomzahl(), helium);
helium->fuege_elektronen_hinzu(1, 2);

// lithium (gruppe 1, Alkalimetalle)
Element* lithium = new Element(this);


...


// ununhexium (gruppe 16, Metalle)
Element* ununhexium = new Element(this);
ununhexium->setObjectName("ununhexium");
ununhexium->setze_element_namen(tr("Ununhexium"));
ununhexium->setze_atomzahl(116);
ununhexium->setze_symbol("Uuh");
ununhexium->setze_masse("292 u");
ununhexium->setze_exakte_masse("292.19979");
ununhexium->setze_ionisation("-1");
ununhexium->setze_elektronenaffinitaet("292.19979");
ununhexium->setze_elektronen_negativitaet("-1");
ununhexium->setze_kovalenter("-1");
ununhexium->setze_van_der_waals_radius("-1");
ununhexium->setze_schmelzpunkt(QString::number(INT_MAX));
ununhexium->setze_siedepunkt(QString::number(INT_MAX));
ununhexium->setze_familie(tr("other metal"));
ununhexium->setze_gruppenfarbe(gruppenfarben.at(16));
ununhexium->setze_entdeckungsjahr(2000);
ununhexium->setze_position(15, 7);
elementliste.insert(ununhexium->atomzahl(), ununhexium);
ununhexium->fuege_elektronen_hinzu(1, 2);
ununhexium->fuege_elektronen_hinzu(2, 8);
ununhexium->fuege_elektronen_hinzu(3, 18);
ununhexium->fuege_elektronen_hinzu(4, 32);
ununhexium->fuege_elektronen_hinzu(5, 32);
ununhexium->fuege_elektronen_hinzu(6, 18);
ununhexium->fuege_elektronen_hinzu(7, 6);

ЗЫ: итого порядка 3000 строк кода

http://sourceforge.net/projects/qperiodictable

3dEyes**, 31 Декабря 2009

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

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

+167.6

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

--------------хтмл заголовок, ява скрипты -------------------
function CheckFields(){

  	if(document.feedback.password.value!='0A23BD671'){
		alert('Пароль неправильный!');
		document.feedback.password.focus();
		document.feedback.password.select();
		return false;
	} 	
 }	
------------------хтмл-------------------------------------------------

		<form name="feedback" action="/handlers/get_prz.php" method=POST onSubmit="return CheckFields()">
			<tr bgcolor="#dfefef" >
				<td align="right"><b>Пароль: </b></td>
				<td valign="top"><INPUT TYPE="PASSWORD" NAME="password" SIZE="9" value=""></b>
				<INPUT TYPE="HIDDEN" NAME="ftpzip" SIZE="9" value="finans.zip"></b></td>
			</tr>
			<tr bgcolor="#dfefef" >
				<td colspan=2 align="center">
				<INPUT TYPE="submit" VALUE="Скачать 2.8Mb" style="color:#cc0000;font-weight:bold;background=#dfefef">
				</td></tr></form>

иф пассворд не равен пассворд..... а с какого сайта я скопикомуниздил этот код, типа ERP система Компас... серьезная софтина для крупных предпрятий... а на сайте такое твориться... как теперь можно доверить такой софтине, сайт для которой студенты писали... ды, нет, думаю студенты не такие дубы чтоб такое писать, школьники наверное....
оригинал кода например тут - http://www.compas.ru/solutions/prz_fin.php
там почти все файлы типа через пароль качать...

LuCiFer, 31 Декабря 2009

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

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

+63.4

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

/* This construction seems to be more optimiser friendly.
       (without it gcc does the isDIGIT test and the *s - '0' separately)
       With it gcc on arm is managing 6 instructions (6 cycles) per digit.
       In theory the optimiser could deduce how far to unroll the loop
       before checking for overflow.  */
    if (++s < send) {
      int digit = *s - '0';
      if (digit >= 0 && digit <= 9) {
        value = value * 10 + digit;
        if (++s < send) {
          digit = *s - '0';
          if (digit >= 0 && digit <= 9) {
            value = value * 10 + digit;
            if (++s < send) {
              digit = *s - '0';
              if (digit >= 0 && digit <= 9) {
                value = value * 10 + digit;
		        if (++s < send) {
                  digit = *s - '0';
                  if (digit >= 0 && digit <= 9) {
                    value = value * 10 + digit;
                    if (++s < send) {
                      digit = *s - '0';
                      if (digit >= 0 && digit <= 9) {
                        value = value * 10 + digit;
                        if (++s < send) {
                          digit = *s - '0';
                          if (digit >= 0 && digit <= 9) {
                            value = value * 10 + digit;
                            if (++s < send) {
                              digit = *s - '0';
                              if (digit >= 0 && digit <= 9) {
                                value = value * 10 + digit;
                                if (++s < send) {
                                  digit = *s - '0';
                                  if (digit >= 0 && digit <= 9) {
                                    value = value * 10 + digit;
                                    if (++s < send) {
                                      /* Now got 9 digits, so need to check
                                         each time for overflow.  */
                                      digit = *s - '0';
                                      while (digit >= 0 && digit <= 9
                                             && (value < max_div_10
                                                 || (value == max_div_10
                                                     && digit <= max_mod_10))) {
                                        value = value * 10 + digit;
                                        if (++s < send)
                                          digit = *s - '0';
                                        else
                                          break;
                                      }
                                      if (digit >= 0 && digit <= 9
                                          && (s < send)) {
                                        /* value overflowed.
                                           skip the remaining digits, don't
                                           worry about setting *valuep.  */
                                        do {
                                          s++;
                                        } while (s < send && isDIGIT(*s));
                                        numtype |=
                                          IS_NUMBER_GREATER_THAN_UV_MAX;
                                        goto skip_value;
                                      }
                                    }
                                  }
				 }
                              }
                            }
                          }
                        }
                      }
                    }
                  }
                }
              }
            }
          }
	 }
      }
    }

Проверка числа в Perl-модуле. Судя по всему стояла задача оптимизировать под что-то.

Thomas_55, 30 Декабря 2009

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

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

+102

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

begin
  tru:=false;
  if    tru=false then
  begin
    ObrMatritsa(Sigm,Sigm_transp);
    tru:=true;
    det:=abs(Determinant(Sigm));
    if det=0 then
    begin
      ShowMessage('Determinant = 0');
      tru:=false;
      Application.Terminate;
      exit;
    end;
    //if  det=0 then begin result:=0 ; exit end;
    sum:=1/sqrt(power(2*pi,kol_prizn)*det* exp(Mas));
    if  sum=0 then result:=0 ;
  end else exit;
end;

TAX, 30 Декабря 2009

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

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

+92

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

{$R-}
fal:=true;
ty:= length(mas1);
while(fal=true)  do
  for j:=0 to kp-1 do
  begin
    for i:=i+k+j to length(mas2)-1 do
    begin
      while mas2[i+k]= ty do
      begin
        s:=s+c[i,j];
        inc(k);
      end;
    end;
    nl:=mas1[ty-1];
    if nl=0 then
      mas_Mj[ty]:=0
    else
      mas_Mj[ty-1][j]:=s/nl;
    s:=0;
  end;

TAX, 30 Декабря 2009

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

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

+144.9

1
2
3

foreach ($quote as $quote_key => $quote_value) {
$tpl[$key] = $value;
}

$quote = $tpl; как бы слишком коротко..

Deromanok, 29 Декабря 2009

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

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

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

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

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

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

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

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

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

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

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

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