Optimized the exception message of Fragment

DylanCaiCoding · Jan 4, 2022 · 83a27bb · 83a27bb
1 parent 8f83a68
commit 83a27bb
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 5 deletions.
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) {