Extract method refactoring leads to inconsistent code behavior.

[provide a description of the issue]

Environment

• Operating System: Windows11
• JDK version: 17
• Visual Studio Code version: 1.88
• Java extension version: 1.29

Steps to Reproduce:

1. select "return;"
2. click “Refacator-Extract to method”

public class OriginalClass {
public void foo() {
Object runnable= null;
Object[] disposeList= null;
for (int i=0; i < disposeList.length; i++) {
if (disposeList [i] == null) {
disposeList [i] = runnable;
// extract method
return;
}
}
}
}

refactoring result:

Reason: The extraction method does not contain a return statement. If inline refactoring is performed on the extraction method, the behavior of the statement before and after refactoring is inconsistent.

Expected behavior: Check for statements like return, provide a prompt before refactoring, and do not allow extraction method refactoring.

提供されたOriginalClassの例と、指定されたVS CodeおよびJDKのバージョンを使用して、Java拡張機能でこのケースを再現します。Extract to methodリファクタリングのエントリポイントを追跡します。完了の条件は、リファクタリングがreturn文を検出し、必要に応じてプロンプトを表示し、安全でない抽出を許可しないことです。