yegor256/qulice

Constants should be on the right side of comparison

Open

#716 geöffnet am 16. Feb. 2016

Auf GitHub ansehen
 (6 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Java (122 Forks)github user discovery
bughelp wantedpostponed

Repository-Metriken

Stars
 (320 Stars)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

When comparing variables with constants there are two possible approaches: if (null == variable) or if (variable == null)

Let's enforce the second one, so that constant value (string literal, null or number) is on the right side of comparison.

Contributor Guide

Research-Richtung
Untersuche vorhandene Checkstyle Prüfungen für Vergleichsreihenfolge (z.B. 'EqualsAvoidNull') und implementiere eine neue Prüfung, die Konstanten auf der rechten Seite erzwingt.
Tech Stack
java
Domain
developer experiencetooling
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitJava
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70