- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
- 15
- 16
- 17
- 18
- 19
- 20
- 21
- 22
- 23
- 24
- 25
- 26
private void ProcessActionsLookup(object sender)
{
LookUpEdit lookUpEdit = sender as LookUpEdit;
if (lookUpEdit != null)
{
if (lookUpEdit.EditValue != null)
{
if (lookUpEdit.EditValue.Equals(currentActionLookupValue))
{
if (actionsGridControlBindingSource.Current != null &&
!_presenter.ActionHasProperParameters(actionsGridView.FocusedRowHandle))
{
SetDataSourceForInnerGrid(sender);
}
}
else
{
SetDataSourceForInnerGrid(sender);
}
}
else
{
SetDataSourceForInnerGrid(sender);
}
}
}
Мы пойдем другим путем.
Некоторые скажут что в представленном контексте абсолютно нормальный код, я все же не соглашусь
guest 03.12.2009 12:50 # 0
guest 03.12.2009 14:32 # 0
dmtr 03.12.2009 14:46 # 0
if (lookUpEdit != null)
SetDataSourceForInnerGrid(sender)
И никаких длинных ифов
guest 03.12.2009 15:39 # 0
if (actionsGridControlBindingSource.Current != null &&
!_presenter.ActionHasProperParameters(ac tionsGridView.FocusedRowHandle))
guest 03.12.2009 21:14 # 0
Не говнокод.
SuBear 04.12.2009 03:07 # 0
Я с гордостью признаю себя автором этого говнокода
Когда с утра увидел своё творение на говнокоде в приступе ярости чуть не убил двух джуниоров которые это запостили (хорошо они сказали что это были не они)
Код рабочий и был написан в спешке (не прощает конечно )) )
Возможно стоит сократить if до следующего
if (lookUpEdit != null)
{
if (lookUpEdit != null)
{
if (!(lookUpEdit.EditValue != null && lookUpEdit.EditValue.Equals(currentActio nLookupValue)) ||
(actionsGridControlBindingSource.Current != null &&
!_presenter.ActionHasProperParameters(ac tionsGridView.FocusedRowHandle)))
{
SetDataSourceForInnerGrid(sender);
}
}
}
Какой if красивее, решать вам ;)
SuBear 04.12.2009 03:08 # 0
if (lookUpEdit != null)
{
)))
guest 07.12.2009 12:30 # 0
guest 04.12.2009 11:26 # 0