webdevqa.jp.net

フラグメントには本当に空のコンストラクタが必要ですか?

複数の引数を取るコンストラクタを持つFragmentがあります。開発中に私のアプリは正常に機能しましたが、実稼働環境ではこのクラッシュが発生することがあります。

Android.support.v4.app.Fragment$InstantiationException: Unable to instantiate fragment 
make sure class name exists, is public, and has an empty constructor that is public

このエラーメッセージが示すように空のコンストラクターを作成することもできますが、それ以降はFragmentのセットアップを完了するために別のメソッドを呼び出す必要があります。

私はこのクラッシュがたまにしか起こらない理由について興味があります。 ViewPagerを間違って使用している可能性がありますか?すべてのFragmentsを自分でインスタンス化し、Activity内のリストに保存します。 FragmentManagerトランザクションは使用していません。これは、私が見たViewPagerの例では必要ではなく、開発中にすべてが機能しているように見えたためです。

243
user1234813

はい、彼らがやります。

とにかくコンストラクターをオーバーライドするべきではありません。 newInstance()静的メソッドを定義し、引数を介してパラメーターを渡す必要があります(バンドル)

例えば:

public static final MyFragment newInstance(int title, String message) {
    MyFragment f = new MyFragment();
    Bundle bdl = new Bundle(2);
    bdl.putInt(EXTRA_TITLE, title);
    bdl.putString(EXTRA_MESSAGE, message);
    f.setArguments(bdl);
    return f;
}

そしてもちろん、この方法で引数を取得します:

@Override
public void onCreate(Bundle savedInstanceState) {
    title = getArguments().getInt(EXTRA_TITLE);
    message = getArguments().getString(EXTRA_MESSAGE);

    //...
    //etc
    //...
}

次に、次のようにフラグメントマネージャーからインスタンス化します。

@Override
public void onCreate(Bundle savedInstanceState) {
    if (savedInstanceState == null){
        getSupportFragmentManager()
            .beginTransaction()
            .replace(R.id.content, MyFragment.newInstance(
                R.string.alert_title,
                "Oh no, an error occurred!")
            )
            .commit();
    }
}

このようにして、オブジェクトの状態をデタッチおよび再アタッチした場合、引数を使用して保存できます。インテントに添付されたバンドルによく似ています。

理由-追加の読み値

なぜだろうと思っている人に、なぜ説明するのかと思った。

チェックした場合: https://Android.googlesource.com/platform/frameworks/base/+/master/core/Java/Android/app/Fragment.Java

Fragmentクラスのinstantiate(..)メソッドがnewInstanceメソッドを呼び出すことがわかります。

public static Fragment instantiate(Context context, String fname, @Nullable Bundle args) {
    try {
        Class<?> clazz = sClassMap.get(fname);
        if (clazz == null) {
            // Class not found in the cache, see if it's real, and try to add it
            clazz = context.getClassLoader().loadClass(fname);
            if (!Fragment.class.isAssignableFrom(clazz)) {
                throw new InstantiationException("Trying to instantiate a class " + fname
                        + " that is not a Fragment", new ClassCastException());
            }
            sClassMap.put(fname, clazz);
        }
        Fragment f = (Fragment) clazz.getConstructor().newInstance();
        if (args != null) {
            args.setClassLoader(f.getClass().getClassLoader());
            f.setArguments(args);
        }
        return f;
    } catch (ClassNotFoundException e) {
        throw new InstantiationException("Unable to instantiate fragment " + fname
                + ": make sure class name exists, is public, and has an"
                + " empty constructor that is public", e);
    } catch (Java.lang.InstantiationException e) {
        throw new InstantiationException("Unable to instantiate fragment " + fname
                + ": make sure class name exists, is public, and has an"
                + " empty constructor that is public", e);
    } catch (IllegalAccessException e) {
        throw new InstantiationException("Unable to instantiate fragment " + fname
                + ": make sure class name exists, is public, and has an"
                + " empty constructor that is public", e);
    } catch (NoSuchMethodException e) {
        throw new InstantiationException("Unable to instantiate fragment " + fname
                + ": could not find Fragment constructor", e);
    } catch (InvocationTargetException e) {
        throw new InstantiationException("Unable to instantiate fragment " + fname
                + ": calling Fragment constructor caused an exception", e);
    }
}

http://docs.Oracle.com/javase/6/docs/api/Java/lang/Class.html#newInstance() インスタンス化時に、アクセサーがpublicであることと、そのクラスローダーはそれへのアクセスを許可します。

全体として非常に厄介なメソッドですが、FragmentMangerを使用して、Fragmentsを強制終了して状態を再作成できます。 (AndroidサブシステムはActivitiesで同様のことを行います)。

クラスの例

newInstanceの呼び出しについてよく聞かれます。これをクラスメソッドと混同しないでください。このクラス全体の例は、使用法を示す必要があります。

/**
 * Created by chris on 21/11/2013
 */
public class StationInfoAccessibilityFragment extends BaseFragment implements JourneyProviderListener {

    public static final StationInfoAccessibilityFragment newInstance(String crsCode) {
        StationInfoAccessibilityFragment fragment = new StationInfoAccessibilityFragment();

        final Bundle args = new Bundle(1);
        args.putString(EXTRA_CRS_CODE, crsCode);
        fragment.setArguments(args);

        return fragment;
    }

    // Views
    LinearLayout mLinearLayout;

    /**
     * Layout Inflater
     */
    private LayoutInflater mInflater;
    /**
     * Station Crs Code
     */
    private String mCrsCode;

    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        mCrsCode = getArguments().getString(EXTRA_CRS_CODE);
    }

    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) {
        mInflater = inflater;
        return inflater.inflate(R.layout.fragment_station_accessibility, container, false);
    }

    @Override
    public void onViewCreated(View view, Bundle savedInstanceState) {
        super.onViewCreated(view, savedInstanceState);
        mLinearLayout = (LinearLayout)view.findViewBy(R.id.station_info_accessibility_linear);
        //Do stuff
    }

    @Override
    public void onResume() {
        super.onResume();
        getActivity().getSupportActionBar().setTitle(R.string.station_info_access_mobility_title);
    }

    // Other methods etc...
}
330
Chris.Jenkins

この質問でCommonsWareが述べているように、 https://stackoverflow.com/a/16064418/1319061 、このエラーは、フラグメントの匿名サブクラスを作成している場合にも発生する可能性があります。 。

Fragmentの匿名サブクラスを作成しないでください:-)

16
JesperB

はい、サポートパッケージがフラグメントをインスタンス化するのを見ることができます(それらが破棄されて再度開かれたとき)。 Fragmentサブクラスにはパブリックな空のコンストラクターが必要です。これはフレームワークによって呼び出されるものだからです。