Apply var to method invocations
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Anfängerfreundlichkeit
- 35/100
Rechercherichtung
Beginnen Sie mit der Durchsicht der bestehenden grundlegenden var-Arbeit, auf die in #217 verwiesen wird, und ihrer Rezepttests. Verfolgen Sie anschließend, wie Rückgabetypen von Java-Methoden und generische Aufrufe dargestellt werden. Verwenden Sie die Vorher-Nachher-Beispiele als Akzeptanzfälle, einschließlich der angegebenen Ausschlüsse für short, byte und nicht übereinstimmende Rückgabetypen. Die Issue nennt keine Implementierungsdateien oder Testpfade; diese müssen daher zuerst ermittelt werden.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
What problem are you trying to solve?
While developing basic var usage with #217 java reveals that local variable type inference is a big field, so we decided to split.
One open area is applying var to variable declarations initialized by method invocations.
What precondition(s) should be checked before applying this recipe?
varis not applicable if
- the method has return type ´short´ or
byte - the method's return type does not match the type definition in the left-hand side
Describe the situation before applying the recipe
class A {
String getHello() {
return "Hello Rewrite";
}
void simple(String bar) {
String msg = getHello(); // (1)
System.out.println(msg);
}
void generic(String bar) {
List<String> msgs = List.of("Hello", "Rewrite"); // (2)
System.out.println(msgs);
}
}
Describe the situation after applying the recipe
class A {
String getHello() {
return "Hello Rewrite";
}
void simple(String bar) {
var msg = getHello(); // (1)
System.out.println(msg);
}
void generic(String bar) {
var msgs = List.of("Hello", "Rewrite"); // (2)
System.out.println(msgs);
}
}
Have you considered any alternatives or workarounds?
An alternative would be to no support var in combination with methods
Any additional context
It may be a good idea to implement this as an independent recipe, this would increase separation of concern and make configuration easier.
Are you interested in contributing this recipe to OpenRewrite?
Yes, but need additional support understanding generics.
- Vorherrschende Sprache
- Java
- Sterne
- 160
- Forks
- 133
- Ø Merge
- 4 Std. 45 Min.
- Gemergte PRs (30 T.)
- 108
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus openrewrite/rewrite-migrate-java
-
good first issue
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 30/100
openrewrite/rewrite-migrate-java#1330 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
UseMapOf: Skip refactoring large maps to Map.ofEntries to prevent javac compiler hangs (add maxEntries option)Evtl. vergeben @Yuren0312 hat das vor 2 Tagen übernommen. Offenbug
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 55/100
openrewrite/rewrite-migrate-java#1310 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 55/100
openrewrite/rewrite-migrate-java#1307 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 68/100
openrewrite/rewrite-migrate-java#1246 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Feedback from a 152k-line Java 8 to 25 migration: the silent behavioural changes that cost the most, and which have no recipeEvtl. vergeben Ein verknüpfter Pull Request ist offen oder bereits gemergt. Offen
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 32/100
openrewrite/rewrite-migrate-java#1227 · 4 Kommentare · 1 Reaktion ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in openrewrite/rewrite-migrate-java
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 80/100
OpenAPITools/openapi-generator#25454 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Component - Build
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 77/100
navikt/mock-oauth2-server#1149 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
oracle/graalvm-reachability-metadata#11318 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
Stirling-Tools/Stirling-PDF#8709 · 1 Reaktion ·
Maintainer antworten meist innerhalb von 1 Tag