传递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的方法？