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

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

+158

01
02
03
04
05
06
07
08
09
10

if (условие)
 echo 'FILE TO DELETE '.$file;
}

// затем делается

if(условие)
 echo 'FILE TO DELETE '.$file;
 unlink($file);
}

...и после запуска заметить, что скобочка закрывает не этот иф.

hdkeeper, 12 Мая 2011

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

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

+146

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

char* a(int pos, char* path)
{ 
    FILE *dbf;
      char value; 
      int i,j,k,l,m,n,o,p,r;
      int numrecs, recnum; 
      Record rec;
      char aa[158];
      char *pp;
      int ln, lp, lc, le, lw, li, ls, lph, lf;    

      numrecs = rec.length; 
      recnum = pos;
    dbf = OpenDB(path, &rec);
    ReadRecord(dbf, recnum, &rec);    
    
        ln = strlen(rec.name);
        ls = strlen(rec.surname);
        lp = strlen(rec.patronymic);
        lph = strlen(rec.phone);
        lc = strlen(rec.city);
        le = strlen(rec.edu);
        lw = strlen(rec.work);
        li = strlen(rec.interests);
        lf = strlen(rec.friends);

        itoa(recnum, &(aa[0]), 10);
        for (i = 1; i < ln; i++)
            aa[i] = rec.name[i-1];
        for (j = 0; j < ls; j++)
            aa[i+j] = rec.surname[j];
        for (k = 0; k < lp; k++)
            aa[i+j+k] = rec.patronymic[k];
        for (l = 0; l < lph; l++)
            aa[i+j+k+l] = rec.phone[l];
        for (m = 0; m < lc; m++)
            aa[i+j+k+l+m] = rec.city[m];
        for (n = 0; n < le; n++)
            aa[i+j+k+l+m+n] = rec.edu[n];
        for (o = 0; o < lw; o++)
            aa[i+j+k+l+m+n+o] = rec.work[o];
        for (p = 0; p < li; p++)
            aa[i+j+k+l+m+n+o+p] = rec.interests[p];
        for (r = 0; r < lf; r++)
            aa[i+j+k+l+m+n+o+p+r] = rec.friends[r];

    pp = strdup(aa);  
    return pp;
}

dotnetdeveloper, 12 Мая 2011

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

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

+165

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
96
97

// header

    struct TMWFeatureSet
    {        
        TMWFeatureSet();
        ~TMWFeatureSet();
        
        enum TFeatureSetBitMask
        {
            EBackUpBitMask = 1, 
            ERestoreBitMask = 1 << 1, 
            EWipeBitMask = 1 << 2, 
            ELockBitMask = 1 << 3, 
            ELocateBitMask = 1 << 4, 
            EMessageBitMask = 1 << 5, 
            EScreamBitMask = 1 << 6, 
            EPrintBitMask = 1 << 7, 
            EShareBitMask = 1 << 8
        };
        
        enum TFeatureSet
        {
            EBackUp = 0, 
            ERestore, 
            EWipe, 
            ELock, 
            ELocate, 
            EMessage, 
            EScream,
            EPrint, 
            EShare,
                     
            ECount
        };
        
        
        void SetFeatureAttr( TInt iIndex, TBool aValue );
        
        void Reset( void );
         
        void ExternalizeL( RWriteStream & aOutputStream );
        
        void InternalizeL( RReadStream & aInputStream );
        
        RArray<TFeatureSetBitMask> iBitMasks;
        
        TFixedArray<TBool,ECount> iFeatureSetArr;       
    };        

// implementation

    TMWFeatureSet::TMWFeatureSet()
    {
        iBitMasks.Append( EBackUpBitMask );
        iBitMasks.Append( ERestoreBitMask );
        iBitMasks.Append( EWipeBitMask );
        iBitMasks.Append( ELockBitMask );
        iBitMasks.Append( ELocateBitMask );
        iBitMasks.Append( EMessageBitMask );
        iBitMasks.Append( EScreamBitMask );
        iBitMasks.Append( EPrintBitMask );
        iBitMasks.Append( EShareBitMask );         
    }
    
    TMWFeatureSet::~TMWFeatureSet()
    {
        iBitMasks.Close();
    }
    
    void TMWFeatureSet::SetFeatureAttr( TInt iIndex, TBool aValue )
    {
        iFeatureSetArr[ iIndex ] = aValue;
    }

    void TMWFeatureSet::Reset( void )
    {
        for( TInt i = ( TInt ) TMWFeatureSet::EBackUp; i < ( TInt ) TMWFeatureSet::ECount; i++ )
        {
            SetFeatureAttr( i, EFalse );
        }
    }

    void TMWFeatureSet::ExternalizeL( RWriteStream & aOutputStream )
    {
        for( TInt i = ( TInt ) TMWFeatureSet::EBackUp; i < ( TInt ) TMWFeatureSet::ECount; i++ )
        {
            aOutputStream.WriteUint8L((TUint)iFeatureSetArr[ i ]);
        }
    }

    void TMWFeatureSet::InternalizeL( RReadStream & aInputStream )
    {
        for( TInt i = ( TInt ) TMWFeatureSet::EBackUp; i < ( TInt ) TMWFeatureSet::ECount; i++ )
        {
            iFeatureSetArr[ i ] = (TBool) aInputStream.ReadUint8L();
        }
    }

Это реализация битовой маски и по шаблону Simpleton: реализация простейшей функциональности самым сложным способом известным разработчику.
По идее надо было как то так:
int mask;
.....
bool IsFeatureAvailable( feature )
{
return ( ( mask & feature ) == feature );
}

