JavaFx Zufallsfarbe beim Button-Klick

Sandro95 · 11. Aug 2020

guten tag leute ,
ich versuche seit gestern eine Zufallsfarbe per Button Klick zu implementieren.
Habe mich schlau gemacht , dennoch klappt es bei mir nicht und ich weiß nicht woran das liehgt .
Andere aus dem Netz haben das ebenso so gelöst.Vlt kann ja einer kurz drüber schauen und mir einen Tipp geben

viele dank leute!

VfL_Freak · 11. Aug 2020

Moin,

bitte poste Deinen Code mit Code-Tags (siehe meine Signatur)!
Nicht jeder wird 'irgendeine' ZIP-Datei öffnen wollen, außerdem ist leserlicher und besser zitierbar!

Danke und Gruß
Klaus

Sandro95 · 11. Aug 2020

meinste so ?

Java:

import java.util.Random;
import java.util.Scanner;

import com.sun.prism.paint.Color;

import javafx.application.Application;
import javafx.event.ActionEvent;
import javafx.geometry.Insets;
import javafx.geometry.Pos;
import javafx.scene.Scene;
import javafx.scene.control.Button;
import javafx.scene.layout.BorderPane;
import javafx.scene.layout.HBox;

import javafx.scene.shape.Rectangle;
import javafx.stage.Stage;

public class Fenster extends Application {
   
    public  void start(Stage primstage) {
       
        //Fenster art
        BorderPane border = new BorderPane();
       
        //Hbox für den Button
        HBox box = new HBox();
        //die Hbox nach unten setzen
        border.setBottom(box);
        //die HBox unten mittig setzen
        box.setAlignment(Pos.CENTER);
        // abstand zu den rändern
        box.setPadding(new Insets(10));
       
       
       
        Rectangle viereck = new Rectangle();
       
       
        viereck.setX(125);
        viereck.setY(100);
        viereck.setWidth(150);
        viereck.setHeight(150);
       
        Button wechsel = new Button("Farbenwechsel");
       
        wechsel.setPadding(new Insets(10));
       
       


        //das Viereck an der Pane hinzufügen
        border.getChildren().add(viereck);
        //den Button an der Hbox hinzufügen
        box.getChildren().add(wechsel);
       
        Scene szene = new Scene(border,400,400);
        primstage.setScene(szene);
        primstage.show();
       
        wechsel.setOnAction((ActionEvent e)-> {
            float farbe1 = (float) Math.random();
            float farb2  = (float) Math.random();
            float farbe3  = (float) Math.random();
           
            Random rnd = new Random();
         Color color = new Color(rnd.nextInt(),rnd.nextInt(),rnd.nextInt());
           
           
           
        });
       
       

       
       
       
       
    }

    public static void main(String[] args) {
        launch();
    }

   
}

NoName2 · 11. Aug 2020

Hab es mir angesehen, die setOnAction ist noch nicht richtig, hier mal mit 27 unterschiedlichen Farben:

Java:

	private static List<int[]> rgbs = new ArrayList<>();

	private static void iniRnd(int i, int[] rgb) {
		if (i == 3) {
			rgbs.add(rgb.clone());
			return;
		}
		for (int j = 0; j < 3; j++) {
			if (j == 0) {
				rgb[i] = 0;
			} else {
				rgb[i] = (j * 128) - 1;
			}
			iniRnd(i + 1, rgb);
		}
	}

	public static Color getRnd() {
		if (rgbs.isEmpty()) {
			iniRnd(0, new int[3]);
		}
		Collections.shuffle(rgbs);
		int[] a = rgbs.get(0);
		return new Color(a[0], a[1], a[2]);
	}

	public static void main(String[] args) {
		System.out.println(getRnd().toString());
		System.out.println(rgbs.size());
	}

VfL_Freak · 11. Aug 2020

Sandro95 hat gesagt.:
meinste so ?

Nicht wirklich, da ohne Code-Tags!
Wie ich schon schrieb: wirf einen Blick in meine Signatur !!

