Performance

[mar]

Hallo an alle,

ich habe hier ein Problem, und brauche eure Hilfe wenn das prog startet
schießt die CPU Auslastung enorm nach oben, hat jemand von euch eine IDEE
wie ich die Auslastung runterkriege.
Danke

import java.io.File;
import java.io.FileReader;
import java.io.IOException;
import java.util.TimerTask;
import java.awt.Color;
import java.awt.Dimension;
import java.io.FileReader;
import java.io.IOException;
import java.util.Timer;
 import java.util.Timer;
import java.util.TimerTask;
 import java.io.File;
import java.io.FileReader;
import java.io.IOException;
import java.util.TimerTask;
import javax.swing.JTextField;
import javax.swing.JFrame;
import javax.swing.JScrollPane;
import javax.swing.JTextField;
 
public class Main extends JFrame {
    private static final long serialVersionUID = 1L;
 
    private JTextField tf;
   
    public Main() {
         setDefaultCloseOperation(DO_NOTHING_ON_CLOSE);
 
        

        setAlwaysOnTop(rootPaneCheckingEnabled);
        // setzt die Position des Jrames
        setLocation(500, 80);
        // nimmt min. max. close weg
        setUndecorated(true);
        // JTextArea textArea = new JTextArea();
        tf = new JTextField();
        // kannst du nicht bearbeiten
        tf.setEditable(false);
 
        //JScrollPane scrollPane = new JScrollPane(tf);
        //scrollPane.setPreferredSize(new Dimension(230, 50));

        tf.setPreferredSize(new Dimension(200,30));

        
        add(tf);
        pack();
        setVisible(true);
        tf.setCursor(null);
 
        tf.setBackground(Color.red);
        tf.setForeground(Color.green);
        tf.setCaretPosition(tf.getText().length());
        tf.setHighlighter(null);
       
        new Timer().schedule(new TextMarqueeTimer(tf), 0L, 200L);
    }
 
    public String readLine(FileReader reader) throws IOException {
        String line = null;                                              //-1
        for (int readInt = 0; (readInt = reader.read()) != 13 && readInt != -1;) {
            String s = String.valueOf((char) readInt);
            if (line == null)
                line = s;
            else
                line += s;
        }
        reader.read();
        return (line);
    }
 
    public static void main(String[] args) {
        new Main();
    }
}

 
class CheckFileContentTimer extends TimerTask {
    private static final File FILE = new File("C:/loi/data.txt");
   
    private long lastMod;
    private TextMarqueeTimer textMarqueeTimer;
   
    CheckFileContentTimer(TextMarqueeTimer textMarqueeTimer) {
        super();
        this.textMarqueeTimer = textMarqueeTimer;
    }
    public void run() {
        while(true) {
            try {
                long atmLastMod = FILE.lastModified();
 
                if(atmLastMod > lastMod) {
                    FileReader fileReader = new FileReader(FILE);
                   
                    StringBuilder stringBuilder = new StringBuilder();
                    for (String incomingLine = null; !(incomingLine = readLine(fileReader)).equals(".");) {
                        stringBuilder.append(incomingLine.concat);
                    }
                    fileReader.close();
                    lastMod = atmLastMod;
                    textMarqueeTimer.setNextText(stringBuilder.toString());
                }       
            }
            catch (IOException e) {
                e.printStackTrace();
            }
        }
    }
    public String readLine(FileReader reader) throws IOException {
        StringBuilder builder = new StringBuilder();
        for (int readInt = 0; (readInt = reader.read()) != 13 && readInt != -1;) {
            builder.append((char) readInt);
        }
        reader.read();
        return (builder.toString());
    }
}

 
class TextMarqueeTimer extends TimerTask {
   
    private String atmText = "";
    private String nextText = "";
    private JTextField textField;
   
    TextMarqueeTimer(JTextField textField) {
        new  Timer().schedule(new CheckFileContentTimer(this), 0L, 100L);
        this.textField = textField;
    }
    public void run() {
        if(atmText != null && nextText != null) {
            if(!atmText.equals(nextText)) {
                atmText = nextText;
                textField.setText(atmText);
            } else {
                if(atmText.length() > 0) {
                    String changedText = textField.getText();
                    String withoutFirstChar = changedText.substring(1, changedText.length());
 
                    char firstChar = changedText.charAt(0);
                    String newText = withoutFirstChar.concat(String.valueOf(firstChar));
                    textField.setText(newText);
                }
            }          
        }
    }
    public void setNextText(String nextText) {
        this.nextText = nextText;
    }
}

 

[Geeeee]

Naja, alle 100 ms die gesamte Datei durchlesen ist wohl etwas aufwendig. Da kommt er einfach nicht mehr zur Ruhe würde ich sagen.

 

[Ark]

Ohne alles in seiner Einzelheit gesehen zu haben: Verzichte auf deine [c]readLine()[/c]-Methoden und benutze [c]BufferedReader.readLine()[/c]. Das löst zwar wahrscheinlich nicht das grundsätzliche Problem, macht das Programm als Ganzes jedoch genügsamer, was Ressourcen angeht.

Ark

 

[Empire Phoenix]

        String line = null;                                              //-1
        for (int readInt = 0; (readInt = reader.read()) != 13 && readInt != -1;) {
            String s = String.valueOf((char) readInt);
            if (line == null)
                line = s;
            else
                line += s;
        }

Benutze string builder, der wird so lange brachen ejdesmal die dinger zu allokieren. In java sind strings constanten, die werden jedesmal neu erstellt.

 

[mar]

Hallo,
erstmal danke für die schnellen antworten.
Ich werde es gleich mal ausprobieren.
Gruß
Schönen abend noch

 

[mar]

Hallo ich nochmal,
ich habe da noch ein anderes Problem,
zur Zeit läuft die Eingabe so, das ich immer ein Punkt in der .txt Datei machen muss um
die Eingabe anzuzeigen, hat jemand einen Rat für mich wie ich das anders lösen kann.

Danke
gruß