- 1
- 2
- 3
TabToShow currentTabToShowInShortForm = tab;
string currentTemplateNameToShowInShortForm = templateName;
currentShortFormToShow = new ShortForm(currentTabToShowInShortForm, currentTemplateNameToShowInShortForm);
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+21
TabToShow currentTabToShowInShortForm = tab;
string currentTemplateNameToShowInShortForm = templateName;
currentShortFormToShow = new ShortForm(currentTabToShowInShortForm, currentTemplateNameToShowInShortForm);
Чем длиннее названия переменных, тем солиднее выглядит код
[Ф]
+26
if (Program.showLoginDialog &&
clientVersion != serverVersion &&
DialogResult.No == MessageBox.Show
(
String.Format
(
"The version {0} of the Server does not match the version of the Client {1}. Would you like to proceed?",
serverVersion, clientVersion
),
"Warning", MessageBoxButtons.YesNo, MessageBoxIcon.Warning
)
)
MessageBox показывается в if
[Ф]
+16.8
return typeof(Nullable<>).MakeGenericType(new Type[] { valueType }).GetConstructor(new Type[] { valueType }).Invoke(new object[] { value });
У афтора было желание сэкономить строчки кода
[Ф]
+25
DateTime startTime = DateTime.Now;
DateTime endTime = DateTime.Now;
foreach (BillablePhas bi in CreateBillablePhaseForFixedPrice.Phases)
{
if (BillablePhase.Equals(bi))
{
errorLbl.Visible = true;
savedLbl.Visible = false;
TimeSpan spread = endTime - startTime; //пик быдлокода
bi.RatioSpredDateTime = Math.Ceiling(Convert.ToDouble(spread.Days*bi.Ratio/100));
}
}
endTime - startTime АААА!!!
+15
if (ValidatorBillablePhase())
{
// строк 40-50 кода
}
private static bool ValidatorBillablePhase()
{
return true;
}
я в шоке от такой валидации ;-)
+3
public void SetDirty()
{
ViewState.SetDirty(false);
}
Отличное название для функции ;-)
+8.5
void delete_ServerClick(object sender, ImageClickEventArgs e)
{
Parent.FindControl(ID).Visible = false;
}
автоматически генерируемый контрол, метод находится непосредственно в самом контроле
+2.4
int a=0;
if (a != 0)
{
a=0;
}
else a=0;
+14.3
bool exitstatus = true;
if (!extremalexit) { exitstatus = true; }
На всякий случай присвоим еще раз
[Ф]
+929.5
if (a == null == false) {
}
когда проверял чужой код пол часа смотрел пока понял что чувак просто не умеет писать if (а != null)