anyone knows why the lambda listener for `setOnClickLIstener` doesn’t works on point 1, but works in...

# codereview

[miqbaldc]

anyone knows why the lambda listener for setOnClickLIstener doesn’t works on point 1, but works in point 2,

DebouncingOnClickListener is an interface that override View.OnClickListener#onClick(View) to prevent spamming click

see this code below:

// 1. this doesn't works
fun View.setDebouncingOnClickListener(onSafeClick: (View) -> Unit) {
    setOnClickListener(DebouncingOnClickListener {
        onSafeClick(it)
    })
}

// 2. this works
fun View.setDebouncingOnClickListener(onSafeClick: (View) -> Unit) {
    val listener = DebouncingOnClickListener {
        onSafeClick(it)
    }
    setOnClickListener(listener)
}

[arekolek]

for me it works, I just had to add a ) that was missing from setOnClickListener after the }

[arekolek]

btw, I think you could also do

setOnClickListener(DebouncingOnClickListener(onSafeClick))

[miqbaldc]

*sorry, typo for this

[arekolek]

so what do you mean by "doesn't work" exactly?

[arekolek]

it works fine as far as I can see

[miqbaldc]

“doesn’t work” my onClick method doesn’t overrided by DebouncingClickListener, unless I choose option 2, here’s my code DebouncingClickListener interface (written in java):

public interface DebouncingOnClickListener extends View.OnClickListener {

    Clicked clicked = new Clicked(0);

    void doClick(View v);

    @Override
    default void onClick(View v) {
        if (!clicked.get()) {
            clicked.set(SystemClock.elapsedRealtime());
            doClick(v);
        }
    }
}

// when I tried to invoke using options 1
viewInKotlin.setDeouncingClickListener {
  // pressed twice, will show toast twice
}