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

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

+149

1
2
3
4
5
6

if ($result->fetch()) {
    return $result->get('num_flags');
}
else {
    return 666;
}

Верующий программист :)

xakip, 17 Декабря 2013

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

Си / Говнокод #14236

+141

1
2
3
4
5

if (connfailed) {
			KSOCKET_CALLBACK(so, disconnected, error);
		} else {
			KSOCKET_CALLBACK(so, connectfailed, error);
		}

https://github.com/joyent/illumos-joyent/blob/master/usr/src/uts/common/fs/sockfs/socknotify.c

myaut, 17 Декабря 2013

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

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

+159

1
2
3
4
5
6
7
8
9

var lines = content.Split(new[] { Environment.NewLine }, StringSplitOptions.RemoveEmptyEntries).ToList();

// remove first and last tags
lines = lines.Skip(2).Take(lines.Count - 3).ToList();           //    <------------   ОНО
for (var i = 0; i < lines.Count; i++)
{
        // remove one indent from each line
        lines[i] = lines[i].Substring(indentation, lines[i].Length - indentation);
}

Покоробило от такого подхода...
Я бы написал for от 1 до lines.Count-1 :)

ddv_demon, 17 Декабря 2013

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

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

+10

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

#include <iostream>
#include <list>
#include <queue>
#include <memory>
#include <mutex>
#include <condition_variable>
#include <type_traits>
#include <assert.h>
using namespace std;

template<class Data>
class UnboundedQueueForNonThrowMovable
{
	static_assert(std::is_nothrow_move_constructible<Data>::value, "Data must be nonthrow movable type.");
	static_assert(!std::is_array<Data>::value, "Data must not be c-array.");
	
public:
	typedef Data value_type;
	
private:
	typedef std::queue<Data, std::list<Data>> Queue;
	typedef std::unique_lock<std::mutex> Lock;
	Queue _queue;
	std::mutex _lockQueue;
	std::condition_variable _pushToQueue;
	
	UnboundedQueueForNonThrowMovable(const UnboundedQueueForNonThrowMovable&) = delete;
	UnboundedQueueForNonThrowMovable(UnboundedQueueForNonThrowMovable&&) = delete;
	UnboundedQueueForNonThrowMovable& operator=(const UnboundedQueueForNonThrowMovable&) = delete;
	UnboundedQueueForNonThrowMovable& operator=(UnboundedQueueForNonThrowMovable&&) = delete;
public:
	UnboundedQueueForNonThrowMovable(void){}
	
	void push(Data&& data)
	{
		Lock lockerQueue(this->_lockQueue);
		this->_queue.push(std::move(data));
		this->_pushToQueue.notify_all();//_condition.notify_one(); most optimal, but can cause deadlock.
	}
	
	void push(Data& data)
	{
		this->push(std::move(data));
	}
	
	bool emptyUnstable(void) const
	{
		Lock lockerQueue(this->_lockQueue);
		return this->_queue.empty();
	}
	
	Data pop(void)
	{
		Lock lockerQueue(this->_lockQueue);
		this->_pushToQueue.wait(lockerQueue, [this](void){return !this->_queue.empty();});
		assert(!this->_queue.empty());
		Data result = std::move(this->_queue.front());
		this->_queue.pop();
		return result;
	}
	
	template< class Rep, class Period>
	Data pop(const std::chrono::duration<Rep, Period> WaitDuration)
	{
		Lock lockerQueue(this->_lockQueue);
		if(!this->_pushToQueue.wait(lockerQueue, WaitDuration, [this](void){return !this->_queue.empty();}))
			return Data();
		assert(!this->_queue.empty());
		Data result = std::move(this->_queue.front());
		this->_queue.pop();
		return result;
	}
};

template<class Data>	
using UnboundedQueueForUniquePtr = UnboundedQueueForNonThrowMovable<std::unique_ptr<Data>>;
template<class Data>
using UnboundedQueueForSharedPtr = UnboundedQueueForNonThrowMovable<std::shared_ptr<const Data>>;
template<class Data>
using UnboundedQueue = UnboundedQueueForUniquePtr<Data>;

