Try-with-resources quando si chiama super-costruttore

Question

C'è un buon modo di usare try-with-resources quando si apre un InputStream in un costruttore e quindi lo si passa a un super costruttore?

Fondamentalmente quello che voglio fare è questo:

public class A { 
    public A(InputStream stream) { 
     // Do something with the stream but don't close it since we didn't open it 
    } 
} 

public class B { 
    public B(File file) { 
     // We open the stream so we need to ensure it's properly closed 
     try (FileInputStream stream = new FileInputStream(file)) { 
      super(new FileInputStream(file)); 
     } 
    } 
} 

Ma, naturalmente, dal momento che super deve essere la prima istruzione nel costruttore questo non è permesso. C'è un buon modo per ottenere questo?

Answer 1

Considerare l'utilizzo di un metodo factory statico anziché utilizzare direttamente il costruttore. Fare almeno B 's costruttore privato, e creare un metodo come

private B(InputStream is) { 
    super(is); 
    // Whatever else is needed 
} 

public static B newInstance(File file) { 
    B result; 
    try (FileInputStream stream = new FileInputStream(file)) { 
     result = new B(stream); 
    } 
    // Further processing 
    return result; 
} 

Answer 2

Un'altra strada da percorrere:

public class A { 
    protected A(){ 
     // so he can't be called from the outside, subclass ensure that init is done properly. 
    } 

    public A(InputStream stream) { 
     init(stream); 
    } 
    // not be able to call it from outside 
    protected final init(InputStream is){ 
     //here goes the code 
    } 
} 

public class B { 
    public B(File file) { 
     // We open the stream so we need to ensure it's properly closed 
     try (FileInputStream stream = new FileInputStream(file)) { 
      init(stream); 
     } 
    } 
} 

sto postando questo qui come una possibile risposta, tuttavia io sono qui consdering:

1. È possibile aggiornare il codice di un
2. Si sta spostando il codice del costruttore per un metodo init, grazie alle protetta costruttore vuoto arg, solo le sottoclassi devono gestire correttamente la chiamata per iniziare. Alcuni potrebbero vederlo non così ben progettato. Il mio punto di vista è che presto la tua sottoclasse qualcosa, devi sapere di più su questo solo quando lo usi.

Answer 3

Purtroppo non ho un compilatore a disposizione per testare ma non potresti fare come segue.

public class B { 
    private static InputStream file2stream(File f){ 
     // We open the stream so we need to ensure it's properly closed 
     try (FileInputStream stream = new FileInputStream(file)) { 
      return stream; 
     }catch(/*what you need to catch*/){ 
      //cleanup 
      // possibly throw runtime exception 
     } 
    } 
    public B(File file) { 
     super(file2stream(file)) 
    } 
}