[SOLUCIONADO] Volver a repintar el area

[Jose789]

A través de un formulario debo de dibujar 2 figuras geométricas, el problema es el siguiente si en el Main borro las 2 lineas siguientes y las coloco en el formulario.java se dibuja pero en diferentes pantallas cuando tiene que estar las 2 figuras en las misma.

De esta forma no consigo que pinte nada en la ventana.

Main.java

Código Java:

pintar pintar = new pintar();
     pintar.setVisible(true);
 

Y a través del formulario.java llamo al constructor

Código Java:

pintar pintar = new pintar(tipo,clor,x,y,height,width);
 

pintar.java

Código Java:

package graficos2d;
 
import javax.swing.*;
import java.awt.*;
 
public class pintar extends JFrame {
 
    private String figura, clor;
    private int alto, ancho, x, y;
    
    pintar(){
        setSize(400, 400);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setLocationRelativeTo(null);
        setVisible(true);
    }
    
    public pintar(String figura, String clor, int x, int y, int alto, int ancho) {
        
        this.figura = figura;
        this.clor = clor;
        this.x = x;
        this.y = y;
        this.alto = alto;
        this.ancho = ancho;
 
        System.out.println(clor + " " + figura + " " + x + " " + y + " " + alto + " " + ancho);
        
    }
 
    @Override
    public void paint(Graphics g) {
        super.paint(g);
        
        if (figura == "Circulo") {
            g.setColor(Color.BLUE);
            g.fillOval(x, y, ancho, alto);
        } else if (figura == "Rectangulo") {
            g.setColor(Color.RED);
            g.fillRect(x, y, ancho, alto);
        }
        
    }
 
}
 

Gracias de antemano.

[CalgaryCorpus]

No tengo idea la respuesta, pero mirando el codigo que presentas:
- el constructor por default ( el que no tiene parametros) tiene un setVisible(true), mientras que el otro constructor no tiene tal setVisible, alguna razon?
- el constructor por default no estable que tipo de figura es y por tanto no va a dibujar nada.
- cuando usas el constructor por default (las lineas que dices que borras), el constructor hace su setVisible y tu luego lo vuelves a hacer visible haciendo un pintar.setVisible(true), por que?
- por que dices que deberian aparecer 2 figuras? el codigo esta preparado para dibujar 2 formas, pero tal como tienes la clase, el objeto que crees solo sera 1 de ellos, no los 2.

[Jose789]

Iniciado por CalgaryCorpus

No tengo idea la respuesta, pero mirando el codigo que presentas:
- el constructor por default ( el que no tiene parametros) tiene un setVisible(true), mientras que el otro constructor no tiene tal setVisible, alguna razon?
- el constructor por default no estable que tipo de figura es y por tanto no va a dibujar nada.
- cuando usas el constructor por default (las lineas que dices que borras), el constructor hace su setVisible y tu luego lo vuelves a hacer visible haciendo un pintar.setVisible(true), por que?
- por que dices que deberian aparecer 2 figuras? el codigo esta preparado para dibujar 2 formas, pero tal como tienes la clase, el objeto que crees solo sera 1 de ellos, no los 2.

Soy novato en esto de la programacion, y no tengo ni P*ta de idea de como hacerlo si me pudieras echar alguna mano me seria de gran ayuda

cada figura (Circulo, rectangulo) seria en 2 métodos distintos?

[chuidiang]

Hola:

Tal cual lo tienes, pintar es una ventana que pinta una sola figura. El método paint() solo pinta una figura.

Si quieres que la misma ventana pinte varias figuras tendrías que:
- Dentro de pintar, figura, clor, ancho, alto, x e y deberían ser arrays o listas, de foram que el índice 0 de cad uno de ellos sea una figura, el indice 1 otra, etc.
- En vez de pasar los datos en el constructor de pintar, pon un método añadeFigura(......) que añada una nueva figura a una nueva posición del array (en vez de array, igual sería más comodo un ArrayList o LinkedList, que permite añadir).
- El paint() debería tener un bucle para los arrrays, pintando cada figura, más o menos con el código que tienes ahora.

Para que quede mejor, en vez de 6 arrays (figura, clor, x, y, ancho y alto), podías hacer una clase Figura con estos 6 parámetros. Así tu clase pintar tendría un array de Figura y quedaría el código más limpio.

Saludos.