int main() {
	cout<<"ok"<<endl;
	{
		//UnboundedQueueForSharedPtr<int>::value_type == std::shared_ptr<const int>
		UnboundedQueueForSharedPtr<int> queueSharedPtr;
		//auto a = std::make_shared<const int>(45);
		auto a = std::make_shared<int>(45);
		assert(a);
		queueSharedPtr.push(a);
		assert(!a);//Fired if you use "auto a = std::make_shared<int>(45)" below. It is compiler bug, I think, because previus code line must cause compile error.
		auto b = queueSharedPtr.pop();//std::shared_ptr<const int>
		assert(b);
		cout<<*b<<endl;
		assert(*b==45);

http://ideone.com/qdsWJi
Немного глупый вопрос, почему в 90 строчке не получаем ошибку компиляции если закомментировать 87-ую строку и разкомментировать 88-ую?

laMer007, 16 Декабря 2013

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

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

+161

1
2
3
4
5
6
7

//надо показать элемент каталога во всей красе
$_CENTER="show_category_item(".$newParts[0].");";
eval($_CENTER);

// ...
$_LEFT='get_main_category($cat_id, $new_path);';
$_CENTER="show_category_item_list($".'newParts'.");";

Вот с таким адом мне приходится работать.

oooZinka, 16 Декабря 2013

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

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

+74

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

for (int i = 0; i != nl.length(); i++)
{
    out.write(nl.charAt(i));
}

for (int i = 0; i != footerStart.length(); i++)
{
    out.write(footerStart.charAt(i));
}

for (int i = 0; i != type.length(); i++)
{
    out.write(type.charAt(i));
}

for (int i = 0; i != footerTail.length(); i++)
{
    out.write(footerTail.charAt(i));
}

for (int i = 0; i != nl.length(); i++)
{
    out.write(nl.charAt(i));
}

Зачем писать функцию, когда можно успешно копипастить циклы

http://grepcode.com/file/repo1.maven.org/maven2/org.bouncycastle/bcpg-jdk16/1.45/org/bouncycastle/bcpg/ArmoredOutputStream.java

roman-kashitsyn, 15 Декабря 2013

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

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

+7

1
2
3
4
5

auto r = [&](){
for(auto i: a)
  if(i==k)
    return f(i);
}();

Однажды мне знакомый рассказывал, что во многих языках плохие грязные циклы. Мол настоящие чистые циклы должны возвращать значение. Я написал ему вот это. Он многозначительно подумал и замолчал. Через две с половиной недели он уволился.

LispGovno, 14 Декабря 2013

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

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

+72

1
2
3
4

SecretKeyFactory factory = SecretKeyFactory.getInstance("PBEWITHMD5ANDTRIPLEDES");
PBEKeySpec spec = new PBEKeySpec(password, salt, 1024, 128);
SecretKey key = factory.generateSecret(spec);
hexdump(key.getEncoded());

http://ideone.com/bVElQG

Не, ну я все понимаю, PKCS #5 1.5 аля PBE, MD5 и DES не считаются безопасными алгоритмами... но не настолько же...

bormand, 14 Декабря 2013

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

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

+137

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

if (!string.IsNullOrEmpty(date))
			{
				var result = date.Split('.');
				filter.TenderDate.Start = new DateTime(
					Convert.ToInt16(result[2]),
					Convert.ToInt16(result[1]),
					Convert.ToInt16(result[0]),
					0,
					0,
					0);
				filter.TenderDate.End = new DateTime(
					Convert.ToInt16(result[2]),
					Convert.ToInt16(result[1]),
					Convert.ToInt16(result[0]),
					23,
					59,
					59);
			}

Парсинг дат? Не, не слышал.

xumix, 14 Декабря 2013

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

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