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

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

+188

1
2
3
4
5
6

typedef enum
  {
    FFALSE = 0,
    TTRUE,
    MMAYBE
  } Truth_t;

ну почти квантовое программирование.

ЗЫ да, это из С++ программы.

Dummy00001, 21 Июня 2011

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

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

+175

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

class BalanceValue
{
  /* ... */
  BalanceValue( int pFamilyGroupId,
                int ContractId,
                int pProfileId,
                int pSncode,
                long pPurchaseSeqNo,
                int pBundledProductId,
                time_t pCreationDate,
                time_t pNextresetDate,
                char pState,
                double pAggregate,
                double pCredit,
                double pInitialCredit,
                double pReservation,
                char pColor,
                double pProrateFactor,
                int pCurrencyId,
                int pUomId,
                time_t pSnapshotDate,
                unsigned long pSequenceNumber,
                time_t pValidFrom,
                time_t pValidTo,
                int pPricingAlternative,
                int pCocTariffId,
                time_t pLastresetDate,
                double pLateCallAggr );
  /* ... */
  void getAllValues( int&            pContractId,
                     int&            pProfileId,
                     int&            pSncode,
                     long&           pPurchaseSeqNo,
                     int&            pBundledProductId,
                     time_t&         pNextResetDate,
                     char&           pState,
                     double&         pAggregate,
                     double&         pCredit,
                     double&         pInitialCredit,
                     double&         pReservation,
                     char&           pColor,
                     double&         pProRateFactor,
                     int&            pCurrencyId,
                     int&            pUomId,
                     time_t&         pSnapShotDate ,
                     unsigned long&  pSequenceNumber,
                     time_t&         pValidFrom,
                     time_t&         pValidTo,
                     int&            pPricingAlternative,
                     int&            pCocTariffId,
                     time_t&         pLastResetDate,
                     double&         pLateCallAggr );
  /* ... */
};

чудо интерфейс. 25 параметров у конструктора, 23 параметров (рефернсы!) у геттера. кто больше?

ЗЫ как оба реализованы можете сами догадатся. и еще несколько других методов в подобном стиле.
ЗЗЫ да, есть и нормальные геттеры.
ЗЗЗЫ нет, другого конструктора нету.
ЗЗЗЗЫ нет, разнести значение по мелким структурам, более приемлимым человеческому мозгу, народ не догадывается. не наш так сказать стиль. (да, есть естественная групировка параметров по источнику откуда они берутся.)

Dummy00001, 21 Июня 2011

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

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

+154

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

