- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
List<int> arr = new List<int>();
List<int> tmpArr = new List<int>();
for (int i = 0; i < arr.Count; i++)
{
if (arr[i] > 100)
{
}
else
tmpArr.Add(arr[i]);
}
arr = tmpArr;-
C# / Говнокод #1507
+131
Удаляем плохие элементы со списка или кто создал дурацкий for???
Запостил:
62316e,
10 Августа 2009
Follow us!
List<int> arr = new List<int>() {100, 95, 68, 12, 102, 10};
for (int i = arr.Count - 1; i > -0; i--)
{
if (arr[i] <= 100)
arr.RemoveAt(i);
}
Можно и так сократить (зная List<> и всякую херню):
arr.RemoveAll(100.Equals);
List<int> arr = new List<int>() {100, 95, 68, 12, 102, 10};
for (int i = arr.Count - 1; i >=0; i--)
{
if (arr[i] <= 100)
arr.RemoveAt(i);
}
это нормально тут нет говнокода
говнопостер лохабот
человек потратил 0.01 секунду на удаление кода из ифа
код при этом хуже не сталю По кр мере он избежал тестирования и прочей хуйни
А тут доброжелатели предлагают переписать все.
за переписывание рабочего кода ребята вам никто не будет $$$ платить.
Но если встречается код типа
if(hyi){
//some commented code
}else{
DoJob();
}
То это не говнокод просто тот кто его писал не ЗАДРОТ и понимает что переписывание ИФа в 100 раз опаснее коментариев внутри
а рядом индус 1 секунду закоментил 1 секунду раскоментил 1 секунду снова закоментил
Поэтому переписывальщиков и не любят среди них 99% задротов которые будут по 1000 раз рефакторить то где нахер рефакторинг не нужен а чсерьезные ошибки не заметят....
//todo переписать !!! :) и нормуль
просто это не говнокод а типа заглушка
// .NET 2.0
List<int> intList = IntListFactory.GetRandom();
intList.RemoveAll(delegate(int number) { return number > 100; });
РаФакторинг сила! ^_^