Android onClick方法不适用于自定义视图

时间:2011-04-18 13:22:36

标签: android android-view android-custom-view

我已经开始开发应用了。我昨天建立了菜单但是onClick方法不起作用! 我创建了一个扩展View并调用她的MainMenuObject的类 - 该类用于主菜单中的任何对象(按钮,徽标等)。我为他们创建了一个特殊的类,因为我在菜单启动时正在做动画。在构建了MainMenuObject类之后,我构建了另一个扩展View的类(OpeningTimesView),它将包含主菜单的所有按钮,并将作为主要活动的布局。

一切都很好,动画非常好,我想把听众放在我的按钮上,所以我在onClickListener中添加了一个实现onClickListener,并覆盖了onClick方法。然后我用setOnClickListener(this)和setClickable(true)将监听器添加到按钮,但它不起作用!我已经尝试了一切!请帮我弄清楚我做错了什么。我已经为onClick方法添加了一个toast,它不依赖于任何“if”但它不会显示。

(BTW有没有办法将屏幕宽度和高度定义为所有类都可以访问的变量?它不能是静态的,因为你从显示对象获得高度和宽度但必须有另一种方式)

这是代码:

public class OpeningTimesView extends View implements OnClickListener{
    private MainMenuObjectView searchButton;
    private MainMenuObjectView supportButton;
    private MainMenuObjectView aboutButton;
    private int screenWidth;
    private int screenHeight;
    public OpeningTimesView(Context context, Display dis) {
        super(context);

        this.screenWidth = dis.getWidth();
        this.screenHeight = dis.getHeight();

        searchButton = new MainMenuObjectView(context, 200, MovingMode.RIGHT, R.drawable.search, dis);
        supportButton = new MainMenuObjectView(context, 400, MovingMode.LEFT, R.drawable.support, dis);
        aboutButton = new MainMenuObjectView(context, 600, MovingMode.RIGHT, R.drawable.about, dis);

        searchButton.setClickable(true);
        supportButton.setClickable(true);
        aboutButton.setClickable(true);

        searchButton.setOnClickListener(this);
        supportButton.setOnClickListener(this);
        aboutButton.setOnClickListener(this);
    }

    @Override
    public void onClick(View view){
        Toast.makeText(getContext(), "Search button pressed", Toast.LENGTH_SHORT).show();
        if(view == searchButton){
            Toast.makeText(getContext(), "Search button pressed", Toast.LENGTH_SHORT).show();
        }
        else if(view == supportButton){
            Toast.makeText(getContext(), "Support button pressed", Toast.LENGTH_SHORT).show();
        }
        else Toast.makeText(getContext(), "About button pressed", Toast.LENGTH_SHORT).show();
    }
    @Override
    public void onDraw(Canvas canvas)
    {
        // Drawing the buttons
        this.searchButton.onDraw(canvas);
        this.aboutButton.onDraw(canvas);
        this.supportButton.onDraw(canvas);
    }

提前谢谢,Elad!

10 个答案:

答案 0 :(得分:21)

我遇到了同样的问题 - 我创建了一个自定义视图,当我通过调用v.setOnClickListener(new OnClickListener() {...});在活动中为它注册一个新的监听器时,监听器就没有被调用。

在我的自定义视图中,我还覆盖了public boolean onTouchEvent(MotionEvent event) {...}方法。问题是我没有调用View类的方法 - super.onTouchEvent(event)。这解决了这个问题。因此,如果您想知道为什么您的监听器没有被调用,您可能忘记调用超类onTouchEvent方法

答案 1 :(得分:15)

如果您对UI Framework的运行方式不满意,在Android中创建自定义控件可能会非常棘手。如果您还没有,我建议您阅读以下内容:

http://developer.android.com/guide/topics/ui/declaring-layout.html

http://developer.android.com/guide/topics/ui/custom-components.html

http://developer.android.com/guide/topics/ui/layout-objects.html

请注意,在XML中声明布局时,元素是嵌套的。这将创建一个布局层次结构,在仅使用Java代码自定义组件时,必须创建自己的布局层次结构。

很可能你会陷入Android的触控层次。与其他一些流行的移动平台不同,Android从View层次结构的顶部开始提供触摸事件,并逐渐降低。传统上占据层次结构较高级别(活动和布局)的类具有逻辑,可以转发他们自己不会消耗的触摸。

因此,我建议您更改OpeningTimesView以扩展ViewGroup(所有Android布局的超类)或特定布局(LinearLayoutRelativeLayout等等)并添加您的按钮作为孩子。现在,似乎没有一个已定义的层次结构(按钮在容器中并没有真正“包含”,它们只是成员),这可能会使事件真正发生的问题混淆不清。

