Sonar-Flag "Schlechte Praxis": Verwendung von GetResource in XYZ kann unsicher sein, wenn Klasse erweitert wird

Question

Sonar zeigt diesen Fehler in unserem Java-Projekt an. Jeder Vorschlag für eine sichere Programmierung ist willkommen!

URL url = getClass().getResource("/myWonderfulResource.txt"); 
if (url == null) { 
    throw new IllegalStateException("File not found: /myWonderfulResource.txt"); 
} 

Answer 1

Machen Sie die Klasse final so dass es nicht erweitert werden kann. Die Warnung soll verhindern, dass die erweiterte Klasse (möglicherweise) versucht, nicht vorhandene Ressourcen zu verwenden.

Answer 2

Das einzige, was ich mir vorstellen kann, warum Sonar diese Nachricht ausspuckt, ist, weil eine abgeleitete Klasse möglicherweise von einem anderen Klassenlader geladen wird, so dass Ihre Textdatei dann möglicherweise nicht gefunden wird. Ich würde dies einfach ignorieren, machen die Klasse final, wie vorgeschlagen, oder eine .class wörtliche getClass() statt

Answer 3

Die Fehlermeldung von Sonar nicht verwenden scheinen viel Sinn zu machen, da die Ressource mit einem Schrägstrich beginnt, und so wird im Stammverzeichnis des Klassenpfads nachgeschlagen. Allerdings Sonar möglicherweise nicht überprüfen, was in dem Ressource-String ist, und es wird dann der Pfad annehmen könnte ein relativer Pfad sein ..

Stellen Sie sich vor, was Sie eine Zeichenfolge ohne einen Schrägstrich geschrieben würde geschehen hat:

URL url = getClass().getResource("myWonderfulResource.txt"); 

Die URL würde im aktuellen Paket auf myWonderfulResource.txt zeigen. Angenommen, Sie haben die Klasse in einem anderen Paket erweitert.

package com.example; 
public class Wonderous {...} 

package com.example.awesome; 
public class Awesome extends Wonderous {...} 

Wenn eine Instanz von Awesome versucht, die wunderbaren Textdatei zu erhalten, sieht es auf dem Klassenpfad in com/example nach oben/genial. Aber Wonderful 's Ressource ist in com/example. Awesome wird es nicht finden.

übrigens dieser Fehlerbericht kommt aus FindBugs und die documentation für diesen speziellen Fehler ist:

UI: Verwendung von GetResource unsicher sein kann, wenn Klasse erweitert wird (UI_INHERITANCE_UNSAFE_GETRESOURCE)

diesen Aufruf. getClass(). getResource (...) könnte andere als erwartete Ergebnisse liefern, wenn diese Klasse um eine Klasse in einem anderen Paket erweitert wird.