Ungerade Zufallszahlen in JAVA

[JavaUndC-Prog]

Hallo,

Ich möchte mir gerne zufällige Werte in einem Intervall generieren lassen und dabei sollen diese Werte ungerade sein.

Ich habe selbst ein bisschen gebastelt, aber leider bekomme ich des öfteren eine 0 geliefert und das kann eigentlich nicht sein, da die 0 nicht im Intervall inbegriffen ist.

import java.util.*;
public class Zufallszahlen_Tester {

	private static Random rand = new Random();
	
	public static int gen_zufzahl(int min, int max){
		return rand.nextInt(max - min + 1) + min;		
	}
	
	public static int gen_zufzahl_ungerade(int min, int max){
		int var;
		
		var = gen_zufzahl(min, max);
		int x = 0;
		if((var % 2 == 1)){
			x = var;
		}
		return x;
	}
	
	public static void main(String[] args){
		for(int i = 0; i < 10; i++){
			System.out.println(gen_zufzahl_ungerade(3, 11));
		}
	}
}

das ganze sollte bitte ohne Do While erfolgen, damit ist es nämlich einfacher ;-)
Danke für eure Hilfe

Alles sollte eigentlich in der Methode. public static int gen_zufzahl_ungerade(int min, int max)
Ablaufen

 

[VfL_Freak]

Moin,

var = gen_zufzahl( min, max );
int x = 0;
if( (var % 2 == 1) )
{
     x = var;
}
return x;

ist doch logisch ... immer dann, wenn Deine if-Bedingung nicht erfüllt ist, gibst Du hier einfach 'x' (also '0') zurück !
Vielleicht solltest Du Dir einen gescheiten else-Fall überlegen

Gruß Klaus

 

[JavaUndC-Prog]

Ja genau da liegt der Fehler, mir fällt leider keine gescheite Else Abfrage ein, außer goto und das möchte ich nicht.

 

[VfL_Freak]

Moin,

nun, ich weiß auch nicht, was Du in einem solchen Fall machen willst/sollst :bahnhof:

Du könntest bspw. den gefundenen Wert 'var' in- oder dekrementieren, um dann auf jeden Fall wieder eine ungerade Zahl zu bekommen.
Oder eben doch halt eine Schleife drumlegen, die solange läuft, bis ein 'echter' ungerader Wert ermittelt wurde .....

Your choice :bae:

Gruß Klaus

EDIT

das ganze sollte bitte ohne Do While erfolgen, damit ist es nämlich einfacher

was ist denn für an Schleifen so schwer ????

 

[JavaUndC-Prog]

Nein gerade weil die do while so einfach ist, möchte ich das nicht... ;-)

Ich hab mir etwas kluges überlegt switch case
Das haut hin, aber gibts noch eine "bessere" Lösung ?

 

[MisterBu]

Ungefähr so:

var [COLOR=#339933]=[/COLOR] gen_zufzahl[COLOR=#009900]([/COLOR] min/2, max/2 [COLOR=#009900])[/COLOR][COLOR=#339933];
var = 2 * var + 1;
[/COLOR]

Wobei man sich noch genauer mit den Grenzen beschäftigen sollte

 

[VfL_Freak]

Moin,

Nein gerade weil die do while so einfach ist, möchte ich das nicht... ;-)

hää ??

Ich hab mir etwas kluges überlegt switch case

Ob nun "switch" oder "if-else" ist doch (zumindest in diesem banalen Fall) völlig unerheblich! Was machst Du denn in den einzelnen Switches ???:L

gibts noch eine "bessere" Lösung?

den tenären Operator (Javabeginners - Ternaerer Operator) !!

Gruß Klaus

 

[JavaUndC-Prog]

Naja ein switch case der dann eine neue zahl generiert

 

[VfL_Freak]

aha, und wenn es dann wieder ein gerade Zahl wird ???
Irgendwie schreit das doch dann geradezu nach einer Schleife (ob nun WHILE oder FOR ist auch egal) ..... :noe:

 

[JavaUndC-Prog]

Stimmt ich habs mir nochmal genauer angesehen also auch nochmal in den Debugger geschaut. Du hast recht !

