Develop Reference

how to assign a condition to an imageView by using if else?

I tried to show a random imageView out of 2(one,two) with this
binding.imageView.setImageResource(oneandtwo[random.nextInt(oneandtwo.size)]) it works fine
and
i wanted to increase score when i clicked on imageView
but score increases independent to that, sometimes increases when i clicked on imageView2 and sometimes imageView, i want to increase score when i only clicked on imageView. i couldnt figure out. Thanks in advance.
var score = 0
val oneandtwo: IntArray = intArrayOf(
R.drawable.ic_baseline_looks_one_24,
R.drawable.ic_baseline_looks_two_24
)
binding.imageView.setOnClickListener{
val random = Random
binding.imageView.setImageResource(oneandtwo[random.nextInt(oneandtwo.size)])
if (oneandtwo[random.nextInt(oneandtwo.size)]==(R.drawable.ic_baseline_looks_one_24)){
score++
binding.textView.text = score.toString()
}
}

The number you gave in the image and the number you checked in the if block may not match and will not give the result you want. If you change code like this. Probably your problem will be solved.
binding.imageView.setOnClickListener{
val random = Random().nextInt(oneandtwo.size)
binding.imageView.setImageResource(oneandtwo[random])
if (oneandtwo[random]==(R.drawable.ic_baseline_looks_one_24)){
score++
binding.textView.text = score.toString()
}
}

What you are doing is checking the resourceId of the newly generated image, not the one you just clicked. That's why it not giving the result you want ,i.e, increment on the click of imageView and not on click of imageView2. Try below code. it should work
var score = 0
val oneandtwo: IntArray = intArrayOf(R.drawable.ic_baseline_looks_one_24,R.drawable.ic_baseline_looks_two_24)
/*Initalize the initial image and tag either here or in xml file*/
val random = Random().nextInt(oneandtwo.size)
binding.imageView.setImageResource(oneandtwo[random])
binding.imageView.Tag = oneandtwo[random]
binding.imageView.setOnClickListener{
val imageTag = binding.imageView.Tag
if (imageTag == (R.drawable.ic_baseline_looks_one_24)) {
score++
binding.textView.text = score.toString()
}
val random = Random().nextInt(oneandtwo.size)
binding.imageView.setImageResource(oneandtwo[random])
binding.imageView.Tag = oneandtwo[random]
}

You've got two problems that both the answers cover - if clicking a particular image is meant to give you points, you have to check the image before you change it. And if you're using random items, you need to pick one and keep a reference to it. This:
binding.imageView.setImageResource(oneandtwo[random.nextInt(oneandtwo.size)])
if (oneandtwo[random.nextInt(oneandtwo.size)]==(R.drawable.ic_baseline_looks_one_24))
picks two completely independent numbers which may not match - and they're supposed to be referencing the same item, right? Get your random thing once, use it twice
RahulK's answer should work but here's another way you could do it, with an explicit listener object so you can throw a state variable in there:
binding.imageView.setOnClickListener(object : View.OnClickListener {
// keep track of whether the current image adds to the score when clicked
var givesPoints = false
override fun onClick(view: View) {
// first, we just got clicked, so add to the score if appropriate
if (givesPoints) score++
// you can just call random() on a collection to get a random element from it
val resId = oneAndTwo.random()
// set the image - might be better to do (view as ImageView).setImageResource
// so it sets it on -whatever was clicked- so it's easier to reuse
binding.imageView.setImageResource(resId)
// now set whether this new image gives points or not
givesPoints = resId == R.drawable.ic_baseline_looks_one_24
}
})
So this way, every time you set a new image, the listener knows whether to give points for it next time it's clicked
I don't know how you have this set up, you're only initialising things when the image is clicked so if you need to set them up beforehand (so you can have an image displayed that you an click for points) you probably want everything in a separate function you can call when clicked and during setup:
/** Assigns a random picture to this ImageView - returns true if it's a point-scoring pic */
fun assignRandomPic(imageView: Imageview): Boolean {
val resId = oneAndTwo.random()
imageView.setImageResource(resId)
return resId == R.drawable.ic_baseline_looks_one_24
}
// set an initial image, storing whether it scores points
val scoreMe = assignRandomPic(binding.imageView)
binding.imageView.setOnClickListener(object : View.OnClickListener {
// initialise this as appropriate for the image we just set up
var givesPoints = scoreMe
override fun onClick(view: View) {
if (givesPoints) score++
// set a new pic and store its point-scoring state
givesPoints = assignRandomPic(view as ImageView)
}
})
or you could just do var givesPoints = assignRandomPic(binding.imageView) and init the image inside the click listener, whatever feels better

Anko ListItem setOnClickListener

I'm trying to play around with some Kotlin and Anko (more familiar with iOS) and taking from their example, there is this code:
internal open class TextListWithCheckboxItem(val text: String = "") : ListItem {
protected inline fun createTextView(ui: AnkoContext<ListItemAdapter>, init: TextView.() -> Unit) = ui.apply {
textView {
id = android.R.id.text1
text = "Text list item" // default text (for the preview)
isClickable = true
setOnClickListener {
Log.d("test", "message")
}
init()
}
checkBox {
id = View.generateViewId()
setOnClickListener {
Log.d("hi", "bye")
}
init()
}
}.view
My row appears how I want with a checkbox and textview. But I want to bind an action to the row selection not the checkbox selection. Putting a log message in both, I see that I get a log message when the row is selected which flips the checkbox. It does not, however, log my "test:message" from the textView click handler. Is there a way to get around this?

Apparently your issue has been addressed here. As the checkbox is consuming all the focus of ListItem you should set the CheckBox's focusable flag to false:
checkBox {
focusable = View.NOT_FOCUSABLE
}
Unfortunately setFocusable call requires at least API 26, but you could define view .xml and inflate the view manually as described here:
<CheckBox
...
android:focusable="false" />
Alternatively you could try setting a onTouchListener returning false which means the touch event will be passed to underlying views.
Let me know if it works ;)

