如何更改此函数以摆脱冗余 SAM 构造函数？

Question

所以，我想制作 SetOnClickListener 来负责打开gallery。它可以工作，但是当我尝试将代码添加到 commit 到 github 时，我不能这样做，因为我使用了冗余 SAM-Constructor。所以我的问题是，如何更改我的代码，这样它就可以在没有它的情况下工作？

class MainActivity : AppCompatActivity() {
private var dialogView: View? = null

private val getPreviewImage = registerForActivityResult(ActivityResultContracts.GetContent(), ActivityResultCallback {
    it?.let { uri ->
        dialogView?.findViewById<ImageView>(R.id.imageChange)?.setImageURI(it)
    }?:run {
        Log.e("MainActivity", "URI not present")
    }
})

private val getPreviewVideo = registerForActivityResult(ActivityResultContracts.GetContent(), ActivityResultCallback {
it?.let { uri ->
    dialogView?.findViewById<VideoView>(R.id.videoChange)?.setVideoURI(it)
}?: run{
    Log.e("MainActivity", "URI not present")
    }
})

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

    bottomNavigationView.background = null
    bottomNavigationView.menu.findItem(R.id.placeholder).isEnabled = false
    replaceFragment(HomeFragment())

    bottomNavigationView.setOnItemSelectedListener {
        when (it.itemId) {
            R.id.home -> replaceFragment(HomeFragment())
            R.id.player -> replaceFragment(PlayerFragment())
            R.id.profile -> replaceFragment(ProfileFragment())
            R.id.settings -> replaceFragment(SettingsFragment())
        }
        true
    }

    popupAddButton.setOnClickListener {
        showDialog()
    }
}

private fun replaceFragment(fragment: Fragment) {
    val transaction = supportFragmentManager.beginTransaction()
    transaction.replace(R.id.fragment_container, fragment)
    transaction.commit()
}
@SuppressLint("InflateParams")
private fun showDialog() { //this is for popupWindow
    dialogView = layoutInflater.inflate(R.layout.popup, null)
    val dialog = Dialog(this)
    val titleEditText = dialogView?.findViewById<EditText>(R.id.titleEdit) //popUp edit field title
    val descEditText = dialogView?.findViewById<EditText>(R.id.description) //popUp edit field description

    dialogView?.addImage?.setOnClickListener {
        getPreviewImage.launch("image/*")

    }
    dialogView?.addVideo?.setOnClickListener {
        getPreviewVideo.launch("video/*")
    }

    dialogView?.addButton?.setOnClickListener {
        AppCompatDelegate.setCompatVectorFromResourcesEnabled(true)
        if (titleEditText?.text?.isEmpty() == true || descEditText?.text?.isEmpty() == true){
            Toast.makeText(applicationContext, "add required data", Toast.LENGTH_SHORT).show()
        }else{
            Toast.makeText(applicationContext, "Added", Toast.LENGTH_SHORT).show()
        }
    }
    dialog.setContentView(dialogView!!)
    dialog.show()
   }
}

Answer 1

我猜你的意思是你尝试推送的 Git 存储库的某些 lint 规则不允许冗余 SAM 构造函数。

SAM 构造函数是指您使用 Kotlin 为函数式接口提供的假“构造函数”。对于任何 Java 函数式接口 (SAM)（或 Kotlin fun 接口），Kotlin 都会创建一个看起来像构造函数的隐式内联函数。它使用该接口的名称作为函数名称，并采用与 SAM 签名匹配的函数引用参数。

例如，如果您在 Java 中定义此接口

public interface Foo {
    public int bar(String value);
}

，或者如果您在 Kotlin 中定义它，则

fun interface Foo {
    fun bar(value: String): Int
}

Kotlin 会隐式创建此函数。你在任何地方都看不到它，但你可以使用它。它只是内联的，因此不能在 Java 中或通过反射访问。

inline fun Foo(crossinline function: (value: String)->Int) = object: Foo {
    override fun bar(value: String): Int = function(value)
}

当您想要创建该接口的实例以存储在属性中时，可以使用此假构造函数。当您传递 lambda 作为函数的最后一个参数时，不需要使用此 SAM 构造函数，因为您可以直接传递 lambda，而不调用此构造函数。 Kotlin 自动知道要从 lambda 构建什么接口。*

因此，您的代码

private val getPreviewImage = registerForActivityResult(ActivityResultContracts.GetContent(), ActivityResultCallback {
    it?.let { uri ->
        dialogView?.findViewById<ImageView>(R.id.imageChange)?.setImageURI(it)
    }?:run {
        Log.e("MainActivity", "URI not present")
    }
})

正在使用冗余 SAM 构造函数，并且可以替换为

private val getPreviewImage = registerForActivityResult(ActivityResultContracts.GetContent()) { 
    it?.let { uri ->
        dialogView?.findViewById<ImageView>(R.id.imageChange)?.setImageURI(it)
    }?:run {
        Log.e("MainActivity", "URI not present")
    }
}

Youalsoused a Redundant SAM constructor for val getPreviewVideo.

* 一个例外是当函数重载采用不同的接口作为最后一个参数时。然后，您将需要假构造函数来区分构造函数。

---

旁注：像这样链接作用域函数是不好的做法 (?.let...? ：运行）。除了难以阅读之外，很容易意外地执行某些操作，从而导致两个块都被执行。例如，如果 URI 不为 null，但 dialogView 为 null，则您的 run 块无论如何都会执行，因为 let 的计算结果为 null。我会这样写：

private val getPreviewImage = registerForActivityResult(ActivityResultContracts.GetContent()) { uri ->
    if (uri != null) {
        dialogView?.findViewById<ImageView>(R.id.imageChange)?.setImageURI(uri)
    } else {
        Log.e("MainActivity", "URI not present")
    }
}