7

我有以下字符串：是可选的，所以
A:B:1111;domain:80;a;b
也是有效的输入。也是可选的， 或者也是有效的输入 我想要的最终结果是： AB:1111;domain:80;a;b
:80B:1111;domain;a;b:1111;domain;a;b
String[]

s[0] = "A";  
s[1] = "B";  
s[2] = "1111";  
s[3] = "domain:80"  
s[4] = "a"  
s[5] = "b"

我这样做如下： 

List<String> tokens = new ArrayList<String>();  
String[] values = s.split(";");  
String[] actions = values[0].split(":");   

for(String a:actions){  
    tokens.add(a);  
}  
//Start from 1 to skip A:B:1111
for(int i = 1; i < values.length; i++){  
    tokens.add(values[i]);  
}  
String[] finalResult = tokens.toArray();

我想知道有没有更好的方法来做到这一点？我还能如何更有效地做到这一点？

4

5 回答 5

2

这里没有太多效率问题，我所看到的都是线性的。

无论如何，您可以使用正则表达式或手动标记器。

你可以避开这个清单。values你知道and的长度actions，所以你可以做

String[] values = s.split(";");  
String[] actions = values[0].split(":");
String[] result = new String[actions.length + values.length - 1];
System.arraycopy(actions, 0, result, 0, actions.legnth);
System.arraycopy(values, 1, result, actions.length, values.length - 1);
return result;

它应该是相当有效的，除非你坚持split自己实施。

未经测试的低级方法（确保在使用前进行单元测试和基准测试）：

// Separator characters, as char, not string.
final static int s1 = ':';
final static int s2 = ';';
// Compute required size:
int components = 1;
for(int p = Math.min(s.indexOf(s1), s.indexOf(s2));
  p < s.length() && p > -1;
  p = s.indexOf(s2, p+1)) {
    components++;
}
String[] result = new String[components];
// Build result
int in=0, i=0, out=Math.min(s.indexOf(s1), s.indexOf(s2));
while(out < s.length() && out > -1) {
  result[i] = s.substring(in, out);
  i++;
  in = out + 1;
  out = s.indexOf(s2, in);
}
assert(i == result.length - 1);
result[i] = s.substring(in, s.length());
return result;

注意：此代码以疯狂的方式进行了优化，它:只会在第一个组件中考虑 a。处理最后一个组件有点棘手，就像outvalue一样-1。

我通常不会使用最后一种方法，除非性能和内存非常重要。很可能其中仍然存在一些错误，并且代码相当不可读，特别是与上面的代码相比。

于 2012-05-16T13:08:21.377 回答
1

通过对可接受字符的一些假设，此正则表达式提供验证以及拆分为您想要的组。

Pattern p = Pattern.compile("^((.+):)?(.+):(\\d+);(.+):(\\d+);(.+);(.+)$");
Matcher m = p.matcher("A:B:1111;domain:80;a;b");
if(m.matches())
{
    for(int i = 0; i <= m.groupCount(); i++)
        System.out.println(m.group(i));
}
m = p.matcher("B:1111;domain:80;a;b");
if(m.matches())
{
    for(int i = 0; i <= m.groupCount(); i++)
        System.out.println(m.group(i));
}

给出：

A:B:1111;domain:80;a;b // ignore this
A: // ignore this
A // This is the optional A, check for null
B
1111
domain
80
a
b

和

B:1111;domain:80;a;b // ignore this
null // ignore this
null // This is the optional A, check for null
B
1111
domain
80
a
b
于 2012-05-16T13:06:17.710 回答
0

你可以做类似的事情

String str = "A:B:1111;domain:80;a;b";
String[] temp;

/* delimiter */
String delimiter = ";";
/* given string will be split by the argument delimiter provided. */
temp = str.split(delimiter);
/* print substrings */
for(int i =0; i < temp.length ; i++)
System.out.println(temp[i]);
于 2012-05-16T13:03:04.527 回答
0

除非这是您代码中的瓶颈，并且您已经验证不必太担心效率，因为这里的逻辑是合理的。您可以避免创建临时数组列表，而是直接创建您知道所需大小的数组。

于 2012-05-16T13:03:14.547 回答
0

如果您想将域和端口保持在一起，那么我相信您将需要您需要两个拆分。您也许可以使用一些正则表达式魔术来做到这一点，但我怀疑您是否会从中看到任何真正的性能提升。

如果您不介意拆分域和端口，则：

  String s= "A:B:1111;domain:80;a;b";
  List<String> tokens = new ArrayList<String>();
  String[] values = s.split(";|:");

  for(String a : values){
      tokens.add(a);
  }
于 2012-05-16T13:11:18.937 回答