java.lang.Objectをパラメータとして渡します

Question

オブジェクトを渡すのは良い方法ではありません。しかし、この場合、それは私にとって最高の解決策であるようです。

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

パス2つの上の方法で呼び出すことができる別private方法に移動

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

。

---

これは// 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. 
} 

と同様に起動します。PDDocumentので

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

Answer 2

のみメソッドへのパラメータとして

PDDocument.load(specificallyTypedVariable) 

の結果で

Answer 3

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を返すメソッドでその処理を入れない理由

---

また、私は理解していませんか？