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

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

+78

1
2
3
4

int strlenw(const wchar_t *s)
{
	return (int)((int)(strlen(s)) % sizeof(wchar_t));
};

Даже и не знаю что сказать. Это было в доставшемся мне проекте. Мне кажется это говнокод.

Говногость, 11 Января 2010

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

ActionScript / Говнокод #2394

−433.5

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

[1:02:57] Dmitriy Artemyev: if (s == rb.getString('pricesForm.oneAdult'))
      ((((p.getChildAt(0) as HBox).getChildAt(0) as Form).getChildAt(0) as FormItem).getChildAt(0) as NumericTextInput).value = newS;
     if (s == rb.getString('pricesForm.twoAdults'))
      ((((p.getChildAt(0) as HBox).getChildAt(0) as Form).getChildAt(1) as FormItem).getChildAt(0) as NumericTextInput).value = newS;
     if (s == rb.getString('pricesForm.extraAdultMarkup'))
      ((((p.getChildAt(0) as HBox).getChildAt(0) as Form).getChildAt(2) as FormItem).getChildAt(0) as NumericTextInput).value = newS;
     
     
     if (s == rb.getString('pricesForm.children') + model.hotelsModule.hotelTotalInfo.iChildAge1)
      ((((p.getChildAt(0) as HBox).getChildAt(1) as Form).getChildAt(0) as FormItem).getChildAt(0) as NumericTextInput).value = newS;
     if (s == rb.getString('pricesForm.children') + model.hotelsModule.hotelTotalInfo.iChildAge2)
      ((((p.getChildAt(0) as HBox).getChildAt(1) as Form).getChildAt(1) as FormItem).getChildAt(0) as NumericTextInput).value = newS;
     if (s == rb.getString('pricesForm.children') + model.hotelsModule.hotelTotalInfo.iChildAge3)
      ((((p.getChildAt(0) as HBox).getChildAt(1) as Form).getChildAt(2) as FormItem).getChildAt(0) as NumericTextInput).value = newS;

Делаю код ревью и рефакторинг в одном старом коде, вот на что нарвался и выпал в даун )

dimas_art, 11 Января 2010

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

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

+145.6

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

extend(Object, {
    extend:        extend,
    inspect:       inspect,
    toJSON:        toJSON,
    toQueryString: toQueryString,
    toHTML:        toHTML,
    keys:          keys,
    values:        values,
    clone:         clone,
    isElement:     isElement,
    isArray:       isArray,
    isHash:        isHash,
    isFunction:    isFunction,
    isString:      isString,
    isNumber:      isNumber,
    isUndefined:   isUndefined
  });

Из Prototype JS.

Cr@ZyBoY, 10 Января 2010

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

bash / Говнокод #2391

−372.7

1

ssh root@localhost

вместо su

jdponomarev, 10 Января 2010

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

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

+156.8

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
84
85
86
87
88
89
90
91
92

<?php
session_start();
$lgn=$_SESSION['lgn'];
$pswd=$_SESSION['pswd'];
//  print_r($_GET);
include_once('spyLib.php');
$dbConn=connectToSpy();
mysql_query("SET NAMES cp1251");
$option=$_REQUEST['option'];
  
