adapterについて教えてください

Question

実現したいこと

kotlin初心者です。GitHubで公開されているリポジトリを検索するアプリを作成しています。recyclerView.adapterを使い、RepoAdapter(repos)でreposを引数として渡したいのですがRepoAdapter側の記述の仕方が分かりません。ご教授いただけると幸いです。

発生している問題・分からないこと

引数にToo many arguments for public constructor RepoAdapter() defined in com.example.yourprojectname.RepoAdapterと表示されてしまう

エラーメッセージ

Too many arguments for public constructor RepoAdapter() defined in com.example.yourprojectname.RepoAdapter

該当のソースコード

import android.content.Intent
import android.os.Bundle
import android.widget.Button
import android.widget.EditText
import android.widget.Toast
import androidx.appcompat.app.AppCompatActivity
import androidx.recyclerview.widget.LinearLayoutManager
import androidx.recyclerview.widget.RecyclerView
import com.example.github.GitHubService
import com.example.github.R
import com.example.github.RepoSearchResponse
import com.example.yourprojectname.RepoAdapter
import retrofit2.Call
import retrofit2.Callback
import retrofit2.Response
import retrofit2.Retrofit
import retrofit2.converter.gson.GsonConverterFactory

class MainActivity : AppCompatActivity() {
    val searchButton: Button = findViewById(R.id.search_button)
    val searchEditText: EditText = findViewById(R.id.search_edit_text)
    val recyclerView: RecyclerView = findViewById(R.id.recycler_view)
    private val retrofit = Retrofit.Builder()
        .baseUrl("https://api.github.com/")
        .addConverterFactory(GsonConverterFactory.create())
        .build()


    private val service = retrofit.create(GitHubService::class.java)

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




        searchButton.setOnClickListener {
            val query = searchEditText.text.toString()
            service.searchRepositories(query).enqueue(object : Callback<RepoSearchResponse> {
                override fun onResponse(call: Call<RepoSearchResponse>, response: Response<RepoSearchResponse>) {
                    val repos = response.body()?.items.orEmpty()
                    recyclerView.adapter = RepoAdapter(repos) { repos ->
                        val intent = Intent(this@MainActivity, DetailActivity::class.java)
                        intent.putExtra("url", repos)
                        startActivity(intent)
                    }

                    recyclerView.layoutManager = LinearLayoutManager(this@MainActivity)
                }

                override fun onFailure(call: Call<RepoSearchResponse>, t: Throwable) {
                    Toast.makeText(this@MainActivity, "Error: ${t.message}", Toast.LENGTH_SHORT).show()
                }
            })
        }
    }
}

class DetailActivity : AppCompatActivity() {
    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContentView(R.layout.activity_detail)

        val url = intent.getStringExtra("url")
        webView.loadUrl(url)
    }
}

package com.example.yourprojectname

import android.view.LayoutInflater
import android.view.View
import android.view.ViewGroup
import android.widget.TextView
import androidx.paging.PagingDataAdapter
import androidx.recyclerview.widget.DiffUtil
import androidx.recyclerview.widget.RecyclerView
import com.example.github.R

class RepoAdapter(private val repos: List<Repo>) : PagingDataAdapter<Repo, RepoAdapter.RepoViewHolder>(DIFF_CALLBACK) {

    override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): RepoViewHolder {
        val view = LayoutInflater.from(parent.context).inflate(R.layout.repo_item, parent, false)
        return RepoViewHolder(view)
    }

    override fun onBindViewHolder(holder: RepoViewHolder, position: Int) {
        val repo = getItem(position)
        if (repo != null) {
            holder.nameTextView.text = repo.name
            holder.descriptionTextView.text = repo.description
        }
    }

    class RepoViewHolder(view: View) : RecyclerView.ViewHolder(view) {
        val nameTextView: TextView = view.findViewById(R.id.repo_name)
        val descriptionTextView: TextView = view.findViewById(R.id.repo_description)
    }

    companion object {
        val DIFF_CALLBACK = object : DiffUtil.ItemCallback<Repo>() {
            override fun areItemsTheSame(oldItem: Repo, newItem: Repo): Boolean {
                return oldItem.id == newItem.id
            }

            override fun areContentsTheSame(oldItem: Repo, newItem: Repo): Boolean {
                return oldItem == newItem
            }
        }
    }
}

試したこと・調べたこと

• teratailやGoogle等で検索した
• ソースコードを自分なりに変更した
• 知人に聞いた
• その他

上記の詳細・結果

コンストラクタに受け取る引数を設定していないことが原因だとわかりましたが記述する内容が分かりませんでした。

補足

特になし

Comments

[jimbe]

MainActivity は kotlin で書いて、アダプタは java で書いているんですか?
アダプタも kotlin で書いたほうが良いのでは。

[soruto]

ありがとうございます。kotlinに変更しました。

Answer 1

パッと見だけですが、 repos が URL ならこんなでしょうか。

  public RepoAdapter(List<URL> repos, Consumer<URL> consumer) {

あ、 java.util.function.Consumer を import してください。

---

class RepoAdapter(private val repos: List<Repo>, private val consumer: (URL) -> Unit)

Comments

[soruto]

頂いたアドバイスをもとに
class RepoAdapter(repos: List<URL>, private val onRepoClick: (URL) -> Unit) : PagingDataAdapter<Repo, RepoAdapter.RepoViewHolder>(DIFF_CALLBACK) {
に変更しました。

[jimbe]

回答の編集と入違っちゃいましたが、その変更で如何なったでしょうか。

[soruto]

引数に関しては問題なく渡せるようになりました。回答ありがとうございました。重ねて質問なのですがメインアクティビティの66行目で使用しているwebViewでUnresolved reference: webViewが表示されています。宣言するのであればどのように記述したらよいのでしょうか？一例でも教えていただけたら幸いです。

[soruto]

申し訳ありません、val webView: WebView = findViewById(R.id.webView)が抜けているだけでした。この度はご回答ありがとうございました。

[jimbe]

お疲れさまでした^^