String.replace上的java.lang.OutOfMemoryError

时间:2015-07-23 20:01:49

标签: java android out-of-memory

我有一个文本文件,其中包含234行string1 @ string2 @ string3。

kya@きゃ@キャ
kyu@きゅ@キュ
kyo@きょ@キョ
sha@しゃ@シャ
shu@しゅ@シュ
...so 234 lines

我正在编写转换器,它将带有string2或string3的单词转换为带有string1的单词。

InputStream is = context.getResources().openRawResource(R.raw.kanatoromaji);
InputStreamReader isr = new InputStreamReader(is);
BufferedReader reader = new BufferedReader(isr);
String line;
try {
    while ((line = reader.readLine()) != null) {
        String[] parts = line.split("@");
        String romaji = parts[0];
        String hiragana=parts[1];
        String katakana = parts[2];

        if (hiragana!=null&&word.contains(hiragana)) {
                word = word.replace(hiragana, romaji);//in this line getting outOfMemory error
        }
        if (word.contains(katakana)) {
            word = word.replace(katakana, romaji);
        }
    }

} catch (IOException e) {
    e.printStackTrace();
}

我多次调用此方法(一次运行200k-300k次)。所以这导致了这个错误: 07-24 00:52:32.859 10848-10848 / net.joerichard.test E / AndroidRuntime:FATAL

EXCEPTION: main
    java.lang.OutOfMemoryError
            at java.lang.AbstractStringBuilder.enlargeBuffer(AbstractStringBuilder.java:94)
            at java.lang.AbstractStringBuilder.append0(AbstractStringBuilder.java:132)
            at java.lang.StringBuilder.append(StringBuilder.java:124)
            at java.lang.String.replace(String.java:1367)
            at net.joerichard.test.Converter.kanaToRomaji(Converter.java:32)
            at net.joerichard.test.MainActivity.migrateKanaKanji(MainActivity.java:222)
            at net.joerichard.test.MainActivity.access$700(MainActivity.java:29)
            at net.joerichard.test.MainActivity$10.onClick(MainActivity.java:131)
            at android.view.View.performClick(View.java:4240)
            at android.view.View$PerformClick.run(View.java:17721)
            at android.os.Handler.handleCallback(Handler.java:730)
            at android.os.Handler.dispatchMessage(Handler.java:92)
            at android.os.Looper.loop(Looper.java:137)
            at android.app.ActivityThread.main(ActivityThread.java:5103)
            at java.lang.reflect.Method.invokeNative(Native Method)
            at java.lang.reflect.Method.invoke(Method.java:525)
            at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:737)
            at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:553)
            at dalvik.system.NativeStart.main(Native Method)

我的代码有什么问题?怎么解决?

更新

根据JFPicard的回答,我将我的String字转换为StringBuilder sbWord。然后我尝试使用StringBuilder替换string2和string3。我之前从未替换过StringBuilder。我在谷歌搜索并发现这个solution来替换我的StringBuilder部分。现在我的代码看起来像这样:

public static String kanaToRomaji(Context context, String word) {

    String kana = word;
    StringBuilder sbWord = new StringBuilder(word);

    InputStream is = context.getResources().openRawResource(R.raw.kanatoromaji);
    InputStreamReader isr = new InputStreamReader(is);
    BufferedReader reader = new BufferedReader(isr);
    String line;
    try {
        while ((line = reader.readLine()) != null) {
            String[] parts = line.split("@");
            String romaji = parts[0];
            String hiragana=parts[1];
            String katakana = parts[2];

            if (hiragana!=null&&word.contains(hiragana)) {
                sbWord = replaceAll(sbWord, hiragana, romaji);//in this line getting outOfMemory error
            }
            if (word.contains(katakana)) {
                sbWord = replaceAll(sbWord, katakana, romaji);
            }
        }

    }
    catch (IOException e) {
        e.printStackTrace();
    }

    char[] chars = kana.toCharArray();
    for (char character: chars)
    {
        if(sbWord.toString().contains(String.valueOf(character)))
        {
            new MyLog(kana+":"+sbWord.toString());
            break;
        }
    }

    return word;
}

