これは、「これは安全か、愚かか、または完全に不要か」という質問です。このコードパターンが妥当かどうかを調べようとしています。
public class SomeClass {
private String someField;
// Other fields here omitted for clarity
private SomeClass() {
someField = "some initialization goes here";
// More initialization of omitted fields
}
public static void doSomething(MyObject myObject) {
SomeClass someClass = new SomeClass();
// Do things with myObject and someClass instance
}
}
このコードに明らかな(または非自明な)問題がありますか?一般的な目標は、メソッド内のアクティビティの複雑さをコード内の他の場所から分離することでしたが、柔軟性のためにクラスを開いdoSomething
たままにしておくことでした。SomeClass
多分?
このようなもので同時実行/同期の問題はありますか?
非常にあいまいなサンプルコードをお詫び申し上げます。