CODEWITHSUNDEEP
javaandroidonclicklistenerontouchlistener
G.V.
G.V.

Reputation: 719

OnClickListener to be executed only once

On an image, I want to set a click listener in order to performe an action after a click.

The code below is called more than once even after one click.

How can I prevent that? (I have used either an onTouchListener but the result was the same...)

   Image1.setOnClickListener(new View.OnClickListener() {

        public void onClick(View v) {
            Log.d("DisplayImage", "on click");
            DisplayOneImage doi=new DisplayOneImage(DisplayImage.this,"Filtered image 1");
            doi.show();
        }
    });

The xml file:

<?xml version="1.0" encoding="utf-8"?>
<ScrollView xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:tools="http://schemas.android.com/tools"
    android:id="@+id/scrollView1"
    android:layout_width="match_parent"
    android:layout_height="fill_parent"

>

<LinearLayout

    android:orientation="vertical"
    android:layout_height="fill_parent"
    android:layout_width="fill_parent"
    >

    <!-- \/ \/ \/ \/ To be deleted Menu \/ \/ \/ \/ -->
    <!--include
        layout="@layout/app_bar_main"
        android:layout_width="match_parent"
        android:layout_height="match_parent" /-->
    <!-- /\ /\ /\ /\ To be deleted Menu /\ /\ /\ /\  -->

    <LinearLayout
        android:id="@+id/img_original"
        android:layout_width="fill_parent"
        android:layout_height="fill_parent"
        android:orientation="vertical"
        android:padding="10dp"
        android:layout_weight="1">


        <TextView
            android:layout_width="match_parent"
            android:layout_height="wrap_content"
            android:text="Original Image:" />

        <ImageView
            android:id="@+id/initial_photo"

            android:layout_width="match_parent"
            android:layout_height="match_parent"
            android:layout_weight="1"/>

        </ImageView>

    </LinearLayout>
</ScrollView>

Upvotes: 4

Views: 5120

Answers (4)

leonheess
leonheess

Reputation: 21361

In case someone wants an onClickListener that can only be clicked every second (1000 milliseconds):

private boolean clicked = false;

element.setOnClickListener(v -> {
    if(!clicked){
        clicked = true;
        doSomething(v.getSomething());
        new Handler().postDelayed(() -> clicked = false, 1000);
    }
});

You can of course change the 1000 to any number of milliseconds you'd like to see.

Upvotes: 4

J. Tenni&#233;
J. Tenni&#233;

Reputation: 348

Use this:

 Image1.setOnClickListener(new View.OnClickListener() {
            boolean clicked = false;
            public void onClick(View v) {
                if (!clicked) {
                    Log.d("DisplayImage", "on click");
                    DisplayOneImage doi=new DisplayOneImage(DisplayImage.this,"Filtered image 1");
                    doi.show();
                    clicked = true;
                }
            }
    });

Upvotes: 0

seekingStillness
seekingStillness

Reputation: 5093

Another, more invasive option

   Image1.setOnClickListener(new View.OnClickListener() {

    public void onClick(View v) {
        Log.d("DisplayImage", "on click");
        DisplayOneImage doi=new DisplayOneImage(DisplayImage.this,"Filtered image 1");

        doi.show();
        Image1.setVisibility(View.GONE);
    }
});

Upvotes: 0

Zain
Zain

Reputation: 2372

I'd set the OnClickListener to null inside the onClick method -as follows.

  Image1.setOnClickListener(new View.OnClickListener() {

    public void onClick(View v) {
        Log.d("DisplayImage", "on click");
        DisplayOneImage doi=new DisplayOneImage(DisplayImage.this,"Filtered image 1");
        doi.show();
        v.setOnClickListener(null);//Remove setOnClickListener
    }
});

Upvotes: 10

Related Questions