Raster mit zufälligen Farben

[Misch'l]

Hallo. Ich möchte ein Raster mit verschiedenfarbigen Rechtecken machen. D.h. dass jedes Rechteck zufällig eingefärbt wird. Komischerweise wählt das Programm die Farben nur Spaltenweise zufällig aus und ich habe keinen Plan warum.


So siehts aus:

   int maxX = getSize().width-getInsets().left-getInsets().right;
   int maxY = getSize().height-getInsets().top-getInsets().bottom;
      for(int x=1; x<=anzahl_zeilen;x++){
        for(int y=1; y<=anzahl_spalten; y++){
          g.setColor(new Color((int)(Math.random()*255),(int)(Math.random()*255),(int)(Math.random()*255)));
          g.drawRect((int) (x*maxX/anzahl_zeilen-maxX/anzahl_zeilen+getInsets().left), (int)(y*maxY/anzahl_spalten-y*maxY/anzahl_spalten+getInsets().top),(int) (maxX/anzahl_zeilen),(int) (y*maxY/anzahl_spalten));
        }
      }
   }

 

[Misch'l]

Habe oben einen Fehler gemacht. Ist natürlich g.fillRect und nicht drawRect. Hab's nur falsch kopiert.

 

[SlaterB]

vielleicht malst du jedes Mal die komplette Spalte?
mit so unverständlichen Code kann man das kaum erkennen,

baue doch ein Logging ein:
System.out.println(dies);
System.out.println(das);

dann bekommst du eine schöne Ausgabe:
male Zeile 1
male Feld 1,1 an Position x = .., y=.., Breite = .., Höhe = .., Farbe = r,g, b .., .., ..
male Feld 1,2 an Position x = .., y=.., Breite = .., Höhe = .., Farbe = r,g, b .., .., ..
male Zeile 2
usw.............

zum Anfang schön einfach mit wenigen Spalten, wenigen Zeilen,
vielleicht nur eine der drei Komponenten der Farbe zufällig,
für x, y, Breite, Höhe einfache Formeln a la zeilenindex * 50, Breite festgelegt auf Wert 50

man kann es sich so einfach machen oder auch so schwer wie in deinem Programm..

 

[kleiner_held]

Dein y Parameter in fillRect ist immer getInsets().top

(int)(y*maxY/anzahl_spalten-y*maxY/anzahl_spalten+getInsets().top)

 

[Misch'l]

Danke schön, so hats funktioniert. War wohl ein kleiner Flüchtigkeitsfehler. Ich frag mich nur, wieso es dann überhaupt die ganze Fläche gezeichnet hat :shock: