navado: (Default)
[personal profile] navado
Авторов таких творчеств нужно отлавливать и пороть до тех пор, пока код не сможет пройти ревью.

Вот пример:
private void doSomething(bool toggle, bool on, bool silent)
{
if ((someCtrlElement.Text == "some static string" && toggle == true) || (toggle == false && on == true))
{
... простыня кода...
}
... еще простыня ...
}

Date: 2011-05-14 06:49 pm (UTC)
From: [identity profile] navado.livejournal.com
Рулит, рулит...
Есть еще ЖЖ-шный http://code-wtf.livejournal.com/

Date: 2011-05-14 06:57 pm (UTC)
From: [identity profile] and-toxa.livejournal.com
Здравствуй, Навадо.

Тысчу лет, как тысячу зим!

Date: 2011-05-14 07:07 pm (UTC)
From: [identity profile] navado.livejournal.com
Привет!
Только не говори про личное знакомство с автором...

Re: личное з.

Date: 2011-05-14 07:20 pm (UTC)
From: [identity profile] navado.livejournal.com
Да ничего особенного. кривовато шо ппц.

должно быть как минимум так:
if ((someCtrlElement.Text == "some static string" && toggle) || (toggle&& on))

А вообще State Machines можно понятнее реализововать. Инуменраторы всякие и т.д.

С человеком надеюсь все в порядке? а то пороть и пороть...

Re: личное з.

Date: 2011-05-14 07:26 pm (UTC)
From: [identity profile] and-toxa.livejournal.com
не, пороть - это не ко мне. хотя даже просто способность понять этот код была бы для меня комплиментом, хехе.

Re: личное з.

Date: 2011-05-14 07:30 pm (UTC)
From: [identity profile] navado.livejournal.com
А!... не парься, это почти для всех, кроме автора комплимент. Ибо как минимум непонятно.

Собственно поэтому и пороть.

Date: 2011-05-14 10:46 pm (UTC)
From: [identity profile] sirujin.livejournal.com
Я бы посетовал на большие кусти кода в ifах, а "==true" читать код не мешает. Мне, во всяком случае.в

Date: 2011-05-15 03:53 am (UTC)
From: [identity profile] navado.livejournal.com
Я про стиль, большие куски только часть его.

Там много всего хоть и работает. Ведь не в комерческий продакшен встраивается.

Profile

navado: (Default)
navado

January 2012

S M T W T F S
1234567
8910111213 14
15161718192021
22232425262728
293031    

Style Credit

Expand Cut Tags

No cut tags
Page generated Sep. 20th, 2017 04:27 pm
Powered by Dreamwidth Studios