0

Estoy tratando de hacer una aplicacion que me dibuje varios circulos en un JFrame con multiples Threads pero a la hora de correr el programa solo me dibuja 2 circulos y no encuentro el problema si alguien podria apoyarme le dejo el codigo abajo.

package juego2;

import java.awt.Color;
import java.awt.Graphics;
import java.util.Scanner;
import javax.swing.JFrame;
import java.util.Random;

public class simulacion extends JFrame implements Runnable {
    Scanner scanner = new Scanner(System.in);
    int cantidad;
    int aleatoriox,aleatorioy,aleatoriox1,aleatorioy1;

    public simulacion() {    
        System.out.println("¿# de Agentes?");
        cantidad = scanner.nextInt();
        Thread agentes[]= new Thread[cantidad];
        for(int i=0;i<cantidad;i++) {
            agentes[i] = new Thread(this);
            agentes[i].start();
            System.out.println("Hilo: "+i);
        }   
    }

    public void paint(Graphics g) {

        aleatoriox = (int) (Math.random() * 375) + 15;
        aleatorioy = (int) (Math.random() * 500) + 100;
        aleatoriox1 = (int) (Math.random() * 375) + 15;
        aleatorioy1 = (int) (Math.random() * 500) + 100;
        g.drawOval(aleatoriox,aleatorioy,aleatoriox1,aleatorioy1);

    }

    public static void main(String[] args) {
        simulacion frame = new simulacion();
        frame.setSize(400,600);
        frame.setVisible(true);
    }

    @Override
    public void run() {
        repaint();
    }
}

1 respuesta 1

0

Todos los threads trabajan con el mismo JFrame, modificando concurrentemente sus variables de instancia:

int aleatoriox,aleatorioy,aleatoriox1,aleatorioy1;

En lugar de eso, podrias mover el codigo de inicializacion de los threads al main, y el codigo de pintado al metodo run() usando variables locales:

public static void main(String[] args) {
    simulacion frame = new simulacion();
    frame.setSize(1000, 800);
    frame.setVisible(true);

    System.out.println("¿# de Agentes?");
    Scanner scanner = new Scanner(System.in);
    int cantidad = scanner.nextInt();
    Thread agentes[] = new Thread[cantidad];
    for (int i = 0; i < cantidad; i++) {
        agentes[i] = new Thread(frame);
        agentes[i].start();
        System.out.println("Hilo: " + i);
    }
}

@Override
public void run() {
    Graphics g = this.getGraphics();
    int aleatoriox = (int) (Math.random() * 375) + 15;
    int aleatorioy = (int) (Math.random() * 500) + 100;
    int aleatoriox1 = (int) (Math.random() * 375) + 15;
    int aleatorioy1 = (int) (Math.random() * 500) + 100;
    g.drawOval(aleatoriox, aleatorioy, aleatoriox1, aleatorioy1);
}

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.