  1. 触摸应该更自然地向下流向按钮,允许您点击事件触发
  2. 您可以利用Android的布局机制绘制视图,而不是依赖绘图代码来完成所有这些操作。
  3. 选择一个布局类,可以帮助您将按钮放在最终位置。你可以使用Android中的动画框架或自定义绘图代码(就像你现在一样)来设置动画,无论如何你都喜欢它们。如果需要,按钮的位置和当前绘制按钮的位置允许非常不同,这就是当前动画框架在Android中的工作方式(3.0之前)......但这是一个单独的问题。您还有AbsoluteLayout,它允许您在任何您喜欢的位置放置和替换对象...但请注意您的应用在所有Android设备上的外观(给定不同的屏幕尺寸)。

    关于显示信息的第二点。 最简单的方法可能就是在需要的地方使用Context.getResources().getDisplayMetrics()Activity继承自Context,因此他们可以直接调用此方法。视图总是有Context,您可以使用getContext()进行访问。您可以在构造中将Context作为参数传递给任何其他类(这是Android中的常见模式,您会看到许多对象需要Context,主要是访问Resources

    这是一个跳跃开始的例子。这只是将三个水平排列一次作为最终位置:

    Public class OpeningTimesView extends LinearLayout implements OnClickListener {
        private MainMenuObjectView searchButton;
        private MainMenuObjectView supportButton;
        private MainMenuObjectView aboutButton;
        private int screenWidth;
        private int screenHeight;
    
        public OpeningTimesView(Context context) {
            this(context, null);
        }
    
        //Thus constructor gets used if you ever instantiate your component from XML
        public OpeningTimesView(Context context, AttributeSet attrs) {
            super(context, attrs);
    
            /* This is a better way to obtain your screen info
            DisplayMetrics display = context.getResources().getDisplayMetrics();
            screenWidth = display.widthPixels;
            screenHeight = display.heightPixels;
            */
            //This way works also, without needing to customize the constructor
            WindowManager wm = (WindowManager)context.getSystemService(Context.WINDOW_SERVICE);
            Display dis = wm.getDefaultDisplay();
            screenWidth = dis.getWidth();
            screenHeight = dis.getHeight();
    
            searchButton = new MainMenuObjectView(context, 200, MovingMode.RIGHT, R.drawable.search, dis);
            supportButton = new MainMenuObjectView(context, 400, MovingMode.LEFT, R.drawable.support, dis);
            aboutButton = new MainMenuObjectView(context, 600, MovingMode.RIGHT, R.drawable.about, dis);
    
            //Even if they don't extend button, if MainMenuObjectView is always clickable
            // this should probably be brought into that class's constructor
            searchButton.setClickable(true);
            supportButton.setClickable(true);
            aboutButton.setClickable(true);
    
            searchButton.setOnClickListener(this);
            supportButton.setOnClickListener(this);
            aboutButton.setOnClickListener(this);
    
            //Add the buttons to the layout (the buttons are now children of the container)
            setOrientation(LinearLayout.HORIZONTAL);
            LinearLayout.LayoutParams params = new LinearLayout.LayoutParams(LayoutParams.WRAP_CONTENT, LayoutParams.WRAP_CONTENT);
            addView(searchButton, params);
            addView(supportButton, params);
            addView(aboutButton, params);       
        }
    
        @Override
        public void onClick(View view){
            Toast.makeText(getContext(), "Search button pressed", Toast.LENGTH_SHORT).show();
            if(view == searchButton){
                Toast.makeText(getContext(), "Search button pressed", Toast.LENGTH_SHORT).show();
            }
            else if(view == supportButton){
                Toast.makeText(getContext(), "Support button pressed", Toast.LENGTH_SHORT).show();
            }
            else Toast.makeText(getContext(), "About button pressed", Toast.LENGTH_SHORT).show();
        }
    
        @Override
        public void onDraw(Canvas canvas)
        {
            //Drawing the buttons
            // This may only be necessary until they are in place, then just call super.onDraw(canvas)
            this.searchButton.onDraw(canvas);
            this.aboutButton.onDraw(canvas);
            this.supportButton.onDraw(canvas);
        }    
    }
    

    您可以从那里进行自定义。也许启动可见性设置为View.INVISIBLE的按钮,直到您使用绘图代码或自定义动画对象为它们设置动画,然后使它们在最终的休息位置可见。

    但是,这里的关键是布局足够聪明,知道当它收到触摸事件时,它应该将它转发给相应的孩子。您可以在没有此项的情况下创建自定义视图,但是您必须拦截容器上的所有触摸并进行数学运算以确定将事件手动转发到哪个子视图。如果你真的无法让布局管理员工作,那么这就是你的追索权。

    希望有帮助!

答案 2 :(得分:9)

您可以在自定义视图的onTouchEvent中调用performClick()。

在自定义视图中使用此功能:

    @Override
    public boolean onTouchEvent(final MotionEvent event) {
        if(event.getAction() == MotionEvent.ACTION_UP){
            return performClick();
        }
        return true;
    }

答案 3 :(得分:7)

我是这样做的:

public class YourView extends LinearLayout implements OnClickListener {
    OnClickListener listener;

    //... constructors

    public void setOnClickListener(OnClickListener listener) {
        this.listener = listener;
    }

    @Override
    public void onClick(View v) {
        if (listener != null)
             listener.onClick(v);
    }
}

答案 4 :(得分:3)

onClickListener课程中实施MainMenuObjectView,因为这些是响应点击的对象。
另一种方法是扩展Button而不是View,因为你只使用那里的按钮


更新:完整示例


这是将其直接实现到可点击视图中的想法。有一个TestView类可以扩展View并覆盖onDraw,因为您需要它,并且还会响应点击次数。我遗漏了任何动画实现,因为你有这个部分,它与ClickListener讨论无关 我在Eclair模拟器中对其进行了测试,它按预期工作(单击后播放Toast消息)。

file:Test.java

package com.aleadam.test;

import android.app.Activity;
import android.os.Bundle;
import android.widget.LinearLayout;
import android.widget.TextView;

public class Test extends Activity {
    /** Called when the activity is first created. */
    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        LinearLayout ll = new LinearLayout(this);
        ll.setOrientation(LinearLayout.VERTICAL);
        TextView label = new TextView(this);
        LinearLayout.LayoutParams layoutParams = new LinearLayout.LayoutParams(
             LinearLayout.LayoutParams.WRAP_CONTENT, LinearLayout.LayoutParams.WRAP_CONTENT);
        label.setText("Click the circle!");
        TestView testView = new TestView(this);
        ll.addView(label, layoutParams);
        ll.addView(testView, layoutParams);
        setContentView(ll);
    }
}

file:TestView.java

package com.aleadam.test;

import android.content.Context;
import android.graphics.Canvas;
import android.graphics.Color;
import android.graphics.Paint;
import android.view.View;
import android.view.View.OnClickListener;
import android.widget.Toast;

public class TestView extends View implements OnClickListener {
    Context context;

