Al hacer Scroll
en un RecyclerView, las vistas se reciclan (dependendo del itemViewCacheSize
definido).
El RecyclerView, define el siguiente ciclo para reciclar sus vistas:
Cuando una vista se desplaza fuera de la pantalla y ya no se muestra, se
convierte en una vista de desecho (Scrap View).
La vista de desecho (Scrap View) se coloca en un grupo y se
convierte en una vista de reciclaje (Recycle View). Este conjunto
es un caché de vistas que muestra el mismo tipo de datos.
Cuando se muestra un nuevo elemento, se toma una vista del grupo de
reciclaje para su reutilización. Debido a que el adaptador debe volver
a unir esta vista antes de mostrarse, se denomina vista sucia (Dirty
View).
La vista sucia (Dirty View) se recicla: el adaptador ubica los
datos para el siguiente elemento que se va a mostrar y copia estos
datos en las vistas de este artículo. Las referencias de estas
vistas se obtienen del titular de la vista asociado con la vista
reciclada.
La vista reciclada se agrega a la lista de elementos en el
RecyclerView que están a punto de aparecer en la pantalla.
La vista reciclada aparece en la pantalla mientras el usuario desplaza
el RecyclerView al siguiente elemento de la lista. Mientras
tanto, otra vista se desplaza fuera de la vista y se recicla de
acuerdo con los pasos anteriores.
En pocas palabras, el RecyclerView
amontona las vistas que desaparecen y toma esas vistas para formar las que van apareciendo nuevas, por lo que las vistas que tengan propiedades alteradas, debes volverla a su estado original, ya que si no lo haces asi, las vistas que se crearan con base a la reciclada (las que no estan visibles), se les asignara propiedades que quizás no quieres, digase, visibility, backgrounds, sizes, animaciones, etc...
Dicho esto, puedes estar cometiendo un error importante:
No defines el caso de ruptura o reestablecimiento de la vista.
Segun veo tu código, siempre estas asignando una propiedad a un elemento y no estas removiéndola, por ende, al hacer Scroll
, siempre se mostrara una vista reciclada (que tiene o puede tener propiedades de quien la creo).
La solucion aqui, es definir todos los casos posibles con una condicion, si quieres que tome cada caso como quieras. Por ejemplo, si quieres que una vista cambie de color, dependiendo de la posicion, debes hacer todos los casos y el caso de ruptura, por ejemplo:
if(position == 1)
// Si la posicion es 1, debe ser rojo
else if(position == 2)
// Si la posicion es dos, debe ser azul
else
// todos los demas deben ser negros, este es el caso de ruptura o reestablecimiento
Si no defines un caso de ruptura, notaras que al hacer scroll puede ser que se vea rojo o azul donde no quieres, porque la vista que se creo, se ha recreado con una vista reciclada, que a lo mejor era roja o azul.
En tu caso deberías hacer lo siguiente para llegar a la solución:
if(alerta.getUrl() != null) {
// Tu codigo tal como esta
}
else
// Volver mTvAlerta a su color original y remover el onClickListener