рекомендации по созданию кода для строковых манипуляций

Question

Поэтому я много думал о логике оператора if if в строке 6 моего кода. Но я надеялся получить некоторые отзывы от более опытных разработчиков. Вы, ребята, думаете, что я просто перепутал свой код, и если да, то как бы вы написали это более кратким?

Answer 1

Over-усложнению? Да, вы на самом деле :-)

Я бы просто использовать гораздо проще:

public String seeColor (String color) { 
    if (color.startsWith("red")) return "red"; 
    if (color.startsWith("blue")) return "blue"; 
    return ""; 
} 

Следующая полная программа показывает в действии:

public class Test 
{ 
    public static String seeColor (String color) { 
     if (color.startsWith("red")) return "red"; 
     if (color.startsWith("blue")) return "blue"; 
     return ""; 
    } 

    public static void main(String[] args) { 
     String[] testData = { "redxx", "xxred", "blueTimes", "NoColor", 
      "red", "re", "blu", "blue", "a", "", "xyzred" }; 
     for (String s: testData) 
      System.out.println("[" + s + "] -> [" + seeColor(s) + "]"); 
    } 
} 

Выходом этой программы бытии , как и ожидалось:

[redxx] -> [red] 
[xxred] -> [] 
[blueTimes] -> [blue] 
[NoColor] -> [] 
[red] -> [red] 
[re] -> [] 
[blu] -> [] 
[blue] -> [blue] 
[a] -> [] 
[] -> [] 
[xyzred] -> [] 

---

Если вы хотите легко расширяемой в будущем, вы можете выбрать что-то вроде:

public static String seeColor (String color) { 
    String[] allow = {"red", "blue"}; 
    for (String s: allow) 
     if (color.startsWith(s)) 
      return s; 
    return ""; 
} 

Затем, добавляя цвет к allow массив является единственным шагом вам нужно принять, чтобы он признал.

Answer 2

Лично я думаю, что вы могли бы упростить ее с чем-то вроде

if (str != null) { 
    if (str.equals("red")) return "red"; 
    else if (str.equals("blue")) return "blue"; 
} 
return ""; 

или чуть более сложным switch

if (str != null) { 
    switch (str) { 
    case "red": 
     return "red"; 
    case "blue"; 
     return "blue"; 
    } 
} 
return ""; 

Тестирование length, а затем выполняя substring(), как представляется, выполнять много относительно дорогого String манипуляции для меня.

Редактировать

Если вам необходимо проверить только начиная символов, я бы предпочел

if (str != null) { 
    if (str.startsWith("red")) return "red"; 
    else if (str.startsWith("blue")) return "blue"; 
} 
return ""; 

Answer 3

Мне кажется, что вы бы лучше герметизирующего свои названия цветов в потоке, так что он может быть легко расширена, вместо того, чтобы добавить новые, если ветви:

return Stream.of("red", "blue", "green") 
    .filter(colourName::startsWith).findAny().orElse(""); 

Обратите внимание, что это решение основано на Java 8 потоках и ссылках на методы.

Answer 4

правильный выражение решение. ^ означает начало матча.

public static String GetColor(String input) { 
    String[] colors = {"red", "blue"}; 
    for(String color : colors) { 
     if(input.matches(String.format("^%s.*", color))) 
      return color; 
    } 
    return ""; 
}