    public TestView(Context context) {
        super(context);
        this.context = context;
        setOnClickListener(this);
    }

    public void onClick(View arg0) {
        Toast.makeText(context, "View clicked.", Toast.LENGTH_SHORT).show();
    }

    @Override
    public void onDraw (Canvas canvas) {
        super.onDraw(canvas);
        this.setBackgroundColor(Color.LTGRAY);
        Paint paint = new Paint (Paint.ANTI_ALIAS_FLAG);
        paint.setColor(Color.RED);
        canvas.drawCircle(20, 20, 20, paint);
    }
}

如果你需要一些可点击的和一些不可点击的,你可以添加一个带有的构造函数 boolean参数,用于确定是否将ClickListener附加到View:

public TestView(Context context, boolean clickable) {
    super(context);
    this.context = context;
    if (clickable)
        setOnClickListener(this);
}

答案 5 :(得分:3)

我有一个解决方案! 它不是这个特定问题的真正解决方案,而是一种全新的方法。 我把这个帖子发给了我认识的人,他告诉我要使用android所拥有的动画SDK(比如提到的无线设计),所以我只使用一个扩展的类,而不是使用4个类的主菜单页面。 Activity,Animation类提供了许多动画选项。 我要感谢你们两位帮助我,你们很棒。 我正在添加代码,如果有人会遇到同样的问题或其他问题:

package elad.openapp;

import android.app.Activity;
import android.content.Intent;
import android.os.Bundle;
import android.view.HapticFeedbackConstants;
import android.view.View;
import android.view.View.OnClickListener;
import android.view.Window;
import android.view.animation.Animation;
import android.view.animation.AnimationSet;
import android.view.animation.ScaleAnimation;
import android.view.animation.TranslateAnimation;
import android.widget.ImageView;
import android.widget.Toast;

public class OpeningTimes extends Activity implements OnClickListener{
    /** Called when the activity is first created. */
    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
    // Disabling the title bar..
    requestWindowFeature(Window.FEATURE_NO_TITLE);
    setContentView(R.layout.main);

    // Create the buttons and title objects
    ImageView title = (ImageView)findViewById(R.id.title_main);
    ImageView search = (ImageView)findViewById(R.id.search_button_main);
    ImageView support = (ImageView)findViewById(R.id.support_button_main);
    ImageView about = (ImageView)findViewById(R.id.about_button_main);

    // Setting the onClick listeners
    search.setOnClickListener(this);
    support.setOnClickListener(this);
    about.setOnClickListener(this);

