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

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

+157

1
2
3
4
5
6
7

function printVar($var) {
    ob_start();
    print_r($var);
    $var = ob_get_contents();
    ob_end_clean();
    return $var;
 }

Вдруг версия php < 4.2.0? :)
Найдено в фреймворке DoPHP

PANACEA, 17 Марта 2011

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

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

+118

1
2
3
4

string s=textBox1.Text,d="";
if (s.lenght > 0)
for (int i = 0; i < s.lenght- 1; i++)
d += s[i];

Код, присланный товарищем мне на проверку. Смех и грех - этот человек учится на третьем курсе политеха на специальности ИТ.

SpeedyWizard, 17 Марта 2011

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

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

+149

01
02
03
04
05
06
07
08
09
10
11

$message = array(
		'Message from blablabla',
		'',
		'Name: '.$_POST['name'],
		'Title: '.$_POST['title'],
		'City, State: '.$_POST['city'],
		'Enrollment: '.$_POST['enrol'],
		'Email: '.$_POST['email'],
		'Phone: '.$_POST['phone'],
	);
	$body = implode('<br>',$message);

Чувак, вроде опытный программист, в 2006 году написал сие произведение искусства:

twtwtw, 16 Марта 2011

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

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

+75

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

private void configComponents(/* params */) throws MyException {
	String err_msg = null;
	try {
		// some code here...
		return;
	} catch (ComponentConfigurationException e) {
		err_msg = e.getMessage();
	} catch (MyException e) {
		err_msg = e.getMessage();
	} catch (Exception e) {
		err_msg = setupProcessErrorMessage;
	}
	throw new MyException(err_msg);
}

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

mdcool, 16 Марта 2011

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

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

+169

01
02
03
04
05
06
07
08
09
10

function ValidateIPAdress($value){
    if (preg_match('/^[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}$/', $value)){
        return true;
    }
    return false;
}
if( ValidateIPAdress("192.168.0.1") )
    echo("This is an IPAdress");
else
    echo("This is not an IPAdress");

Функция проверки IP адреса.

vdmodule, 16 Марта 2011

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

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

−183

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

def validate_line(line):
    if len(line) <= 2: return False
    if line[0] == '""': return False
    if line[0] == "''": return False
    if line[0] == '+!': return False
    if line[0] == '-?': return False
    if line[0] == '- (': return False
    if line[0] == '-(': return False
    if line[0] == '-)': return False

    if line[0] == '-6a': return False
    if line[0] == '-6a )': return False
    if line[0] == '-a': return False
    if line[0] == '-aa': return False

    if line[0] == '-:': return False
    if line[0] == ':': return False
    if line[0][0] == '#': return False
    if line[0][0] == "'": return False
    if line[0][0] == '&': return False
    if line[0][0] == '(': return False
    return True

Жил был один маленький портальчик одной немаленькой компании. Портал внутренний, пользовался секретамками, манагерами и прочим планктоном. С задачей своей справлялся, не глючил и вообще проблем не приносил. И вот задумали в него добавить пару "фишек". А писался он давно, программиста уже не найти. Полезли в код, а там ... В общем вот; ВЕСЬ код написан в так вот стиле...

ossa, 16 Марта 2011

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

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

+162

01
02
03
04
05
06
07
08
09
10
11

/**
	 * Test to see if the cache storage is available.
	 *
	 * @static
	 * @access public
	 * @return boolean  True on success, false otherwise.
	 */
	function test()
	{
		return true;
	}

Описание метода в коментах.
Joomfish

tranquillity, 16 Марта 2011

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

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

+154

1
2
3
4
5

__int64 timestamp;

	// Get nanoseconds since 1601
	timestamp = rand() + (((__int64)rand()) << 32);
	timestamp /= 100;

очень вольная работа со временем...

kubrinsky, 16 Марта 2011

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

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

+114

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

[Serializable()]
	public class Vendor
	{
		#region Constructors

		public Vendor(long vendorID)
		{
			LoadData(vendorID);
		}

		internal Vendor(Vendor argVendor)
		{
			if (argVendor != null)
			{
				this.ID = argVendor.ID;
				this.VendorName = argVendor.VendorName;
				this.Account = argVendor.VendorAccount;
				this.EIN = argVendor.VendorEIN;
				this.Address = argVendor.VendorAddress;
				this.City = argVendor.VendorCity;
				this.State = argVendor.VendorState;
				this.PostalCode = argVendor.VendorZip;
				this.Phone = argVendor.VendorPhone;
				this.ContactName = argVendor.VendorContact;
				this.VendorCode = argVendor.VendorCode;
				this.Country = argVendor.VendorCountry;
				this.FaxNumber = argVendor.VendorFax;
				this.Email = argVendor.Email;
			}
		}
		#endregion

		public void LoadData(long vendorID)
		{
			POMRepositoryDataClassesDataContext db = new POMRepositoryDataClassesDataContext();
			var ven = (from v in db.Vendors
					   where v.ID == vendorID
					   select v).SingleOrDefault();
			if (ven != null)
			{
                populateMe(ven);
			}
		}
        public void LoadDataByVendorCode(string argVendorCode)
        {
            POMRepositoryDataClassesDataContext db = new POMRepositoryDataClassesDataContext();
            var ven = (from v in db.Vendors
                       where v.VendorCode == argVendorCode
                       select v).Take(1).SingleOrDefault();
            if (ven != null)
            {
                populateMe(ven);
            }
        }
        private void populateMe(Vendor ven)
        {
            this.ID = ven.ID;
            this.VendorName = ven.VendorName;
            this.Account = ven.VendorAccount;
            this.EIN = ven.VendorEIN;
            this.Address = ven.VendorAddress;
            this.City = ven.VendorCity;
            this.State = ven.VendorState;
            this.PostalCode = ven.VendorZip;
            this.Phone = ven.VendorPhone;
            this.ContactName = ven.VendorContact;
            this.VendorCode = ven.VendorCode;
            this.Country = ven.VendorCountry;
	    this.FaxNumber = ven.VendorFax;
	    this.Email = ven.Email;
        }
}

No comments!!!

Arterius, 16 Марта 2011