2016-01-17 50 views
0

我想在某人赢了或者输了一个二十一点游戏后重写一个文件。我每次运行游戏并获得胜利,我得到这个错误:内存不足错误

Exception in thread "AWT-EventQueue-0" java.lang.OutOfMemoryError: Java heap space 
    at java.util.Arrays.copyOf(Unknown Source) 
    at java.lang.AbstractStringBuilder.expandCapacity(Unknown Source) 
    at java.lang.AbstractStringBuilder.ensureCapacityInternal(Unknown Source) 
    at java.lang.AbstractStringBuilder.append(Unknown Source) 
    at java.lang.StringBuilder.append(Unknown Source) 
    at User.setWinnings(User.java:278) 
    at Blackjack.addWinnings(Blackjack.java:182) 
    at Blackjack.showWinMessage(Blackjack.java:323) 
    at Blackjack.showOutcome(Blackjack.java:308) 
    at Blackjack.actionPerformed(Blackjack.java:401) 
    at javax.swing.AbstractButton.fireActionPerformed(Unknown Source) 
    at javax.swing.AbstractButton$Handler.actionPerformed(Unknown Source) 
    at javax.swing.DefaultButtonModel.fireActionPerformed(Unknown Source) 
    at javax.swing.DefaultButtonModel.setPressed(Unknown Source) 
    at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(Unknown Source) 
    at java.awt.Component.processMouseEvent(Unknown Source) 
    at javax.swing.JComponent.processMouseEvent(Unknown Source) 
    at java.awt.Component.processEvent(Unknown Source) 
    at java.awt.Container.processEvent(Unknown Source) 
    at java.awt.Component.dispatchEventImpl(Unknown Source) 
    at java.awt.Container.dispatchEventImpl(Unknown Source) 
    at java.awt.Component.dispatchEvent(Unknown Source) 
    at java.awt.LightweightDispatcher.retargetMouseEvent(Unknown Source) 
    at java.awt.LightweightDispatcher.processMouseEvent(Unknown Source) 
    at java.awt.LightweightDispatcher.dispatchEvent(Unknown Source) 
    at java.awt.Container.dispatchEventImpl(Unknown Source) 
    at java.awt.Window.dispatchEventImpl(Unknown Source) 
    at java.awt.Component.dispatchEvent(Unknown Source) 
    at java.awt.EventQueue.dispatchEventImpl(Unknown Source) 
    at java.awt.EventQueue.access$500(Unknown Source) 
    at java.awt.EventQueue$3.run(Unknown Source) 
    at java.awt.EventQueue$3.run(Unknown Source) 

这是setWinnings方法

public void setWinnings(int winnings2) { 
    winnings = winnings2; 

    IO.openInputFile("users.txt"); 
    String line = IO.readLine(); 
    StringBuilder newFile = new StringBuilder(); 

    while(line != null){ 
     String tokens[] = line.split("%"); 

     if(tokens.length > 0){ 

      if(tokens[0].equalsIgnoreCase(this.getUserID())){ 
       String newLine = tokens[0] + tokens[1]+ tokens[2] + winnings2 + tokens[4]+ tokens[5]+ tokens[6]; 

       newFile.append(newLine); 
       newFile.append("\n"); 
      } else { 
       newFile.append(line); 
       newFile.append("\n"); 
      } 

     } 
    } 

    IO.closeInputFile(); 
    IO.createOutputFile("users.txt"); 
    IO.println(newFile.toString()); 
    IO.closeOutputFile(); 

} 

我已经尝试进入控制面板,并改变任何该字段代码被称为java下的-Xms2048m什么的。任何想法为什么这不起作用?

编辑:请注意,users.txt的大小只有82个字节。

+0

可能重复[什么是OutOfMemoryError,以及如何调试和修复它](http://stackoverflow.com/questions/24510188/what-is-an-outofmemoryerror-and-how-do-i-debug - 修复它) – Raedwald

回答

3

你有一个循环

while(line != null) { 
    ... 
} 

但在循环中,你从来不看另一行。循环从不终止,并且您继续将相同的数据附加到newFile。最终你会消耗所有的内存。

+0

Brb kms ...多么愚蠢的错误。我会尝试添加一条新的readline语句 –

0

您赋值给变量line只有一次,在申报阶段:String line = IO.readLine();while循环之前sin'ceit发生,内环路您百达使用相同的第一line没有从输入文件中的新行重新分配它。

换句话说,while循环会无限地运行,或者直到内存用完为止,就像您的情况一样。

您必须在while循环范围结束之前添加line = IO.readLine();