C# — Говнокод.ру

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

+134

01
02
03
04
05
06
07
08
09
10

private static void GetUserData(User user)
    {
        if (user == null)
            return;

        var firm = user.Firm;

        if (firm == null)
            return;
    }

Бессмысленно и беспощадно.

Gizz, 13 Января 2011

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

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

+127

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

public IQueryable<AuditEntry> ViewAudit()
{
	var items = _operationEntities.AuditItems.AsQueryable();

	return items.Select((item) => new AuditEntry()
	{
		OperationID = item.Id,
		CreationTime = item.CreationTime,
         
		Description = item.AuditItemType.StringFormat.Replace(
			"{0}", item.InsertionStrings.Where((x) => x.StringId == 0).Select((x) => x.String).FirstOrDefault() ?? String.Empty).Replace(
			"{1}", item.InsertionStrings.Where((x) => x.StringId == 1).Select((x) => x.String).FirstOrDefault() ?? String.Empty).Replace(
			"{2}", item.InsertionStrings.Where((x) => x.StringId == 2).Select((x) => x.String).FirstOrDefault() ?? String.Empty).Replace(
			"{3}", item.InsertionStrings.Where((x) => x.StringId == 3).Select((x) => x.String).FirstOrDefault() ?? String.Empty).Replace(
			"{4}", item.InsertionStrings.Where((x) => x.StringId == 4).Select((x) => x.String).FirstOrDefault() ?? String.Empty).Replace(
			"{5}", item.InsertionStrings.Where((x) => x.StringId == 5).Select((x) => x.String).FirstOrDefault() ?? String.Empty).Replace(
			"{6}", item.InsertionStrings.Where((x) => x.StringId == 6).Select((x) => x.String).FirstOrDefault() ?? String.Empty).Replace(
			"{7}", item.InsertionStrings.Where((x) => x.StringId == 7).Select((x) => x.String).FirstOrDefault() ?? String.Empty),
                
		CreatedBy = item.CreatedBy,
		OperationType = item.AuditItemType.MessageType.Name,
		OperationCategory = item.AuditItemType.AuditCategory.Name,
	});
}

Паровозик. Реализация string.Format() в виде linq-запроса

_volt, 13 Января 2011

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

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

+119

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

public enum PageType
{
    Default,
    Billing,
    UserCabinet
}

public PageType GetPageType(int code)
{
     if(code == 1){return PageType.Default;}
     if(code == 2){return PageType.Billing;}
     if(code == 3){return PageType.UserCabinet;}
     return PageType.Default;
}

private int PageType
{
    get
    {
            int page;
            Int32.TryParse(Request["page"], out page);
            return page;
    }
}

Думаю что объяснять тут ничего не нужно...

Arnezami, 13 Января 2011

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

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

+112

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

static Concept()
        {
            s_deserialize = new Dictionary<string, deserializationHandler>
                                {{"1.0", (deserialize_1_0)}};
        }

        private static void serialize(Concept concept, BinaryWriter writer)
  {
            writer.Write ( concept.Name );
            writer.Write ( concept.Score );

            writer.Write ( concept.Documents.Count );
            foreach (var doc in concept.Documents)
                writer.Write(doc.ToString());

            writer.Write ( concept.Nested.Count );
            foreach (var nested in concept.Nested)
                serialize ( nested, writer );
}

       public static List<Concept> Deserialize(Stream stream)
        {
            using (var reader = new BinaryReader(stream, Encoding.UTF8))
            {
                var version = reader.ReadString();
                if (!s_deserialize.ContainsKey(version))
                    throw new NotSupportedException(String.Format("Unsupported version of format. Version: {0}", version));

                var concepts = new List<Concept>(50);
                s_deserialize[version](null, concepts, reader);
                return concepts;
            }
        }

private delegate void deserializationHandler(Concept parent, IList<Concept> concepts, BinaryReader reader);

private static readonly Dictionary<string, deserializationHandler> s_deserialize;

  private static int s_lastId = 0;
       private static int getNextConceptId()
        {
            if (s_lastId == Int32.MaxValue)
                s_lastId = 0;
            return System.Threading.Interlocked.Increment(ref s_lastId);
        }

