RecyclerView.Adapterで同じpositionのgetItemViewTypeが複数回呼ばれる

Question

前提・実現したいこと

androidで、RecyclerViewを使用したリスト表示のプログラムを作っています。

発生している問題・エラーメッセージ

リストの一部行を、ランダム表示にしたいのですが、スクロールすると、ランダム
表示の場所が何度も更新ががかかってしまいます。

具体的には、スクロールしたさいに、getItemViewTypeが呼ばれるまではいいのですが、
同じpositionのものを大量(55回連続)で呼んでいます。

getItemViewType: position:8, type : 1
getItemViewType: position:8, type : 5
getItemViewType: position:8, type : 1
getItemViewType: position:8, type : 4
・
・
・
getItemViewType: position:8, type : 1

getItemViewTypeを同じpositionのものをするロール毎に1回だけ呼ぶように
できないでしょうか？

該当のソースコード

class dataAdapter(private val fragment: Fragment)
    : RecyclerView.Adapter<RecyclerView.ViewHolder>() {
    init {
    }

    override fun getItemViewType(position: Int): Int {
        val type:Int = if(8 != position%9 ){
            -1
        }else {
            Random(SystemClock.currentThreadTimeMillis()).nextInt(4)
        }
        return type
    }

    override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): RecyclerView.ViewHolder {
        Util.log.d("onCreateViewHolder")
        val itemView: View
        val viewHolder: RecyclerView.ViewHolder

        // レイアウト設定
        when (viewType) {
            0 -> {
                itemView = LayoutInflater.from(parent.context).inflate(
                        R.layout.item_0, parent, false)
                viewHolder = ZeroViewHolder(fragment, itemView)
            }
            1 -> {
                itemView = LayoutInflater.from(parent.context).inflate(
                        R.layout.item_1, parent, false)
                viewHolder = OneViewHolder(fragment, itemView)
            }
            2 -> {
                itemView = LayoutInflater.from(parent.context).inflate(
                        R.layout.item_2, parent, false)
                viewHolder = TwoViewHolder(fragment, itemView)
            }
            3 -> {
                itemView = LayoutInflater.from(parent.context).inflate(
                        R.layout.item_3, parent, false)
                viewHolder = ThreeViewHolder(fragment, itemView)
            }
            4 -> {
                itemView = LayoutInflater.from(parent.context).inflate(
                        R.layout.4, parent, false)
                viewHolder = fourViewHolder(fragment, itemView)
            }fragment
            else -> {
                itemView = LayoutInflater.from(parent.context).inflate(
                        R.layout.item_other, parent, false)
                viewHolder = OtherViewHolder(fragment, itemView)
            }
        }
        return viewHolder
    }

    override fun onBindViewHolder(viewHolder: RecyclerView.ViewHolder, position: Int) {
        (viewHolder as ViewHolderInterface).onBindViewHolder(position)
    }

    override fun getItemCount(): Int {
        return this.rssList.size
    }
}

補足情報（FW/ツールのバージョンなど）

android 8.0
kotlin 1.3.50
android studio

Comments

[jimbe]

getItemViewType だけでなく, Adapter 全体をご提示頂くほうが良いように思います.

[tamago83]

ありがとうございます。
追記させて頂きました。

[jimbe]

Adapter を見た感じでは, そのような動作になるような原因は無さそうです.
といいますか, getItemViewType が何回呼ばれても, 最終的なランダム表示が実現されていれば大丈夫なようにも見えますが, 結果の表示の方は如何なのでしょうか.
「ランダムさ」が思っていたのとは違うということでしたら, getItemViewType の呼ばれる回数を気にされるよりも, 思われている「ランダムさ」を実現する方法を考えるほうが良いかと思いますが.

Answer 1

getItemViewTypeを同じpositionのものをするロール毎に1回だけ呼ぶように

できないでしょうか？

できません、基本的には AndroidSdk 側からいつ/何度呼ばれても良いように実装する必要があります。

やりたいことはおそらく以下のような感じかとmm

class dataAdapter(private val fragment: Fragment)
    : RecyclerView.Adapter<RecyclerView.ViewHolder>() {

    override fun getItemViewType(position: Int): Int {
        if(position == 0){
          return 0
        } else if (position == 1) {
          return 1
        } else if (position == 2) {
          return 2
        } else if (position == 3) {
          return 3
        } else if (position == 4) {
          return 4
        } else if (position % 9 == 8) { 
          return this.rssList[position].viewType // rssListの要素にviewTypeに予めランダムな値を設定しておく
        } else { 
          return -1   
        }
    }
    // ...
}

class RssListItem {
  val viewType = Random(SystemClock.currentThreadTimeMillis()).nextInt(4)
  // ...
}