Недействительные методы возврата, которые изменяют состояние их аргумента как анти-шаблон?

Question

Есть методы, которые возвращают void, но изменяют состояние своих аргументов (т. Е. Предоставляют скрытое или неявное возвращаемое значение), как правило, плохая практика?

Мне кажется, что их сложно насмехаться, что предполагает, что они, возможно, являются признаком плохого дизайна.

Какие шаблоны существуют для их устранения?

Очень надуманный пример:

public interface IMapper 
{ 
    void Map(SourceObject source, TargetObject target); 
} 

public class ClassUnderTest 
{ 
    private IMapper _mapper; 

    public ClassUnderTest(IMapper mapper) 
    { 
     _mapper = mapper; 
    } 

    public int SomeOperation() 
    { 
     var source = new SourceObject(); 
     var target = new TargetObject(); 

     _mapper.Map(source, target); 

     return target.SomeMappedValue; 
    } 
} 

Answer 1

Да в некоторой степени.

То, что вы описываете, является типичным побочным эффектом. Побочные эффекты затрудняют понимание программ, поскольку информация, которую вы должны понимать, не содержится в стеке вызовов. Вам нужна дополнительная информация, т. Е. Какие методы были вызваны до (и в каком) порядке.

Решение заключается в программном обеспечении без побочных эффектов. Это означает, что вы не меняете переменные, поля или что-то еще. Вместо этого вы вернете новую версию того, что вы обычно меняете.

Это основной принцип функционального программирования.

Конечно, этот способ программирования имеет свои собственные проблемы. Просто рассмотрите ввод-вывод.

Answer 2

Ваш код whould будет намного проще, чтобы проверить, если вы сделаете это:

public interface IMapper 
{ 
    TargetObject Map(SourceObject source); 
} 

public class ClassUnderTest 
{ 
    private IMapper _mapper; 

    public ClassUnderTest(IMapper mapper) 
    { 
     _mapper = mapper; 
    } 

    public int SomeOperation(SourceObject source) 
    { 
     var target = _mapper.Map(source, target); 
     return target.SomeMappedValue; 
    } 
} 

Теперь вы можете проверить вас Карта opperation и SomeOperation seperatly. Проблема в том, что вы можете изменить состояние объекта, что затрудняет предоставление заглушки для тестирования. При возврате нового объекта вы можете вернуть тестовый заглушка цели и протестировать свой метод вызова.