private static void deserialize_1_0(Concept parent, IList<Concept> concepts, BinaryReader reader)
		{
            for (var i = reader.ReadInt32(); i > 0; i--)
			{
                var c = new Concept(reader.ReadString(), reader.ReadDouble(), parent);
				concepts.Add(c);

                for (var d = reader.ReadInt32(); d > 0; d--)
                    c.Documents.Add(Convert.ToInt32(reader.ReadString()));

				deserialize_1_0(c, c.Nested, reader);
			}
        }
}

Человек в очередной раз изобрел свой велосипед, на этот раз как сериализовать _свои_ классы - с поддержкой версий! По тому что некоторые переменные в классе были помечены [NonSerialized] было похоже что он что то слышал про уже имеющиеся но решил как обычно что самописный - всегда лучше. Но аттрибуты оставил - на всякий пожарный :)

eval_2009, 13 Января 2011

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

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

+117

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

public class A
{
    private int field;
    public int Field { get { return field; } }

    public void Foo()
    {
         B.CreateInstance().A.field = 4;
    }
}

public class B
{
    private static _b = null;
    private A _a = new A();
    public A { get { return _a; } }

    private B() {}
    public static B CreateInstance()
    {
         if (_b == null)
         {
              _b = new B();
         }
         return _b;
    }
}

Встретил сегодня. Не мог не поделиться. Кода там много, поэтому выписал суть. CreateInstance() - это реальное название метода.

t36, 12 Января 2011

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

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

+116

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

public RabotaJurnal(string ceh, string kod, string nameN, string nameM, string nameP, string shet, string tip, string statya, string obj, string nomen, string edIzm, double normaTime, double normaVir, double rascenka, DateTime dateFrom, DateTime dateTo)
{
               Ceh = ceh;
               Kod = kod;
               NameN = nameN;
               NameM = nameM;
               NameP = nameP;
               Shet = shet;
               Tip = tip;
               Statya = statya;
               Obj = obj;
               Nomen = nomen;
               EdIzm = edIzm;
               NormaTime = normaTime;
               NormaVir = normaVir;
               Rascenka = rascenka;
               DateFrom = dateFrom;
               DateTo = dateTo;
}

Конструктор =)

Guid, 12 Января 2011

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

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

+122

1
2
3
4
5
6
7

/// <summary>
/// I hate you guys.
/// You guys are assholes.
/// Especially author of this class.
/// I hate him the most.
/// </summary>
public partial class ProfileEditor

Semargl, 11 Января 2011

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

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

+104

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

public class BuildActivity     
 {
        private class RequestLimit : ICloneable
        {
             //несколько свойств
        }

        private class Request : IDisposable
        {
           // оберка вокруг stream      
        }

        private class Response
        {
            // несколько методов
        }

        private class Document        
        {
            // 4 простых свойства
        }

       //примерно с десяток методов   
}

еще один пример навороченных структур на ровном месте. В достаточно простом классе имеем 4 (sic!) private вложенных класса - которые ниоткуда никем больше не используются. Все потому что человек не умел дизайнить классы так что бы не приходилось временные данные передавать из одной процедуры в другую. Страшно представить что он будет писать когда узнает про Tupels :)

eval_2009, 11 Января 2011

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

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

+120

1
2
3
4

if (Game1.clou == true)
            {Game1.clou = false;}
            else
            { Game1.clou = true; }

Не знаю, как могло моё усталое воображение родить это, вместо
Game1.clou = !Game1.clou

Falcon, 10 Января 2011

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

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

+123

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

WebClient webClient = new WebClient();
            int i = 1;
            while (true)
            {
                try
                {
                    webClient.DownloadFile("http://sayt.ru/" + i.ToString() + ".exe", @i.ToString() + ".exe");
                    number += 1;
                                       
                }
                catch
                {
                 break; 
                }
         }

qbasic, 09 Января 2011

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

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