Sehen Sie eine Schwäche in dieser Java-Funktion?

Question

Können Sie mir sagen, ob Sie eine Schwäche in diesem Code sehen?

private String SplitCategory(String categoryName){ 

    String[] categories = categoryName.split("\\-"); 
    String categorySet = "Set Category: "; 

    if(categories.length <= 1) 
     categorySet += categories[0].trim(); 
    else 
     categorySet += categories[categories.length - 1].trim(); 

    return categorySet; 
} 

Die params für diese Funktion wie folgt aussieht: „CategoryOne“, „CategoryOne - Unterkategorie A“

Answer 1

• Funktionen Namen beginnen in der Regel in Kleinbuchstaben: splitCategory
• Die Funktion aufgeteilt nicht wirklich die Kategorien (ich würde erwarten, dass es ein String [] zurück, wenn es getan hat). So könnte man es vielleicht noch etwas anderes nennen, wie getLowestLevelCategory
• Überprüfen Sie, dass der Parameter nicht null ist, wie bereits von anderen gesagt
• Was meinen Sie wirklich nicht if(categories.length <= 1) ist aber if(categories.length == 1)

Answer 2

Erstens ist es zu einer Nullpointer verwundbar, weil man nie prüfen, ob die param categoriesName vor nicht null ist es Aufspaltung .

Zweitens der Block:

if(categories.length <= 1) 
    categorySet += categories[0].trim(); 

ist zu einem ArrayIndexOutOfBoundsException verwundbar.

Auch dies wäre wahrscheinlich besser auf codereview.stackexchange.com

Answer 3

wenn yourcategories.length Null ist ein Laufzeitfehler haben hier (CategorySet + = Kategorien [0] .trim();) da Es gibt keinen Index 0 in einem Null-Längen-Array.

Answer 4

sollten Sie

setzen

private String splitCategory(String categoryName){ 
String[] categories = categoryName.split("\\-"); 
String categorySet = "Set Category: "; 

if(categories.length == 1) 
    categorySet += categories[0].trim(); 
else if (categories.length == 2) 
    categorySet += categories[categories.length - 1].trim(); 

return categorySet; 

}

in Ihrem vorherigen Code Nullpointer auftreten können.