$ID=autentID($lgn, $pswd);
//$ID='45';
if ($ID=='8') {

  if ($option=='editTable'){
  include_once("bd/editTable.php");
  }elseif($option=="addRecord"){
  include_once( 'bd/addRecord.php' );
  }elseif($option=="processAdd"){
  include_once( 'bd/processAdd.php' );
  }elseif($option=="deleteRecord"){
  include_once( 'bd/deleteRecord.php' );
  }elseif($option=="editRecord"){
  include_once( 'bd/editRecord.php' );
  }elseif($option=="updateRecord"){
  include_once( 'bd/updateRecord.php' );  
  }else{
  
  print <<<HERE
<link href="./style.css" rel="stylesheet" type="text/css">
  <!-----------главная страница------------------------------------->
<img src="./img/fonpageleft.png" style="position: absolute; top:50px; left: 50px;" width="670" height="370">
<div style="position: absolute; top:70px; left: 100px;">

<form action="index.php" method="post">
  <input name="ok" type="image" src="img/admin.gif" class="bulleon" alt="Перейти на сайт">
</form>
<table border="0" cellpadding="0" cellspacing="0">
  <tr>
  <td valign="top" width="458">
  <div class="text_10">Страница находится в разработке.</div>
  <form action="admin.php">
  <input name="option" type="hidden" value="editTable">
  <input name="page" type="hidden" value="admin">
  <input name="back" type="hidden" value="false">
  <table cellspacing="1" cellpadding="3" width='100%'>
  <tr  class="title">
  <td>
  </td>
  <td class='title2'>
  Выберите таблицу для редактирования
  </td>
  <td>
  </td>
  </tr>
  <tr class='table_top'>
  <td width='30' nowrap="nowrap">
  </td>
  <td  align="left">
  <select name="tableName" size=5>
  <option value="kompleks">комплексы</option>
  <option value="glav">главная страница</option>
  <option value="bulletin">новости</option>
  <option value="forleader">для руководителей</option>
  <option value="lunchoffice">обеды в офис</option>
  <option value="partner">партнеры</option>
  <option value="contact">контакты</option>
  <option value="review">отзывы</option>
  </select>
  </td>
  <td width='30' nowrap="nowrap">
  </td>
  </tr>
  <tr>
  <td colspan='3' align='left'>
  <input name="option2" type="image" value="1" src="./img/ok4.gif" class="bulleon">
  </td>
  </tr>
  </table>
  </form>
  </td>
    </tr>
  </table>
</div>
<!---!--------главная страница------------------------------------->
HERE;
  }//if

}else{
  print "<p class=\"mes_small_error\">Попробуйте ввести пороль и логин еще раз</p>";
}
?>

что тут комментировать - аффтар даже русский с трудом осилил: "Попробуйте ввести пороль"(sic!)

xXx_totalwar, 09 Января 2010

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

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

+151.7

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

/**
	 * bool _validate_mail(string mail)
	 * Validate an email address
	 * @access private
	 * @param string mail The email address string
	 * @return bool
	 */
	function _validate_mail($mail)
	{
		if (ereg('^[-!#$%&\'*+\\./0-9=?A-Z^_`a-z{|}~]+'.'@'.'[-!#$%&\'*+\\/0-9=?A-Z^_`a-z{|}~]+\.'.'[-!#$%&\'*+\\./0-9=?A-Z^_`a-z{|}~]+$',$mail)){
			return true;
		}
		return $this->_debug(4, $mail);
	}

no comments...

InTech, 09 Января 2010

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

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

+922.2

1
2
3
4
5
6

__declspec(noreturn) int FastAdd(const int a, const int b)
{
  __asm mov eax, a;
  __asm mov ebx, b;
  __asm add eax, ebx;
};

Встречаеться множество подобных функций в одном С++ проекте, который в данный момент разгребаю. Хочеться что-нибудь сказать автору. Например: "на XYZ"?

Говногость, 08 Января 2010

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

ActionScript / Говнокод #2386

−190.8

1
2
3
4

if (flash.net.getClassByAlias("foo.bar.VO") == null){
          flash.net.registerClassAlias("foo.bar.VO", foo.bar.VO);}
      } catch (e:Error) {
          flash.net.registerClassAlias("foo.bar.VO", foo.bar.VO); }

Так во флексовом фреймворке регистрируются алиасы для RPC классов... Нелогичность ситуации не сразу очевидна, поэтому опишу:
При загрузке модулей может возникнуть ситуация, когда алиас уже зарегистрирован для другого класса, поэтому нужно проверить, а не был ли алиас зарегистрирован раньше. Иначе, его нужно зарегистрировать. Исторически, этой проверки сначала не было, данный код - это фикс вышеописаной проблемы. Пытаясь разобраться, что же все-таки случилось - скорее всего "писатель" не подозревал, что getClassByAlias() не может вернуть null - а выяснил он это, когда код вывалился с ошибкой, вот он не долго думая завернул это все в try-catch...
Из мелких деталей: в рамках AS3 сравнение сложных типов с null - бессмысленная, и более того вредная операция, т.как выражение в условии всегда кастуется к Boolean, и компилятор не умеет делать оптимизации для таких случаев.

wvxvw, 08 Января 2010

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

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