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

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

+156

1
2
3
4
5

<?php

if ($user_id == 0 || $user_id == '0') {
    /*НИЧЕГО*/
}

1С Битрикс, этим всё сказано...

P.S давненько ничего не постил...

qbbr, 09 Октября 2013

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

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

+135

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

private void Button2_Click(object sender, EventArgs e)
    {
      this.Label4.Text = new StreamReader("C:\\Program Files\\Microsoft Visual Studio 10.0\\VB\\VBProjects\\датаметр\\датаметр\\bin\\Release\\Key.txt").ReadLine();
      if (Operators.CompareString(this.Label4.Text, "vrc5rhhgyuuoxr45", false) == 0)
      {
        this.Button1.Visible = true;
        this.TextBox1.Visible = true;
        this.Label1.Visible = true;
        this.Label4.Visible = false;
        this.Button2.Visible = false;
        this.Label3.Visible = true;
        int num = (int) Interaction.MsgBox((object) "Ключ верный.\r\nНе сообщайте его никому !!\r\nИ не потеряйте.", MsgBoxStyle.OkOnly, (object) null);
      }
      else
        this.Label4.Text = "Ключ не верный !!\r\nПоменяйте его в файле \"Key\"";
    }

Классное расположение файла. Причём в архиве лежит пустой файл Key.txt рядом с программой...
Как надо было извратиться, чтобы в коде оказался не просто абсолютный путь, но и вот такой-вот с Program Files, ведь по умолчанию VS создаёт проекты в Моих документах

http://www.cyberforum.ru/vb-net/thread971437.html

Qwertiy, 08 Октября 2013

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

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

+105

1

<div class="стиль38" style="position:relative; background:#333333; height:10px;"></div>

Работа фрилансЁра. И потом удивляются, что я их не люблю.

Vasiliy, 08 Октября 2013

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

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

−4

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

static Singleton * g_pInstance = NULL;

Singleton* Singleton::sharedInstance()
{
    if (g_pInstance)
        return g_pInstance;
    else {
        g_pInstance = new Singleton();
        if (g_pInstance->init()) {
            return g_pInstance;
        } else {
            delete g_pInstance;
            g_pInstance = NULL;
        }
        return g_pInstance;
    }
}

Ещё из жизни синглтонов. Даёшь больше return'ов!

Deacon, 08 Октября 2013

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

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

+16

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

// in .h file
class Singleton
{
public:
    Singleton();
    ~Singleton();
private:
    static Singleton* m_Instance;
    friend Singleton& GetInstance();
};

inline Singleton& GetInstance()
{
    assert(Singleton::m_Instance);
    return *Singleton::m_Instance;
}

// in .cpp file
Singleton* Singleton::m_Instance = NULL;

Singleton::Singleton()
{
    assert(!m_Instance);
    m_Instance = this;
}

Singleton::~Singleton()
{
    m_Instance = NULL;
}

Вот такую реализацию синглтона увидел в одном проекте.
ЗЫ: Для его корректной работы, в main было написано конечно же:
main() {
Singleton* s = new Singleton;
...
delete s;
}

Deacon, 08 Октября 2013

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

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

+8

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

void SaveEncryptedFile( const char *text, int len, const char* filename )
{
	char* pEncryptedText = new char[strlen(text)+1];
	string x1 = "you'll";
	string x2 = "never";
	string x3 = "get a";
	string x4 = "password";
	char l_pBuf[255];
	sprintf(l_pBuf,"%d",30*11/3);
	string result = x1+x2+x2+x1+l_pBuf+x3;
	encryptString(text,pEncryptedText,result.c_str(),strlen(text),result.length()); // там внутри xor

	FILE* pFile = fopen(filename, "wb");
	if (pFile)
	{
		fwrite(pEncryptedText,sizeof(char),len,pFile);
		fclose(pFile);
	}

	delete[] pEncryptedText;
}

Нашёл в рабочем проекте. Для "расшифровки" файла используется ещё одна такая же функция.

Deacon, 07 Октября 2013

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

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

+8

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

const Registry & Registry::getInstance()
{
	Registry *instance = RegistrySingleton::instance();
	if (!instance->mRootNode) {
		instance->load();
	}
	return *instance;
}

void Registry::load()
{
	try {
		// ...
		if (!mReader) {
			mReader = XMLReaderFactory::createXMLReader();
		}
		// ...
		mReader->parse( ... );
	} catch (...) {
		// ...
		throw; // удачи всем пользователям обрабатывать исключения xerces...
	}
}

боян синглтонно-абстрактный для чтения xml конфигурации с помощью xerces.

и не только ошибки не обрабатаешь (потому что getInstance() их бросает, угадай какой именно вызов из сотен загружает конфигурацию), но и в добавок народ не впечатал как многопоточность сделать правильно (RegistrySingleton это специализация шаблона который синхронизирует инициализацию mInstance переменной, и только).

Dummy00001, 07 Октября 2013

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

Python / Говнокод #13904

−100

1

2/3

ПИТОНОПРОБЛЕМЫ ;)

P.S. Да, я читал доки. Не меня в них тыкать носом.

bormand, 07 Октября 2013

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

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

+138

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

if ("A" == Key.ToUpper().Substring(startIndex, 1))
        num = 11L;
      else if ("B" == Key.ToUpper().Substring(startIndex, 1))
        num = 12L;
      else if ("C" == Key.ToUpper().Substring(startIndex, 1))
        num = 13L;
      else if ("D" == Key.ToUpper().Substring(startIndex, 1))
        num = 14L;
      else if ("E" == Key.ToUpper().Substring(startIndex, 1))
        num = 15L;
      else if ("F" == Key.ToUpper().Substring(startIndex, 1))
        num = 16L;
      else if ("0" == Key.ToUpper().Substring(startIndex, 1))
        num = 0L;
      else if ("1" == Key.ToUpper().Substring(startIndex, 1))
        num = 1L;
      else if ("2" == Key.ToUpper().Substring(startIndex, 1))
        num = 2L;
      else if ("3" == Key.ToUpper().Substring(startIndex, 1))
        num = 3L;
      else if ("4" == Key.ToUpper().Substring(startIndex, 1))
        num = 4L;
      else if ("5" == Key.ToUpper().Substring(startIndex, 1))
        num = 5L;
      else if ("6" == Key.ToUpper().Substring(startIndex, 1))
        num = 6L;
      else if ("7" == Key.ToUpper().Substring(startIndex, 1))
        num = 7L;
      else if ("8" == Key.ToUpper().Substring(startIndex, 1))
        num = 8L;
      else if ("9" == Key.ToUpper().Substring(startIndex, 1))
      {
        num = 9L;
      }

Программист, писавший ЭТО считал себя очень большим талантом и был даже тех.диром, пока не уволили....))

CraxyFright, 07 Октября 2013

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

Objective C / Говнокод #13902

−105

1
2
3
4

- (void)reloadTableViewOnSecondaryThread
{
	[self performSelectorOnMainThread:@selector(reloadTableViewOnMainThread) withObject:nil waitUntilDone:NO];
}

Разглядывая код от заказчика нашёл вот такой перл!

ProFFeSSoR, 06 Октября 2013

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

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