- 1
- 2
- 3
- 4
ПолучитьСтроку();
Пока КоличествоСтрок() <> 0 Цикл
УдалитьСтроку();
КонецЦикла;
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
−128
ПолучитьСтроку();
Пока КоличествоСтрок() <> 0 Цикл
УдалитьСтроку();
КонецЦикла;
Новый способ удаления всех строк из документа. Похоже, что разработчик не ведает о существовании метода УдалитьСтроки();
+161
var s = '';
s += (hours) ? ( (hours>9) ? hours + ':' : '0' + hours + ':') : '00:';
s += (minute) ? ( (minute>9) ? minute + ':' : '0' + minute + ':') : '00:';
s += (second) ? ( (second>9) ? second + '.' : '0' + second + '.') : '00.';
почему бы не сделать так:
var s = '';
s += (hours<10) ? '0' + hours : hours) + ':';
// и так далее...
+120
private void comboBox1_SelectedIndexChanged(object sender, EventArgs e)
{
ComboBox
cb = sender as ComboBox;
TextBox
tb = new TextBox();
if (cb == comboBox1)
{
tb = textBox7;
}
if (cb == comboBox2)
{
tb = textBox6;
}
if (cb == comboBox12)
{
tb = textBox2;
}
if (cb == comboBox3)
{
tb = textBox8;
}
if (cb == comboBox4)
{
tb = textBox9;
}
if (cb == comboBox5)
{
tb = textBox10;
}
if (cb == comboBox6)
{
tb = textBox11;
}
if (cb == comboBox7)
{
tb = textBox12;
}
if (cb == comboBox8)
{
tb = textBox13;
}
if (cb == comboBox11)
{
tb = textBox14;
}
tb.Enabled = !(cb.SelectedIndex > 0);
tb.Text = (cb.SelectedIndex > 0) ? "" : tb.Text;
}
+71
if(time.compareTo(startCheckTime) < 0)
return time.compareTo(startCheckTime);
else if(time.compareTo(endCheckTime) > 0)
return time.compareTo(startCheckTime);
time.compareTo(startCheckTime);
return 0;
−130
Если Конт.Сумма <> КонтСумма Тогда
Конт.Сумма = КонтСумма;
КонецЕсли;
Встретил в типовой конфигурации одной из небольших фирм-франчайзи. Много думал ...
+151
public function GetCurrentCultureID(){
return 1;
}
ой как лень дописывать поддержку мультиязычность в устаревшие модули....
+116
var result = resultDate.ToString("yyyy-MM-dd");
result = result.Replace("-", "");
+111
/// -----------------------------------------------------------------------------
/// <summary>
/// Page_Load runs when the control is loaded
/// </summary>
/// -----------------------------------------------------------------------------
protected void Page_Load(object sender, System.EventArgs e)
{
...
}
да ну!! серьезно что-ли???
+151
std::string wbfunc(std::string& hash)
{
std::string line;
std::ifstream f("/home/hromjo/q");
int l = 0;
while (std::getline(f, line))
{
std::cout << "Перебор: " << l << "паролей." << "\n";
l++;
if (md5(line) == hash)
{
return line;
break;
}
}
}
Возможно слегка не в тему.
Писал я и хочу проверить, говнокод ли это.
И что можно исправить в этом говнокоде.
+113
const string newObjectName = "Новый объект";
// формирует имя нового объекта
string BuildNewObjectName()
{
var namesTaken = from node in objectAdapters where node.Name.Contains(newObjectName) select node.Name;
int n = 0;
// ищем максимальное число в конце имени
if (namesTaken.Any())
n = namesTaken.Aggregate(n, (acc, name) =>
{
int current;
return (int.TryParse(name.Split().Last(), out current) && current > acc) ? current : acc;
});
// возвращаем следующее
return newObjectName + " " + (n+1).ToString();
}
Получение имени для нового объекта. Смесь различных техник. Не читаемо.