<?php
 if (isset($_POST['update'])) {
			 $kolcen=mysql_real_escape_string($_POST['kolcen']);
			 $tovar=mysql_real_escape_string($_POST['tovar']);
			 $descr=mysql_real_escape_string($_POST['descr']);
			 $price=intval($_POST['price']);
			 $price2=intval($_POST['price2']);
			 $price3=intval($_POST['price3']);
			 $id=intval($_POST['update']); 
			 $zag1=mysql_real_escape_string($_POST['zag1']);
			 $zag2=mysql_real_escape_string($_POST['zag2']);
			 $zag3=mysql_real_escape_string($_POST['zag3']);
			 
			  switch($kolcen){ 
		 case '3':$updatetovar = mysql_query ("UPDATE `tovar3` SET `tovar`='$tovar',`descr`='$descr',`price`='$price',`price2`='$price2',`price3`='$price3' WHERE `id`='$id' and `id`='$serv'");
		 break; 
		 case '2': $updatetovar = mysql_query ("UPDATE `tovar2` SET `tovar`='$tovar',`descr`='$descr',`price`='$price',`price2`='$price2' WHERE `id`='$id' and `id`='$serv'");
		 break; 
		 case '1':$updatetovar = mysql_query ("UPDATE `tovar` SET `tovar`='$tovar',`descr`='$descr',`price`='$price' WHERE `id`='$id' and `pizzaid`='$service'");
		 break;
		 case '2images': $updatetovar = mysql_query ("UPDATE `2images` SET `tovar`='$tovar',`descr`='$descr',`price`='$price',`price2`='$price2',`price3`='$price3',`img`='$img',`zag1`='$zag1',`zag2`='$zag2',`zag3`='$zag3' WHERE `id`='$id' and `id`='$serv'");
		 break;
		 }
?>

Народ зацените мой код плиз на предмет говнокода, и если чето не так то дайте совет . Этот код для админки добавления товаров. Есть 4 типа категорий, товар с одной ценой, с двумя, с тремя, и товар с двумя картинками. В зависимости от типа категории делаем запрос к базе

frie, 21 Июня 2011

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

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

+159

1
2
3
4
5
6
7

protected function readConfig($configPath) {
    $ini = parse_ini_file($configPath);
    foreach ($ini as $key => $value) {
        $config[$key] = $value;
    }
    return $config;
}

xarper, 21 Июня 2011

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

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

+120

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

var trimmedKey = Regex.Split(key, @"\.").Last();
                if (_options.Any(o => o == ModelBinderOptions.ExpectUnderLineSymbolAsPrefixDelimiter))
                    trimmedKey = Regex.Split(trimmedKey, "_").Last();

                if (_allRequiredParameters.Any(p => p.Key.ToLower() == trimmedKey.ToLower()))
                {
                    var param = _allRequiredParameters.Single(p => p.Key.ToLower() == trimmedKey.ToLower());

                    try
                    {
                        if (param.Value != typeof(string))
                        {
                            if (Nullable.GetUnderlyingType(param.Value) != null)
                            {
                                try
                                {
                                    var parseMethod = Nullable.GetUnderlyingType(param.Value).GetMethods().Where(m => m.Name == "Parse").First(m => m.GetParameters().Count() == 1 && m.GetParameters().First().ParameterType == typeof(string));
                                    var value = parseMethod.Invoke(null, new object[] { form[key] });
                                    formValues.Add(param.Key, value);
                                }
                                catch(Exception)
                                {
                                    formValues.Add(param.Key, null);
                                }
                            }
                            else
                            {
                                var parseMethod = param.Value.GetMethods().Where(m => m.Name == "Parse").First(m => m.GetParameters().Count() == 1 && m.GetParameters().First().ParameterType == typeof(string));
                                var value = parseMethod.Invoke(null, new object[] { form[key] });
                                formValues.Add(param.Key, value);
                            }
                            
                        }
                        else
                        {
                            formValues.Add(param.Key, form[key]);
                        }
                    }
                    catch (Exception)
                    {
                        // Если произошла ошибка парсинга - печально, но ничего не поделать
                    }
                }

Фееричный парсер

dans, 21 Июня 2011

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

JavaScript / Говнокод #7015

+171

1

var e=("article,aside,footer,header,nav,section").split(',');

Найдено в дебрях одного сайта.

lucidfox, 21 Июня 2011

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

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

+150

1
2
3
4
5
6

$black = "eval|file_get_contents|mysql_(.*?)|include|require|readfile|show_source|highlight|import_request_variables|extract|parse_str|assert|passthru|exec|system|shell_exec|proc_open|fopen|echo";

if(preg_match("#\<\?(.*?)($black)(.*?)\?\>#i", $template)){
exit ("1");
break;
}

substr, 20 Июня 2011

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

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

+159

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

#include <math.h>
#include <iostream>
using namespace std;
void main()
{ int month,day;
        cin>>month,day;
        int den=1;
        switch(month)
{       case 1: cout<<"month: "<<month<<endl<<"day: "<<day+1;
if(day==31){cout<<"month: "<<month+1<<endl<<den;}
break;
        case 2: cout<<"month: "<<month<<endl<<"day: "<<day+1;
        if(day==28){cout<<"month: "<<month+1<<endl<<"day: "<<den;}
break;
        case 3:cout<<"month: "<<month<<endl<<"day: "<<day+1;
                if(day==31){cout<<"month: "<<month+1<<endl<<"day: "<<den;}
break;
        case 4:cout<<"month: "<<month<<endl<<"day: "<<day+1;
                if(day==30){cout<<"month: "<<month+1<<endl<<"day: "<<den;}
break;
        case 5: cout<<"month: "<<month<<endl<<"day: "<<day+1;
                if(day==31){cout<<"month: "<<month+1<<endl<<"day: "<<den;}
break;
        case 6: cout<<"month: "<<month<<endl<<"day: "<<day+1;
                if(day==30){cout<<"month: "<<month+1<<endl<<"day: "<<den;}
break;
        case 7: cout<<"month: "<<month<<endl<<"day: "<<day+1;
                if(day==31){cout<<"month: "<<month+1<<endl<<"day: "<<den;}
break;
        case 8: cout<<"month: "<<month<<endl<<"day: "<<day+1;
                if(day==31){cout<<"month: "<<month+1<<endl<<"day: "<<den;}
break;
        case 9: cout<<"month: "<<month<<endl<<"day: "<<day+1;
                if(day==30){cout<<"month: "<<month+1<<endl<<"day: "<<den;}
break;
        case 10: cout<<"month: "<<month<<endl<<"day: "<<day+1;
                if(day==31){cout<<"month: "<<month+1<<endl<<"day: "<<den;}
break;
        case 11: cout<<"month: "<<month<<endl<<"day: "<<day+1;
                if(day==30){cout<<"month: "<<month+1<<endl<<"day: "<<den;}
break;
        case 12: cout<<"month: "<<month<<endl<<"day: "<<day+1;
                if(day==31){cout<<"month: "<<month+1<<endl<<"day: "<<den;}
                
                                
}       
 
cin.get();
        cin.get();
}

Juris_Kabanis, 20 Июня 2011

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

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

+82

1
2
3

/*
        todo it seems that code below will be right, when client will be changed
*/

Вот такой емкий комментарий перед 200 закоментированными строками кода

galak, 20 Июня 2011

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

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

+147

1
2
3

if ($a = 1) {
	...бла-бла-бла
}

nislova, 20 Июня 2011