Индо-китайский говнокод

От:	alexanderfedin	http://alexander-fedin.pixels.com/
Дата:	15.03.13 17:24
Оценка:

string tempAssembly;
tempAssembly = deliveryAgent.Split(char.Parse("]"));
...

Respectfully,
Alexander Fedin.

Re: Индо-китайский говнокод

	От:	MT-Wizard
	Дата:	15.03.13 17:33
	Оценка:

Здравствуйте, alexanderfedin, Вы писали:

A>

A>...
A>

А вот такое видели:

1. var array = list.AsParallel().ToArray()

2. var array = new List<Campaign>().ToArray();

Это всё из кода, который приходил на этой неделе на ревью. Нового кода!

А ти, москалику, вже приїхав (с)

Re: Индо-китайский говнокод

	От:	G-Host
	Дата:	15.03.13 17:48
	Оценка:

Здравствуйте, alexanderfedin, Вы писали:

A>

A>string tempAssembly;
A>tempAssembly = deliveryAgent.Split(char.Parse("]"));
A>...
A>

за такое по голове надо стучать! кто же так, сразу Parse вызывает? надо с проверкой:

string tempAssembly;
char c;
if (char.TryParse("]", out c))
  tempAssembly = deliveryAgent.Split(c));

Re[2]: Индо-китайский говнокод

От:	alexanderfedin	http://alexander-fedin.pixels.com/
Дата:	15.03.13 18:26
Оценка:

Здравствуйте, G-Host, Вы писали:

GH>Здравствуйте, alexanderfedin, Вы писали:

A>>

A>>string tempAssembly;
A>>tempAssembly = deliveryAgent.Split(char.Parse("]"));
A>>...
A>>

GH>за такое по голове надо стучать! кто же так, сразу Parse вызывает? надо с проверкой:

GH>

GH>string tempAssembly;
GH>char c;
GH>if (char.TryParse("]", out c))
GH>  tempAssembly = deliveryAgent.Split(c));
GH>

Нет предела совершенству!

Respectfully,
Alexander Fedin.

Re[2]: Индо-китайский говнокод

От:	alexanderfedin	http://alexander-fedin.pixels.com/
Дата:	15.03.13 18:27
Оценка:

Здравствуйте, MT-Wizard, Вы писали:

MW>Здравствуйте, alexanderfedin, Вы писали:

A>>

A>>...
A>>

MW>А вот такое видели:

MW>

MW>1. var array = list.AsParallel().ToArray()

MW>2. var array = new List<Campaign>().ToArray();
MW>

MW>Это всё из кода, который приходил на этой неделе на ревью. Нового кода!
Какая изобретательность!

Respectfully,
Alexander Fedin.

Re[2]: Индо-китайский говнокод

	От:	UA
	Дата:	15.03.13 18:40
	Оценка:

Здравствуйте, MT-Wizard, Вы писали:

MW>Это всё из кода, который приходил на этой неделе на ревью. Нового кода!

Если компилится значить можно!

Re[2]: Индо-китайский говнокод

	От:	3V
	Дата:	15.03.13 19:04
	Оценка:

Здравствуйте, UA, Вы писали:
UA>Если компилится значить можно!

+1

Здравствуйте, MT-Wizard, Вы писали:
MW>Это всё из кода, который приходил на этой неделе на ревью. Нового кода!

Лол. У вас ревью кода делают.

А вот мы негров линчуем и вообще у нас код "гос. важности".

З.Ы. Это так... сарказм сквозь слезы если что...

Re[2]: Индо-китайский говнокод

	От:	Цыба
	Дата:	15.03.13 20:35
	Оценка:

Здравствуйте, G-Host, Вы писали:

GH>

GH>string tempAssembly;
GH>char c;
GH>if (char.TryParse("]", out c))
GH>  tempAssembly = deliveryAgent.Split(c));
GH>

А что в else?

Re[3]: Индо-китайский говнокод

	От:	G-Host
	Дата:	15.03.13 21:35
	Оценка:

Здравствуйте, Цыба, Вы писали:

Ц>А что в else?

return конечно же

Re: Индо-китайский говнокод

	От:	DreamMaker
	Дата:	16.03.13 06:07
	Оценка:

Здравствуйте, alexanderfedin, Вы писали:

вот такое все-равно не переплюнуть:

bool flag;
//....
if(flag.ToString().Length==4)

In P=NP we trust.

Re[3]: Индо-китайский говнокод

	От:	Kaifa
	Дата:	16.03.13 06:46
	Оценка:

UA>Если компилится значить можно!

к тому же почти уверен что оптимизирует как надо.

Re[2]: Индо-китайский говнокод

	От:	G-Host
	Дата:	16.03.13 15:19
	Оценка:

Здравствуйте, DreamMaker, Вы писали:

DM>Здравствуйте, alexanderfedin, Вы писали:

DM>вот такое все-равно не переплюнуть:

DM>

DM>bool flag;
DM>//....
DM>if(flag.ToString().Length==4)
DM>

а мне так больше нравится, строковые литералы рулезз

if (flag.ToString() == "True")

Re[2]: Индо-китайский говнокод

	От:	IObserver
	Дата:	16.03.13 16:26
	Оценка:	+2

Здравствуйте, DreamMaker, Вы писали:

DM>

DM>bool flag;
DM>//....
DM>if(flag.ToString().Length==4)
DM>

Во-первых, из кода не ясно что за число "4". Во-вторых, не нужно нарушать старую добрую традицию: константу пишем слева. Ну и в третьих, нужно указать культуру. Нужно отрефакторить.

public const int TrueLength = 4;

//....

if(TrueLength == flag.ToString(CultureInfo.InvariantCulture).Length)

А вообще, раз уж на то пошло, неплохо бы левую часть оформить как Extension-метод:

public static class BoolExtension
    {
        public static int GetBoolInvLen(this bool source)
        {
            return source.ToString(CultureInfo.InvariantCulture).Length;
        }
    }

После такого небольшого улучшения, теперь везде можем писать намного проще и понятнее:

public const int TrueLength = 4;

//....

if(TrueLength == flag.GetBoolInvLen())

Re[2]: Индо-китайский говнокод

	От:	Кодт
	Дата:	17.03.13 13:31
	Оценка:

Здравствуйте, DreamMaker, Вы писали:

DM>вот такое все-равно не переплюнуть:
DM>

DM>bool flag;
DM>//....
DM>if(flag.ToString().Length==4)
DM>

А в какой локали?
Если ru_RU, то "ложь".Length == 4, как раз...

Перекуём баги на фичи!

Re[3]: Индо-китайский говнокод

	От:	DreamMaker
	Дата:	17.03.13 14:40
	Оценка:

Здравствуйте, Кодт, Вы писали:

DM>>

DM>>bool flag;
DM>>//....
DM>>if(flag.ToString().Length==4)
DM>>

К>А в какой локали?
К>Если ru_RU, то "ложь".Length == 4, как раз...

так им и надо, будут знать как за путина голосовать

и вообще, локаль.. не слишком ли умное слово для простого индийского парня?

In P=NP we trust.

Переместить
Удалить
Выделить ветку

Пока на собственное сообщение не было ответов, его можно удалить.

Заголовок: