Android Button listener implementation

Question

Hey guys i'm new in Android's dev and for a project, I have to implement a listener for a button. But unfortunately, he can't detect the button I think. Here is the java code :

public class Touch extends AppCompatActivity implements OnTouchListener,OnClickListener {


private button boutonCompteur = null;

@Override
public void onCreate(Bundle savedInstanceState)
{

    super.onCreate(savedInstanceState);
    setContentView(R.id.Compteur);
    boutonCompteur = (button) findViewById(R.id.Compteur);
    boutonCompteur.setOnTouchListener(this);
    boutonCompteur.setOnClickListener(this);

}

@Override
public boolean onTouch(View v, MotionEvent event)
{
    return true;
} }

and here is the XML :

<Button
    android:layout_width="wrap_content"
    android:layout_height="wrap_content"
    android:text="appuyez ici pour compter votre nombre de touch"
    android:id="@+id/Compteur"
    android:height="130dp"
    android:textColor="#1818e3"
    android:textColorHighlight="#cd5555"
    android:textSize="22dp"
    android:textStyle="italic"
    android:layout_marginTop="150dp"
    android:layout_alignParentStart="true" />

Thanks for your answers :)

you don't need to add both onTouch and onClick to the button. just use onClick — kandroidj
– kandroidj, Commented Feb 25, 2016 at 18:19
Try this: stackoverflow.com/questions/25803727/… (setOnClickListner); or this, from the official docs: developer.android.com/reference/android/widget/Button.html — TheLettuceMaster
– TheLettuceMaster, Commented Feb 25, 2016 at 18:19
Thanks for your answers, it was really helpful ! I learnt things thanks to you guys :) — Matnako
– Matnako, Commented Feb 25, 2016 at 19:49

Community · Accepted Answer · 2020-06-20 09:12:55Z

2

I am going to use some visual help to answer your question and identify why is not working... take a look at the image below and note:

You need a layout for your activity and this is not the same as the button
You need to find the button in the layout, that is odne by the id you used in the xml file, and is not the same as the Layout!!
you need to set the rigth listener to get the onclick, android has for that the OnClickListener interface
the word button between parenthesis is a casting, (android will try to convert something to a class, therefore the class is the class button, and not the name of your variable)
all the code that you write inside the onClick is what will be executed once the button is pressed.

Conclusion

Take the image as a reference and implement it in your code.

edited Jun 20, 2020 at 9:12

CommunityBot

11 silver badge

answered Feb 25, 2016 at 18:29

ΦXocę 웃 Пepeúpa ツ

48.4k17 gold badges77 silver badges102 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Matnako Over a year ago

Thanks for your answer, it was really helpful and i learnt things thanks to you :)

John Joe · Accepted Answer · 2016-02-25 18:30:00Z

0

Remove onTouch , use onClick only.

Remove this line

boutonCompteur.setOnTouchListener(this);

and change public boolean onTouch(View v, MotionEvent event) to

 public void onClick(View v) {
           // Do whatever you want
        }

And also change private button boutonCompteur = null; to private Button boutonCompteur; . (I don't think you need to add =null when doing button declaration).

edited Feb 25, 2016 at 18:30

answered Feb 25, 2016 at 18:22

John Joe

12.8k19 gold badges80 silver badges142 bronze badges

1 Comment

Matnako Over a year ago

thanks for your answer. I haven't read the whole android courses so i didn't know that i just needed onClick. Anyway, thanks again for a so fast answer :D

Collectives™ on Stack Overflow

Android Button listener implementation

2 Answers 2

Conclusion

1 Comment

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Conclusion

1 Comment

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related