intentでオブジェクトを渡したい

Question

前提

Android kotlinでFirebaseのReal Time Databaseを使用して、簡易的なチャットアプリを作ってます。

実現したいこと

ユーザー間でのメッセージ送信

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

原因としては、以下のChatLogActivityのuserがnullになっているためです。

val user = intent.getParcelableExtra<User>(NewMessageActivity.USER_KEY)
val toId = user?.uid

NewMessageActivityのfetchUsersメソッドにintentを実装し、オブジェクトを渡していますが、これが上手くいってないみたいで、これ以上の原因が分からずです。

intent.putExtra(USER_KEY, holder.msgList.text)

以下はエラーメッセージです。

java.lang.NullPointerException
        at com.ChatApp.ChatLogActivity.performSendMessage(ChatLogActivity.kt:64)
        at com.ChatApp.ChatLogActivity.onCreate$lambda-0(ChatLogActivity.kt:44)

該当のソースコード

package com.ChatApp

import androidx.appcompat.app.AppCompatActivity
import android.os.Bundle
import android.util.Log
import android.widget.Button
import android.widget.EditText
import androidx.appcompat.widget.Toolbar
import androidx.recyclerview.widget.LinearLayoutManager
import androidx.recyclerview.widget.RecyclerView
import com.google.firebase.auth.FirebaseAuth
import com.google.firebase.database.FirebaseDatabase
import java.sql.Timestamp

class ChatLogActivity : AppCompatActivity() {
    private lateinit var recyclerView: RecyclerView

    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContentView(R.layout.activity_chat_log)

        val toolbar = findViewById<Toolbar>(R.id.toolbar_chat_log)
        val user = intent.getParcelableExtra<User>(NewMessageActivity.USER_KEY)
        println("user1: $user")

        setSupportActionBar(toolbar)
        getSupportActionBar()?.setTitle(user?.uid)
        getSupportActionBar()?.setDisplayHomeAsUpEnabled(true)

        val textList = listOf(
            "Hello",
            "Hello",
            "Hello",
            "Hello"
        )

        recyclerView = findViewById(R.id.chat_log_list)
        recyclerView.adapter = ChatFromRecyclerAdapter(textList)
        recyclerView.adapter = ChatToRecyclerAdapter(textList)
        recyclerView.layoutManager = LinearLayoutManager(this)

        findViewById<Button>(R.id.sendButton).setOnClickListener {
            Log.d("ChatLogActivity", "Send message...")
            performSendMessage()
        }
    }

    class ChatLog(val id: String, val text: String, val fromId: String, val toId: String, timestamp: Long)

    private fun performSendMessage() {
        val text = findViewById<EditText>(R.id.editSendText).text.toString()
        val fromId = FirebaseAuth.getInstance().uid
        val user = intent.getParcelableExtra<User>(NewMessageActivity.USER_KEY)
        val toId = user?.uid

        println("fromId: $fromId")
        println("toId: $toId")
        println("user2: $user")

        if (fromId == null) return

        val ref = FirebaseDatabase.getInstance().getReference("/messages").push()

        val chatMessage = ChatLog(ref.key!!, text, fromId, toId!!, System.currentTimeMillis() / 1000)

        ref.setValue(chatMessage)
            .addOnSuccessListener {
                Log.d("ChatLogActivity", "Saved our chatMessage: ${ref.key}")
            }
    }
}

package com.ChatApp

import android.content.Intent
import android.os.Bundle
import android.util.Log
import android.view.MenuItem
import androidx.appcompat.app.AppCompatActivity
import androidx.appcompat.widget.Toolbar
import androidx.recyclerview.widget.LinearLayoutManager
import androidx.recyclerview.widget.RecyclerView
import com.google.firebase.database.*


class NewMessageActivity : AppCompatActivity() {

    private lateinit var database: DatabaseReference
    private lateinit var recyclerView: RecyclerView
    var nameList = mutableListOf<String>(
        "テスト太郎"
    )
    var adapter = NewMessageRecyclerAdapter(nameList)

    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContentView(R.layout.activity_new_message)

        val databaseRef: DatabaseReference = FirebaseDatabase.getInstance().reference

        val toolbar = findViewById<Toolbar>(R.id.toolbar_new_message)
        setSupportActionBar(toolbar)
        getSupportActionBar()?.setTitle("友達リスト")
        getSupportActionBar()?.setDisplayHomeAsUpEnabled(true)

        fetchUsers()
    }

    companion object {
        const val USER_KEY = "USER_KEY"
    }

    private fun fetchUsers(){
        val ref = FirebaseDatabase.getInstance().getReference("/users")
        ref.addListenerForSingleValueEvent(object: ValueEventListener {

            override fun onDataChange(snapshot: DataSnapshot) {

                for(i in snapshot.children){
                    Log.d("NewMessageActivity",i.toString())

                    val userName = i.child("username").value

                    if (userName != null){
                        nameList.add(userName as String)
                    }
                }

                recyclerView = findViewById(R.id.new_message_list)
                recyclerView.layoutManager = LinearLayoutManager(this@NewMessageActivity)
                recyclerView.adapter = adapter

                adapter.itemClickListener = object: NewMessageRecyclerAdapter.OnItemClickListener {
                    override fun onItemClick(holder: NewMessageViewHolderList) {
                        val intent = Intent(this@NewMessageActivity, ChatLogActivity::class.java)
                        intent.putExtra(USER_KEY, holder.msgList.text)
                        startActivity(intent)
                    }
                }

            }

            override fun onCancelled(p0: DatabaseError) {

            }
        })
        println("fetchUsers called")
    }


    //オプションメニュー用
    //override fun onCreateOptionsMenu(menu: Menu?): Boolean {}

    override fun onOptionsItemSelected(item: MenuItem): Boolean {

        //戻るボタンの処理
        if (item.itemId == android.R.id.home) {
            finish()
        }

        if (item.itemId == R.id.action_new_message) {
            //toolbarのアイコンがタップされた時の処理
        }
        return super.onOptionsItemSelected(item)
    }


}

Comments

[hoshi-takanori]

Android OS の建前上、各 Activity は独立したメモリ空間で動くように作ることになっているため、Intent (とういか Bundle) の内容はシリアライズされるので、生きたオブジェクトは渡せません。
Intent では id などを渡して、それに基づいてオブジェクトを取得し直す必要があるかと。

[jimbe]

関係無いですが、
> recyclerView = findViewById(R.id.chat_log_list)
> recyclerView.adapter = ChatFromRecyclerAdapter(textList)
> recyclerView.adapter = ChatToRecyclerAdapter(textList)
このようなことをしても RecyclerView が2つ表示される訳ではありません。
指定できるアダプタは1つだけで、この場合は後から指定した To のほうが表示されるでしょう。

[eotw]

hoshi-takanori さん
ありがとうございます。
今回の場合だとどのように書けば良いのかイメージがつかず。。

Answer 1

過去のご質問 から holder である NewMessageViewHolderList を探しますと、 msgList は TextView です。
従いまして、 NewMessageActivity の以下のコードの msgList.text は CharSequence です。

  val intent = Intent(this@NewMessageActivity, ChatLogActivity::class.java)
  intent.putExtra(USER_KEY, holder.msgList.text)
  startActivity(intent)

一方、ChatLogActivity では USER_KEY で入っているデータを User オブジェクトとして取り出そうとしています。

  val user = intent.getParcelableExtra<User>(NewMessageActivity.USER_KEY)

型が合っていませんので null が返されているものと思います。