傳遞java.lang.Object作爲參數

Question

我知道傳遞一個Object不是一個好習慣。但在這種情況下，這似乎是對我來說最好的解決方案。

public void doSomething(final Object obj) { 
    // some code 
    ... 
    if (obj instanceof InputStream) { 
      document = PDDocument.load((InputStream) obj); 
    } else if (obj instanceof File) { 
     document = PDDocument.load((File) obj); 
    } else { 
     throw new IllegalArgumentException("Argument must be an instance of " + InputStream.class.getName() + " or " + " " + File.class.getName() + "."); 
    // more code 
    ... 
    } 
} 

另一種解決辦法有更多的重複代碼（之前和PDDocument.load(obj);畢竟行）

public void doSomething(final InputStream obj) { 
    // some code 
    ... 
    document = PDDocument.load(obj); 
    // more code 
    ... 
    } 
} 

public void doSomething(final File obj) { 
    // some code 
    ... 
    document = PDDocument.load(obj); 
    // more code 
    ... 
    } 
} 

由於重複的代碼，我更喜歡第一個解決方案。

您是否知道解決此問題的更好解決方案？

Answer 1

通行證中的

PDDocument.load(specificallyTypedVariable) 

結果作爲參數傳遞給該方法。

---

這假定// some code是不是做某種設置爲load電話。如果是這樣的話，你可以在一個Function<? super T, PDDocument>通過與T你會從加載它一起：

public <T> void doSomething(final T obj, Function<? super T, PDDocument> loader) { 
    // some code 
    PDDocument document = loader.apply(obj); 
    // other code. 
} 

和調用，如：

doSomething(someFile, PDDocument::load); 
doSomething(someInputStream, PDDocument::load); 

Answer 2

移動

// some code 
... 
document = PDDocument.load(obj); 
// more code 

到一個單獨的private方法，其可以僅由上述的兩種方法被稱爲

Answer 3

由於PDDocument可以從InputStream加載和你可以從一個File的InputStream反正，我建議：

public void doSomething(final InputStream in) 
{ 
    // some code 
    document = PDDocument.load(in); 
    // more code 
} 

public void doSomething(final File file) 
{ 
    try (
     final InputStream in = new FileInputStream(file); 
    ) { 
     doSomething(in); 
    } catch (IOException e) { 
     throw new RuntimeException(e); 
    } 
} 

賽道中e，相應地處理錯誤！

---

另外，我不明白爲什麼你不回的document，並將該處理返回void的方法？