    setButtonsAnimation(title, search, support, about);

}

@Override
public void onClick(View v) {
    if(v.getId()==R.id.search_button_main){
        v.performHapticFeedback(HapticFeedbackConstants.LONG_PRESS);
        startActivity(new Intent(this,SearchPage.class));
    }
    else if(v.getId()==R.id.support_button_main){
        v.performHapticFeedback(HapticFeedbackConstants.LONG_PRESS);
        Toast.makeText(this, "Coming soon...", Toast.LENGTH_LONG).show();
    }
    else if(v.getId()==R.id.about_button_main){

        v.performHapticFeedback(HapticFeedbackConstants.LONG_PRESS);
        Toast.makeText(this, "Coming soon...", Toast.LENGTH_LONG).show();
    }
}

// Setting the animation on the buttons
public void setButtonsAnimation(ImageView title, ImageView search, ImageView support, ImageView about){
    // Title animation (two animations - scale and translate)
    AnimationSet animSet = new AnimationSet(true);

    Animation anim = new ScaleAnimation(0.0f, 1.0f, 0.0f, 1.0f);
    anim.setDuration(750);
    animSet.addAnimation(anim);

    anim = new TranslateAnimation(Animation.RELATIVE_TO_SELF, 0.0f, Animation.RELATIVE_TO_SELF, 0.0f,
            Animation.RELATIVE_TO_SELF, -1.0f, Animation.RELATIVE_TO_SELF, 0.0f);
    anim.setDuration(750);
    animSet.addAnimation(anim);

    title.startAnimation(animSet);

    // Search button animation
    anim = new TranslateAnimation(Animation.RELATIVE_TO_SELF, -1.5f, Animation.RELATIVE_TO_SELF, 0.0f,
            Animation.RELATIVE_TO_SELF, 0.0f, Animation.RELATIVE_TO_SELF, 0.0f);
    anim.setDuration(750);

    search.startAnimation(anim);

    // Support button animation
    anim = new TranslateAnimation(Animation.RELATIVE_TO_SELF, 1.5f, Animation.RELATIVE_TO_SELF, 0.0f,
            Animation.RELATIVE_TO_SELF, 0.0f, Animation.RELATIVE_TO_SELF, 0.0f);
    anim.setDuration(750);

    support.startAnimation(anim);

    // About button animation
    anim = new TranslateAnimation(Animation.RELATIVE_TO_SELF, 0.0f, Animation.RELATIVE_TO_SELF, 0.0f,
            Animation.RELATIVE_TO_SELF, 3f, Animation.RELATIVE_TO_SELF, 0.0f);
    anim.setDuration(750);

    about.startAnimation(anim);
}
}

答案 6 :(得分:1)

就是这么简单:

public class FancyButton
 extends FrameLayout
 implements View.OnClickListener { ..

    void yourSetupFunction(Context context, @Nullable AttributeSet attrs) {
        ..
        super.setOnClickListener(this); // NOTE THE SUPER
    }

    OnClickListener consumerListener = null;
    @Override
    public void setOnClickListener(@Nullable OnClickListener l) {
        consumerListener = l;
        // DO NOT CALL SUPER HERE
    }

    @Override
    public void onClick(View v) {
        Log.i("dev","perform my custom functions, and then ...");
        if (consumerListener != null) { consumerListener.onClick(v); }
    }
  1. 我们实现了 View.OnClickListener,因此有一个
  2. onClick(View v)
  3. 在setOnClickListener中,记住外界设置的监听器
  4. 将实际侦听器设置为我们(使用“超级”...)
  5. 在 onClick 中自定义东西,然后调用外界的 onClick

答案 7 :(得分:0)

我有同样的问题。在我的情况下,我将LinearLayout作为自定义视图的根元素,其中clickablefocusable设置为true,并且自定义视图标签本身(用于片段的布局)也设置为clickablefocusable。原来,我要做的唯一一件事就是从XML内删除所有clickablefocusable属性:)违反直觉的,但是行得通。

答案 8 :(得分:0)

只需将callOnClick()添加到您的onTouchEvent()方法中

public boolean onTouchEvent(MotionEvent event) {
        .....YOURCODE.....
        callOnClick();
        return boolean;
    }

答案 9 :(得分:0)

在我的情况下,我的自定义视图中有一个RelativeLayout作为父级,而使其起作用的唯一方法是在RelativeLayout中将focusableclickable设置为true自定义视图的构造函数,在扩大布局后,添加以下内容:

View view = inflate(getContext(), R.layout.layout_my_custom_view, this);

view.findViewById(R.id.theparent).setOnClickListener(new OnClickListener() {
        @Override
        public void onClick(View view) {
            performClick();
        }
    });