#527 [fix] 카메라 권한 체크 변수에서 함수로 변경

Author: yjooooo

app/src/main/java/com/spark/android/ui/auth/profile/ProfileBottomSheet.kt

@@ -34,20 +34,6 @@ class ProfileBottomSheet : BottomSheetDialogFragment() {
     private var _binding: BottomSheetProfileBinding? = null
     val binding get() = _binding ?: error(getString(R.string.binding_error))
 
-    private val checkCameraPermission by lazy {
-        ContextCompat.checkSelfPermission(
-            requireContext(),
-            android.Manifest.permission.CAMERA
-        ) == PackageManager.PERMISSION_GRANTED
-    }
-
-    private val checkCameraPermissionUnderQ by lazy {
-        checkCameraPermission && ContextCompat.checkSelfPermission(
-            requireContext(),
-            android.Manifest.permission.WRITE_EXTERNAL_STORAGE
-        ) == PackageManager.PERMISSION_GRANTED
-    }
-
     private lateinit var imgUri: Uri
 
     private val fromAlbumActivityLauncher = registerForActivityResult(
@@ -70,7 +56,6 @@ class ProfileBottomSheet : BottomSheetDialogFragment() {
         dismiss()
     }
 
-
     override fun onCreateView(
         inflater: LayoutInflater,
         container: ViewGroup?,
@@ -122,7 +107,7 @@ class ProfileBottomSheet : BottomSheetDialogFragment() {
     private fun initFromCameraBtnClickListener() {
         binding.tvProfileBottomFromCamera.setOnClickListener {
             if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
-                if (checkCameraPermission) {
+                if (checkCameraPermission()) {
                     takePicture()
                 } else {
                     ActivityCompat.requestPermissions(
@@ -132,7 +117,7 @@ class ProfileBottomSheet : BottomSheetDialogFragment() {
                     )
                 }
             } else {
-                if (checkCameraPermissionUnderQ) {
+                if (checkCameraPermissionUnderQ()) {
                     takePicture()
                 } else {
                     ActivityCompat.requestPermissions(
@@ -148,6 +133,19 @@ class ProfileBottomSheet : BottomSheetDialogFragment() {
         }
     }
 
+    private fun checkCameraPermission() =
+        ContextCompat.checkSelfPermission(
+            requireContext(),
+            android.Manifest.permission.CAMERA
+        ) == PackageManager.PERMISSION_GRANTED
+
+
+    private fun checkCameraPermissionUnderQ() =
+        checkCameraPermission() && ContextCompat.checkSelfPermission(
+            requireContext(),
+            android.Manifest.permission.WRITE_EXTERNAL_STORAGE
+        ) == PackageManager.PERMISSION_GRANTED
+
     private fun takePicture() {
         try {
             imgUri = getImgUri(requireContext().contentResolver)

app/src/main/java/com/spark/android/ui/certify/CertifyBottomSheet.kt

@@ -35,20 +35,6 @@ class CertifyBottomSheet : BottomSheetDialogFragment() {
     private lateinit var imgUri: Uri
     private val certifyViewModel by activityViewModels<CertifyViewModel>()
 
-    private val checkCameraPermission by lazy {
-        ContextCompat.checkSelfPermission(
-            requireContext(),
-            android.Manifest.permission.CAMERA
-        ) == PackageManager.PERMISSION_GRANTED
-    }
-
-    private val checkCameraPermissionUnderQ by lazy {
-        checkCameraPermission && ContextCompat.checkSelfPermission(
-            requireContext(),
-            android.Manifest.permission.WRITE_EXTERNAL_STORAGE
-        ) == PackageManager.PERMISSION_GRANTED
-    }
-
     private val fromAlbumActivityLauncher = registerForActivityResult(
         ActivityResultContracts.StartActivityForResult()
     ) { result: ActivityResult ->
@@ -132,7 +118,7 @@ class CertifyBottomSheet : BottomSheetDialogFragment() {
     private fun initFromCameraBtnClickListener() {
         binding.tvCertifyCamera.setOnClickListener {
             if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
-                if (checkCameraPermission) {
+                if (checkCameraPermission()) {
                     takePicture()
                 } else {
                     ActivityCompat.requestPermissions(
@@ -142,7 +128,7 @@ class CertifyBottomSheet : BottomSheetDialogFragment() {
                     )
                 }
             } else {
-                if (checkCameraPermissionUnderQ) {
+                if (checkCameraPermissionUnderQ()) {
                     takePicture()
                 } else {
                     ActivityCompat.requestPermissions(
@@ -158,6 +144,19 @@ class CertifyBottomSheet : BottomSheetDialogFragment() {
         }
     }
 
+    private fun checkCameraPermission() =
+        ContextCompat.checkSelfPermission(
+            requireContext(),
+            android.Manifest.permission.CAMERA
+        ) == PackageManager.PERMISSION_GRANTED
+
+
+    private fun checkCameraPermissionUnderQ() =
+        checkCameraPermission() && ContextCompat.checkSelfPermission(
+            requireContext(),
+            android.Manifest.permission.WRITE_EXTERNAL_STORAGE
+        ) == PackageManager.PERMISSION_GRANTED
+
     private fun takePicture() {
         try {
             imgUri = getImgUri(requireContext().contentResolver)