Extract duplicated string literals into constants in main code (java:S1192)

Offen Anfängerfreundlich
#738 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Anfängerfreundlichkeit
70/100
Issue-Typ
Refactoring
Klarheit
Größtenteils klar
Aktivitätsstatus
Aktiv
Tech-Stack
java
Bereich
tooling

Rechercherichtung

Lies CONTRIBUTING.md und prüfe die verlinkten SonarCloud-Befunde, beginnend mit den benannten Duplikaten in ExplorePage und Compute. Wähle ein Modul aus, vermerke deine Wahl in einem Kommentar und lagere geeignete wiederholte Namen oder Schlüssel in Konstanten aus; lass HTML-Fragmente unverändert, wenn eine Konstante die Lesbarkeit beeinträchtigen würde. Abgeschlossen ist die Aufgabe, wenn ein PR für dieses Modul vorliegt, das Verhalten unverändert bleibt und ausgelassene Befunde erklärt werden.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

good first issue

SonarCloud flags duplicated string literals: inspector 7 (ExplorePage), writer 5, reader 2 (Compute:55, :56). List: java:S1192 open issues.

Out of scope: fbs-gen and proto-gen (32 findings). Those are code-generator templates where a constant would hurt readability.

How: pick one module, say so in a comment here, and send one PR. Use judgment: a constant for a name or key repeated across methods is good; for fragments like "</td>" in an HTML builder it may not be, so leave those and explain on the Sonar finding. Behavior must not change. See CONTRIBUTING.md.

Vorherrschende Sprache
Java
Sterne
3
Forks
2
Ø Merge
9 Std. 19 Min.
Gemergte PRs (30 T.)
154

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus dfa1/vortex-java

Alle Issues in dfa1/vortex-java

Ähnliche Issues

Weitere Issues zu Java

Neue Issues direkt in Ihr Postfach

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