我替换我的StringBuilder部件的方法:

public static StringBuilder replaceAll(StringBuilder builder, String from, String to)
{
    int index = builder.indexOf(from);
    while (index != -1)
    {
        builder.replace(index, index + from.length(), to);
        index += to.length(); // Move to the end of the replacement
        index = builder.indexOf(from, index);
    }
    return builder;
}

现在我收到此错误: 07-24 01:23:52.890 13512-13512 / net.joerichard.test E / AndroidRuntime:FATAL

  

例外:主要       java.lang.OutOfMemoryError               at java.lang.AbstractStringBuilder.move(AbstractStringBuilder.java:397)               at java.lang.AbstractStringBuilder.insert0(AbstractStringBuilder.java:356)               at java.lang.AbstractStringBuilder.replace0(AbstractStringBuilder.java:442)               在java.lang.StringBuilder.replace(StringBuilder.java:637)               at net.joerichard.test.Converter.replaceAll(Converter.java:65)               at net.joerichard.test.Converter.kanaToRomaji(Converter.java:33)               at net.joerichard.test.MainActivity.migrateKanaKanji(MainActivity.java:222)               at net.joerichard.test.MainActivity.access $ 700(MainActivity.java:29)               at net.joerichard.test.MainActivity $ 10.onClick(MainActivity.java:131)               在android.view.View.performClick(View.java:4240)               在android.view.View $ PerformClick.run(View.java:17721)               在android.os.Handler.handleCallback(Handler.java:730)               在android.os.Handler.dispatchMessage(Handler.java:92)               在android.os.Looper.loop(Looper.java:137)               在android.app.ActivityThread.main(ActivityThread.java:5103)               at java.lang.reflect.Method.invokeNative(Native Method)               在java.lang.reflect.Method.invoke(Method.java:525)               在com.android.internal.os.ZygoteInit $ MethodAndArgsCaller.run(ZygoteInit.java:737)               在com.android.internal.os.ZygoteInit.main(ZygoteInit.java:553)               在dalvik.system.NativeStart.main(本地方法)

在这种情况下该怎么办?

3 个答案:

答案 0 :(得分:6)

我想我有个主意。

这一行word = word.replace(hiragana, romaji);

和这一个word = word.replace(katakana, romaji);

每次都创建一个新的String。 String是不可变的,因此它消耗内存。在你的情况下很多。

尝试使用StringBuilder。

答案 1 :(得分:1)

使用InputStream时,您永远不会释放资源。如果在finally块中不再需要该流,则应关闭该流。

另外,如果你把这个方法称为200k次,也许你不应该一直解析这个文件?据我所知,文件总是一样的,为什么不在首次运行后将结果缓存到某个地方?

答案 2 :(得分:1)

我在JFPicard的help找到了这个解决方案,我很欣赏。正如我所说的那样,我的问题就在这一行:

word = word.replace(hiragana, romaji);

我的错,我没注意到空的string2行。我很抱歉,我不知道这可能会引起问题。我有26行,strin2为空。

  

无线@@ウィ

错误发生了,因为它被替换了26次因为它是空的并且可以通过我的if子句

        if (hiragana != null && word.contains(hiragana)) {
            word = word.replace(hiragana, romaji);
        }

现在我将我的代码更改为此代码并且正在运行=)

        String[] parts = line.split("@");
        String romaji = parts[0];
        String hiragana = parts[1];
        if(hiragana.equals(""))
        {
            hiragana=null;
        }
        String katakana = parts[2];
        if(katakana.equals(""))
        {
            katakana=null;
        }

        if (hiragana != null && word.contains(hiragana)) {
            word = word.replace(hiragana, romaji);
        }
        if (katakana!=null && word.contains(katakana)) {
            word = word.replace(katakana, romaji);
        }