Array-Problem

[kartofelbauer]

so ich hoff das mir einer hier hilft, hab eben schon ne frage gestellt un es kam recht flott ne antwort
war aber ziemlich billig...

so jetzt zu dem prob.
ich brauch 32 buttons, diese will ich nicht alle per hand in den container einfügen, is verständlich oder
(diese buttons haben immer den gleichen abstand)

daher hab ich ein 2d Array erstellt und dieses gefüllt, aber ich glaube dabei is mir ein fehler unterlaufen,
weil ich immer nur 4 componenten aus dem kompletten array zeigen kann!
ich habe nur 4 JButtons erzeugt, das muss doch reichen oder muss ich echt 32 stück erstellen?
des kann doch nit sein, desmuss doch schneller gehen

wenn hier einer es gelöst hat kann derjenige mir dann bitte auch noch erklären warum es jetzt so ist und nicht so wie ich es gemacht habe?
ich bin eigentlich davon überzeugt, das es so funktionieren müßte! (macht es nur nicht .......................)

hier sind die quelltexte:

import javax.swing.*;
import java.awt.Color;

/**
 * @author Markus Ritter
 *
 * TODO To change the template for this generated type comment go to
 * Window - Preferences - Java - Code Style - Code Templates
 */
public class buttons {
   
    public static JButton bt1, bt2, bt3, bt4;
    public static JComponent[] rowIndex;
    public static JButton[][] neueButtons; 

    
    public buttons(){
        createnewButtons();   
        
    }   
    
   public static JButton[][] createnewButtons(){
       bt1 = new JButton("");
       bt1.setBackground(new Color(255,255,255));
       bt2 = new JButton("");
       bt2.setBackground(new Color(255,255,255));
       bt3 = new JButton("");
       bt3.setBackground(new Color(255,255,255));
       bt4 = new JButton("");
       bt4.setBackground(new Color(255,255,255));
       
      neueButtons = new JButton[8][4];
      
      for (int i=0; i<8; i++){
          neueButtons[i][0] = bt3;
          neueButtons[i][1] = bt2;
          neueButtons[i][2] = bt3;
          neueButtons[i][3] = bt4;
      }
      
      return neueButtons;
   }
    
    
}

import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
/**
 * @author Markus Ritter
 *
 * TODO To change the template for this generated type comment go to
 * Window - Preferences - Java - Code Style - Code Templates
 */
public class mainFile extends JFrame{
    
    Container c;
    JButton[][] array;
    
    public mainFile(){
        
        c = getContentPane();
        c.setLayout(null);
        
        this.setSize(500,600);
        this.setVisible(true);
        this.setTitle("Mastermind");
        
        buttons neuesButtonArray = new buttons();
        
        // hinzufügen der JButtons, die, die Farben,
        // die man vermutet darstellen sollen
        array = buttons.createnewButtons();     
        
        for(int k =0, y=70; k<8; k++, y=y+50){
            
            c.add(array[k][0]);
        
        	for (int i=0, x = 180; i<4; i++, x = x+50){                
            	    	c.add(array[k][i]);
            	    	array[k][i].setBounds(x,y,30,20);            	
        	}
        }
        
        this.show();
    }
    
    public static void main(String[] args){
        mainFile fu = new mainFile();
    }

}

 

[Illuvatar]

Du musst sogar nur 1 mal im Code einen Button erstellen , und zwar so:

 public static JButton[][] createnewButtons(){ 
    neueButtons = new JButton[8][4];     
    for (int i = 0; i < 4; i++)
      for (int j=0; j<8; j++){ 
          neueButtons[j][i] = new JButton("");
          neueButtons[j][i].setBackground(new Color(255,255,255));
      }       
    return neueButtons; 
  }

Zu deinem Problem:

array = buttons.createnewButtons(); //static/nicht static bisschen komisch vermischt in der buttons-Klasse, könnte auch array = buttons.neueButtons heißen. Btw: Klassennamen Großschreiben!
for(int k =0, y=70; k<8; k++, y=y+50){ 
            
            c.add(array[k][0]); //Das hier ist unnötig, da i in der nächsten Schleife mit 0 beginnt
        
           for (int i=0, x = 180; i<4; i++, x = x+50){                
                      c.add(array[k][i]); 
                      array[k][i].setBounds(x,y,30,20);                
           } 
        }

Mehr seh ich nicht. Aber ich denke eigentlich auch, deins müsste funktionieren ???:L

Edit 1: j und i waren vertauscht.

Edit 2: So klappt es jetzt, der Fehler lag doch in der createnewButtons-Methode, es waren nur vier Objekte, und deshalb wurde die setBounds-Methode immer wieder auf den gleichen Objekten ausgeführt.

 

[kartofelbauer]

super dankeschön..
des hab ich übersehen hrhr , war nur eine forschleife da man man man, naja irgendwann lässt das konzentrations vermögen halt doch nach...

danke