2013-02-01 20 views
0

我有android应用程序,它从telnet客户端获取巨大的String对象。稍后我只使用大字符串的一小部分。我用Android上的字符串复制

new String(Part of old string);

从旧字符串的字符数组分离新的字符串的字符数组。所以旧的字符串应该被垃圾收集,但是新的字符串仍然有一个对旧对象的引用。我可以在“Eclipse Memory Analyzer”中看到它。 而且这很快就溢出了我的小16meg应用程序内存。

如何避免这种情况?

private WifiChannel parse1(String channLine){ 
    //scanning with "iwlist wlan0 scanning" the getChans1 method 
    String[] input = channLine.split(System.getProperty("line.separator")); 
    if (input.length < 4); 
    String segment[]; 
    String segment2[]; 
    WifiChannel chan = new WifiChannel(); 
    try { 
     if (input.length > 5){ 
      chan.setMacAddress(new String(input[0])); 
      segment = input[1].split(":"); 
      chan.setChannel(Integer.parseInt(segment[1].trim())); 
      segment = input[3].split(" "); 
      segment2 = segment[20].split("="); 
      chan.setQuality(new String(segment2[1])); 
      segment2 = segment2[1].split("/"); 
      chan.setSignalStrength((Integer.parseInt(segment2[0].trim())*100)/Integer.parseInt(segment2[1].trim())+"%"); 
      segment2 = segment[23].split("="); 
      try{chan.setSignalLevel(Integer.parseInt(segment2[1].trim()));}catch(Exception e){chan.setSignalLevel(0);} 
      segment = input[5].split(":"); 
      chan.setName(new String(segment[1].replaceAll("^\"|\"$", ""))); 
      for (int i = 6;i<input.length;i++) 
       if (input[i].contains("Mode")) 
        segment = input[i].split(":"); 
      chan.setChannelMode(new String(segment[1])); 
      String band = ""; 
      if(channLine.contains("5.5 Mb/s"))band = band +"b"; 
      if(channLine.contains("12 Mb/s"))band = band +"g"; 
      chan.setBand(new String(band)); 
     } 
    }catch (Exception e){Log.e("","",e);} 
    return chan; 
} 

方法输入也是更大的字符串的一部分。

+0

尝试使用'String.substring()'方法。 – Thommy

+0

你能告诉我们所有的代码吗?你什么时候检查内存使用情况?不保证旧的字符串会立即清除。如果您不知道,只要您重新分配旧字符串被破坏的字符串(在下一次垃圾回收迭代时),字符串就是不可改变的。 – Grambot

+0

不能你做oldString = null; ? –

回答

3

每个字符串实例由一个字符数组支持:

public class String { 

    private final char[] value; 

    ... 
} 

由于效率的原因,在一根绳子上的一些操作可以可以创建一个共享char[]与原始字符串的新字符串实例。这是可能的,因为所有字符串都是不可变的。这方面的一个例子是substring()方法:

public String substring(int start) { 
    if (start == 0) { 
     return this; 
    } 
    if (start >= 0 && start <= count) { 
     return new String(offset + start, count - start, value); // !!! 
    } 
    throw indexAndLength(start); 
} 

与我的评论添加的行调用不创建char[]的拷贝构造函数,而是直接引用它。你不能直接调用此构造方法,因为它是包专用,因此它是由方法,如子仅在内部使用:现在

/* 
* Internal version of the String(char[], int, int) constructor. 
* Does not range check, null check, or copy the character array. 
*/ 
String(int offset, int charCount, char[] chars) { 
    this.value = chars; 
    this.offset = offset; 
    this.count = charCount; 
} 

,如果你从一个很长的字符串创建一个简短的字符串,然后短的仍然参考原始大char[]。即使原始字符串被垃圾回收后,其大数组仍然停留在内存中,即使只有一小部分现在可以被新的子字符串访问。这有效地创建了一个内存泄漏

要解决此问题,通常的技巧是使用复制构造函数从子字符串中创建一个新字符串,该复制构造函数仅复制原始char[]所需的范围。

String longString = "1234567890"; 
// backed by char[] of length 10 

String substring = longString.substring(5); 
// value is "67890", but still backed by the original char[] of length 10 

String copy = new String(substring); 
// also has value "67890", but now backed only by char[] of length 5 

编辑:

为了完整,这就是拷贝构造器的来源。正如你所看到的,如果原始字符串引用与字符串本身相同长度的数组,那么该数组不需要被复制,因为它中没有“死亡字符”。但是,如果数组大于字符串的长度,则会执行数组的“活动”范围的副本。

public String(String toCopy) { 
    value = (toCopy.value.length == toCopy.count) 
     ? toCopy.value 
     : Arrays.copyOfRange(toCopy.value, toCopy.offset, 
      toCopy.offset + toCopy.length()); 
    offset = 0; 
    count = value.length; 
} 

注: 所有上述源代码来自Android的API 15

+0

为什么这可能不适用于android? – Balvonas

+0

我不确定。 String.split似乎根据source [here]使用了substring()方法(https://android.googlesource.com/platform/libcore/+/refs/heads/master/luni/src/main/java /java/lang/String.java)和[here](https://android.googlesource.com/platform/libcore/+/refs/heads/master/luni/src/main/java/java/util/regex/ Splitter.java),但是你大量使用复制构造函数应该可以消除这种情况。 “Eclipse Memory Analyzer”可以显示代码中的哪些特定对象受此影响? – Natix