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

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

+116

1
2

var result = resultDate.ToString("yyyy-MM-dd");
result = result.Replace("-", "");

zonder, 04 Августа 2010

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

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

+111

1
2
3
4
5
6
7
8
9

/// ----------------------------------------------------------------------------- 
/// <summary> 
/// Page_Load runs when the control is loaded 
/// </summary> 
/// ----------------------------------------------------------------------------- 
protected void Page_Load(object sender, System.EventArgs e)
{
   ...
}

да ну!! серьезно что-ли???

Coffeeholic, 04 Августа 2010

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

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

+151

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

std::string wbfunc(std::string& hash)
{
   std::string line;
   std::ifstream f("/home/hromjo/q");
   int l = 0;
   while (std::getline(f, line))
   {
      std::cout << "Перебор: " << l << "паролей." << "\n";
      l++;
      if (md5(line) == hash)
      {
        return line;
        break;
      }
   }
}

Возможно слегка не в тему.
Писал я и хочу проверить, говнокод ли это.
И что можно исправить в этом говнокоде.

hromjo, 04 Августа 2010

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

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

+113

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

const string newObjectName = "Новый объект";

        // формирует имя нового объекта
        string BuildNewObjectName()
        {
            var namesTaken = from node in objectAdapters where node.Name.Contains(newObjectName) select node.Name;

            int n = 0;

            // ищем максимальное число в конце имени
            if (namesTaken.Any())
                n = namesTaken.Aggregate(n, (acc, name) =>
                    {
                        int current;
                        return (int.TryParse(name.Split().Last(), out current) && current > acc) ? current : acc;
                    });

            // возвращаем следующее
            return newObjectName + " " + (n+1).ToString();
        }

Получение имени для нового объекта. Смесь различных техник. Не читаемо.

Lehox, 04 Августа 2010

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

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

+146

01
02
03
04
05
06
07
08
09
10

$module=(!empty($_GET['mod']))?$_GET['mod']:"static";

if(is_file("modules/".$module.".class.php"))
{
    include_once("modules/".$module.".class.php");
}
else
{
    include_once("modules/error.class.php");
}

от такая хитрожопая система, позволяющая подгружать "модули"

zloiia, 04 Августа 2010

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

ActionScript / Говнокод #3869

−85

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

switch (arr_brick[Y][X].bonus) {
  case "1" :
   bonus_mc.showBonus.gotoAndStop(1);
   break;
  case "2" :
   bonus_mc.showBonus.gotoAndStop(2);
   break;
  case "3" :
   bonus_mc.showBonus.gotoAndStop(3);
   break;
  case "4" :
   bonus_mc.showBonus.gotoAndStop(4);
   break;
  case "5" :
   bonus_mc.showBonus.gotoAndStop(5);
   break;
  case "6" :
   bonus_mc.showBonus.gotoAndStop(6);
   break;
  case "7" :
   bonus_mc.showBonus.gotoAndStop(7);
   break;
  case "8" :
   bonus_mc.showBonus.gotoAndStop(8);
   break;
  case "9" :
   bonus_mc.showBonus.gotoAndStop(9);
   break;
  case "10" :
   bonus_mc.showBonus.gotoAndStop(10);
   break;
  case "11" :
   bonus_mc.showBonus.gotoAndStop(11);
   break;
  case "12" :
   bonus_mc.showBonus.gotoAndStop(12);
   break;
  case "13" :
   bonus_mc.showBonus.gotoAndStop(13);
   break;
  case "14" :
   bonus_mc.showBonus.gotoAndStop(14);
   break;
  case "15" :
   bonus_mc.showBonus.gotoAndStop(15);
   break;
  case "16" :
   bonus_mc.showBonus.gotoAndStop(16);
   break;
  }
  bonAnim(bonus_mc, arr_brick[Y][X].bonus);
 }

dimas_art, 04 Августа 2010

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

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

+113

1
2
3
4
5
6
7
8

if (((productOrder.DataSet).ProductOrder[0].RowState != DataRowState.Deleted) &&
                (productOrder.DataSet).ProductOrder[0].IsOrderReferenceNull() &&
                WebOrderType.IsIngestion() &&
                (ingestOrder != null) && (ingestOrder.IngestOrder.Count > 0) &&
                !(ingestOrder).IngestOrder[0].IsOrderReferenceNull()){
                (productOrder.DataSet).ProductOrder[0].OrderReference =
                    (ingestOrder).IngestOrder[0].OrderReference;
            }

