Süper kurucuyu çağırırken kaynaklar ile çalışın

Question

Bir kurucuda bir InputStream açarken ve sonra bir süper kurucuya geçirirken kaynakları kullanmaya çalışmak için iyi bir yol var mı? super yapıcı ilk deyimi olması gerektiğinden elbette bu izin verilmez

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

Ama:

Temelde ne yapmak istediğinizi budur. Bunu başarmanın iyi bir yolu var mı?

Answer 1

Doğrudan kurucu kullanmak yerine statik bir fabrika yöntemi kullanmayı düşünün. Ancak burada ben değilim,

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

Muhtemel bir cevap olarak buraya ekliyorum.Böylece: En az B 'ın yapıcı gizli yap ve bir yöntem gibi

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

gitmek için başka bir yol oluşturmak consdering:

1. Sen bir init yöntemi şantiye kodunu taşıyoruz
2. A'nın kodunu güncelleyebilir, korumalı boş arg yapıcı sayesinde sadece alt sınıflar init için çağrıyı doğru bir şekilde kullanmalıdır. Bazıları bu kadar iyi tasarlanmadığını görebilir. Benim amacım, kısa bir süre sonra bir şeyleri alt sınıflara ayırmak, sadece onu kullandığınız zaman hakkında daha çok şey bilmek zorundasınız.

Answer 3

Ne yazık ki test etmek için elimde bir derleyici yok, ancak aşağıdakileri yapamaz mıydınız?

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