Optimized the exception message of Fragment

DylanCaiCoding · DylanCaiCoding · commit 83a27bb1ac7b · 2022-01-04T23:10:51.000+08:00
diff --git a/viewbinding-ktx/src/main/java/com/dylanc/viewbinding/Fragment.kt b/viewbinding-ktx/src/main/java/com/dylanc/viewbinding/Fragment.kt
@@ -41,7 +41,7 @@ class FragmentBindingProperty<VB : ViewBinding>(private val clazz: Class<VB>) :
         if (binding is ViewDataBinding) binding.lifecycleOwner = thisRef.viewLifecycleOwner
       }
     } catch (e: IllegalStateException) {
-      throw IllegalStateException("The binding property has been destroyed.")
+      throw IllegalStateException("The property of ${property.name} has been destroyed.")
     }
 }
 
@@ -55,7 +55,7 @@ class FragmentInflateBindingProperty<VB : ViewBinding>(private val clazz: Class<
         binding = (clazz.getMethod("inflate", LayoutInflater::class.java).invoke(null, thisRef.layoutInflater) as VB)
           .also { binding -> if (binding is ViewDataBinding) binding.lifecycleOwner = thisRef.viewLifecycleOwner }
       } catch (e: IllegalStateException) {
-        throw IllegalStateException("The binding property has been destroyed.")
+        throw IllegalStateException("The property of ${property.name} has been destroyed.")
       }
       thisRef.viewLifecycleOwner.lifecycle.addObserver(object : DefaultLifecycleObserver {
         override fun onDestroy(owner: LifecycleOwner) {
diff --git a/viewbinding-ktx/src/main/java/com/dylanc/viewbinding/ViewBinding.kt b/viewbinding-ktx/src/main/java/com/dylanc/viewbinding/ViewBinding.kt
@@ -19,7 +19,6 @@
 package com.dylanc.viewbinding
 
 import android.view.LayoutInflater
-import android.view.View
 import android.view.ViewGroup
 import androidx.viewbinding.ViewBinding
 
diff --git a/viewbinding-nonreflection-ktx/src/main/java/com/dylanc/viewbinding/nonreflection/Fragment.kt b/viewbinding-nonreflection-ktx/src/main/java/com/dylanc/viewbinding/nonreflection/Fragment.kt
@@ -39,7 +39,7 @@ class FragmentBindingDelegate<VB : ViewBinding>(private val bind: (View) -> VB)
         if (binding is ViewDataBinding) binding.lifecycleOwner = thisRef.viewLifecycleOwner
       }
     } catch (e: IllegalStateException) {
-      throw IllegalStateException("The binding property has been destroyed.")
+      throw IllegalStateException("The property of ${property.name} has been destroyed.")
     }
     return binding
   }
@@ -55,7 +55,7 @@ class FragmentInflateBindingDelegate<VB : ViewBinding>(private val inflate: (Lay
           if (binding is ViewDataBinding) binding.lifecycleOwner = thisRef.viewLifecycleOwner
         }
       } catch (e: IllegalStateException) {
-        throw IllegalStateException("The binding property has been destroyed.")
+        throw IllegalStateException("The property of ${property.name} has been destroyed.")
       }
       thisRef.viewLifecycleOwner.lifecycle.addObserver(object : DefaultLifecycleObserver {
         override fun onDestroy(owner: LifecycleOwner) {

Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,7 @@ class FragmentBindingProperty<VB : ViewBinding>(private val clazz: Class<VB>) :`
`41`	`41`	`if (binding is ViewDataBinding) binding.lifecycleOwner = thisRef.viewLifecycleOwner`
`42`	`42`	`}`
`43`	`43`	`} catch (e: IllegalStateException) {`
`44`		`- throw IllegalStateException("The binding property has been destroyed.")`
	`44`	`+ throw IllegalStateException("The property of ${property.name} has been destroyed.")`
`45`	`45`	`}`
`46`	`46`	`}`
`47`	`47`
`@@ -55,7 +55,7 @@ class FragmentInflateBindingProperty<VB : ViewBinding>(private val clazz: Class<`
`55`	`55`	`binding = (clazz.getMethod("inflate", LayoutInflater::class.java).invoke(null, thisRef.layoutInflater) as VB)`
`56`	`56`	`.also { binding -> if (binding is ViewDataBinding) binding.lifecycleOwner = thisRef.viewLifecycleOwner }`
`57`	`57`	`} catch (e: IllegalStateException) {`
`58`		`- throw IllegalStateException("The binding property has been destroyed.")`
	`58`	`+ throw IllegalStateException("The property of ${property.name} has been destroyed.")`
`59`	`59`	`}`
`60`	`60`	`thisRef.viewLifecycleOwner.lifecycle.addObserver(object : DefaultLifecycleObserver {`
`61`	`61`	`override fun onDestroy(owner: LifecycleOwner) {`
Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,7 @@ class FragmentBindingDelegate<VB : ViewBinding>(private val bind: (View) -> VB)`
`39`	`39`	`if (binding is ViewDataBinding) binding.lifecycleOwner = thisRef.viewLifecycleOwner`
`40`	`40`	`}`
`41`	`41`	`} catch (e: IllegalStateException) {`
`42`		`- throw IllegalStateException("The binding property has been destroyed.")`
	`42`	`+ throw IllegalStateException("The property of ${property.name} has been destroyed.")`
`43`	`43`	`}`
`44`	`44`	`return binding`
`45`	`45`	`}`
`@@ -55,7 +55,7 @@ class FragmentInflateBindingDelegate<VB : ViewBinding>(private val inflate: (Lay`
`55`	`55`	`if (binding is ViewDataBinding) binding.lifecycleOwner = thisRef.viewLifecycleOwner`
`56`	`56`	`}`
`57`	`57`	`} catch (e: IllegalStateException) {`
`58`		`- throw IllegalStateException("The binding property has been destroyed.")`
	`58`	`+ throw IllegalStateException("The property of ${property.name} has been destroyed.")`
`59`	`59`	`}`
`60`	`60`	`thisRef.viewLifecycleOwner.lifecycle.addObserver(object : DefaultLifecycleObserver {`
`61`	`61`	`override fun onDestroy(owner: LifecycleOwner) {`