Problem bei Modulen

[Meteor86]

Hallo zusammen,

hab ein Problemchen bei einer Aufgabe. Und zwar müssen wir zwei Klassen erstellen, Würfel und Ausgabe. In Würfel soll die ganze Berechnung implementiert werden und in Ausgabe eben die Ausgabe 😉. Hab das auch soweit hinbekommen, bis auf eine Sache. Eine Aufgabenstellung besteht darin, auszugeben, welcher Würfel größer bzw. kleiner ist. Testweise hab ich bei nur einer Klasse mit der if-schleife auch hinbekommen, nur bei zwei Klassen hab ich ein Problem, dass er manchmal die falsche Aussage ausgibt. Hier meine zwei Quellcodes:
[Java]package tools;

public final class Wuerfel {
private Wuerfel() {};
public static int wuerfel6() {
return (int) (java.lang.Math.random()*6+1);
}
public static int wuerfel20() {
return (int) (java.lang.Math.random()*20+1);
}
public static int wuerfel2d6() {
int WuerfelSumme= wuerfel6()+ wuerfel6();
return WuerfelSumme;
}
}[/code]

und

 package spiel;
import tools.Wuerfel;
public final class Ausgabe {
	public static void main(String[] args) {
		ausgabeD6();
		ausgabeD20();
		ausgabegroeßer();
	}
	private static void ausgabegroeßer() {
		if (Wuerfel.wuerfel2d6() > Wuerfel.wuerfel20()){			
			System.out.println("*** Die D6 Würfel haben gewonnen***");
		}	
			else {			
				System.out.println("*** Der D20 Würfel hat gewonnen***");
		        }
	}
	private static void ausgabeD20() {
		System.out.println("Die Augenzahl des D20 ist: "+Wuerfel.wuerfel20());
	}
	private static void ausgabeD6() {
		System.out.println("Die Summe der Augenzahlen der beiden D6-Würfel ist: "
				+Wuerfel.wuerfel2d6());
	}
}

Er gibt also manchmal "Der D20 Würfel hat gewonnen" obwohl die D6 Würfel eine höhere Zahl ausgeben. Hoffe, ich hab es verständlich erklährt^^. Paar Tipps wären sehr hilfreich😉.

mfg

 

[AwsmDude]

Falls du nur die Texte vergleichst:
Deine ausgegebenen Würfelwerte haben auch nichts mit denen zu tun die du vergleichst.
Beim vergleichen lässt du dir neue Zahlen generieren die du nie ausgibst.

 

[preachie]

Hier ist ein Tipp:

Jedes mal wenn Du die statische Methode wuerfel6() bzw. wuerfel20() aufrufst, wird ein neu generierter Zuffalswert zurückgeliefert.
Daher wird der Wert des Würfels beim Vergleich welcher Wurf größer ist ein andere sein, der ursprünglich bei der Ausgabe erzeugt wurde.
Du müsstest in der Ausgabe Klasse die Methode wuerfel6() und wuerfel20() nur einmal aufrufen und die Ergebnisse in einer lokalen Variablen zur weiteren Verarbeitung schreiben.

Davon abgesehen, gehört das mit zur Aufgabe, dass Du die Würfel() Methoden statisch schreiben sollst?

 

[Meteor86]

ok danke, habs jetzt hinbekommen. :toll: Wußte nicht, dass er die vergleichenden Zahlen einfach neu generiert. Habs mit der lokalen Variable gemacht. Und wegen der statischen Methoden...gibt es da noch ne andere schreibweise? Sind erst am Anfang vom Programmieren, deshalb kenne ich nur diese.
Könnte sein, dass ich später noch ne Frage habe aber ich versuch mich erst noch an dem Problem 😉

 

[preachie]

Auch wenn es in dem simplen Beispiel noch keine große Rolle spielt, würde ich die Würfel schon als "richtige" Objekte betrachten. Du hast zwar eine Klasse "Wuerfel", instanzierst daraus aber kein Objekt, sondern verwendest die enthaltenen Methoden statisch.

Hier mal ein Beispiel, wie ich das ungefähr angehen würde (ohne mir jetzt tiefgreifende Gedanken drüber gemacht zu haben 😉)

Wuerfel.java

public class Wuerfel
{
	private int seiten;
	private int ergebnis;
	
	protected Wuerfel(int seiten)
	{
		this.seiten = seiten;
	}
	
	protected void wurf(int anzahlWuerfel)
	{
		ergebnis = 0;
		
		if (anzahlWuerfel <= 0)
		{
			System.out.println("Man sollte schon mit mindestens einem Würfel würfeln ;-)");
		}
		else
		{
			for (int i = 0; i < anzahlWuerfel; i++)
			{
				ergebnis += (int)(Math.random() * seiten + 1);				
			}
		}
	}
	
	protected int holeErgebnis()
	{
		return ergebnis;
	}
}

Ausgabe.java

