Was ist falsch?

[Phenix]

Hi an alle. Ich habe hier eine Implementierung des ActionListeners. Leider funktioniert er nicht wie gewollt. Wäre nett, wenn jemand mal einen Blick drauf werfen könnte und mir sagen könnte, was ich da falsch mache. Habe zuvor die Arrays erstellt und ActionListener geaddet. Manchmal funktioniert die Methode, manchmal nicht. Aber nie funktioniert sie richtig.

    //The ActionListeners
                 public void actionPerformed(ActionEvent e) 
       {
           if(checkArray[0].isSelected())
           {
               if(descriptionsOn)
               {
                   for(int i=0; i<anzahl; i++)
                   {
                       jLabelDescribesRed[i].setVisible(true);
                       repaint();
                    }
                }
                if(valuesOn)
                {
                    for(int i=0; i<anzahl; i++)
                    {
                        jLabelArrayRed[i].setVisible(true);
                        repaint();
                    }
                }
                for(int i=0; i<anzahl; i++)
                {
                    chartsArrayRed[i].setVisible(true);
                    repaint();
                }
                repaint();
            }
        //-------------    
            if(checkArray[1].isSelected())
           {
               if(descriptionsOn)
               {
                   for(int i=0; i<anzahl; i++)
                   {
                       jLabelDescribesBlack[i].setVisible(true);
                       repaint();
                    }
                }
                if(valuesOn)
                {
                    for(int i=0; i<anzahl; i++)
                    {
                        jLabelArrayBlack[i].setVisible(true);
                        repaint();
                    }
                }
                for(int i=0; i<anzahl; i++)
                {
                    chartsArrayBlack[i].setVisible(true);
                    repaint();
                }
                repaint();
            }
       //-------------      
           if(checkArray[2].isSelected())
           {
               if(descriptionsOn)
               {
                   for(int i=0; i<anzahl; i++)
                   {
                       jLabelDescribesYellow[i].setVisible(true);
                       repaint();
                    }
                }
                if(valuesOn)
                {
                    for(int i=0; i<anzahl; i++)
                    {
                        jLabelArrayYellow[i].setVisible(true);
                        repaint();
                    }
                }
                for(int i=0; i<anzahl; i++)
                {
                    chartsArrayYellow[i].setVisible(true);
                    repaint();
                }
                repaint();
            } 
        //-------------     
           if(checkArray[3].isSelected())
           {
               if(descriptionsOn)
               {
                   for(int i=0; i<anzahl; i++)
                   {
                       jLabelDescribesGreen[i].setVisible(true);
                       repaint();
                    }
                }
                if(valuesOn)
                {
                    for(int i=0; i<anzahl; i++)
                    {
                        jLabelArrayGreen[i].setVisible(true);
                        repaint();
                    }
                }
                for(int i=0; i<anzahl; i++)
                {
                    chartsArrayGreen[i].setVisible(true);
                    repaint();
                }
                repaint();
            }
            
            //-------------     
           if(checkArray[4].isSelected())
           {
               if(descriptionsOn)
               {
                   for(int i=0; i<anzahl; i++)
                   {
                       jLabelDescribesRed2[i].setVisible(true);
                       repaint();
                    }
                }
                if(valuesOn)
                {
                    for(int i=0; i<anzahl; i++)
                    {
                        jLabelArrayRed2[i].setVisible(true);
                        repaint();
                    }
                }
                for(int i=0; i<anzahl; i++)
                {
                    chartsArrayRed2[i].setVisible(true);
                    repaint();
                }
                repaint();
            }
           
           
            
            
            
            if(checkArray[3].isSelected())
           {
               if(descriptionsOn)
               {
                   for(int i=0; i<anzahl; i++)
                   {
                       jLabelDescribesGreen[i].setVisible(true);
                       repaint();
                    }
                }
                if(valuesOn)
                {
                    for(int i=0; i<anzahl; i++)
                    {
                        jLabelArrayGreen[i].setVisible(true);
                        repaint();
                    }
                }
                for(int i=0; i<anzahl; i++)
                {
                    chartsArrayGreen[i].setVisible(true);
                    repaint();
                }
                repaint();
            }
           
           
           
           
           
           //Sets Labels, Valueslabels and Charts according to the Checkbox invisible
            if(checkArray[0].isSelected()==false)
            {
                    for (int i=0; i<anzahl; i++)
                    {
                        chartsArrayRed[i].setVisible(false);
                        jLabelDescribesRed[i].setVisible(false);
                        jLabelArrayRed[i].setVisible(false);
                        repaint();
                    }
            }
            
            if(checkArray[1].isSelected()==false)
            {
                    for (int i=0; i<anzahl; i++)
                    {
                        chartsArrayBlack[i].setVisible(false);
                        jLabelDescribesBlack[i].setVisible(false);
                        jLabelArrayBlack[i].setVisible(false);
                        repaint();
                    }
            }
            
            if(checkArray[2].isSelected()==false)
            {
                    for (int i4=0; i4<anzahl; i4++)
                    {
                        chartsArrayYellow[i4].setVisible(false);
                        jLabelDescribesYellow[i4].setVisible(false);
                        jLabelArrayYellow[i4].setVisible(false);
                        repaint();
                    }
            }
            
            if(checkArray[3].isSelected()==false)
            {
                    for (int i5=0; i5<anzahl; i5++)
                    {
                        chartsArrayGreen[i5].setVisible(false);
                        jLabelDescribesGreen[i5].setVisible(false);
                        jLabelArrayGreen[i5].setVisible(false);
                        repaint();
                    }
            }
            
            if(checkArray[4].isSelected()==false)
            {
                    for (int i2=0; i2<anzahl; i2++)
                    {
                        chartsArrayRed2[i2].setVisible(false);
                        jLabelDescribesRed2[i2].setVisible(false);
                        jLabelArrayRed2[i2].setVisible(false);
                        repaint();
                    }
            }
            
            
            repaint();
        }

