LCD.drawstring and threads

[esmetaman]

Hi, I have developed a example to show how to make a OUTMEMORY using the method LCD.drawstring

Code: Select all

import lejos.nxt.*;

public class NXTCommTest {

   /**
    * @param args
    */
   public static void main(String[] args) {

      try{
         LCDUILite1 LCDUIObj = new LCDUILite1();
         //NXTBotMenu NBM = new NXTBotMenu();
         //NXTCommsListener NCL = new NXTCommsListener();
         LCDUIObj.start();
         //NBM.start();
         //NCL.start();
      }catch(Exception e){
         
      }
   }

}



Code: Select all

import javax.microedition.lcdui.*;
import lejos.nxt.*;

public class LCDUILite1 extends Thread {

   public LCDUILite1(){
      
   }
   
    public void run()
    {
       
        while(true)
        {
            try{
              LCD.drawString("asdf",0,0);
                int battery = (int)(Battery.getVoltageMilliVolt() );
                LCD.drawInt(battery,0,1);
                int memory = (int)(Runtime.getRuntime().freeMemory());
                LCD.drawInt(memory,0,2);
              LCD.refresh();
           }catch(Exception  e){

           }
        }
    }

}


[esmetaman]

Code: Select all

import lejos.nxt.*;

public class BugLCDDrawString {

   /**
    * @param args
    */
   public static void main(String[] args) {
      // TODO Auto-generated method stub
       
        LCD.drawString("Esmeta Labs",0,0);
        LCD.drawString("Bug detected",0,1);
        while(true)
        {
            try{
              LCD.drawString("LCD.drawString()",0,2);
              int battery = (int)(Battery.getVoltageMilliVolt() );
              LCD.drawInt(battery,0,3);
              int memory = (int)(Runtime.getRuntime().freeMemory());
              LCD.drawInt(memory,0,4);
              LCD.refresh();
           }catch(Exception  e){

           }
        }
   }

}


[/code]

[lawrie]

It is not a bug in LCD.drawString that leaks memory. Any use of a string literal in a loop creates a new object each time round the loop. You need to create a named string outside the loop.

[esmetaman]

Thanks Lawrie,

if you test the following example, LCD.drawString doesnt forget any byte.

Code: Select all

import lejos.nxt.*;

public class BLCDDrawString2 {

   /**
    * @param args
    */
   public static void main(String[] args) {
      // TODO Auto-generated method stub
       
      String message;
      
      message = "Esmeta Labs";      
        LCD.drawString(message,0,0);
        message = "Bug detected";
        LCD.drawString(message,0,1);
        message = "LCD.drawString()";
        while(true)
        {
            try{
               
              LCD.drawString(message,0,2);
              int battery = (int)(Battery.getVoltageMilliVolt() );
              LCD.drawInt(battery,0,3);
              int memory = (int)(Runtime.getRuntime().freeMemory());
              LCD.drawInt(memory,0,4);
              LCD.refresh();
           }catch(Exception  e){

           }
        }
   }

}



but if you create a string in a loop, The program will forget memory:

Code: Select all

import lejos.nxt.*;

public class BLCDDrawString3 {

   /**
    * @param args
    */
   public static void main(String[] args) {
      // TODO Auto-generated method stub
       
      String message;
      
      message = "Esmeta Labs";      
        LCD.drawString(message,0,0);
        message = "Bug detected";
        LCD.drawString(message,0,1);
        while(true)
        {
            try{
               message = "LCD.drawString()";   
              LCD.drawString(message,0,2);
              int battery = (int)(Battery.getVoltageMilliVolt() );
              LCD.drawInt(battery,0,3);
              int memory = (int)(Runtime.getRuntime().freeMemory());
              LCD.drawInt(memory,0,4);
              LCD.refresh();
           }catch(Exception  e){

           }
        }
   }

}



Conclussions: It is neccesary to define the messages.

Thanks Lawrie