Custom animation triggers twice

I have a listener that listens and makes focused items little bit bigger with animation.
private fun focus() {
itemView?.setOnFocusChangeListener { _, hasFocus ->
if (hasFocus) {
val anim : Animation = AnimationUtils.loadAnimation(itemView.context, R.anim.scale_in)
itemView.startAnimation(anim)
anim.fillAfter = true
} else {
val anim : Animation = AnimationUtils.loadAnimation(itemView.context, R.anim.scale_out)
itemView.startAnimation(anim)
anim.fillAfter = true
}
}
}
Besides this listener I also made custom function, that when focused item is clicked, it actually changes size back to normal
fun customFunction(): Unit = with(itemView) {
val anim : Animation = AnimationUtils.loadAnimation(itemView.context, R.anim.scale_out)
itemView.startAnimation(anim)
anim.fillAfter = true
}
PROBLEM: focus() and customFunction() functions work alright. Problem is, when I hit enter on focused element (customFunction() triggers) and element changes it size to normal - which is okay. But the moment I navigate to other element, the previous one scales out twice. How do I need to modify my onFocusListener to know that I shouldn't scale out twice if I've triggered customFunction() by clicking some item. Any idea is welcomed.

if i understand right your question you may have to try : itemView. setOnFocusChangeListener(null); in your customFunction()

Visibility Not working with Kotlin

Hi the Given below is my Code, where my button should become INVISIBLE but the INVISIBLE is not working
fun onPlay(view: View){
var play = findViewById(R.id.play) as Button
play.isClickable=false
play.visibility=view.INVISIBLE
}

You have a mistake in your code, visibility constant should be set from Class variable, not from argument variable. Change view.INVISIBLE by View.INVISIBLE
fun onPlay(view: View){
var play = findViewById(R.id.play) as Button
play.isClickable=false
play.visibility= View.INVISIBLE // v letter should be capital
}

use play.visibility=View.VISIBLE to visible and play.visibility=View.GONE to invisible or to hide

Android Scenes & Transitions, logic doesn't work after first scene

I have an AuthorizationActivity with 3 scenes, which are simply different number of fields, so when user comes on Activity runs first "splash scene", with 2 buttons(Sign In, Sign Up), by clicking on each user sees the same activity but different scenes. By clicking on Back button I change scene, for example if user on the Sign In, back click -> change scene to splash.
And now I have a problem, when he at first time went to Sign In, typing text(validation works,logic works, all is cool), then he decides go back, and go again to SignIn (logic is broken, validation doesn't work, things are bad)
Memory is growing, so I guess the problem is a multiple objects of scenes, because if I on SignIn and all fields filled correctly debug shows that my model is empty. I appreciate any help)
go SignUp
fun goSignUpTransition(v: View) {
TransitionManager.beginDelayedTransition(rootContainer_AS, changeBounds)
back.isClickable = true
splashSceneFields.sceneRoot.visibility = View.GONE
val layoutParams = innerArcContainer_SSM.layoutParams
layoutParams.height = signUpHeaderHeight
innerArcContainer_SSM.layoutParams = layoutParams
val pp = localRoot.layoutParams as LinearLayout.LayoutParams
pp.topMargin = signUpMargin
this.signUpContainer.visibility = View.VISIBLE
this.signUpContainer.animate().alpha(1f).start()
this.wellaLogo_SA.visibility = View.GONE
this.wellaLogo_SA.animate().alpha(0f).start()
this.title_SSM.text = getString(R.string.sign_up_text)
val alpha = this.toolbarControllers_SSM.animate().alpha(1f)
alpha.duration = ANIM_DURATION
alpha.start()
TransitionManager.go(signUpScene)
splashSceneFields.sceneRoot.visibility = View.VISIBLE
}
go back
fun goSplashTransition(v: View) {
galleryBottomSheet.photoURI = null
userPhoto_SSM.setImageResource(R.drawable.ic_photo_upload_white_big)
TransitionManager.beginDelayedTransition(rootContainer_AS, changeBounds)
splashSceneFields.sceneRoot.visibility = View.GONE
val layoutParams = innerArcContainer_SSM.layoutParams
layoutParams.height = preAnimDimension
innerArcContainer_SSM.layoutParams = layoutParams
val pp = localRoot.layoutParams as LinearLayout.LayoutParams
pp.topMargin = 0
val animator = wellaLogo_SA.animate()
animator.scaleX(1f).scaleY(1f).duration = ANIM_DURATION
animator.start()
val alpha = this.toolbarControllers_SSM.animate().alpha(0f)
alpha.duration = ANIM_DURATION
alpha.start()
this.signUpContainer.animate().alpha(0f).start()
this.signUpContainer.visibility = View.GONE
this.wellaLogo_SA.visibility = View.VISIBLE
this.wellaLogo_SA.animate().alpha(1f).start()
TransitionManager.go(splashSceneFields)
back.isClickable = false
splashSceneFields.sceneRoot.visibility = View.VISIBLE
}

Found solution by myself, every time when scenes comes, I had to find all my views again, set listeners and so on. Scene has a setEnterAction where you'd like to bind UI or update your model(this I supposed to do) and setExitAction if you have something you have to release/clear/update, do it there.