[Jose789]

Iniciado por chuidiang

Hola:

Tal cual lo tienes, pintar es una ventana que pinta una sola figura. El método paint() solo pinta una figura.

Si quieres que la misma ventana pinte varias figuras tendrías que:
- Dentro de pintar, figura, clor, ancho, alto, x e y deberían ser arrays o listas, de foram que el índice 0 de cad uno de ellos sea una figura, el indice 1 otra, etc.
- En vez de pasar los datos en el constructor de pintar, pon un método añadeFigura(......) que añada una nueva figura a una nueva posición del array (en vez de array, igual sería más comodo un ArrayList o LinkedList, que permite añadir).
- El paint() debería tener un bucle para los arrrays, pintando cada figura, más o menos con el código que tienes ahora.

Para que quede mejor, en vez de 6 arrays (figura, clor, x, y, ancho y alto), podías hacer una clase Figura con estos 6 parámetros. Así tu clase pintar tendría un array de Figura y quedaría el código más limpio.

Saludos.

Gracias por aclararme un poco mas el asunto... pero sigo sin poder mostrar las figuras geométricas. es como si en pintar.java nunca se ejecutara el método paint(Graphicpaints g)

Tengo una nueva clase figura.java

Código Java:

package graficos2d;
 
import java.util.Vector;
 
public class figura {
    
    Vector<String> V_tipo = new Vector();
    Vector<String> V_color = new Vector();
    Vector<Integer> V_x = new Vector();
    Vector<Integer> V_y = new Vector();
    Vector<Integer> V_alto = new Vector();
    Vector<Integer> V_ancho = new Vector();
    
    
    public void AddFigura(String tipo,String clor,int x,int y,int height,int width){
        
      V_tipo.add(tipo);
      V_color.add(clor);
      V_x.add(x);
      V_y.add(y);
      V_alto.add(height);
      V_ancho.add(width);
      
      System.out.println(tipo+" "+clor+" "+x+" "+y+" "+height+" "+width);
      
    }
    
}
 

esto es lo que cambio en pintar.java

Código Java:

@Override
    public void paint(Graphics g) {
        
        super.paint(g);
        figura figura = new figura();
        
        for(int c=0; c<figura.V_tipo.size(); c++){
            
            System.out.println("Pintamos");
            
            g.setColor(Color.BLUE); 
            g.fillOval(figura.V_x.get(c), figura.V_y.get(c), figura.V_ancho.get(c), figura.V_alto.get(c));
            
        }
}
 

formulario.java

Código Java:

String tipo = String.valueOf(figura.getSelectedItem());
        String clor = String.valueOf(color.getSelectedItem());
        
        int x = Integer.parseInt(X.getText());
        int y = Integer.parseInt(Y.getText());
        int height= Integer.parseInt(alto.getText());
        int width = Integer.parseInt(ancho.getText());
        
        figura figura = new figura();
        figura.AddFigura(tipo, clor, x, y, height, width);
 

Y esto seria el Main.java

Código Java:

public static void main(String[] args) {
       
        formulario form = new formulario();//Interfaz Gráfica
        
        pintar pintar = new pintar();//Interfaz Gráfica donde se ven las figuras geométricas dibujadas
        
    }
 

[chuidiang]

Hola:

En el método paint() estás haciendo un "new figura()", es decir, estas creando un nuevo objeto figura al que nunca has llamado a sus métodos AddFigure(), así que paint() no pintera ninguna figura simplemente porque el objeto figura que tiene, recién creado, no tiene datos dentro.

En tu formulario haces un "new figure()" (OTRO objeto figura distinto del anerior) al que llamas una vez a AddFigura. No pones el resto del código, pero si cada vez que en el formulario introduces los datos de una figura haces un new figura(), pierdes todo lo anterior. Y ademas no es el mismo objeto figura que el del paint() donde creas otro mas y encima vacio.

Solo tienes que hacer un new de figura en un sitio y pasárselo a todas las demas clases. En el main podrias poner algo asi

Código:

... main ... {
   figure figura = new figure()
   formulario form = new formulario( figure );
   pintar pintar = new pintar( figure )
}

pintar y figure deben guardarse el objeto figure que reciben en el constructor en algún atributo de la clase y usar ese atributo para AddFigure() y para paint(), sin ahcer más new de figura.

Saludos.

[Jose789]

