[cornetto456]

Hi, I have this problem where I have a box created and want to update the color after it is created but it stays black.

Here is the code:

       private static int ID;
       private static String user;
       private static int level;
       private static  int exp;
       private static  int r;
       private static int gr;
       private static int b;


    public void paintComponent(Graphics g){
         super.paintComponent(g);
        Color clr = new Color(r,gr,B)/>;
      
        g.fillRect(50, 50, 100, 100);   
        g.setColor(clr);
     super.repaint();
    }

    public static void setValues(String username,int ID,int level,int exp,int r,int g,int B)/>{
        createChar.user = username;
        createChar.ID = ID;
        createChar.level = level;
        createChar.exp = exp;
        createChar.r = r;
        createChar.gr = g;
        createChar.b = b;
        System.out.println(username +" " + ID + " " + level + " " + exp + " " + r + " " + g +" " + " " + b );
        System.out.println(createChar.user + createChar.ID + createChar.level+ createChar.exp + createChar.r + createChar.gr + createChar.B)/>;
    }

when I run it it just stays black, any help ?

Thanks!

[smohd]

The setValues() is not related to GUI part, so you will need to repaint after setting new values for your variables

[cornetto456]

smohd, on 23 February 2012 - 03:05 PM, said:

The setValues() is not related to GUI part, so you will need to repaint after setting new values for your variables

like this:

    private void jButton1MouseClicked(java.awt.event.MouseEvent evt) {                                      
    System.out.println(username + ID + level + exp + r +g +B)/>;
    jPanel1.repaint();
    } 

[CasiOo]

Why are you calling super.repaint() from your paintComponent method?

[pbl]

cornetto456, on 23 February 2012 - 05:09 PM, said:

smohd, on 23 February 2012 - 03:05 PM, said:

The setValues() is not related to GUI part, so you will need to repaint after setting new values for your variables

like this:

    private void jButton1MouseClicked(java.awt.event.MouseEvent evt) {                                      
    System.out.println(username + ID + level + exp + r +g +B)/>;
    jPanel1.repaint();
    } 

What will paint() or paintComponent() have to paint differently ?
Your jButton1MouseClicked() will have to change instance variables that paint()/paintComponet() will use to draw a different shapes/states

CasiOo, on 23 February 2012 - 05:18 PM, said:

Why are you calling super.repaint() from your paintComponent method?

Ya looks like an infinie loop to me CasiOo

[macosxnerd101]

Looks to me like use of the NetBeans GUI Builder. Please- write your own GUI by hand. Don't rely on the horrid code generated by the GUI Builder. Also, static variables are not good OO practice.