2017-04-10 5 views
0

ich dieses folgende Fehlermeldung erhalten: Die Methode getCDsByToken() in CDsImplTl.java verwendet nie den Anfangswert auf Linie 141Fortify Testfehler: nie verwendet den Anfangswert sie der Variablen zuweist

auf die Variable paymentCDApp zuweist und das hier ist mein Code:

Ich verstehe nicht die Fehlermeldung so weit, was muss ich nachschlagen? Ich meine, alles wird benutzt, was hier nicht stimmt?

Antwort

2
IfPaymentCDApp paymentCDApp = new IfPaymentCDApp(); 

Im obigen Code vergeben Sie paymentCDApp mit einem Objekt. Aber später, im try-Block, weisen Sie derselben Variablen einen anderen Wert zu;

paymentCDApp = AppUtils.mapStripeCDToExtCD(cD); 

Der Wert, den Sie vor (von IfPaymentCDApp paymentCDApp = new IfPaymentCDApp(); gegeben) zugeordnet ist, nie zwischen dieser Linie und der Linie verwendet, die paymentCDApp = AppUtils.mapStripeCDToExtCD(cD); enthält. Deshalb bekommen Sie einen solchen Fehler.

Sie können die Variable am Anfang auf null initialisieren, was eine viel bessere Praxis ist.

Möglicherweise müssen Sie nach dem try-Block eine Nullprüfung durchführen, um festzustellen, ob der richtige Wert zugewiesen wurde. Ich kann keinen Vorschlag machen, da Sie den Code nach dem try-Block nicht eingefügt haben. Wenn Sie die Variable jedoch auf Null initialisieren, wird Ihr Problem gelöst.

+0

Wenn ich das so implementiere, wie Eclipse möchte, dass ich @SuppressWarnings (null) hinzufüge, muss ich diesen Weg gehen? oder gibt es eine Möglichkeit, die ich ohne SurpressWarnings implementieren könnte – Nali

+0

Deshalb habe ich gesagt, dass Sie einen Null-Check benötigen. 'if (paymentCDApp! = null)' um sicherzustellen, dass deine Sachen gut gelaufen sind. Noch eine Sache, bist du sicher 'Die Methode getCDsByToken() in CDsImplTl.java benutzt nie den Initialwert, den sie der Variablen paymentCDApp in Zeile 141 zuordnet.' Ist ein Fehler? Oder ist es nur eine weitere Warnung? –

+0

Ich mache das Zeug in Fortify und es ist markiert als und Problem, aber auch mit niedriger Priorität markiert, so dass ich nicht weiß, ob ich das lösen muss oder es so lassen – Nali

Verwandte Themen