public class Ausgabe
{
	public static void main(String[] args)
	{
		Wuerfel w6 = new Wuerfel(6);
		Wuerfel w20 = new Wuerfel(20);
		
		w6.wurf(2);
		w20.wurf(1);
		
		System.out.println("Der/die W6er wirft/werfen eine " + w6.holeErgebnis());
		System.out.println("Der/die W20er wirft/werfen eine " + w20.holeErgebnis());
		
		if (w6.holeErgebnis() > w20.holeErgebnis())
		{
			System.out.println("Der/die W6er gewinnt!");
		}
		else if (w6.holeErgebnis() < w20.holeErgebnis())
		{
			System.out.println("Der/die W20 gewinnt!");
		}
		else
		{
			System.out.println("Beide Würfelarten sind gleich!");
		}
	}
}

Man kann es natürlich noch optimieren, beispielsweise die Ergebnisse der einzelnen Würfel zu speichern, wenn mit mehr als einem Würfel gewürfelt wird.
Der Code sollte sich eigentlich von selbst erklären. Bei Fragen einfach her damit.

 

[Meteor86]

erstmal danke für die ganzen antworten :toll:

aber die struktur der klasse wuerfel ist quasi vorgegeben, deshalb kann ich da nicht viel ändern(noch nicht), liegt an der übung. habe das mit dem vergleichen auch hin bekommen. nur hab ich jetzt ein weiterer problem, an dem ich fast verzweifle: man soll den Mittelwert aller würfel bei 20 versuchen berechnen. Also z.b. soll ausgeben werden "der mittelwert des ersten d6 würfels ist 5.4, der mittelwert des zweiten d6 würfels 3.3 und des d20 würfels 5.5"

als vorgabe in der klasse wuerfel ist

//@param
//@return
private static int wuerfel(int n){

}

vorgegeben.

Und ich hab irgendwie keine ahnung, wie man jetzt aus der Klasse Ausgabe alles aufruft, besonders mit der varible n! würde anstatt des n nur 20 stehen (also 20 würfe), dann hät ich ne for schleife reingeknallt wie bei der übung davor und gut is aber das versteh ich irgendwie nicht.
hab in der klasse wuerfel das geschrieben:

//@param
//@return
private static int wuerfel(int n){
return (int) (java.lang.Math.random ()*n+1);
}

ist das soweit richtig? nur wie berechne ich es dann in der klasse ausgabe bzw. wie gib ich es dort aus? :autsch:

 

[preachie]

Ich stelle mir da zuächst 2 Fragen...

1.) Was sagt die Variable n aus? Ist es die Anzahl der Würfe? Ist es die Anzahl der Würfelseiten?
2.) Was soll die Methode wuerfel zurückliefern? Den geworfenenen Wert? Oder einfach nur den Mittelwert aller Würfe?

Ich frage deshalb, weil wenn n = Anzahl der Würfe sein soll und die Methode den geworfenen Wert zurückliefern soll, passt da irgendwas nicht zusammen, da die Methode nur einen einzelnen int Wert zurückliefern kann.

Soll sie allerdings den Mittelwert zurückliefern, würde es ungefähr so aussehen:

private static int wuerfel(int n)
{
	int temp = 0;
	
	for (int i = 0; i < n; i++)
	{
		temp += (int)(java.lang.Math.random ()*<Seitenzahl des Würfels>+1);
	}

	// Aufpassen, dass es keinen Div/0 Fehler geben kann
	if (n != 0)
	{
		return temp / n;
	}
	else
	{
		return 0;
	}
}

Allerdings wäre das Ergebnis dann immer eine Ganzzahl da int und so wie ich Dich verstanden habe, sollen ja wohl auch die Nachkommastellen ausgegeben werden?
Im Moment bin ich ein wenig ratlos, was überhaupt erreicht werden soll, mit den vorgegebenen Informationen. Hast Du eventuell noch mehr Details zu den Vorgaben?

 

[Meteor86]

Also die Aufgabenstellung lautet: Simuliert den Wurf eines Würfels mit n Seiten. Liefert eine zufällige Augenzahl zwischen 1 und n; jede Zahl ist gleich warscheinlich. @param Maximale Augenzahl, @return Zufallszahl zwischen 1 und n.
Und Aufgebaut ist das so: Zwei Klassen, Wuerfel.java mit der Vorgabe

 private static int wuerfel(int n){

}

und der Klasse Ausgabe. In der Klasse soll die Ausgabe stattfinden, ausgegeben werden soll:
- Mittelwert von 20 Würfen des ersten Würfels ist: X.X
- Mittelwert von 20 Würfen des zweiten Würfels ist: X.X
- Mittelwert von 20 Würfen des dritten Würfels ist: X.X

Ist das überhaupt machbar nur mit der Methode oder muss noch eine dazu geschrieben werden? Bin noch selber am Testen, also ganz hab ich noch nicht aufgegeben. 😉

 

[XHelp]

So wie ich die Codeschnipsel weiter oben sehe, kannst du schon einen N-beliebigen Würfel würfeln.
Was du jetzt brauchst ist eine Schleife, eine Summe und eine Division. Die musst du ja natürlich irgendwo haben, also sogesehen musst du schon noch eine Methode schreiben. (auch wenn es die main ist)

 

[Meteor86]

ok thx für die ganzen infos, habs jetzt hinbekommen :toll: