Try-with-resources podczas wywoływania super konstruktora

Question

Czy istnieje jakiś dobry sposób użycia try-for-resources podczas otwierania w konstruktorze obiektu InputStream, a następnie przekazywania go super konstruktorowi?

Zasadniczo co chcę zrobić to:

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)); 
     } 
    } 
} 

Ale, oczywiście, ponieważ super musi być pierwszą instrukcją w konstruktorze to jest niedozwolone. Czy jest jakiś dobry sposób na osiągnięcie tego?

Answer 1

Rozważ użycie statycznej metody fabrycznej zamiast bezpośredniego korzystania z konstruktora. Dokonać przynajmniej B „s konstruktora prywatnych oraz stworzenie metody, takie jak

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

Innym sposobem, aby przejść:

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); 
     } 
    } 
} 

jestem delegowania to tutaj jako możliwą odpowiedź, jednak tu jestem consdering:

1. można aktualizować za kod
2. ruszasz kod konstruktora do metody init dzięki chronionej konstruktora pusty Arg, tylko podklasy muszą poprawnie obsłużyć wywołanie init. Niektórzy mogą zobaczyć, że nie są tak dobrze zaprojektowane. Chodzi mi o to, że jak tylko zaczniesz coś podklasować, musisz wiedzieć o tym więcej, kiedy właśnie go użyjesz.

Answer 3

Niestety nie mam pod ręką kompilatora do testowania, ale nie można wykonać następujących czynności.

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)) 
    } 
}