www.javahispano.org: Tu lenguaje, tu comunidad

Hola gente!

Tengo un problemilla:

He escrito un programillla usando JFrame al que he intregrado recientemente un pequeño Canvas. Pues bien, antes de integrar el canvas el uso de CPU era razonable pero ahora con el añadido el uso del repintado obliga a utilizar la CPU en un 100% continuamente.

Sabeis si hago algo mal?

Se agradece cualquier ayuda.

La única solución que te puedo aportar por el momento es que uses algún debugger o profiler para Java. Todos los IDEs que conozco incorporan alguno y sino en eclipse (que es el que uso) hay bastantes plugins que lo hacen como:

YourKit Java Profiler 5.0

AppPerfect Java Profiler

CodePro Profiler

El que usé fue el último, y por tanto el que te aconsejo pero eso no quiere decir que no haya uno mejor que otro. Hasta que encuentres quien es el comedor de CPU te entretendrás bastante. ¡¡suerte!!

El canvas lo uso para insertar un contador (como string), no ocupa mas que lo justo para ver el tiempo:

{ xx:xx }

Como nota adicional, el contador esta basado en la diferencia entre un tiempo de referencia y el tiempo actual.

Si comento la llamada a repaint del contador, el uso de CPU vuelve a la normalidad (sin incrementar el contador, obviamente).

Gracias, por vuestras respuestas.

Se os ocurre algo mas?

¿cada cuánto tiempo se actualiza el canvas?

Ciertamente RuGI, cambiar el canvas por un JLabel reduciria la carga de CPU pero no sabría como integrar el contador y que se actualice. Tendria que ponerme a hacer pruebas....

Podrias darme unos consejos si crees que asi se soluciona.

Caledor, la llamada a repaint la hago con el metodo sin argumentos, repaint(), por lo que se repinta tantas veces como java tenga el uso del procesador. He probado a cambiar la llamada por el metodo con 1 argumento, x ejemplo, repaint(2000), para que se repinte cada 2 segundos (y tendria que ver como el contador se actualiza cada 2 segundos), pero no es así(misterios de la vida) igualmente se actualiza cada segundo.

Gracias a todos por vuestra ayuda, me haceis un gran favor.

Hola.

Estimo que sólo estás queriendo actualizar el contador, por lo que estas llamando al método repaint() del componente donde se encuentra tu contador, ya sea este un canvas u otro componente, y no al método repaint() de un contenedor del componente del contador.

Si esto es asi, no debería traerte ningun problema de consumo de cpu, implementar un thread que cada segundo llame al método repaint() del canvas.

Si utilzas JFrame como contenedor principal no es dificil implemetar tu contador sobre un JLabel como te recomendo RuGi.

Quiza si me muestras algo de tu código te podria recomendar algo más concreto o te indicaría donde está tu error.

Saludos.

He hecho una clase para que la pruebes, solo asegurate de establecer a false la condicon cuando termine tu aplicacion.

/*
* To change this template, choose Tools | Templates
* and open the template in the editor.
*/
package com.pl;

import java.util.Date;
import javax.swing.JLabel;

/**
*
* @author EOLOS
*/
public class JLabelTimer extends JLabel {

private boolean sigue = true;

public JLabelTimer(){
super("Inicio");
Timer tm = new Timer(this);
tm.start();
}

@Override
public void finalize() {
this.setSigue(false);
System.out.println("Condicion para terminar");
}

public boolean isSigue() {
return sigue;
}

public void setSigue(boolean sigue) {
this.sigue = sigue;
}

class Timer extends Thread {
private JLabelTimer jl;
Timer(JLabelTimer jl) {
this.jl = jl;
}//constructor

public void run() {
while (this.jl.isSigue()) {
this.jl.setText(new Date().toString());
}
System.out.println("El hilo debe morir");
}//run
}
}//class

Hola a todos, siento haber tardado en responder, he estado un poco liado.

 jCalamaro, entendiste bien (ya se que no he sido muy claro en mis explicaciones), invoco a repaint desde el canvas que se inserta sobre un JFrame principal.

Al final he "solucionado" el problema, mas bien lo he parcheado.

Probe una tonteria y sin saber porque ha funcionado, me explico:

En el metodo paint, una vez  realizo todos los pintados necesarios invoco a repaint (con lo que entraria en un bucle). Lo que he hecho es antes de llamar a repaint provocar un sleep de 1 ms con Thread.sleep(1).

Notar que solo uso 1 hilo para toda la aplicacion.

Sin saber porque el uso de CPU se ha reducido al nivel normal y como el sleep es muy pequeño el usuario no puede notar esa pausa.

Que suerte la mia!

Os agradezco a todos vuestra ayuda!

 Os dejo mi mail por si estais interesados en echar un vistazo al codigo (ya por curiosidad mas que nada).

topilloveloz@gmail.com

Mandadme un correo y os adjunto el codigo fuente.

Chao y gracias!

Hola Xeros. No deberias llamar a repaint() desde tu sobreescritura del metodo paint() o por lo menos no es la soloucion mas efectiva.

Lo que te sucedia es que el CPU procesaba a full ya que estas obligando el repintado en un bucle infinito, al darle un sleep de un milisegundo, estas haciendo que el thread le de un respiro a tu procesador cada un milisegundo.

Te envie un mail.