Iniciado por chuidiang

Hola:

En el método paint() estás haciendo un "new figura()", es decir, estas creando un nuevo objeto figura al que nunca has llamado a sus métodos AddFigure(), así que paint() no pintera ninguna figura simplemente porque el objeto figura que tiene, recién creado, no tiene datos dentro.

En tu formulario haces un "new figure()" (OTRO objeto figura distinto del anerior) al que llamas una vez a AddFigura. No pones el resto del código, pero si cada vez que en el formulario introduces los datos de una figura haces un new figura(), pierdes todo lo anterior. Y ademas no es el mismo objeto figura que el del paint() donde creas otro mas y encima vacio.

Solo tienes que hacer un new de figura en un sitio y pasárselo a todas las demas clases. En el main podrias poner algo asi

Código:

... main ... {
   figure figura = new figure()
   formulario form = new formulario( figure );
   pintar pintar = new pintar( figure )
}

pintar y figure deben guardarse el objeto figure que reciben en el constructor en algún atributo de la clase y usar ese atributo para AddFigure() y para paint(), sin ahcer más new de figura.

Saludos.

Mis conocimientos no llegan a mas...

En los constructores de las demás clases donde le paso figure como se haría?

[chuidiang]

Hola:

Código:

public class pintar{
   private figura figura;

   public pintar(figure figura) {
       this.figura = figura;
   }

   public void paint(Graphics g) {
       .... this.figura ....
   }

}

y la otra similar.

Un detalle nada mas que no tiene que ver mucho con esto. En java es habitual poner el nombre de clase la primera con mayúsculas. Tus clases deberían ser Pintar, Figura, etc. No es importante, pero la costumbre está tan arraigada que despista bastante cuando no se ve así, con "figura" no sabemos si se refiere a la clase o la variable.

Saludos.

[Jose789]

Iniciado por chuidiang

Hola:

Código:

public class pintar{
   private figura figura;

   public pintar(figure figura) {
       this.figura = figura;
   }

   public void paint(Graphics g) {
       .... this.figura ....
   }

}

y la otra similar.

Un detalle nada mas que no tiene que ver mucho con esto. En java es habitual poner el nombre de clase la primera con mayúsculas. Tus clases deberían ser Pintar, Figura, etc. No es importante, pero la costumbre está tan arraigada que despista bastante cuando no se ve así, con "figura" no sabemos si se refiere a la clase o la variable.

Saludos.

Muchísimas gracias por la ayuda, ya hace lo que debería de hacer. Y ya para acabar una última cosilla.

De un combo box recogo el nombre del color (String)

Pero como luego con el nombre del color puedo hacer el g.SetColor no sé si me explico.

he pensado en hacer un switch poner los colores que tengo el combo box y directamente establecer el color según cual sea, pero no sé si sería una buena manera

[chuidiang]

Hola:

Lo más sencillo es rellenar un Map con los colores que quieras, algo así (pongo de memoria, tendrás que revisar sintaxis y tal)

Código:

Map <String, Color> colores = new HashMap<>();
colores.put("rojo", Color.red);
colores.put("verde", Color.green);
....

para rellenar el combo, usa colores.keys() y obtendras la lista de strings a meter en el combo. Para recuperar el color a partir del string, basta un colores.get(textoDelCombo).

Color es la clase de java.awt.Color https://docs.oracle.com/javase/7/doc...awt/Color.html

Echale un ojo también a JColorChooser, igual te vale mejor https://docs.oracle.com/javase/tutor...orchooser.html

Saludos.

[Jose789]

Iniciado por chuidiang

Hola:

Lo más sencillo es rellenar un Map con los colores que quieras, algo así (pongo de memoria, tendrás que revisar sintaxis y tal)

Código:

Map <String, Color> colores = new HashMap<>();
colores.put("rojo", Color.red);
colores.put("verde", Color.green);
....

para rellenar el combo, usa colores.keys() y obtendras la lista de strings a meter en el combo. Para recuperar el color a partir del string, basta un colores.get(textoDelCombo).

Color es la clase de java.awt.Color https://docs.oracle.com/javase/7/doc...awt/Color.html

Echale un ojo también a JColorChooser, igual te vale mejor https://docs.oracle.com/javase/tutor...orchooser.html

Saludos.

No se como agradecerte todo lo que me has enseñado, Muchísimas gracias