Aber kann ich eine Schleife nicht in einen IF Block umschrieben und wenn ja, wie in diesem Fall ?

 

[VfL_Freak]

Moin,

Aber kann ich eine Schleife nicht in einen IF Block umschrieben und wenn ja, wie in diesem Fall ?

So direkt sicher nicht, da das zwei Paar Schuhe sind !!
Mal im Pseudocode:

IF( bedingung_A )
    tue_A
ELSE
    tue_was_anderes

vs.

solange Bedingung_A gilt
    tue_was

Gruß Klaus

 

[JavaUndC-Prog]

Nun das verstehe ich, also gibt es keine andere "Einfache" Möglichkeit das mit einem IF und IF ELSE zu lösen ? Sondern nur mit While oder Do While ?

 

[MWin123]

Sondern nur mit While oder Do While ?

Mit einer For-Schleife geht es auch.

 

[VfL_Freak]

Moin,

also gibt es keine andere "Einfache" Möglichkeit das mit einem IF und IF ELSE zu lösen?

Was genau meinst Du eigentlich?
Das habe ich doch oben gepostet .....:noe:

Gruß Klaus

 

[JavaUndC-Prog]

Danke nochmal an alle hier, ihr habt mir gut weitergeholfen !

VfL_Freak deine Lösung mit if Else funktioniert aber nicht so gut, da wird immer noch 0 ausgegeben ?!

 

[VfL_Freak]

Moin,

VfL_Freak deine Lösung mit if Else funktioniert aber nicht so gut, da wird immer noch 0 ausgegeben ?!

(a) was bedeutet "nicht so gut" ???:L:noe:
(b) ist es wirklich so schwer den aktuellen Code zu posten und zu beschrieben was nicht funktioniert?

Gruß Klaus

 

[JavaUndC-Prog]

Es läuft, gibt aber teils 0 aus !

Die Else ist bei mir noch nicht definiert, dafür finde ich keine Lösung

 

[VfL_Freak]

Moin,

warum sollte sowas nicht funktionieren ??

int var = gen_zufzahl( min, max );
int x = 0;
if( (var % 2 == 1) )
{
    x = var;
}
else
{
    x = var++;
    // oder alternativ
    // x = var--;
}
return x;

Gruß Klaus

 

[JavaUndC-Prog]

Danke Klaus, das liefert bei mir aber:

11
11
11
11
4
6
4
4
10
3

 

[VfL_Freak]

Moin,

was meinst Du mit aber ??
Sieht doch gut aus !

Es halt das Prinzip von Zufallszahlen, dass sie zufällig kommen
Mit welchem min/max hast Du denn aufgerufen??

Gruß Klaus

 

[JavaUndC-Prog]

Mit 3 und 11.

Ja zufällig sind sie, mehr oder weniger :-D ;-)

Aber gerade Zahlen kommen auch vor 8, 4 ...

 

[VfL_Freak]

Ich denke, das willst Du im else-Fall ????????

 

[InfectedBytes]

Das Problem ist das "x=var++", in dem Fall sollte es wenn "x=++var" sein.
Denn die Zeile "x=var++;" ist im Grunde das selbe wie "x=var; var++;"

Allerdings wäre es weitaus einfacher folgendes zu machen:

int var = 2*gen_zufzahl( min, max )+1;

 

[JavaUndC-Prog]

Ich möchte nur ungerade zahlen herausbekommen

 

[InfectedBytes]

und genau das erhälst du mit der zeile doch auch

 

[JavaUndC-Prog]

Beil dieser Implementierung ?:

	public static int gen_zufzahl_ungerade(int min, int max){
		int var;
		int x = 0;
		var = gen_zufzahl(min, max);

		if((var % 2 == 1)){
			x = var;
		}
		else
		{
		    //x = var++;
		    
			var = 2*gen_zufzahl( min, max )+1;
		}
		return x;
	}

 

[InfectedBytes]

