Supondo uma interface IReader, uma implementação da interface IReader ReaderImplementation e uma classe ReaderConsumer que consome e processa dados do leitor.
public interface IReader
{
object Read()
}
Implementação
public class ReaderImplementation
{
...
public object Read()
{
...
}
}
Consumidor:
public class ReaderConsumer()
{
public string location
// constructor
public ReaderConsumer()
{
...
}
// read some data
public object ReadData()
{
IReader reader = new ReaderImplementation(this.location)
data = reader.Read()
...
return processedData
}
}
Para testar o ReaderConsumer e o processamento, uso uma simulação do IReader. Então o ReaderConsumer se torna:
public class ReaderConsumer()
{
private IReader reader = null
public string location
// constructor
public ReaderConsumer()
{
...
}
// mock constructor
public ReaderConsumer(IReader reader)
{
this.reader = reader
}
// read some data
public object ReadData()
{
try
{
if(this.reader == null)
{
this.reader = new ReaderImplementation(this.location)
}
data = reader.Read()
...
return processedData
}
finally
{
this.reader = null
}
}
}
Nesta solução, a simulação introduz uma sentença if para o código de produção, pois apenas o construtor da simulação fornece instâncias da interface.
Ao escrever isso, percebo que o bloco try-finally não está relacionado, pois existe para lidar com o usuário alterar o local durante o tempo de execução do aplicativo.
No geral, parece mal cheiroso, como pode ser tratado melhor?
ReaderConsumer
independenteReaderImplementation
?