feat(refactoring): extract method for readable #129

Author: phodal

docs/features/refactoring.md

@@ -52,3 +52,9 @@ In [#132](https://github.com/unit-mesh/auto-dev/issues/132), we provide basic fu
 2. select the variable you want to rename use `Shift` + `F6`
 
 <img src="https://unitmesh.cc/auto-dev/autodev-rename.png" alt="Rename Functions" width="600px"/>
+
+## Resource
+
+In [How to Refactor this Code? An Exploratory Study on Developer-ChatGPT Refactoring Conversations](https://arxiv.org/abs/2402.06013) ,
+talking about how to refactoring to better understand how developers identify areas for improvement in code
+and how ChatGPT addresses developers' needs. 

src/main/kotlin/cc/unitmesh/devti/practise/RenameLookupManagerListener.kt

@@ -18,6 +18,7 @@ import com.intellij.psi.PsiNameIdentifierOwner
 import com.intellij.psi.PsiWhiteSpace
 import com.intellij.psi.impl.source.tree.LeafPsiElement
 import com.intellij.psi.util.PsiEditorUtil
+import kotlinx.coroutines.cancel
 import kotlinx.coroutines.flow.*
 import kotlinx.coroutines.launch
 
@@ -31,21 +32,7 @@ class RenameLookupManagerListener(val project: Project) : LookupManagerListener
         val lookupImpl = newLookup as? LookupImpl ?: return
         val editor = lookupImpl.editor as? EditorEx ?: return
 
-        val startOffset = lookupImpl.lookupOriginalStart
-        val psiFile = PsiEditorUtil.getPsiFile(editor)
-
-        var targetElement: PsiElement? = null
-        if (startOffset >= 0) {
-            targetElement = psiFile.findElementAt(startOffset)
-        }
-
-        if (targetElement == null) {
-            targetElement = psiFile.findElementAt(editor.caretModel.offset)
-        }
-
-        if (targetElement is LeafPsiElement || targetElement is PsiWhiteSpace) {
-            targetElement = targetElement.parent
-        }
+        val targetElement: PsiElement? = lookupElement(lookupImpl, editor)
 
         // maybe user just typing, we should handle for this
         val element = targetElement ?: return
@@ -98,6 +85,28 @@ class RenameLookupManagerListener(val project: Project) : LookupManagerListener
         stringJob.start()
     }
 
+    private fun lookupElement(
+        lookupImpl: LookupImpl,
+        editor: EditorEx
+    ): PsiElement? {
+        val startOffset = lookupImpl.lookupOriginalStart
+        val psiFile = PsiEditorUtil.getPsiFile(editor)
+
+        var targetElement: PsiElement? = null
+        if (startOffset >= 0) {
+            targetElement = psiFile.findElementAt(startOffset)
+        }
+
+        if (targetElement == null) {
+            targetElement = psiFile.findElementAt(editor.caretModel.offset)
+        }
+
+        if (targetElement is LeafPsiElement || targetElement is PsiWhiteSpace) {
+            targetElement = targetElement.parent
+        }
+        return targetElement
+    }
+
     private fun extractSuggestionsFromString(result: String) = result.split("\n").map {
         it.replace(Regex("^\\d+\\."), "")
             .trim()