Code optimieren

[PollerJava]

Hallo,

ich möchte die untere Methode optimieren, komme aber nicht so richtig weiter,
ich lese Werte aus der DB aus (diese Werte stehen dann in dBWerte) und ich muss die Werte dann eben in das Objekt sec (Second) bekommen.
Geht das irgendwie noch schneller?

Vielen Dank für die Antworten!!

lg

 private XYDataset createDataset(final String name, RegularTimePeriod start, List<DBWertReadBean> dBWerte, final int trendCnt, final int xSeite)                                
        {   
        series[xSeite][trendCnt] = null;
        series[xSeite][trendCnt] = new TimeSeries(name, start.getClass());             
        for(DBWertReadBean liste : dBWerte)                            
                {          
                DateTime dateTime = new DateTime(); 
                dateTime.setDateTime(liste.getZeitstempel().toString());
                Second sec = new Second(dateTime.getSecond(), dateTime.getMinute(), dateTime.getHour(), dateTime.getDay(), dateTime.getMonth(), dateTime.getYear());
                series[xSeite][trendCnt].add(sec, liste.getWert());     
                dateTime = null;           
                }
        dataset = new TimeSeriesCollection();   
        dataset.addSeries(series[xSeite][trendCnt]);             
        return dataset;
        }

[/quote]

 

[SlaterB]

evtl. musst du nicht ständig neue DateTime-Objekte erzeugen, sondern verwendest nur eins, ani dem du ständig setDateTime(string) aufrufst,

aber das sind höchstens Microsekunden so wie der ganze Algorithmus doch praktisch keine Zeit dauern dürfte oder beobachtest du ein anderes Verhalten?
normalerweise dürfte allein die DB-Anfrage zigmal länger dauern,

 

[The_S]

Sicher dass es an dieser Methode liegt? Weil abgesehen von dem Hinweis von Slater wird da nicht mehr viel gehen. Wie lange dauert denn der Aufruf dieser Methode? Wie viele Daten hast du? Evtl. gibts nen schnelleren SQL, falls es daran liegt.

 

[PollerJava]

HAbs jetzt schon ein bisschen optimiert, ich übergebe gleich den Timestamp (habe übersehen, dass es so einen Konstruktor auch gibt)
Der Zeitverlust liegt sicher an dieser Methode, ich habe je nach dem um die 10000 bis 40000 Datesätze (x,y-Werte),

Gibts vielleicht noch eine Möglichkeit, dass ich die Second() Objekte nicht bei jeden durchlauf erzeugen muss?

Vielen Dank,

lg

 private XYDataset createDataset(final String name, RegularTimePeriod start, List<DBWertReadBean> dBWerte, final int trendCnt, final int xSeite)                                 // erstellt das dataset
        {
        series[xSeite][trendCnt] = null;
        series[xSeite][trendCnt] = new TimeSeries(name, start.getClass()); 
        for(DBWertReadBean liste : dBWerte)                            
            {                             
            try { 
                Second sec = new Second(liste.getZeitstempel());            
                series[xSeite][trendCnt].add(sec, liste.getWert());     
                sec = null;
                }
            catch (Exception ex) 
                {
                //System.out.println("Fehler Klasse MultipleAxisTrend, Methode createDataset: " + ex);   // hier tritt eine Exception auf, wenn 2 Werte mit den gleichen Zeiten hinzugefügt werden sollen
                }                     
            }      
        dataset = new TimeSeriesCollection();   
        dataset.addSeries(series[xSeite][trendCnt]); 
        return dataset;
        }

 

[The_S]

10000 bis 40000 Sätze? Das ist ja nicht gerade viel. Sollte eigentlich relativ schnell ablaufen. Wie lang benötigt denn so ein Methodenaufruf?

Das Einzige, was ich mir jetzt noch vorstellen könnte, wäre, dass TimeSeries so langsam ist. Was ist das denn für eine Klasse?

Ansonsten überprüfe doch mal, was genau so lange dauert.

 

[PollerJava]

naja für das auslesen verwende ich iBatis und das benötigt um die 60ms je nach Datenanzahl und das erstellen des Datasets benötigt um die 130ms bei 25000 Datensätzen,
Schneller wirds wohl nicht mehr werden,

Der Schwachpunkt ist glaube ich die paint()- Methode von JFreechart
DB und dataset- erstellen geht ja ziemlich schnell (insgesamt nicht langsamer als 300ms) aber das zeichnen benötigt dann noch einige zeit aber da kann ich eh nichts machen,

lg vielen Dank

 

[SlaterB]

du hast doch nur 1000 Bildpunkte, müssen es wirklich 25000 Datensätze sein?

 

[tuxedo]

Er zeichnet mehrere Kurven ...

Aber 25.000 ist dennoch viel (was ich nebenbei gesagt schon vor Wochen erwähnt hab)

- Alex

 

[PollerJava]

Es geht mir ums reinzoomen,
ich habs jetzt hinbekommen, dass meine Methoden um sehr viel schneller arbeitet, Mal schauen, wies bei den Kunden ankommt,

Vielen Dank für Eure Anregungen!!

lg

 

[tuxedo]

Du musst doch für's zoomen nicht so viele Daten (im Chart) vorhalten. Erst wenn gezoomt wird solltest du die Daten holen, bzw ins Chart rein stecken.

- Alex