Eugene, 04 Августа 2010

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

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

+168

1
2
3
4
5

var count = response.data.length;
var all_count = count;
if(all_count > count) {
    count = all_count - count;
}

Уличная магия. О_о (Мапед не мой)

Alex_Slubsky, 04 Августа 2010

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

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

+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
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

public static int count(String s) {
    int res = 0;
    for (int i = 0; i < s.length(); i++) {
        if(s.charAt(i) == '1') res++;
    }
    return res;
}

public List<VDictLiability> get(String currency_id, String usedTrailer, Date actualDate, String contract_class_id, String product_id, String risk_object_type_id) {
    for(int i = 0;i<5;i++){
        int j = 0;
        while(j < 32) {
            if(count(Integer.toBinaryString(j)) == i) {
                String s = Integer.toBinaryString(j);
                while (s.length() != 5) {
                    s = "0" + s;
                }
                List<VDictLiability> result = get(currency_id, usedTrailer, actualDate, contract_class_id, product_id, risk_object_type_id, s);
                if(result.size() > 0) return result;
            }
            j++;
        }
    }
    return Collections.EMPTY_LIST;
}

public List<VDictLiability> get(String currency_id, String usedTrailer, Date actualDate, String contract_class_id, String product_id, String risk_object_type_id, String mask) {
    String territoryCode = App.config.getCurrentBrunchCode().substring(1, 3);
    
...
    
    HashMap<Double, VDictLiability> map = new HashMap<Double, VDictLiability>();
    for (VDictLiability liabilityRow : get()) {
        if(!territoryType.equals(liabilityRow.getterritory_type_id()) && !(mask.charAt(4) == '1' && liabilityRow.getterritory_type_id() == null)){
            continue;
        }
        if(actualDate != null && !CompareUtil.isInside(liabilityRow.getstart_date(), liabilityRow.getend_date(), actualDate)){
            continue;
        }
        if(((risk_object_type_id != null && !CompareUtil.isEquals(liabilityRow.getrisk_object_type_id(), risk_object_type_id)) && !(mask.charAt(0) == '1' && liabilityRow.getrisk_object_type_id() == null))){
            continue;
        }
        if(((product_id != null && !CompareUtil.isEquals(liabilityRow.getproduct_id(), product_id)) && !(mask.charAt(1) == '1' && liabilityRow.getproduct_id() == null))){
            continue;
        }
        if(((contract_class_id != null && !CompareUtil.isEquals(liabilityRow.getcontract_class_id(), contract_class_id)) && !(mask.charAt(2) == '1' && liabilityRow.getcontract_class_id() == null))){
            continue;
        }
        if((liabilityRow.getcurrency_id() != null & !CompareUtil.isEquals(liabilityRow.getcurrency_id(), currency_id)) && !(mask.charAt(3) == '1' && liabilityRow.getcurrency_id() == null)){
            continue;
        }
        if(liabilityRow.getis_used_trailer() != SQLUtils.NULL_INTEGER_VALUE & !CompareUtil.isEquals(""+liabilityRow.getis_used_trailer(), usedTrailer)){
            continue;
        }
        map.put(liabilityRow.getvalue(), liabilityRow);
    }
    
    ArrayList<VDictLiability> result = new ArrayList<VDictLiability>();
    result.addAll(map.values());
...
    return result;
}

public List<VDictLiability> get(String currency_id, boolean usedTrailer, Date actualDate, String contract_class_id, String product_id, String risk_object_type_id) {
    return get(currency_id, usedTrailer?"1":"0", actualDate, contract_class_id, product_id, risk_object_type_id);
}

Это ад

borka, 03 Августа 2010

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

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

+158

1
2
3
4
5
6

$s = "insert content (name,description,type,ltype,contract_id) values ('".addslashes($title[$key])."','".addslashes($description)."','".addslashes($type[$key])."','".addslashes($ltype[$key])."','".addslashes($contracts[$key])."')";
$result = mysql_query($s);
$s2 = "select id from content where ((name='".addslashes($title[$key])."')and(description='".addslashes($description)."')) order by id desc limit 1";
//echo $s2;
$result2 = mysql_query($s2);
$id = mysql_result($result2, 0, "id"); // вот id вставленного контента

(facepalm) получение last_insert_id =)

особо порадовало "order by id desc limit 1", учитывая что в таблице `content` поле `name` UNIQUE

qbbr, 03 Августа 2010

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

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

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

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

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

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

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

ActionScript / Говнокод #3869

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

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

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

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