redhat-developer / redhat-developer/vscode-java

Better quick fix to "Potential null pointer access": check if the variable is not null

Offen
#4,094 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

code action enhancement
Vorherrschende Sprache
TypeScript
Sterne
2.3k
Forks
546
Ø Merge
20 Std. 1 Min.
Gemergte PRs (30 T.)
11

Beschreibung

I have the following code:

String s = null;
// do conditional stuff with "s"
s.length() // "Potential null pointer access" warning

Currently VS Code offers me 2 quick fixes:

  1. "Ignore compiler problem(s)"
  2. "Add @SuppressWarnings 'null' to myMethod"

A better quick fix option would be to add a "not nulll" check before the method calling.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne damit, die bestehenden Schnellkorrekturen für die Warnung „Potential null pointer access“ und die beiden im Issue beschriebenen Optionen nachzuverfolgen. Ermittle, wie sich eine Schnellkorrektur zur Nullprüfung für das Beispiel verhalten sollte, und ergänze anschließend Tests, die zeigen, dass die neue Option angezeigt wird und den erwarteten abgesicherten Aufruf erzeugt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
java, typescript
Bereich
tooling
Issue-Typ
Feature
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.