Android继承ScrollView时，用非静态代码块调用init方法是否正确？

你的代码目前能正常运行，但这种用非静态代码块触发初始化的方式有几个隐藏问题，也不符合Android自定义View的常规实现规范：

主要问题点 #

• 强制类型转换的崩溃风险：你直接把Context强转为Activity，但传入的Context不一定是Activity实例（比如可能是Application Context、ContextWrapper等），这种场景下会直接抛出ClassCastException，导致应用崩溃。
• 代码可读性与维护性差：非静态代码块的执行时机是在所有构造方法的super()调用之后，但它的位置脱离了构造方法本身，后续维护的开发者很容易忽略这个初始化逻辑，增加理解和修改成本。
• 潜在的初始化顺序隐患：虽然当前View的getContext()在super()调用后能正常返回，但如果后续Android系统对View的构造逻辑有调整（概率极低但并非不可能），可能导致getContext()返回null，进而让init()方法执行出错。

更安全、规范的实现方式 #

推荐使用构造方法链来减少重复代码，同时显式调用初始化方法，并且增加类型安全检查：

public class ScrollViewExtended extends ScrollView {
    private Activity activity;
    private UserActivityLogDao userActivityLogDao;
    private String activity_name;

    public ScrollViewExtended(Context context) {
        this(context, null);
    }

    public ScrollViewExtended(Context context, AttributeSet attrs) {
        this(context, attrs, 0);
    }

    public ScrollViewExtended(Context context, AttributeSet attrs, int defStyleAttr) {
        super(context, attrs, defStyleAttr);
        init(context);
    }

    @RequiresApi(api = Build.VERSION_CODES.LOLLIPOP)
    public ScrollViewExtended(Context context, AttributeSet attrs, int defStyleAttr, int defStyleRes) {
        super(context, attrs, defStyleAttr, defStyleRes);
        init(context);
    }

    private void init(Context context) {
        // 先检查Context类型，避免强转崩溃
        if (context instanceof Activity) {
            activity = (Activity) context;
            activity_name = activity.getClass().getSimpleName();
        } else {
            // 可根据需求选择抛出异常或做容错处理
            throw new IllegalArgumentException("ScrollViewExtended requires an Activity Context");
        }
        userActivityLogDao = new UserActivityLogDao();
    }
}

为什么这样更好？ #

• 减少重复代码：通过构造方法链，让所有构造逻辑最终流向少数几个构造方法，避免在每个构造里重复写初始化调用。
• 类型安全：增加instanceof判断，提前拦截非法的Context输入，避免崩溃。
• 代码清晰直观：初始化逻辑显式写在构造方法内部，其他开发者一眼就能找到初始化的位置，维护起来更轻松。

内容的提问来源于stack exchange，提问作者Prashant Jha