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

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

+70

1
2

.
																		buffer.put(Transaction.getTransaction((JSONObject)transactionsData.get(j)).getBytes());

Вложенность, например. Но это нужно видеть целиком.
https://bitbucket.org/JeanLucPicard/nxt-public/src/

rat4, 15 Февраля 2014

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

SQL / Говнокод #14590

−157

1
2
3
4

SELECT ...
FROM ...
WHERE (CASE WHEN big_part = 1 AND blk_flag = 2 THEN -1 ELSE 0 END) <> -1 
;

slbsomeone, 15 Февраля 2014

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

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

+139

1

var date = (DateTime.UtcNow.Date > DateTime.UtcNow ? DateTime.UtcNow.AddSeconds(1) : (DateTime.UtcNow.Date < DateTime.UtcNow ? DateTime.UtcNow.Date.AddSeconds(1) : DateTime.UtcNow));

Обнаружил сегодня в процессе код ревью (получение даты в каком-то тесте, который писал джуниор). Пребываю в состоянии когнитивного диссонанса...

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)

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

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

SQL / Говнокод #14590

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

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

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

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

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

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

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

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