- 1
var date = (DateTime.UtcNow.Date > DateTime.UtcNow ? DateTime.UtcNow.AddSeconds(1) : (DateTime.UtcNow.Date < DateTime.UtcNow ? DateTime.UtcNow.Date.AddSeconds(1) : DateTime.UtcNow));-
C# / Говнокод #14585
+139
Обнаружил сегодня в процессе код ревью (получение даты в каком-то тесте, который писал джуниор). Пребываю в состоянии когнитивного диссонанса...
aleksandr_kesha,
14 Февраля 2014
Комментарии (23) -
Java / Говнокод #14583
+73
- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
- 15
- 16
final double[][] matrix = new double[companies.size() + 1][(sampleSizeTo - sampleSizeFrom) / sampleSizeStep + 2]; for (int i = 0; i < matrix.length; i++) { for (int j = 0; j < matrix[0].length; j++) { if (i == 0 && j == 0) { continue; } if (i == 0) { matrix[i][j] = sampleSizeFrom + (j - 1) * sampleSizeStep; continue; } if (j == 0) { matrix[i][j] = companies.get(i-1); continue; } } }Название функции getSmartDistibution (именно так) какбэ намекает, что дальше будет весело
roman-kashitsyn,
14 Февраля 2014
Комментарии (5) -
C# / Говнокод #14582
+136
- 1
- 2
- 3
- 4
- 5
public static string CreateBrowserCacheExtension(object key) { //...Остальной код опущен для ясности return "Cache=" + Math.Abs(key.GetHashCode()); }Legacy code из проекта над которым я работаю.
Косяк в том, что GetHashCode() иногда возвращает значение, равное System.Int32.MinValue.
А это в свою очередь приводит к OverflowException, в случае с Math.Abs(...);
pikowatt,
14 Февраля 2014
Комментарии (27) -
C++ / Говнокод #14581
+58
- 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
template <typename T> class MySharedPtr{ public: explicit MySharedPtr(T* obj) : _obj(obj){} // --> я дописал MySharedPtr(const MySharedPtr& other) : _obj(other._obj){ inc_ref_count(_obj);} // <-- я дописал MySharedPtr& operator=(const MySharedPtr& other) { // --> я дописал if (this == &other) return *this; // <-- я дописал _obj = other._obj; inc_ref_count(_obj); } ~MySharedPtr(){ dec_ref_count(_obj); } private: static void inc_ref_count(T* obj){ std::lock_guard<std::mutex> lock(_mutex); _ref_count[obj] ++ ; } static void dec_ref_count(T* obj){ std::lock_guard<std::mutex> lock(_mutex); if (--_ref_count[obj]){ delete obj; _ref_count.erase(_ref_count.find(obj)); } } T* _obj; static std::mutex MySharedPtr<T>::_mutex; static std::map<T*,int> MySharedPtr<T>::_ref_count; }; template <typename T> std::map<T*,int> MySharedPtr<T>::_ref_count; template <typename T> std::mutex MySharedPtr<T>::_mutex;сегодня приходил чел-выпускник, написал на листочке shared_ptr, какое ваше мнение?
LispGovno,
14 Февраля 2014
Комментарии (63) -
PHP / Говнокод #14577
+146
- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
$synonyms = array( 1 => 'once', 2 => 'twice', 3 => 'three times', 4 => 'four times', 5 => 'five times', 6 => 'six times', 7 => 'seven times', 8 => 'eight times', 9 => 'nine times', 10 => 'ten times', );Dima, ты лучший!;)
valllllera,
14 Февраля 2014
Комментарии (31) -
PHP / Говнокод #14576
+155
- 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
function compileStr($arrStrings) { if (is_array($arrStrings)) { $part1 = 'a:2:{s:5:"VALUE";a:'; $part2 = 's:11:"DESCRIPTION";a:'; $part3 = '}'; $ornums = array(); foreach ($arrStrings as $str) { $exp = explode(",", str_replace(" ", "", $str)); $ornums = array_merge($ornums, $exp); } $ecount = count($ornums); $s1 = $ecount.':{'; $s2 = $ecount.':{'; for ($i = 0; $i < $ecount; $i++) { $s1 .= 'i:'.$i.';s:'.strlen($ornums[$i]).':"'.$ornums[$i].'";'; $s2 .= 'i:'.$i.';s:0:"";'; } $s1 .= '}'; $s2 .= '}'; $result = $part1.$s1.$part2.$s2.$part3; return $result; } }Нашел в дебрях проекта. Писать php-сериализатор на php - это концептуально )))
если кто не понял функция делает вот это:
if (is_array($arrStrings)) {
$ornums = explode(',', implode(',', str_replace(' ', '', $arrStrings)));
return serialize(array("VALUE" => $ornums, "DESCRIPTION" => array_fill(0, count($ornums), '')));
}
Homess,
14 Февраля 2014
Комментарии (1) -
C# / Говнокод #14575
+134
- 1
- 2
- 3
- 4
- 5
- 6
Random random = new Random(Environment.TickCount); for (int i = 0; i < random.Next(380, 480); i++) { Marshal.AllocHGlobal(random.Next(0x4400, 0xd400)); Thread.Sleep(1); }Из исходников подрядчика...
Заказчиков надо садить на иглу правильно
Stewie,
14 Февраля 2014
Комментарии (5) -
C# / Говнокод #14574
+135
- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
if (currentItem.Type.ActiveProperty != null) { Type controltype = currentControl.GetType(); PropertyInfo pinfo = controltype.GetProperty(currentItem.Type.ActiveProperty.Name); bool enabledtrue = (currentItem.Type.ActiveProperty.Name.ToLower() != "readonly"); if ((r1 != 2) || _page.ReadOnly) enabledtrue = !enabledtrue; pinfo.SetValue(currentControl, enabledtrue, null); }enabledtrue как пример интуитивно-понятного названия переменной
Gocoder,
14 Февраля 2014
Комментарии (5) -
Pascal / Говнокод #14572
+96
- 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
- 93
- 94
- 95
- 96
- 97
- 98
- 99
if movcamn[1] then begin // if movcams[ 2] then begin if Nmr= 2 then goto m1; if movcams[ 3] then begin if Nmr= 3 then goto m1; if movcams[ 4] then begin if Nmr= 4 then goto m1; if movcams[ 5] then begin if Nmr= 5 then goto m1; if movcams[ 6] then begin if Nmr= 6 then goto m1; if movcams[ 7] then begin if Nmr= 7 then goto m1; if movcams[ 8] then begin if Nmr= 8 then goto m1; if movcams[ 9] then begin if Nmr= 9 then goto m1; if movcams[10] then begin if Nmr=10 then goto m1; if movcams[11] then begin if Nmr=11 then goto m1; if movcams[12] then begin if Nmr=12 then goto m1; if movcams[13] then begin if Nmr=13 then goto m1; if movcams[14] then begin if Nmr=14 then goto m1; if movcams[15] then begin if Nmr=15 then goto m1; if movcams[16] then begin if Nmr=16 then goto m1; if movcams[17] then begin if Nmr=17 then goto m1; if movcams[18] then begin if Nmr=18 then goto m1; if movcams[19] then begin if Nmr=19 then goto m1; if movcams[20] then begin if Nmr=20 then goto m1; m1: Summs1:=0; movcams[1]:=False; movcamn[1]:=False; movcam[1,1]:=False; movcam[1,2]:=False; movcam[1,3]:=False; movcam[1,4]:=False; Summs2:=0; movcams[2]:=False; movcamn[2]:=False; movcam[2,1]:=False; movcam[2,2]:=False; movcam[2,3]:=False; movcam[2,4]:=False; if Nmr=2 then goto m2; // <--- Summs3:=0; movcams[3]:=False; movcamn[3]:=False; movcam[3,1]:=False; movcam[3,2]:=False; movcam[3,3]:=False; movcam[3,4]:=False; if Nmr=3 then goto m2; // <--- Summs4:=0; movcams[4]:=False; movcamn[4]:=False; movcam[4,1]:=False; movcam[4,2]:=False; movcam[4,3]:=False; movcam[4,4]:=False; if Nmr=4 then goto m2; // <--- Summs5:=0; movcams[5]:=False; movcamn[5]:=False; movcam[5,1]:=False; movcam[5,2]:=False; movcam[5,3]:=False; movcam[5,4]:=False; if Nmr=5 then goto m2; // <--- Summs6:=0; movcams[6]:=False; movcamn[6]:=False; movcam[6,1]:=False; movcam[6,2]:=False; movcam[6,3]:=False; movcam[6,4]:=False; if Nmr=6 then goto m2; // <--- Summs7:=0; movcams[7]:=False; movcamn[7]:=False; movcam[7,1]:=False; movcam[7,2]:=False; movcam[7,3]:=False; movcam[7,4]:=False; if Nmr=7 then goto m2; // <--- Summs8:=0; movcams[8]:=False; movcamn[8]:=False; movcam[8,1]:=False; movcam[8,2]:=False; movcam[8,3]:=False; movcam[8,4]:=False; if Nmr=8 then goto m2; // <--- Summs9:=0; movcams[9]:=False; movcamn[9]:=False; movcam[9,1]:=False; movcam[9,2]:=False; movcam[9,3]:=False; movcam[9,4]:=False; if Nmr=9 then goto m2; // <--- // Summs10:=0; movcams[10]:=False; movcamn[10]:=False; movcam[10,1]:=False; movcam[10,2]:=False; movcam[10,3]:=False; movcam[10,4]:=False; if Nmr=10 then goto m2; // <--- Summs11:=0; movcams[11]:=False; movcamn[11]:=False; movcam[11,1]:=False; movcam[11,2]:=False; movcam[11,3]:=False; movcam[10,4]:=False; if Nmr=11 then goto m2; // <--- Summs12:=0; movcams[12]:=False; movcamn[12]:=False; movcam[12,1]:=False; movcam[12,2]:=False; movcam[12,3]:=False; movcam[10,4]:=False; if Nmr=12 then goto m2; // <--- Summs13:=0; movcams[13]:=False; movcamn[13]:=False; movcam[13,1]:=False; movcam[13,2]:=False; movcam[13,3]:=False; movcam[10,4]:=False; if Nmr=13 then goto m2; // <--- Summs14:=0; movcams[14]:=False; movcamn[14]:=False; movcam[14,1]:=False; movcam[14,2]:=False; movcam[14,3]:=False; movcam[10,4]:=False; if Nmr=14 then goto m2; // <--- Summs15:=0; movcams[15]:=False; movcamn[15]:=False; movcam[15,1]:=False; movcam[15,2]:=False; movcam[15,3]:=False; movcam[10,4]:=False; if Nmr=15 then goto m2; // <--- Summs16:=0; movcams[16]:=False; movcamn[16]:=False; movcam[16,1]:=False; movcam[16,2]:=False; movcam[16,3]:=False; movcam[10,4]:=False; if Nmr=16 then goto m2; // <--- Summs17:=0; movcams[17]:=False; movcamn[17]:=False; movcam[17,1]:=False; movcam[17,2]:=False; movcam[17,3]:=False; movcam[10,4]:=False; if Nmr=17 then goto m2; // <--- Summs18:=0; movcams[18]:=False; movcamn[18]:=False; movcam[18,1]:=False; movcam[18,2]:=False; movcam[18,3]:=False; movcam[10,4]:=False; if Nmr=18 then goto m2; // <--- Summs19:=0; movcams[19]:=False; movcamn[19]:=False; movcam[19,1]:=False; movcam[19,2]:=False; movcam[19,3]:=False; movcam[10,4]:=False; m2: end; end; end; end; end; end; end; end; end; end; // 20 штук end; end; end; end; end; end; end; end; end; end;Попалось в одном проекте, который приходилось расчищать.
Особенно позабавило заботливое "// 20 штук" в конце.
qb60,
14 Февраля 2014
Комментарии (2) -
JavaScript / Говнокод #14571
+151
- 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
$('.catalogue-list .in-cart a, .items-slider .in-cart a').click(function() { var src_pic = $(this).parent('div').parent().children('.pic_ajax').children().children().attr('src'); var name_pic = $(this).parent('div').parent().children('.pic').children().children().attr('alt'); var colors = $(this).parent('div').children('.colors_values').children('input'); var colors_for_ajax = ""; $.each( colors, function() { colors_for_ajax = colors_for_ajax+$(this).val()+','; }); if(colors_for_ajax) var url=$(this).attr('href'); $.fancybox({ type : 'ajax', href : '/catalogue/ajax_add_to_cart_popup.php?NAME='+encodeURI(name_pic)+'&SRC='+encodeURI(src_pic)+'&COLORS='+encodeURI(colors_for_ajax)+'&URL_ADD='+encodeURI(url), fitToView : false, width : '336px', height : '234px', autoSize : false, autoDimensions: false, openEffect : 'fade', closeEffect : 'none', margin : 0, padding : 0 }); if(!colors_for_ajax) { $(this).parent('.in-cart').find('.ajax-load').show('fast'); var url=$(this).attr('href'); $('.top-line .block .cart').load(url,'',onAjaxCart); } return false; });Осталось от прошлого разраба в проекте. Аяксовое добавление товара в корзину...
Зачем получать имя,свойства,картинку из ID элемента?! Я лучше это передам параметрами.
kopoBko,
14 Февраля 2014
Комментарии (3)
Follow us!