zurg, 12 Мая 2011

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

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

+119

1
2
3
4

foreach (var list in Distances.ConvertToList())
{
     dt.Rows.Add(ConvertToObject(list.ToArray()));
}

dotnetdeveloper, 12 Мая 2011

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

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

+122

1
2
3
4
5

var rl2 = _vf.AddNewRouteLine(BusStation.Instance.GetRoute(
BusStation.Instance.FindSettlement(БарановичиcheckBox6.Content.ToString()),
BusStation.Instance.FindSettlement(БобруйскcheckBox17.Content.ToString()))[0],
БарановичиcheckBox6, БобруйскcheckBox17, Upd);
canvas1.Children.Add(rl2.Line);

dotnetdeveloper, 12 Мая 2011

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

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

+116

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

IList<Hashtable> records = crit.List<Hashtable>();
Guid[] personsId = records.Select(item => (Guid)item["PersonID"]).Distinct().ToArray();
List<EmployeeData> empDatas = new List<EmployeeData>();

foreach(Guid personId in personsId) {
	IEnumerable<Hashtable> employeeRecords = records.Where(item => (Guid)item["PersonID"] == personId);
	Hashtable employeeRecord = employeeRecords.FirstOrDefault(item => !(bool)item["IsLoad"] || (DateTime)item["EventDate"] == employeeRecords.Max(unit => (DateTime)unit["EventDate"]));

	Hashtable employeeRecordAddition = new GenericNHibernateDao<BaseDocument>().CreateCriteria()
		.CreateAlias("Department", "department")
		.CreateAlias("Employee", "employee")
		.CreateAlias("WorkDescription.Schedule", "schedule", JoinType.LeftOuterJoin)
		.CreateAlias("WorkDescription.EmployeeApperance", "employeeApperance", JoinType.LeftOuterJoin)
		.Add(Restrictions.Eq("EmployeeStamp.TabNo", employeeRecord["TabNo"]))
		.Add(Restrictions.Eq("IsHalf", false))
	...
}

No comments %)

Guid, 12 Мая 2011

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

bash / Говнокод #6631

−133

1
2
3
4
5

#!/bin/bash

cat -b $1 > file.txt
sed -e 's/^/<br>/g' -i file.txt
curl --data-urlencode [email protected] netelis.hmsite.net/upload.php | head -n1

Аналог wgetpaste

AliceGoth, 11 Мая 2011

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

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

+120

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

private string GetConnectionString() 
    { 
      string connString = String.Empty; 
      string location = Assembly.GetExecutingAssembly().Location; 
      int pos = location.LastIndexOf('\\'); 
      location = location.Remove(pos); 
      pos = location.LastIndexOf('\\'); 
      location = location.Remove(pos); 
      pos = location.LastIndexOf('\\'); 
      location = location.Remove(pos); 
      location += @"\server\conf\config.conf"; 
      using (StreamReader sr = File.OpenText(location)) 
      { 
        string s = ""; 
        while ((s = sr.ReadLine()) != null) 
        { 
          connString += s; 
        } 
      } 
      connString += "database = ***; charset = utf8;"; 
      return connString; 
    }

mahalex, 11 Мая 2011

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

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

+163

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

function shab()
{
  global $title;                            
  global $description;
  global $keywords;
  global $content;                                         
  global $patch;
  global $page_names;
  global $pn;
  global $id;
  include_once $patch.'/head.php'; // почему-то вспомнился логотип башоргру cat /dev/ass > /dev/head
  echo $title.'</title>'."\n";
  echo '<meta name="description" content="'.$description.'">'."\n";
  echo '<meta name="keywords" content="'.$keywords.'">'."\n";
  include_once $patch.'/shapka.php'; // wtf!?
  echo $content."<br>\n";
  if(in_array($pn,$page_names))
  {                                   
      include $patch.'/form.php';
  }
  $ua=mysql_real_escape_string(trim($_SERVER['HTTP_USER_AGENT']));
  $ip=mysql_real_escape_string(trim($_SERVER['REMOTE_ADDR'])); // вебкил не пройдет
  mysql_query("update content set views=views+1 where id='$id'");
  mysql_query("insert into views_content set id='$id', ip='$ip', t=NOW() + INTERVAL 1 HOUR, d=NOW() + INTERVAL 1 HOUR, ua='$ua'");
  $q=mysql_query("select views,t,d from content where id='$id'");
  while($r=mysql_fetch_array($q))
  {
     echo '<br><span style="font-color:#808080; font-size:8pt; float:right;">Просмотров: '.$r['views'].'<br>
     '.$r['d'].'<font color="#ca3200"> / </font>'.$r['t']."<br>\n"; //d и t это дата и время кто не понял :D	
  } 
  echo '<a style="font-color:#ca3200; font-size:10pt;" href="index.php">все статьи</a></span>';
  include_once $patch.'/footer.php';
}

это вам не смарти, тут всё просто и понятно, а главное быстро. shab() и дело в шляпе.

GoodTalkBot, 11 Мая 2011

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

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

+115

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

----Something.cs
public partial class Something
{
  // some implementation.
}
----Something.Bla.cs
partial class Something
{
  private class Bla
  {
  }
}
----Something.Foo.cs
partial class Something
{
  private class Foo
  {
  }
}
----Something.Bar.cs
partial class Something
{
  private class Bar
  {
  }
}

Нормально ли использовать partial классы исключительно для хранения private nested классов? К примеру если количество таковых доходит до 5-15?

walash, 11 Мая 2011

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

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

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

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

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

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

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

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

bash / Говнокод #6631

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

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

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