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

    +105

    1. 1
    2. 2
    3. 3
    4. 4
    if (row["Enable"].ToString().ToLower() == "true" || row["Enable"].ToString() == "1")
         chState.Checked = true;
 else if (row["Enable"].ToString().ToLower() == "false" || row["Enable"].ToString() == "0")
         chState.Checked = false;

    В начале метода такая вот проверка.
    Дальше да же читать не стал.

    pipjaka, 03 Октября 2013

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

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

    +139

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    public Agent[] GetChildAgents(Agent parent)
{
	if (parent == null)
		throw new NullReferenceException();
		
	// ...some code...	
}

    Перед смертью не надышишься, зачем оттягивать неизбежное.

    yamamoto, 02 Октября 2013

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

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

    +129

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    private List<Ball> _balls;
private Interaction GetInteraction(Ball ball) {
  foreach (var b2 in _balls.SkipWhile(b => b != ball).Skip(1)) {
      /* ... */
  }
}

var table = _balls.Select(GetInteraction);

    Откопал в своей проге.
    нужно было определить, какой шарик с кем взаимодействует раньше всего, притом если уже рассчитано взаимодействие 1 и 2 шарика (при рассчёте взаимодействий 1 и 2, 1 и 3, 1 и 4, ... и выделено самое раннее взяимодействие шарика 1), то взимодействие 2 и 1 рассчитываеть не надо. Таким образом, нужно было пройти все пары шариков так, чтоб первый встретился в списке раньше второго.
    Массивы, индексы и циклы? Не, не знаем такого. Только Select, только ФП, только колдовство со SkipWhile(b => b!=ball).Skip(1).

    Jean-Esther, 01 Октября 2013

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

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

    +109

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    BusinessLogic.Entity_InheritanceSetting.Save(
                    Session.SessionSettings.Settings.ContactID.Value,
                    Session.SessionSettings.Settings.SessionID.Value, copyID.Value,
                    true, true, true, true, true, true, true, true, true, true, true, true, true, true, true, true,
                    true, true, true, true, true, true, true, true, true, true, true, true, true, true, true, true,
                    true, true, true, true, true, true, true, true, true, true, true, true, true, true, true, true, true,
                    true, true, true, true, true, true, true, true, true, true, true, true, true, true, true,
                    true, true, true, true, true, true, true, true, true, true, true, true, true, true, true);

    kore_sar, 30 Сентября 2013

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

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

    +142

    1. 1
    2. 2
    3. 3
    4. 4
    public static string Guid_GetAnimalUniqueNameToString(int i)
{
	return string.Format("{0}{0}{0}{0}{0}{0}{0}{0}-{0}{0}{0}{0}-{0}{0}{0}{0}-{0}{0}{0}{0}-{0}{0}{0}{0}{0}{0}{0}{0}{0}{0}{0}{0}", i);
}

    Этот код мне знакомый скинул, говорит в цикле for сюда передавались значения :-)

    Smekalisty, 27 Сентября 2013

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

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

    +129

    1. 1
    2. 2
    var tiffFileName = GetRandomNameForTiffFile() +
	currentObject.FileName.Substring(currentObject.FileName.LastIndexOf(".", StringComparison.Ordinal) > 0 ? currentObject.FileName.LastIndexOf(".", StringComparison.Ordinal) : 0);

    Это наверное тоже бояный пример говнокода, но всё же меня порадовал

    Smekalisty, 27 Сентября 2013

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

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

    +137

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    public static ListBox GetListBox()
{
    var list = _customList as ListBox;

    if (list != null)
    {
        return list;
    }

    return null;
}

    Наверное это бояный пример говнокода, но все же я скопировал его собственными руками

    Smekalisty, 27 Сентября 2013

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

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

    +127

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    <Grid x:Name="LayoutRoot" >
	<Grid.ColumnDefinitions>
		<ColumnDefinition />
	</Grid.ColumnDefinitions>
	<Grid.RowDefinitions>
		<RowDefinition />
	</Grid.RowDefinitions>
	
	// .. разные контролы
	
</Grid>

    Это тоже кусочек XAML (WPF)

    Smekalisty, 27 Сентября 2013

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

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

    +129

    1. 1
    2. 2
    Margin="0"
mc:Ignorable="d" Margin="0,0,-191,-31" d:DesignHeight="477.612" d:DesignWidth="671.642" >

    Это кусочек XAML (WPF). Даже не знаю что сказать, наверное мысли были такие: "Мне нужно именно туда и именно так"

    Smekalisty, 27 Сентября 2013

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

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

    +126

    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
    private string _itemName;
public string ItemName
{
	get { return _itemName; }
	set
	{
		if (_itemName != value)
		{
			_itemName = value;
			RaisePropertyChanged("ItemName");
			CustomValue = GetCustomValue(ItemName);
		}
	}
}

    Незнаю насколько этот код можно считать говнокодом, но его неочевидность сразу бросилась в глаза

    Smekalisty, 27 Сентября 2013

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