Как избежать дублирования кода здесь?

Question

У меня есть две разновидности метода в классе, один с дополнительным параметром

первый:

public override void CalcV(IV iv) 
{ 
    initializations 
    otherOperations 

    for (int i=0; i < NUM; ++i) 
    { 
     SomeOtherOperations 
     double v = GetV(a,b,c); 
     SomeOtherOperationsUsing_v 
    } 

    restOfOperations 
} 

и второй один:

public override void CalcV(IV iv, int index) 
{ 
    initializations 
    otherOperations 

    for (int i=0; i < NUM; ++i) 
    { 
     SomeOtherOperations 
     double v = GetV(a,b,c, index); 
     SomeOtherOperationsUsing_v 
    } 

    restOfOperations 
} 

, как вы можете увидеть только разницу что первый вызывает GetV() с 3 параметрами, а второй вызывает перегрузку GetV() с 4 параметрами.

Насколько я могу избежать дублирования кода здесь?

Спасибо!

Answer 1

Если вы используете .NET 4.0, вы можете сделать это необязательный параметр:

public override void CalcV(IV iv, int index = -1) 
{ 
    .... 
    double v = index > -1 ? GetV(a,b,c, index) : GetV(a,b,c); 

    .... 
} 

Answer 2

Имея догадку на то, что делает GetV (вам нужно изменить в соответствии с:

public override void CalcV(IV iv) 
{ 
    CalcV(iv, 0); 
} 


public override void CalcV(IV iv, int index) 
{ 
    initializations 
    otherOperations 

    for (int i=0; i < NUM; ++i) 
    { 
     SomeOtherOperations 
     double v = GetV(a,b,c, index); 
     SomeOtherOperationsUsing_v 
    } 

    restOfOperations 
} 

Answer 3

Я полагаю, index 0 на основе и положительны:

public override void CalcV(IV iv, int index) 
{ 
    initializations 
    otherOperations 

    for (int i=0; i < NUM; ++i) 
    { 
    SomeOtherOperations 
    double v = index == -1 ? GetV(a, b, c) : GetV(a,b,c, index); 
    SomeOtherOperationsUsing_v 
    } 

    restOfOperations 
} 

Тогда вы называете функцию с индексом -1 вы хотите использовать GetV с тремя параметрами или «правильным» индексом, если вы хотите позвонить GetV с четырьмя параметрами.

public override void CalcV(IV iv) 
{ 
    return CalcV(iv, -1); 
} 

Answer 4

public override void CalcV(IV iv, int? index = null) 
{ 
    initializations 
    otherOperations 

    for (int i=0; i < NUM; ++i) 
    { 
     SomeOtherOperations 
     double v = index != null ? GetV(a,b,c, index) : GetV(a,b,c); 
     SomeOtherOperationsUsing_v 
    } 

    restOfOperations 
} 

Затем вы можете удалить первую переопределение, и это будет иметь дело с обоих сценариев.

Answer 5

Предполагая, что вы не знаете, то по умолчанию, очень простой способ будет:

public override void CalcV(IV iv) 
{ 
    CalcV(iv, null); 
} 

public override void CalcV(IV iv, int? index) 
{ 
    ... 
    double v = index.HasValue ? GetV(a,b,c,index.Value) : GetV(a,b,c); 
    ... 
}