¿Está bien pasar una secuencia a varios métodos?

Question

que tienen una interfaz define como:

public interface IClientFileImporter 
{ 
    bool CanImport(Stream stream); 
    int Import(Stream stream); 
} 

La idea es tomar cualquier secuencia de archivo y ejecutarlo a través de una serie de implementaciones de esta interfaz para determinar cuál debe manejar el archivo. Algunas de las implementaciones pueden buscar una determinada fila de encabezado, mientras que otras pueden buscar una determinada secuencia de bytes, etc. ...

Mi pregunta es, ¿está bien pasar una secuencia así siempre que nunca cierre ¿eso? Cada método debería ser responsable de restablecer el flujo a la posición 0 si es necesario, pero ¿hay algún otro problema potencial (aparte de la seguridad del hilo)? Este código realmente huele, IMO, pero no estoy seguro de una mejor manera de hacerlo.

Answer 1

Para evitar que se modifique la secuencia subyacente, cree una secuencia contenedora que se deriva de Stream y reenvía solo las llamadas seguras a la secuencia envolvente. Además, no asuma que los métodos Import/CanImport restablecen la posición de la secuencia. La persona que llama de esos métodos debe restablecer el flujo a un estado válido antes de pasarlo a Import/CanImport.

Answer 2

Si cada función devuelve la transmisión tal como lo obtuvo, no creo que haya ningún problema.

Answer 3

Esto no debería ser un problema.

A pesar de que probablemente reestructurar ligeramente:

public interface IClientFileImporter 
{ 
    int Import(Stream stream); 
} 

Entonces tendría el método de importación devuelve un -1 si no era capaz de hacerlo. Podría hacer que tu otro código sea un poco más simple.

Answer 4

Está perfectamente bien pasar la misma secuencia a varios métodos.

Tenga cuidado con las transmisiones no buscables: hay transmisiones que no puede restablecer. El comentario de Andre Loker tiene buenos consejos para incluir a Stream, de modo que los métodos de CanImport no alteren la transmisión real.

También puede considerar proporcionar explícitamente una parte del encabezado de la secuencia a los métodos de CanImport, también los hará menos flexibles.

Answer 5

Si le preocupa pasar un flujo continuo porque podría terminar ejecutando código externo que podría no ser confiable, entonces lo mejor que puede hacer es crear una nueva secuencia de solo lectura y pasarla para que ningún código externo pueda cambie el contenido del archivo hasta que esté seguro de querer dejarlo.

public class ReadonlyStream : Stream 
{ 
    public ReadonlyStream(Stream baseStream) 
    { 
     ownerStream = baseStream; 
    } 

    private Stream ownerStream; 

    public override bool CanWrite 
    { 
     get { return false; } 
    } 

    public override int Write(byte[] bits, int offset, int count) 
    { 
     throw new InvalidOperationException(); 
    } 

    // Other code ommitted 
}