nein, entweder der unnötig komplizierte ansatz

	public static int gen_zufzahl_ungerade(int min, int max){
		int var;
		int x = 0;
		var = gen_zufzahl(min, max);
 
		if((var % 2 == 1)){
			x = var;
		}
		else
		{
		    x = ++var;
		}
		return x;
	}

oder nur die eine zeile:

var = 2*gen_zufzahl( min, max )+1;

bzw. in einer Mthode:

public static int gen_zufzahl_ungerade(int min, int max){
  return 2*gen_zufzahl( min, max )+1;
}

p.s. die grenzen min bzw. max sollten ggf noch angepasst werden

 

[VfL_Freak]

Moin,

Das Problem ist das "x=var++", in dem Fall sollte es wenn "x=++var" sein.
Denn die Zeile "x=var++;" ist im Grunde das selbe wie "x=var; var++;"

ok, hast Recht, in dem Fall natürlich Pre-Inkrement ....
Das war wohl VHIDT :lol:

Gruß Klaus

 

[JavaUndC-Prog]

Danke euch,

also nochmal zum letzte Kommentar, die letzte Lösung liefert Wert größer > der Eingabe

	public static int gen_zufzahl_ungerade(int min, int max){
		int var;
		int x = 0;
		var = gen_zufzahl(min, max);
 
		if((var % 2 == 1)){
			x = var;
		}
		else
		{
		    x = ++var;
		}
		return x;
	}

Ist die Lösung das haut hin. Danke, großes LOB !

 

[InfectedBytes]

Deswegen schrieb ich folgendes:

p.s. die grenzen min bzw. max sollten ggf noch angepasst werden

Ein bisschen eigeninitiative von dir wäre schon nicht schlecht^^
Im Grunde musst du die min/max Eingabe anpassen

 

[JavaUndC-Prog]

Alles gut !

Ich hab das ja in der while und do while locker und einfach gelöst, weil es da auch einfach ist ;-)
Ich bevorzuge aber immer gerne mal den komplizierten Weg, ich hab hierbei z.b. noch was von euch lernen können !
Danke sehr !

 

[CSHW89]

Danke euch,

also nochmal zum letzte Kommentar, die letzte Lösung liefert Wert größer > der Eingabe

	public static int gen_zufzahl_ungerade(int min, int max){
		int var;
		int x = 0;
		var = gen_zufzahl(min, max);
 
		if((var % 2 == 1)){
			x = var;
		}
		else
		{
		    x = ++var;
		}
		return x;
	}

Ist die Lösung das haut hin. Danke, großes LOB !

Bei dieser Lösung gibt es aber zwei Probleme.
1. Falls 'max' eine gerade Zahl ist (sagen wir 6), und gen_zufzahl 6 zurückgibt, wird 1 addiert und 7 zurückgegeben, was nicht in den Grenzen liegt.
2. Falls 'min' eine ungerade Zahl ist, ist die Wahrscheinlichkeit, diese zu ziehen, geringer. Wenn z.b. 'min = 1' und 'max = 3' ist, sind mögliche Werte von gen_zufzahl 1, 2, 3. 1 bleibt unverändert (33%), 2 und 3 kommt 3 raus (66%).

Mit ein bisschen Arithmetik kann man beide Probleme einfach lösen:

public static int gen_zufzahl_ungerade(int min, int max){
	// 'number' gibt an, wie viele ungerade Zahlen es zwischen 'min' und 'max gibt.
	// Berechne zunächst '(max - min + 1) / 2'.
	// Es gibt aber noch den Spezialfall, dass 'min' und 'max' beide ungerade Zahlen sind.
	// Dann gibt es noch eine weitere ungerade Zahl.
	// Dies macht der hintere etwas kompliziertere Part.
	// Könnte man natürlich auch mit einem 'if' machen.
	int number = (max - min + 1) / 2 + ((min % 2 == 1) ? (max % 2) : 0);
	int i = gen_zufzahl(0, number-1);
	// Nun gebe die i-te ungerade Zahl im Intervall aus.
	// Falls 'min' gerade ist, muss man noch 1 addieren: '(1-min%2)'.
	return min + (i*2) + (1-min%2);
}

Drei Zeilen und fertig
lg Kevin