Und beschreibe ggf. auch auftretende Fehler und poste den Stacktrace!
VG Klaus

Sandro95 · 11. Aug 2020

NoName2 hat gesagt.:

Hab es mir angesehen, die setOnAction ist noch nicht richtig, hier mal mit 27 unterschiedlichen Farben:

Java:

    private static List<int[]> rgbs = new ArrayList<>();

    private static void iniRnd(int i, int[] rgb) {
        if (i == 3) {
            rgbs.add(rgb.clone());
            return;
        }
        for (int j = 0; j < 3; j++) {
            if (j == 0) {
                rgb[i] = 0;
            } else {
                rgb[i] = (j * 128) - 1;
            }
            iniRnd(i + 1, rgb);
        }
    }

    public static Color getRnd() {
        if (rgbs.isEmpty()) {
            iniRnd(0, new int[3]);
        }
        Collections.shuffle(rgbs);
        int[] a = rgbs.get(0);
        return new Color(a[0], a[1], a[2]);
    }

    public static void main(String[] args) {
        System.out.println(getRnd().toString());
        System.out.println(rgbs.size());
    }

geht das ganze nicht einfacher ?🧐

NoName2 · 11. Aug 2020

Sandro95 hat gesagt.:
geht das ganze nicht einfacher

Man könnte selbstverständlich die Zufallszahlen direkt generieren bei jedem Durchlauf, anstatt die Liste zu mischen, aber ansonsten nein. 😉

Stackoverflow empfiehlt auch, die möglikchen Farben vorher anzulegen.

Sandro95 · 11. Aug 2020

NoName2 hat gesagt.:
Man könnte selbstverständlich die Zufallszahlen direkt generieren bei jedem Durchlauf, anstatt die Liste zu mischen, aber ansonsten nein. 😉

Stackoverflow empfiehlt auch, die möglikchen Farben vorher anzulegen.

Wie würde das dann aussehen ?

MoxxiManagarm · 11. Aug 2020

Wieso nicht einfach

Java:

Color.color(Math.random(), Math.random(), Math.random())

Siehe https://docs.oracle.com/javase/8/ja.../paint/Color.html#color-double-double-double-

Sandro95 · 11. Aug 2020

MoxxiManagarm hat gesagt.:
Wieso nicht einfach

Java:

Color.color(Math.random(), Math.random(), Math.random())

Siehe https://docs.oracle.com/javase/8/ja.../paint/Color.html#color-double-double-double-

das funktioniert so iwie nicht , die Methode ist undefiniert steht dort dann

MoxxiManagarm · 11. Aug 2020

Sandro95 hat gesagt.:
das funktioniert so iwie nicht , die Methode ist undefiniert steht dort dann

Ist ja auch klar, du benutzt nicht JavaFX Color (blame myself). Ich kann aber die docs zu com.sun.prism.paint.Color nicht finden spontan, hast du docs dazu?

Edit:
Ah habe zumindest etwas SourceCode gefunden.

Java:

new Color(Math.random(), Math.random(), Math.random(), 1f)

Siehe https://jar-download.com/artifacts/...11/source-code/com/sun/prism/paint/Color.java

Das sollte passen. Der letzte Wert ist Transparenz. Wenn der auch zufällig sein soll musst du den noch ändern.

Sandro95 · 11. Aug 2020

MoxxiManagarm hat gesagt.:
Ist ja auch klar, du benutzt nicht JavaFX Color (blame myself). Ich kann aber die docs zu com.sun.prism.paint.Color nicht finden spontan, hast du docs dazu?

Edit:
Ah habe zumindest etwas SourceCode gefunden.

Java:

new Color(Math.random(), Math.random(), Math.random(), 1f)

Siehe https://jar-download.com/artifacts/...11/source-code/com/sun/prism/paint/Color.java

Das sollte passen. Der letzte Wert ist Transparenz. Wenn der auch zufällig sein soll musst du den noch ändern.

Funktioniert ebenfalls nicht, wieder undefiniert ..

