- 1
- 2
- 3
- 4
- 5
public bool IsStandart
{
get { return IsStandart; }
set { IsStandart = value; }
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+96
public bool IsStandart
{
get { return IsStandart; }
set { IsStandart = value; }
}
+101
private void FillLocalService(int SelectedPageIndex)
{
//Set category by search
string search = inputSearch.Value == inputSearchValue ? string.Empty : inputSearch.Value;
if (inputSearch.Value != inputSearchValue && !string.IsNullOrWhiteSpace(inputSearch.Value))
{
if (inputSearch.Value.StartsWith("landscap", StringComparison.InvariantCultureIgnoreCase))
{
SelectedCategory = "Landscapers";
search = string.Empty;
}
else if (inputSearch.Value.StartsWith("baby sitter", StringComparison.InvariantCultureIgnoreCase))
{
SelectedCategory = "Baby sitters";
search = string.Empty;
}
else if (inputSearch.Value.StartsWith("doctor", StringComparison.InvariantCultureIgnoreCase))
{
SelectedCategory = "Doctor";
search = string.Empty;
}
else if (inputSearch.Value.StartsWith("cleaning service", StringComparison.InvariantCultureIgnoreCase))
{
SelectedCategory = "Cleaning services";
search = string.Empty;
}
else if (inputSearch.Value.StartsWith("pet sitter", StringComparison.InvariantCultureIgnoreCase))
{
SelectedCategory = "Pet sitters";
search = string.Empty;
}
else if (inputSearch.Value.StartsWith("handyman", StringComparison.InvariantCultureIgnoreCase))
{
SelectedCategory = "Handyman";
search = string.Empty;
}
else if (inputSearch.Value.StartsWith("electrical service", StringComparison.InvariantCultureIgnoreCase))
{
SelectedCategory = "Electrical services";
search = string.Empty;
}
else if (inputSearch.Value.StartsWith("tutor", StringComparison.InvariantCultureIgnoreCase))
{
SelectedCategory = "Tutor";
search = string.Empty;
}
else if (inputSearch.Value.StartsWith("dog walker", StringComparison.InvariantCultureIgnoreCase))
{
SelectedCategory = "Dog walker";
search = string.Empty;
}
else if (inputSearch.Value.StartsWith("plumb", StringComparison.InvariantCultureIgnoreCase))
{
SelectedCategory = "Plumbing";
search = string.Empty;
}
else if (inputSearch.Value.StartsWith("Caterer", StringComparison.InvariantCultureIgnoreCase))
{
SelectedCategory = "Caterers";
search = string.Empty;
}
else if (inputSearch.Value.Equals("misc", StringComparison.InvariantCultureIgnoreCase))
{
SelectedCategory = "Misc";
search = string.Empty;
}
}
200+ строк кода
}
тот же девелопер, та же .aspx страница)))))
+112
switch (SelectedCategory)
{
case "Landscapers":
inputSearch.Value = "Landscapers";
break;
case "Baby sitters":
inputSearch.Value = "Baby sitters";
break;
case "Doctor":
inputSearch.Value = "Doctor";
break;
case "Cleaning services":
inputSearch.Value = "Cleaning services";
break;
case "Pet sitters":
inputSearch.Value = "Pet sitters";
break;
case "Handyman":
inputSearch.Value = "Handyman";
break;
case "Electrical services":
inputSearch.Value = "Electrical services";
break;
case "Tutor":
inputSearch.Value = "Tutor";
break;
case "Dog walker":
inputSearch.Value = "Dog walker";
break;
case "Plumbing":
inputSearch.Value = "Plumbing";
break;
case "Caterers":
inputSearch.Value = "Caterers";
break;
case "Misc":
inputSearch.Value = "Misc";
break;
}
обычный switch
+102
return (m.Id == Id) ? (true) : (false);
походу платят за символы
+110
PropertyAdminFormModel data = new PropertyAdminFormModel
{
DateFinish = rep.DateFinish,
Name = rep.Name,
ReportingCircleId = rep.ReportingCircleId
};
if (data != null)
{
Id = reportId;
Name = data.Name;
DateFinish = data.DateFinish;
ReportingCircleId = data.ReportingCircleId;
}
А вдруг!? =)
+90
public string Материал_Clean { get; set; }
:[
+93
for (int i = 0; i < wastePeriods.Length; i++)
{
wastePeriods = 0;
docPeriods = 0;
wastePeriodsYear = 0;
}
Диггеры-рефакторщики раскопали очередной оригинальный цикл.
wastePeriods - массив с over300+ записями.
docPeriods, wastePeriodsYear - тоже массивы
+96
string AutoWait;
int AutoStop;
/*...*/
RegistryKey sKey = Registry.LocalMachine.OpenSubKey("Platform\\Settings");
AutoStop=(int)Key.GetValue("AutoStop");
if (AutoStop == 1) { AutoWait = "yes"; wait.Enabled = false; } else { AutoWait = "no"; }
/*...*/
if ((km4<=2)&(onTheWay)&(AutoWait=="yes"))
{
Waiting=true;
label3.BackColor = Color.Red;
label4.BackColor = Color.Black;
}else
if ((km4 > 2) & (onTheWay))
/*...*/
Классика жанра - использование переменной String в качестве bool. Про ООП автор вообще не слышал, весь код оформлен одной большой портянкой с кучей повторений сброса/установки подобных "флажков".
+96
private void textBox10_TextChanged(object sender, EventArgs e)
{
}
private void button3_Click(object sender, EventArgs e)
{
textfactorname1.Text = "";
textfactorname2.Text = "";
textfactorname3.Text = "";
textfactorname4.Text = "";
textfactorname5.Text = "";
textfactorname6.Text = "";
textfactorname7.Text = "";
textfactorname8.Text = "";
textfactorname9.Text = "";
...
checkBox9.Checked = false;
checkBox10.Checked = false;
}
private void button2_Click_1(object sender, EventArgs e)
{
int n;
try
{
n = Convert.ToInt32(textItera.Text);
}
catch
{
n = 1;
}
string results;
double[] NPVarray = new double[n];
double[] NPVsquares = new double[n];
double NPVmax,NPVmin,NPVmid,disp,standdev,CV,zeropoint,pxy,xxx,yyy;
int lessthanzero, morethanxxx, lessthanyyy;
try
{
for (int i = 0; i < n; i++)
{
string factorname1;
...
string factorname10;
double fixedvalue1;
...
double fixedvalue10;
double midvalue1;
...
double midvalue10;
double minvalue1;
...
double minvalue10;
double maxvalue1;
...
double maxvalue10;
string varname1;
...
string varname10;
if (textfactorname1.Text == "")
{
factorname1 = "What a fuck?";
}
else
{
factorname1 = textfactorname1.Text;
}
if (textfactorname2.Text == "")
{
factorname2 = "What a fuck?";
}
Вот так вот. Лимит в 100 строк не позаоляет показать все прелести.
+92
Console.Write("Введите координату Xm: ");
double Xm = Convert.ToDouble(Console.ReadLine());
Console.Write("Введите координату Ym: ");
double Ym = Convert.ToDouble(Console.ReadLine());
Console.Write("Введите координату Xa: ");
double Xa = Convert.ToDouble(Console.ReadLine());
Console.Write("Введите координату Ya: ");
double Ya = Convert.ToDouble(Console.ReadLine());
Console.Write("Введите координату Xb: ");
double Xb = Convert.ToDouble(Console.ReadLine());
Console.Write("Введите координату Yb: ");
double Yb = Convert.ToDouble(Console.ReadLine());
Console.Write("Введите координату Xc: ");
double Xc = Convert.ToDouble(Console.ReadLine());
Console.Write("Введите координату Yc: ");
double Yc = Convert.ToDouble(Console.ReadLine());
if (
((((Xb - Xa) * Ym + (Xa * Yb - Xb * Ya) - (Yb - Ya) * Xm <= 0) && ((Xb - Xa) * Yc + (Xa * Yb - Xb * Ya) - (Yb - Ya) * Xc <= 0))
|| (((Xb - Xa) * Ym + (Xa * Yb - Xb * Ya) - (Yb - Ya) * Xm >= 0) && ((Xb - Xa) * Yc + (Xa * Yb - Xb * Ya) - (Yb - Ya) * Xc >= 0)))
&& ((((Xc - Xa) * Ym + (Xa * Yc - Xc * Ya) - (Yc - Ya) * Xm <= 0) && ((Xc - Xa) * Yb + (Xa * Yc - Xc * Ya) - (Yc - Ya) * Xb <= 0))
|| (((Xc - Xa) * Ym + (Xa * Yc - Xc * Ya) - (Yc - Ya) * Xm >= 0) && ((Xc - Xa) * Yb + (Xa * Yc - Xc * Ya) - (Yc - Ya) * Xb >= 0)))
&& ((((Xc - Xb) * Ym + (Xb * Yc - Xc * Yb) - (Yc - Yb) * Xm <= 0) && ((Xc - Xb) * Ya + (Xb * Yc - Xc * Yb) - (Yc - Yb) * Xa <= 0))
|| (((Xc - Xb) * Ym + (Xb * Yc - Xc * Yb) - (Yc - Yb) * Xm >= 0) && ((Xc - Xb) * Ya + (Xb * Yc - Xc * Yb) - (Yc - Yb) * Xa >= 0)))
) Console.WriteLine("Точка ({0}, {1}) входит в треугольник", Xm, Ym);
else Console.WriteLine("Точка ({0}, {1}) не входит в треугольник", Xm, Ym);
Console.ReadKey();
Проверка входит ли точка в треугольник.