1. C# / Говнокод #11992

    +130

    1. 1
    void RetreiveAddressInfo(ref string shipToName, ref string streetAddress, ref string streetAddress2, ref string city, ref string state, ref string zip);

    abatishchev, 25 Октября 2012

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

  2. C# / Говнокод #11989

    +141

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    public sealed class CustomProvider
{
     private readonly object _SyncRoot = new object();
 
    public CustomProvider()
    {
             lock(_SyncRoot)
             {

    А вообще, если честно, создание экземпляра класса предка (object) всех классов, чтобы произвести захват критической секции - это так печально. Одному мне эта техника кажется удобной, но странной? Нет чтобы создать класс CriticalSection или что-то такое. А они создают объект совсем не связанный с синхронизацией. Не самодокументированно и тут явно какая-то переголова скрывается при создании объекта, по затратам памяти и системных ресурсов, что в каждый объект на уровне имплементации языка приходится по критической секции добавлять.
    Кстати, там как реализована эта критическая секция? Хендл этой критической секции фактически системный ресурс и для него по идеи нужно вызывать Dispose, но это не происходит. Тогда почему этих системных хендлов критической секции хватает, хотя ситуация без Dispose похожа на утечку системных ресурсов?

    Если кто подумал про lock(this), то это плохо с точки зрения проектирования класса и его последующего использования, поэтому идея с private _SyncRoot - это правильно. Ведь lock(this) (под this в последнем случае имеется данный экземпляр класса), может использоваться и снаружи класса, что может быть пересинхронизацией.

    HaskellGovno, 24 Октября 2012

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

  3. C# / Говнокод #11987

    +141

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    public static string GetFormURLParams(bool ld, bool размещен, bool зарегистр, bool поступил, bool неДляРазмещения, bool удален,
									  string nomerDoc, string vhIshNom, string naim, string ntd,
									  string vidDoc, string razdel, string tipDok,
									  NullableDateTime датаДок1, bool датаДокПериод, NullableDateTime датаДок2,
									  NullableDateTime датаРег1, bool датаРегПериод, NullableDateTime датаРег2,
									  Территория территория, Улица улица, String дом1, Boolean диапазон, String дом2,
									  string keyWords, bool scYes, bool scNo, bool scCancel)
{
	var str = GetOnlyOwnParams(ld, размещен, зарегистр, поступил, неДляРазмещения, удален,
							   nomerDoc, vhIshNom, naim, ntd,
							   vidDoc, razdel, tipDok,
							   keyWords, scYes, scNo, scCancel);

	str += DatePickerPeriod.GetParams("dt1", датаДок1, датаДокПериод, датаДок2);
	str += DatePickerPeriod.GetParams("dt2", датаРег1, датаРегПериод, датаРег2);
	str += AdresSearch.GetParams("adr", территория, улица, дом1, диапазон, дом2);

	return str;
}

    ни много ни мало

    wAngel, 24 Октября 2012

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

  4. C# / Говнокод #11984

    +136

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    double thresholdDepth = 0;
if (!txtThresholdDepth.Text.Equals(String.Empty) && !txtThresholdDepth.Text.StartsWith(CultureInfo.CurrentCulture.NumberFormat.NegativeSign))
{
	thresholdDepth = Double.Parse(txtThresholdDepth.Text);
}
else
{
	Log.Print("Input threshold value");
	return;
}

    redrick, 24 Октября 2012

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

  5. C# / Говнокод #11960

    +134

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    public static byte[] ConvertToMessage(Stream stream)
        {
            int length = (int) stream.Length;
            byte[] buffer = new byte[length + 4];
            buffer[0] = (byte) (length & 0xff);
            buffer[1] = (byte) ((length & 0xff00) >> 8);
            buffer[2] = (byte) ((length & 0xff0000) >> 16);
            buffer[3] = (byte) ((length & 0xff000000L) >> 24);
            stream.Read(buffer, 4, length);
            return buffer;
        }

    А товарищи знают толк в извращенствах

    Yukiakze, 19 Октября 2012

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

  6. C# / Говнокод #11950

    +137

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    using System;
 
class TLockCriticalSystemResource : IDisposable
{
        public TLockCriticalSystemResource(){Console.WriteLine("Acquire critical system resource");}
        public void Dispose(){Console.WriteLine("Release critical system resource");}
        public bool Property1{private get{return true;}set{throw new Exception();}}
}
 
public class Test
{
        public static void Main()
        {
                using (var file = new TLockCriticalSystemResource()
                        {
                                Property1=true
                        })
                {
                    // Делаем чего-то с ресурсом
                }
        }
}

    Ололо. using не даёт гарантию безопасности с точки зрения исключений:
    http://ideone.com/nHDIJ
    Системный ресурс остался захваченным.

    LispGovno, 17 Октября 2012

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

  7. C# / Говнокод #11946

    +140

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    var actions = new List<Action>();
foreach (var i in Enumerable.Range(1, 3))
{
    actions.Add(() => Console.WriteLine(i));
}

foreach (var action in actions)
{
    action();
}

    Внезапно:
    http://ideone.com/RaiHr
    http://ideone.com/EMQBA

    LispGovno, 17 Октября 2012

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

  8. C# / Говнокод #11941

    +141

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    var x = new { Items = new List<int> { 1, 2, 3 }.GetEnumerator() };
while (x.Items.MoveNext())
{
    Console.WriteLine(x.Items.Current);
}

    http://ideone.com/Qzdki

    LispGovno, 16 Октября 2012

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

  9. C# / Говнокод #11933

    +137

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    public List<int> parseInt(string input)
{
  string res = "";
  int i = 0;
  while (input[i] != ' ')
  {
    res = res + input[i];
    //System.Console.WriteLine(res + " = res[" + i + "]");
    i++;
  }
  i = res.Length - 4;
  List<int> num = new List<int>();
  for (int j = 0; i > -4; i = i - 4, j++)
  {
    try
    {
      num.Add(int.Parse(res.Substring(i, 4)));
    }
    catch (Exception e1)
    {
      try
      {
        num.Add(int.Parse(res.Substring(i + 1, 3)));
      }
      catch (Exception e2)
      {
        try
        {
          num.Add(int.Parse(res.Substring(i + 2, 2)));
        }
        catch (Exception e3)
        {
          num.Add(int.Parse(res.Substring(i + 3, 1)));
        }
      }
    }
  //System.Console.WriteLine(num[j]+" = num["+j+"]");
  }
  return num;
}

    Опять студенты.
    Задание: реализовать длинную арифметику.

    один из примеров чтения длинного числа

    graph_di5, 16 Октября 2012

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

  10. C# / Говнокод #11912

    +139

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    private static string f(bool? arg)
{
    switch (arg)
    {
        case true: return "true";
        case false: return "false";
        case null: return "null";
        default: return "???";
    }
}

    В продолжение историй про bool-shit. Если строку 8 убрать, то Visual Studio пишет:
    "Error 3 'f(bool?)': not all code paths return a value"
    Что должно быть в аргументе, чтобы попасть в ветку default?

    absolut, 11 Октября 2012

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