Danke schonmal

 

[Ebenius]

Mach mal am Ende der Methode

SwingUtilities.invokeLater(new Runnable() {
  @Override
  public void run() {
    revalidate();
    repaint();
  }
});

Funktioniert das? Ansonsten: Beschreib mal das Problem.

 

[0x7F800000]

Wäre nett, wenn jemand mal einen Blick drauf werfen könnte und mir sagen könnte, was ich da falsch mache.

von den 230 zeilen code sind mindestens 200 zuviel. Alles was nicht dry ist, ist dreck. Schmeiß es schleunigst alles raus, wozu gibts denn bitte schleifen??? :roll:

 

[Phenix]

ich weiss aber nich wie ich das anders machen soll.

von den 230 zeilen code sind mindestens 200 zuviel. Alles was nicht dry ist, ist dreck. Schmeiß es schleunigst alles raus, wozu gibts denn bitte schleifen???:roll:

...denn ich muss doch die Checkboxes die in dem CheckArray drin sind enzln überprüfen der nicht? Je nach Checkbox soll ja auch was unterschiedliches gelöscht werden.

 

[0x7F800000]

...denn ich muss doch die Checkboxes die in dem CheckArray drin sind enzln überprüfen der nicht? Je nach Checkbox soll ja auch was unterschiedliches gelöscht werden.

woher soll ich das denn wissen, du hast doch nicht erzählt was es alles ist und wozu es gut sein soll. Aber ich sehe code. SEHR VIEL CODE. Der dazu an jeder stelle fast gleich aussieht. Willst du mir erzählen, dass du völlig ohne jeglichen erkennbaren Muster den code erstellt hast? Denk ich nicht. Falls da aber ein muster drin ist, dann solltest du das Muster ausformulieren, und vom rechner umsetzen lassen, statt es selbst alles auszuschreiben. Dem rechner macht's nichts, der mag solche stumpfsinnigen sachen.

 

[Phenix]

Ok. Ich werd mal drüber nachdenken und das verbessern. Allerdings bleibt noch das Problem, das der das nicht richtig umsetzt. Habe probiert, dass von Ebenius einzufügen, aber hab das irgendwie nich geschafft
Kann das denn daran liegen? Sowas hab ich noch nie benutzt. Was macht das überhaupt genau?

Thx Leutz

 

[diggaa1984]

Allerdings bleibt noch das Problem, das der das nicht richtig umsetzt

glaub du müsstest problem mal definieren
is nicht ganz ersichtlich bei dem code

 

[Phenix]

Ok, also ich hab das Problem, dass irgendwie nichts geschieht, wenn ich diese Checkboxes halt nicht mehr selektiert habe. EIgentlich sollen bei einem Klick auf eine Checkbox mehrere grafische Komponenten verschwinden. Allerdings passiert nichts, obwohl ich die Listener hinzugefügt habe.

 

[java2000]

Weisst du denn auch sicher, dass die Methode actionPerformed abgearbeitet wird? Kannst du ja rausfinden, indem du z.B. ein System.out.println("actionPerformed meldet sich") irgendwo einfügst und dann sollte das irgendwo in der Konsole gedruckt werden, wenn die Methode wirklich abgearbeitet wird.

Ausserdem kannst du unter Umständen noch ein paar repaint()s in den for-Schleifen sparen, am Ende der actionPerformed machst du das ja sowieso.

 

[Phenix]

Hat sich inzwischen erledigt das Problem. Danke an alle.