Sandro95 · 11. Aug 2020

okay jetzt gehts , dennoch wird die Farbe beim klicken nicht geändert

NoName2 · 11. Aug 2020

Sandro95 hat gesagt.:
Wie würde das dann aussehen ?

Naja, all-in-one straightforward so:

Java:

	public static Color getRnd() {
		Random r = new Random();
		int[] rgb = new int[3];
		for (int i = 0; i < rgb.length; i++) {
			rgb[i] = r.nextInt(3) * 128 - 1;
			if (rgb[i] < 0) {
				rgb[i] = 0;
			}
		}
		return new Color(rgb[0], rgb[1], rgb[2]);
	}

mrBrown · 11. Aug 2020

Naja, dein Code macht ja auch nichts mit der Color...

(und bitte verzichte auf alles was aus com.sun.* kommt, die Dinge direkt aus JavaFX sind schon passend 🙂 )

NoName2 · 11. Aug 2020

MoxxiManagarm hat gesagt.:
Wieso nicht einfach

Das bildet die Farben nicht gleichmäßig ab, zum Beispiel hätten Grautöne eine höhere Wahrscheinlichkeit.

NoName2 · 11. Aug 2020

mrBrown hat gesagt.:
und bitte verzichte auf alles was aus com.sun.* kommt, die Dinge direkt aus JavaFX sind schon passend

Wen meinst du denn jetzt? Kenne mich mit JavaFX nicht aus.

mrBrown · 11. Aug 2020

NoName2 hat gesagt.:
Das bildet die Farben nicht gleichmäßig ab, zum Beispiel hätten Grautöne eine höhere Wahrscheinlichkeit.

In deinem Code genauso, ob man jetzt max(0, nextInt(3)*128-1) oder random() nimmt macht für die Verteilung keinen Unterschied.

NoName2 hat gesagt.:
Wen meinst du denn jetzt? Kenne mich mit JavaFX nicht aus.

Keine Sorge, nicht dich, dein Code ist ganz unabhängig davon schlecht 🤷‍♀️

NoName2 · 11. Aug 2020

Wie gesagt, mögliche Farben sollten im Vorhinein festgelegt werden.

lam_tr · 11. Aug 2020

MoxxiManagarm hat gesagt.:
Wieso nicht einfach

Java:

Color.color(Math.random(), Math.random(), Math.random())

Siehe https://docs.oracle.com/javase/8/ja.../paint/Color.html#color-double-double-double-

Hierbei ist noch zu beachten dass rgb jeweils nur bis 0 bis 254 geht.

lam_tr · 11. Aug 2020

Sandro95 hat gesagt.:
okay jetzt gehts , dennoch wird die Farbe beim klicken nicht geändert

An was willst du die Farbe anpassen, wenn es das Rectangle ist dann muss du in den Listener noch rechteck.setbackground machen.

mrBrown · 11. Aug 2020

lam_tr hat gesagt.:
Hierbei ist noch zu beachten dass rgb jeweils nur bis 0 bis 254 geht.

Ne, die Methode nimmt 0 - 1f, also genau das was Math.random produziert 🙂

lam_tr · 11. Aug 2020

mrBrown hat gesagt.:
Ne, die Methode nimmt 0 - 1f, also genau das was Math.random produziert 🙂

Ach Mist, ein Blick in den Code hast das wirklich bestätigt. Okay mein Fehler.

JavaFx Zufallsfarbe beim Button-Klick

Bekanntes Mitglied

Anhänge

Top Contributor

Bekanntes Mitglied

Mitglied

Top Contributor

Bekanntes Mitglied

Mitglied

Bekanntes Mitglied

Top Contributor

Bekanntes Mitglied

Top Contributor

Bekanntes Mitglied

Bekanntes Mitglied

Mitglied

Super-Moderator

Mitglied

Mitglied

Super-Moderator

Mitglied

Top Contributor

Top Contributor

Super-Moderator

Top Contributor

Ähnliche Themen